diff options
author | Daniel Wagner <dwagner@suse.de> | 2021-03-04 19:36:14 +0100 |
---|---|---|
committer | John Kacur <jkacur@redhat.com> | 2021-05-07 12:24:45 -0400 |
commit | a9fc7fac63ec60e5d09c9f75483d26d947a670c1 (patch) | |
tree | 8ac7a5dc1bc0e60cedb8fb08b92f58cbad293644 | |
parent | 3afeb71c96b5d615eed4354aa2e16f9220a6cc9a (diff) | |
download | rt-tests-a9fc7fac63ec60e5d09c9f75483d26d947a670c1.tar.gz |
rt-util: Add rt_init function
rt_init() should be called as first in the test program.
The main job for this function is to copy the command line
before getopt() runs. By default, getopt_long() permutes
the contents of argv as it scans, so that eventually all the
nonoptions are at the end. This is confusing in the JSON
output, thus copy the command line before we call getopt_long().
Signed-off-by: Daniel Wagner <dwagner@suse.de>
Signed-off-by: John Kacur <jkacur@redhat.com>
-rw-r--r-- | src/include/rt-utils.h | 2 | ||||
-rw-r--r-- | src/lib/rt-utils.c | 47 |
2 files changed, 42 insertions, 7 deletions
diff --git a/src/include/rt-utils.h b/src/include/rt-utils.h index 36af92b..1dbd3f1 100644 --- a/src/include/rt-utils.h +++ b/src/include/rt-utils.h @@ -80,6 +80,8 @@ static inline int64_t calctime(struct timespec t) return time; } +void rt_init(int argc, char *argv[]); + void rt_write_json(const char *filename, int argc, char *argv[], void (*cb)(FILE *, void *), void *data); diff --git a/src/lib/rt-utils.c b/src/lib/rt-utils.c index 00907c5..8f0e094 100644 --- a/src/lib/rt-utils.c +++ b/src/lib/rt-utils.c @@ -29,12 +29,15 @@ #include "rt-error.h" #define TRACEBUFSIZ 1024 +#define MAX_COMMAND_LINE 4096 static char debugfileprefix[MAX_PATH]; static char *fileprefix; static int trace_fd = -1; static int tracemark_fd = -1; static __thread char tracebuf[TRACEBUFSIZ]; +static char test_cmdline[MAX_COMMAND_LINE]; +static int rt_init_run; /* * Finds the tracing directory in a mounted debugfs @@ -486,6 +489,34 @@ void disable_trace_mark(void) close_tracemark_fd(); } +void rt_init(int argc, char *argv[]) +{ + int offset = 0; + int len, i; + + test_cmdline[0] = '\0'; + + /* + * getopt_long() permutes the contents of argv as it scans, so + * that eventually all the nonoptions are at the end. Make a + * copy before calling getopt_long(). + */ + for (i = 0; i < argc;) { + len = strlen(argv[i]); + if (offset + len + 1 >= MAX_COMMAND_LINE) + break; + + strcat(test_cmdline, argv[i]); + i++; + if (i < argc) + strcat(test_cmdline, " "); + + offset += len + 1; + } + + rt_init_run = 1; +} + static char *get_cmdline(int argc, char *argv[]) { char *cmdline; @@ -519,7 +550,7 @@ void rt_write_json(const char *filename, int argc, char *argv[], struct timeval tv; char tsbuf[64]; struct tm *tm; - char *cmdline; + char *cmdline = NULL; FILE *f, *s; time_t t; size_t n; @@ -533,10 +564,11 @@ void rt_write_json(const char *filename, int argc, char *argv[], err_exit(errno, "Failed to open '%s'\n", filename); } - cmdline = get_cmdline(argc, argv); - if (!cmdline) - err_exit(ENOMEM, "get_cmdline()"); - + if (!rt_init_run) { + cmdline = get_cmdline(argc, argv); + if (!cmdline) + err_exit(ENOMEM, "get_cmdline()"); + } gettimeofday(&tv, NULL); t = tv.tv_sec; @@ -557,7 +589,7 @@ void rt_write_json(const char *filename, int argc, char *argv[], fprintf(f, "{\n"); fprintf(f, " \"file_version\": 1,\n"); - fprintf(f, " \"cmdline:\": \"%s\",\n", cmdline); + fprintf(f, " \"cmdline:\": \"%s\",\n", rt_init_run? test_cmdline : cmdline); fprintf(f, " \"rt_test_version:\": \"%1.2f\",\n", VERSION); fprintf(f, " \"finished\": \"%s\",\n", tsbuf); fprintf(f, " \"sysinfo\": {\n"); @@ -573,7 +605,8 @@ void rt_write_json(const char *filename, int argc, char *argv[], fprintf(f, "}\n"); - free(cmdline); + if (!rt_init_run) + free(cmdline); if (!filename || strcmp("-", filename)) fclose(f); |