From mboxrd@z Thu Jan  1 00:00:00 1970
From: jeremie.galarneau@efficios.com (=?iso-8859-1?Q?J=E9r=E9mie?= Galarneau)
Date: Fri, 20 Sep 2019 17:53:33 -0400
Subject: [lttng-dev] [PATCH lttng-tools master,
 2.11] Fix: destroy command: put consumer output after destroy
 notifier
In-Reply-To: <20190920214114.1073-1-mathieu.desnoyers@efficios.com>
References: <20190920214114.1073-1-mathieu.desnoyers@efficios.com>
Message-ID: <20190920215333.GA49780@gmail.com>

Merged in master and stable-2.11. Thanks!

J?r?mie

On Fri, Sep 20, 2019 at 05:41:14PM -0400, Mathieu Desnoyers wrote:
> The destroy notifier needs to access the consumer output to format
> the absolute path to the last chunk.
> 
> The observed problematic behavior can be observed by doing a
> rotate and a destroy command in quick succession. Sometimes,
> the resulting path printed by the destroy command is incomplete:
> 
> e.g. /archives/20190920T163616-0400-20190920T163618-0400-1
> 
> when we would expect:
> 
> /home/efficios/lttng-traces/auto-20190920-164425/archives/20190920T164437-0400-20190920T164439-0400-1
> 
> Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers at efficios.com>
> ---
>  src/bin/lttng-sessiond/session.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/bin/lttng-sessiond/session.c b/src/bin/lttng-sessiond/session.c
> index 9072a2ec..7fd4332e 100644
> --- a/src/bin/lttng-sessiond/session.c
> +++ b/src/bin/lttng-sessiond/session.c
> @@ -811,7 +811,6 @@ void session_release(struct urcu_ref *ref)
>  
>  	DBG("Destroying session %s (id %" PRIu64 ")", session->name, session->id);
>  
> -	consumer_output_put(session->consumer);
>  	snapshot_destroy(&session->snapshot);
>  
>  	pthread_mutex_destroy(&session->lock);
> @@ -823,6 +822,7 @@ void session_release(struct urcu_ref *ref)
>  	}
>  	session_notify_destruction(session);
>  
> +	consumer_output_put(session->consumer);
>  	kernel_free_session(ksess);
>  	session->kernel_session = NULL;
>  	if (usess) {
> -- 
> 2.17.1
>