X-Git-Url: https://git.liburcu.org/?a=blobdiff_plain;f=src%2Fcommon%2Factions%2Fstart-session.c;h=da7aad81fc7f0e4773183f5802e112fb50dfcbd4;hb=97535efaa975ca52bf02c2d5e76351bfd2e3defa;hp=e5cbac7267c0b74dd24817c8e68413cc51775bce;hpb=588c4b0d153e8919b8d9159b69ce0db8fbe52af0;p=lttng-tools.git

diff --git a/src/common/actions/start-session.c b/src/common/actions/start-session.c
index e5cbac726..da7aad81f 100644
--- a/src/common/actions/start-session.c
+++ b/src/common/actions/start-session.c
@@ -5,9 +5,9 @@
  *
  */
 
-#include <assert.h>
 #include <common/error.h>
 #include <common/macros.h>
+#include <common/mi-lttng.h>
 #include <lttng/action/action-internal.h>
 #include <lttng/action/rate-policy-internal.h>
 #include <lttng/action/rate-policy.h>
@@ -45,7 +45,7 @@ lttng_action_start_session_internal_get_rate_policy(
 static struct lttng_action_start_session *action_start_session_from_action(
 		struct lttng_action *action)
 {
-	assert(action);
+	LTTNG_ASSERT(action);
 
 	return container_of(action, struct lttng_action_start_session, parent);
 }
@@ -53,7 +53,7 @@ static struct lttng_action_start_session *action_start_session_from_action(
 static const struct lttng_action_start_session *
 action_start_session_from_action_const(const struct lttng_action *action)
 {
-	assert(action);
+	LTTNG_ASSERT(action);
 
 	return container_of(action, struct lttng_action_start_session, parent);
 }
@@ -92,8 +92,8 @@ static bool lttng_action_start_session_is_equal(
 	b = container_of(_b, struct lttng_action_start_session, parent);
 
 	/* Action is not valid if this is not true. */
-	assert(a->session_name);
-	assert(b->session_name);
+	LTTNG_ASSERT(a->session_name);
+	LTTNG_ASSERT(b->session_name);
 	if (strcmp(a->session_name, b->session_name)) {
 		goto end;
 	}
@@ -111,12 +111,12 @@ static int lttng_action_start_session_serialize(
 	size_t session_name_len;
 	int ret;
 
-	assert(action);
-	assert(payload);
+	LTTNG_ASSERT(action);
+	LTTNG_ASSERT(payload);
 
 	action_start_session = action_start_session_from_action(action);
 
-	assert(action_start_session->session_name);
+	LTTNG_ASSERT(action_start_session->session_name);
 
 	DBG("Serializing start session action: session-name: %s",
 			action_start_session->session_name);
@@ -216,7 +216,7 @@ ssize_t lttng_action_start_session_create_from_payload(
 		goto end;
 	}
 
-	assert(policy);
+	LTTNG_ASSERT(policy);
 	status = lttng_action_start_session_set_rate_policy(action, policy);
 	if (status != LTTNG_ACTION_STATUS_OK) {
 		consumed_len = -1;
@@ -233,6 +233,62 @@ end:
 	return consumed_len;
 }
 
+static enum lttng_error_code lttng_action_start_session_mi_serialize(
+		const struct lttng_action *action, struct mi_writer *writer)
+{
+	int ret;
+	enum lttng_error_code ret_code;
+	enum lttng_action_status status;
+	const char *session_name = NULL;
+	const struct lttng_rate_policy *policy = NULL;
+
+	LTTNG_ASSERT(action);
+	LTTNG_ASSERT(IS_START_SESSION_ACTION(action));
+
+	status = lttng_action_start_session_get_session_name(
+			action, &session_name);
+	LTTNG_ASSERT(status == LTTNG_ACTION_STATUS_OK);
+	LTTNG_ASSERT(session_name != NULL);
+
+	status = lttng_action_start_session_get_rate_policy(action, &policy);
+	LTTNG_ASSERT(status == LTTNG_ACTION_STATUS_OK);
+	LTTNG_ASSERT(policy != NULL);
+
+	/* Open action start session element. */
+	ret = mi_lttng_writer_open_element(
+			writer, mi_lttng_element_action_start_session);
+	if (ret) {
+		goto mi_error;
+	}
+
+	/* Session name. */
+	ret = mi_lttng_writer_write_element_string(
+			writer, mi_lttng_element_session_name, session_name);
+	if (ret) {
+		goto mi_error;
+	}
+
+	/* Rate policy. */
+	ret_code = lttng_rate_policy_mi_serialize(policy, writer);
+	if (ret_code != LTTNG_OK) {
+		goto end;
+	}
+
+	/* Close action start session element. */
+	ret = mi_lttng_writer_close_element(writer);
+	if (ret) {
+		goto mi_error;
+	}
+
+	ret_code = LTTNG_OK;
+	goto end;
+
+mi_error:
+	ret_code = LTTNG_ERR_MI_IO_FAIL;
+end:
+	return ret_code;
+}
+
 struct lttng_action *lttng_action_start_session_create(void)
 {
 	struct lttng_action *action = NULL;
@@ -256,7 +312,8 @@ struct lttng_action *lttng_action_start_session_create(void)
 			lttng_action_start_session_is_equal,
 			lttng_action_start_session_destroy,
 			lttng_action_start_session_internal_get_rate_policy,
-			lttng_action_generic_add_error_query_results);
+			lttng_action_generic_add_error_query_results,
+			lttng_action_start_session_mi_serialize);
 
 	status = lttng_action_start_session_set_rate_policy(action, policy);
 	if (status != LTTNG_ACTION_STATUS_OK) {