X-Git-Url: https://git.lttng.org/?p=lttng-tools.git;a=blobdiff_plain;f=src%2Fbin%2Flttng-sessiond%2Fkernel-consumer.c;h=05e30e49cb0b9ff65e88534ae9cec25de2222693;hp=c86d52803b80ac19c254eb3a1e267f3223d6c5d9;hb=1624d5b7ba86633d36f3a5c86ea1df5d308c4360;hpb=ca22feea083301934d1c8511851c86fb008c0697

diff --git a/src/bin/lttng-sessiond/kernel-consumer.c b/src/bin/lttng-sessiond/kernel-consumer.c
index c86d52803..05e30e49c 100644
--- a/src/bin/lttng-sessiond/kernel-consumer.c
+++ b/src/bin/lttng-sessiond/kernel-consumer.c
@@ -26,36 +26,87 @@
 #include <common/defaults.h>
 
 #include "consumer.h"
+#include "health.h"
 #include "kernel-consumer.h"
 
 /*
  * Sending a single channel to the consumer with command ADD_CHANNEL.
  */
-int kernel_consumer_add_channel(int sock, struct ltt_kernel_channel *channel)
+int kernel_consumer_add_channel(struct consumer_socket *sock,
+		struct ltt_kernel_channel *channel, struct ltt_kernel_session *session)
 {
 	int ret;
+	char tmp_path[PATH_MAX];
+	const char *pathname;
 	struct lttcomm_consumer_msg lkm;
+	struct consumer_output *consumer;
 
 	/* Safety net */
 	assert(channel);
+	assert(session);
+	assert(session->consumer);
+
+	consumer = session->consumer;
 
 	DBG("Kernel consumer adding channel %s to kernel consumer",
 			channel->channel->name);
 
+	/* Get the right path name destination */
+	if (consumer->type == CONSUMER_DST_LOCAL) {
+		/* Set application path to the destination path */
+		ret = snprintf(tmp_path, sizeof(tmp_path), "%s/%s",
+				consumer->dst.trace_path, consumer->subdir);
+		if (ret < 0) {
+			PERROR("snprintf metadata path");
+			goto error;
+		}
+		pathname = tmp_path;
+
+		/* Create directory */
+		ret = run_as_mkdir_recursive(pathname, S_IRWXU | S_IRWXG,
+				session->uid, session->gid);
+		if (ret < 0) {
+			if (ret != -EEXIST) {
+				ERR("Trace directory creation error");
+				goto error;
+			}
+		}
+		DBG3("Kernel local consumer tracefile path: %s", pathname);
+	} else {
+		ret = snprintf(tmp_path, sizeof(tmp_path), "%s", consumer->subdir);
+		if (ret < 0) {
+			PERROR("snprintf metadata path");
+			goto error;
+		}
+		pathname = tmp_path;
+		DBG3("Kernel network consumer subdir path: %s", pathname);
+	}
+
 	/* Prep channel message structure */
 	consumer_init_channel_comm_msg(&lkm,
 			LTTNG_CONSUMER_ADD_CHANNEL,
 			channel->fd,
-			channel->channel->attr.subbuf_size,
-			0, /* Kernel */
+			session->id,
+			pathname,
+			session->uid,
+			session->gid,
+			consumer->net_seq_index,
 			channel->channel->name,
-			channel->stream_count);
+			channel->stream_count,
+			channel->channel->attr.output,
+			CONSUMER_CHANNEL_TYPE_DATA,
+			channel->channel->attr.tracefile_size,
+			channel->channel->attr.tracefile_count);
+
+	health_code_update();
 
 	ret = consumer_send_channel(sock, &lkm);
 	if (ret < 0) {
 		goto error;
 	}
 
+	health_code_update();
+
 error:
 	return ret;
 }
@@ -63,7 +114,8 @@ error:
 /*
  * Sending metadata to the consumer with command ADD_CHANNEL and ADD_STREAM.
  */
-int kernel_consumer_add_metadata(int sock, struct ltt_kernel_session *session)
+int kernel_consumer_add_metadata(struct consumer_socket *sock,
+		struct ltt_kernel_session *session)
 {
 	int ret;
 	char tmp_path[PATH_MAX];
@@ -74,6 +126,7 @@ int kernel_consumer_add_metadata(int sock, struct ltt_kernel_session *session)
 	/* Safety net */
 	assert(session);
 	assert(session->consumer);
+	assert(sock);
 
 	DBG("Sending metadata %d to kernel consumer", session->metadata_stream_fd);
 
@@ -115,31 +168,34 @@ int kernel_consumer_add_metadata(int sock, struct ltt_kernel_session *session)
 	consumer_init_channel_comm_msg(&lkm,
 			LTTNG_CONSUMER_ADD_CHANNEL,
 			session->metadata->fd,
-			session->metadata->conf->attr.subbuf_size,
-			0, /* for kernel */
-			"metadata",
-			1);
+			session->id,
+			pathname,
+			session->uid,
+			session->gid,
+			consumer->net_seq_index,
+			DEFAULT_METADATA_NAME,
+			1,
+			DEFAULT_KERNEL_CHANNEL_OUTPUT,
+			CONSUMER_CHANNEL_TYPE_METADATA,
+			0, 0);
+
+	health_code_update();
 
 	ret = consumer_send_channel(sock, &lkm);
 	if (ret < 0) {
 		goto error;
 	}
 
