Re: [lttng-dev] [PATCH v2 2/2] lttng_ust_init_thread: call urcu_register_thread - Mathieu Desnoyers via lttng-dev

From: Mathieu Desnoyers via lttng-dev <lttng-dev@lists.lttng.org>
To: Norbert Lange <nolange79@gmail.com>
Cc: lttng-dev <lttng-dev@lists.lttng.org>
Subject: Re: [lttng-dev] [PATCH v2 2/2] lttng_ust_init_thread: call urcu_register_thread
Date: Mon, 1 Aug 2022 11:13:24 -0400 (EDT)	[thread overview]
Message-ID: <390803304.93584.1659366804771.JavaMail.zimbra@efficios.com> (raw)
In-Reply-To: <20220801143700.160380-2-norbert.lange@andritz.com>

----- On Aug 1, 2022, at 10:37 AM, Norbert Lange nolange79@gmail.com wrote:

> From: Norbert Lange <nolange79@gmail.com>
> 
> Eagerly register the thread, and avoid taking mutex during the
> first tracepoint.

Merged, thanks!

Mathieu

> 
> Signed-off-by: Norbert Lange <nolange79@gmail.com>
> Acked-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
> ---
> src/lib/lttng-ust/lttng-ust-comm.c | 2 ++
> 1 file changed, 2 insertions(+)
> 
> diff --git a/src/lib/lttng-ust/lttng-ust-comm.c
> b/src/lib/lttng-ust/lttng-ust-comm.c
> index 0a039fef..0fe2da46 100644
> --- a/src/lib/lttng-ust/lttng-ust-comm.c
> +++ b/src/lib/lttng-ust/lttng-ust-comm.c
> @@ -447,6 +447,8 @@ void lttng_ust_init_thread(void)
> 	 * this thread attempts to use them.
> 	 */
> 	lttng_ust_common_init_thread(LTTNG_UST_INIT_THREAD_MASK);
> +
> +	lttng_ust_urcu_register_thread();
> }
> 
> int lttng_get_notify_socket(void *owner)
> --
> 2.35.1

-- 
Mathieu Desnoyers
EfficiOS Inc.
http://www.efficios.com
_______________________________________________
lttng-dev mailing list
lttng-dev@lists.lttng.org
https://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev