No objection to the code, but please add a description and comment. Jérémie
On 18 September 2017 at 18:51, Jonathan Rajotte <[email protected]> wrote: > Signed-off-by: Jonathan Rajotte <[email protected]> > --- > src/bin/lttng-sessiond/main.c | 7 ++++--- > 1 file changed, 4 insertions(+), 3 deletions(-) > > diff --git a/src/bin/lttng-sessiond/main.c b/src/bin/lttng-sessiond/main.c > index 45c0270e..0ca72ec8 100644 > --- a/src/bin/lttng-sessiond/main.c > +++ b/src/bin/lttng-sessiond/main.c > @@ -497,9 +497,6 @@ static void stop_threads(void) > ERR("write error on thread quit pipe"); > } > > - /* Dispatch thread */ > - CMM_STORE_SHARED(dispatch_thread_exit, 1); > - futex_nto1_wake(&ust_cmd_queue.futex); > } > > /* > @@ -6306,6 +6303,10 @@ exit_client: > retval = -1; > } > exit_reg_apps: > + /* Instruct the dispatch thread to quit */ > + CMM_STORE_SHARED(dispatch_thread_exit, 1); > + futex_nto1_wake(&ust_cmd_queue.futex); > + > ret = pthread_join(dispatch_thread, &status); > if (ret) { > errno = ret; > -- > 2.11.0 > -- Jérémie Galarneau EfficiOS Inc. http://www.efficios.com _______________________________________________ lttng-dev mailing list [email protected] https://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev
