[PATCH i-g-t 4/5] runner/settings: Serialize command line

[Lucas De Marchi]

Serialize the command line to metadata.txt. The expected format in the
metadata.txt is like below:

	cmdline.argc : 6
	cmdline.argv[0] : ./build/runner/igt_runner
	cmdline.argv[1] : -o
	cmdline.argv[2] : --test-list
	cmdline.argv[3] : /tmp/testlist.txt
	cmdline.argv[4] : build/tests/
	cmdline.argv[5] : /tmp/results

Signed-off-by: Lucas De Marchi <lucas.demarchi at intel.com>
---
 runner/settings.c | 27 +++++++++++++++++++++++++++
 runner/settings.h |  4 ++++
 2 files changed, 31 insertions(+)

diff --git a/runner/settings.c b/runner/settings.c
index 96377f1de..1d65195c9 100644
--- a/runner/settings.c
+++ b/runner/settings.c
@@ -874,6 +874,8 @@ bool parse_options(int argc, char **argv,
 		goto error;
 	}
 
+	settings->cmdline.argc = argc;
+	settings->cmdline.argv = argv;
 
 	return true;
 
@@ -1054,6 +1056,7 @@ static bool serialize_hook_strs(struct settings *settings, int dirfd)
 bool serialize_settings(struct settings *settings)
 {
 #define SERIALIZE_LINE(f, s, name, format) fprintf(f, "%s : " format "\n", #name, s->name)
+#define SERIALIZE_ARRAY_ITEM(f, s, name, _i, format) fprintf(f, "%s[%d] : " format "\n", #name, _i, s->name[_i])
 
 	FILE *f;
 	int dirfd, covfd;
@@ -1122,6 +1125,10 @@ bool serialize_settings(struct settings *settings)
 	SERIALIZE_LINE(f, settings, cov_results_per_test, "%d");
 	SERIALIZE_LINE(f, settings, code_coverage_script, "%s");
 
+	SERIALIZE_LINE(f, settings, cmdline.argc, "%d");
+	for (int i = 0; i < settings->cmdline.argc; i++)
+		SERIALIZE_ARRAY_ITEM(f, settings, cmdline.argv, i, "%s");
+
 	if (settings->sync) {
 		fflush(f);
 		fsync(fileno(f));
@@ -1176,9 +1183,21 @@ static char *parse_str(char **pval)
 		s->field = _f(&val);		\
 		goto cleanup;			\
 	}
+#define PARSE_LINE_ARRAY(s, name, val, field, _f, _max)		\
+	do {							\
+		int idx;					\
+		if (sscanf(name, #field "[%u]", &idx) == 1 &&	\
+		    idx < s->_max) {				\
+			s->field[idx] = _f(&val);		\
+			goto cleanup;				\
+		}						\
+	} while (0)
+
 #define PARSE_INT(s, name, val, field) PARSE_LINE(s, name, val, field, parse_int)
 #define PARSE_UL(s, name, val, field)  PARSE_LINE(s, name, val, field, parse_ul)
 #define PARSE_STR(s, name, val, field) PARSE_LINE(s, name, val, field, parse_str)
+#define PARSE_ARRAY_STR(s, name, val, field, _max) \
+	PARSE_LINE_ARRAY(s, name, val, field, parse_str, _max)
 
 bool read_settings_from_file(struct settings *settings, FILE *f)
 {
@@ -1210,6 +1229,13 @@ bool read_settings_from_file(struct settings *settings, FILE *f)
 		PARSE_INT(settings, name, val, enable_code_coverage);
 		PARSE_INT(settings, name, val, cov_results_per_test);
 		PARSE_STR(settings, name, val, code_coverage_script);
+		PARSE_INT(settings, name, val, cmdline.argc);
+
+		if (settings->cmdline.argc && !settings->cmdline.argv)
+			settings->cmdline.argv = calloc(settings->cmdline.argc,
+							sizeof(*settings->cmdline.argv));
+
+		PARSE_ARRAY_STR(settings, name, val, cmdline.argv, cmdline.argc);
 
 		printf("Warning: Unknown field in settings file: %s = %s\n",
 		       name, val);
@@ -1233,6 +1259,7 @@ cleanup:
 	return true;
 }
 #undef PARSE_LINE
+#undef PARSE_LINE_ARRAY
 
 /**
  * read_env_vars_from_file() - load env vars from a file
diff --git a/runner/settings.h b/runner/settings.h
index f69f09778..2266118a7 100644
--- a/runner/settings.h
+++ b/runner/settings.h
@@ -75,6 +75,10 @@ struct settings {
 	char *code_coverage_script;
 	bool enable_code_coverage;
 	bool cov_results_per_test;
+	struct {
+		int argc;
+		char **argv;
+	} cmdline;
 };
 
 /**
-- 
2.48.0