* Re: [LTTNG-TOOLS PATCH] Fix: don't destroy the sockets if the snapshot was successful
[not found] <1418161085-18739-1-git-send-email-jdesfossez@efficios.com>
@ 2014-12-10 21:37 ` Jérémie Galarneau
0 siblings, 0 replies; 2+ messages in thread
From: Jérémie Galarneau @ 2014-12-10 21:37 UTC (permalink / raw)
To: Julien Desfossez; +Cc: lttng-dev, Jeremie Galarneau
Merged all the way to 2.4.
Thanks!
Jérémie
On Tue, Dec 9, 2014 at 4:38 PM, Julien Desfossez
<jdesfossez@efficios.com> wrote:
> Missing a goto to skip the error condition that was destroying the
> relayd sockets even if a snapshot was successful. We want to keep them
> open to reuse them for the next snapshots.
>
> Signed-off-by: Julien Desfossez <jdesfossez@efficios.com>
> ---
> src/bin/lttng-sessiond/cmd.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/src/bin/lttng-sessiond/cmd.c b/src/bin/lttng-sessiond/cmd.c
> index 11ff9b4..58506b2 100644
> --- a/src/bin/lttng-sessiond/cmd.c
> +++ b/src/bin/lttng-sessiond/cmd.c
> @@ -2979,11 +2979,13 @@ static int record_kernel_snapshot(struct ltt_kernel_session *ksess,
> }
>
> ret = LTTNG_OK;
> + goto end;
>
> error_snapshot:
> /* Clean up copied sockets so this output can use some other later on. */
> consumer_destroy_output_sockets(output->consumer);
> error:
> +end:
> return ret;
> }
>
> --
> 1.9.1
>
--
Jérémie Galarneau
EfficiOS Inc.
http://www.efficios.com
_______________________________________________
lttng-dev mailing list
lttng-dev@lists.lttng.org
http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev
^ permalink raw reply [flat|nested] 2+ messages in thread
* [LTTNG-TOOLS PATCH] Fix: don't destroy the sockets if the snapshot was successful
@ 2014-12-09 21:38 Julien Desfossez
0 siblings, 0 replies; 2+ messages in thread
From: Julien Desfossez @ 2014-12-09 21:38 UTC (permalink / raw)
To: jgalar; +Cc: lttng-dev, Julien Desfossez
Missing a goto to skip the error condition that was destroying the
relayd sockets even if a snapshot was successful. We want to keep them
open to reuse them for the next snapshots.
Signed-off-by: Julien Desfossez <jdesfossez@efficios.com>
---
src/bin/lttng-sessiond/cmd.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/bin/lttng-sessiond/cmd.c b/src/bin/lttng-sessiond/cmd.c
index 11ff9b4..58506b2 100644
--- a/src/bin/lttng-sessiond/cmd.c
+++ b/src/bin/lttng-sessiond/cmd.c
@@ -2979,11 +2979,13 @@ static int record_kernel_snapshot(struct ltt_kernel_session *ksess,
}
ret = LTTNG_OK;
+ goto end;
error_snapshot:
/* Clean up copied sockets so this output can use some other later on. */
consumer_destroy_output_sockets(output->consumer);
error:
+end:
return ret;
}
--
1.9.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
end of thread, other threads:[~2014-12-10 21:37 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <1418161085-18739-1-git-send-email-jdesfossez@efficios.com>
2014-12-10 21:37 ` [LTTNG-TOOLS PATCH] Fix: don't destroy the sockets if the snapshot was successful Jérémie Galarneau
2014-12-09 21:38 Julien Desfossez
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.