From: David Goulet <david.goulet@polymtl.ca>
Date: Tue, 16 Aug 2011 14:24:51 +0000 (-0400)
Subject: Send data to kconsumerd before tracing start
X-Git-Tag: v2.0-pre12~18
X-Git-Url: https://git.lttng.org/?p=lttng-tools.git;a=commitdiff_plain;h=283046e084c1461001c8584c5b4adada02265496;hp=b389abbe14d653f704fcf8f952539cc5ce775c53

Send data to kconsumerd before tracing start

The metadata buffer of the kernel can get overflowed between the start
of the kernel consumer and the kernel start session. Sending the fds
before tracing start fix that problem.

Fixed-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Signed-off-by: David Goulet <david.goulet@polymtl.ca>
---

diff --git a/ltt-sessiond/main.c b/ltt-sessiond/main.c
index 470ab1818..153b215c0 100644
--- a/ltt-sessiond/main.c
+++ b/ltt-sessiond/main.c
@@ -1961,16 +1961,16 @@ static int process_client_msg(struct command_ctx *cmd_ctx)
 				}
 			}
 
-			DBG("Start kernel tracing");
-			ret = kernel_start_session(cmd_ctx->session->kernel_session);
+			ret = start_kernel_trace(cmd_ctx->session->kernel_session);
 			if (ret < 0) {
-				ERR("Kernel start session failed");
 				ret = LTTCOMM_KERN_START_FAIL;
 				goto error;
 			}
 
-			ret = start_kernel_trace(cmd_ctx->session->kernel_session);
+			DBG("Start kernel tracing");
+			ret = kernel_start_session(cmd_ctx->session->kernel_session);
 			if (ret < 0) {
+				ERR("Kernel start session failed");
 				ret = LTTCOMM_KERN_START_FAIL;
 				goto error;
 			}