X-Git-Url: https://git.lttng.org/?p=lttng-tools.git;a=blobdiff_plain;f=src%2Fbin%2Flttng%2Fcommands%2Fsnapshot.c;h=904ebae15c8450b6ff1d887fdce3a883d5c648f4;hp=19b28321f084f26a97a7549bc6c13bb42a122c84;hb=54185c70692f8a6d9c667214743e0ffd07a0afa3;hpb=3da6df223e781decc790cd870e408026f633ec9c

diff --git a/src/bin/lttng/commands/snapshot.c b/src/bin/lttng/commands/snapshot.c
index 19b28321f..904ebae15 100644
--- a/src/bin/lttng/commands/snapshot.c
+++ b/src/bin/lttng/commands/snapshot.c
@@ -179,11 +179,18 @@ static int list_output(void)
 	}
 
 	while ((s_iter = lttng_snapshot_output_list_get_next(list)) != NULL) {
-		MSG("%s[%" PRIu32 "] %s: %s (max-size: %" PRId64 ")", indent4,
-				lttng_snapshot_output_get_id(s_iter),
-				lttng_snapshot_output_get_name(s_iter),
-				lttng_snapshot_output_get_ctrl_url(s_iter),
-				lttng_snapshot_output_get_maxsize(s_iter));
+		if (lttng_snapshot_output_get_maxsize(s_iter)) {
+			MSG("%s[%" PRIu32 "] %s: %s (max size: %" PRIu64 " bytes)", indent4,
+					lttng_snapshot_output_get_id(s_iter),
+					lttng_snapshot_output_get_name(s_iter),
+					lttng_snapshot_output_get_ctrl_url(s_iter),
+					lttng_snapshot_output_get_maxsize(s_iter));
+		} else {
+			MSG("%s[%" PRIu32 "] %s: %s", indent4,
+					lttng_snapshot_output_get_id(s_iter),
+					lttng_snapshot_output_get_name(s_iter),
+					lttng_snapshot_output_get_ctrl_url(s_iter));
+		}
 		output_seen = 1;
 		if (lttng_opt_mi) {
 			ret = mi_lttng_snapshot_list_output(writer, s_iter);
@@ -312,10 +319,16 @@ static int add_output(const char *url)
 
 	MSG("Snapshot output successfully added for session %s",
 			current_session_name);
-	MSG("  [%" PRIu32 "] %s: %s (max-size: %" PRId64 ")",
-			lttng_snapshot_output_get_id(output), n_ptr,
-			lttng_snapshot_output_get_ctrl_url(output),
-			lttng_snapshot_output_get_maxsize(output));
+	if (opt_max_size) {
+		MSG("  [%" PRIu32 "] %s: %s (max size: %" PRIu64 " bytes)",
+				lttng_snapshot_output_get_id(output), n_ptr,
+				lttng_snapshot_output_get_ctrl_url(output),
+				lttng_snapshot_output_get_maxsize(output));
+	} else {
+		MSG("  [%" PRIu32 "] %s: %s",
+				lttng_snapshot_output_get_id(output), n_ptr,
+				lttng_snapshot_output_get_ctrl_url(output));
+	}
 	if (lttng_opt_mi) {
 		ret = mi_lttng_snapshot_add_output(writer, current_session_name,
 				n_ptr, output);
@@ -333,11 +346,23 @@ static int cmd_add_output(int argc, const char **argv)
 	int ret;
 
 	if (argc < 2 && (!opt_data_url || !opt_ctrl_url)) {
+		ERR("An output destination must be specified to add a snapshot output.");
 		ret = CMD_ERROR;
 		goto end;
 	}
 
 	ret = add_output(argv[1]);
+	if (ret < 0) {
+		switch (-ret) {
+		case LTTNG_ERR_SNAPSHOT_UNSUPPORTED:
+			ERR("Session \"%s\" contains a channel that is incompatible with the snapshot functionality.\nMake sure all channels are configured in 'mmap' output mode.",
+					current_session_name);
+			ret = CMD_ERROR;
+			break;
+		default:
+			break;
+		}
+	}
 
 end:
 	return ret;
@@ -350,6 +375,7 @@ static int cmd_del_output(int argc, const char **argv)
 	long id;
 
 	if (argc < 2) {
+		ERR("A snapshot output name or id must be provided to delete a snapshot output.");
 		ret = CMD_ERROR;
 		goto end;
 	}
@@ -493,8 +519,22 @@ static enum cmd_error_code handle_command(const char **argv)
 
 			result = cmd->func(argc, argv);
 			if (result) {
-				switch (-result) {
-				case LTTNG_ERR_SNAPSHOT_NODATA:
+				switch (result) {
+				case CMD_ERROR:
+				case CMD_UNDEFINED:
+				case CMD_FATAL:
+				case CMD_WARNING:
+				case CMD_UNSUPPORTED:
+					/*
+					 * Sub-commands mix lttng_error_codes
+					 * and cmd_error_codes. This should be
+					 * cleaned-up, but in the meantime this
+					 * hack works since the values of the
+					 * two enums do not intersect.
+					 */
+					cmd_ret = result;
+					break;
+				case -LTTNG_ERR_SNAPSHOT_NODATA:
 					WARN("%s", lttng_strerror(result));
 
 					/*  A warning is fine since the user has no control on