+	health_code_update();
+
 	/* Prep stream message structure */
 	consumer_init_stream_comm_msg(&lkm,
 			LTTNG_CONSUMER_ADD_STREAM,
 			session->metadata->fd,
 			session->metadata_stream_fd,
-			LTTNG_CONSUMER_ACTIVE_STREAM,
-			DEFAULT_KERNEL_CHANNEL_OUTPUT,
-			0, /* Kernel */
-			session->uid,
-			session->gid,
-			consumer->net_seq_index,
-			1, /* Metadata flag set */
-			"metadata",
-			pathname,
-			session->id);
+			0); /* CPU: 0 for metadata. */
+
+	health_code_update();
 
 	/* Send stream and file descriptor */
 	ret = consumer_send_stream(sock, consumer, &lkm,
@@ -148,6 +204,8 @@ int kernel_consumer_add_metadata(int sock, struct ltt_kernel_session *session)
 		goto error;
 	}
 
+	health_code_update();
+
 error:
 	return ret;
 }
@@ -155,12 +213,11 @@ error:
 /*
  * Sending a single stream to the consumer with command ADD_STREAM.
  */
-int kernel_consumer_add_stream(int sock, struct ltt_kernel_channel *channel,
-		struct ltt_kernel_stream *stream, struct ltt_kernel_session *session)
+int kernel_consumer_add_stream(struct consumer_socket *sock,
+		struct ltt_kernel_channel *channel, struct ltt_kernel_stream *stream,
+		struct ltt_kernel_session *session)
 {
 	int ret;
-	char tmp_path[PATH_MAX];
-	const char *pathname;
 	struct lttcomm_consumer_msg lkm;
 	struct consumer_output *consumer;
 
@@ -168,6 +225,7 @@ int kernel_consumer_add_stream(int sock, struct ltt_kernel_channel *channel,
 	assert(stream);
 	assert(session);
 	assert(session->consumer);
+	assert(sock);
 
 	DBG("Sending stream %d of channel %s to kernel consumer",
 			stream->fd, channel->channel->name);
@@ -175,41 +233,14 @@ int kernel_consumer_add_stream(int sock, struct ltt_kernel_channel *channel,
 	/* Get consumer output pointer */
 	consumer = session->consumer;
 
-	/* Get the right path name destination */
-	if (consumer->type == CONSUMER_DST_LOCAL) {
-		/* Set application path to the destination path */
-		ret = snprintf(tmp_path, sizeof(tmp_path), "%s/%s",
-				consumer->dst.trace_path, consumer->subdir);
-		if (ret < 0) {
-			PERROR("snprintf stream path");
-			goto error;
-		}
-		pathname = tmp_path;
-		DBG3("Kernel local consumer tracefile path: %s", pathname);
-	} else {
-		ret = snprintf(tmp_path, sizeof(tmp_path), "%s", consumer->subdir);
-		if (ret < 0) {
-			PERROR("snprintf stream path");
-			goto error;
-		}
-		pathname = tmp_path;
-		DBG3("Kernel network consumer subdir path: %s", pathname);
-	}
-
 	/* Prep stream consumer message */
-	consumer_init_stream_comm_msg(&lkm, LTTNG_CONSUMER_ADD_STREAM,
+	consumer_init_stream_comm_msg(&lkm,
+			LTTNG_CONSUMER_ADD_STREAM,
 			channel->fd,
 			stream->fd,
-			stream->state,
-			channel->channel->attr.output,
-			0, /* Kernel */
-			session->uid,
-			session->gid,
-			consumer->net_seq_index,
-			0, /* Metadata flag unset */
-			stream->name,
-			pathname,
-			session->id);
+			stream->cpu);
+
+	health_code_update();
 
 	/* Send stream and file descriptor */
 	ret = consumer_send_stream(sock, consumer, &lkm, &stream->fd, 1);
@@ -217,6 +248,8 @@ int kernel_consumer_add_stream(int sock, struct ltt_kernel_channel *channel,
 		goto error;
 	}
 
+	health_code_update();
+
 error:
 	return ret;
 }
@@ -224,7 +257,7 @@ error:
 /*
  * Send all stream fds of kernel channel to the consumer.
  */
-int kernel_consumer_send_channel_stream(int sock,
+int kernel_consumer_send_channel_stream(struct consumer_socket *sock,
 		struct ltt_kernel_channel *channel, struct ltt_kernel_session *session)
 {
 	int ret;
@@ -234,6 +267,7 @@ int kernel_consumer_send_channel_stream(int sock,
 	assert(channel);
 	assert(session);
 	assert(session->consumer);
+	assert(sock);
 
 	/* Bail out if consumer is disabled */
 	if (!session->consumer->enabled) {
@@ -244,7 +278,7 @@ int kernel_consumer_send_channel_stream(int sock,
 	DBG("Sending streams of channel %s to kernel consumer",
 			channel->channel->name);
 
-	ret = kernel_consumer_add_channel(sock, channel);
+	ret = kernel_consumer_add_channel(sock, channel, session);
 	if (ret < 0) {
 		goto error;
 	}
@@ -269,7 +303,8 @@ error:
 /*
  * Send all stream fds of the kernel session to the consumer.
  */
-int kernel_consumer_send_session(int sock, struct ltt_kernel_session *session)
+int kernel_consumer_send_session(struct consumer_socket *sock,
+		struct ltt_kernel_session *session)
 {
 	int ret;
 	struct ltt_kernel_channel *chan;
@@ -277,6 +312,7 @@ int kernel_consumer_send_session(int sock, struct ltt_kernel_session *session)
 	/* Safety net */
 	assert(session);
 	assert(session->consumer);
+	assert(sock);
 
 	/* Bail out if consumer is disabled */
 	if (!session->consumer->enabled) {