diff options
| author | Daniel Vetter <daniel.vetter@ffwll.ch> | 2017-08-23 15:30:44 +0200 |
|---|---|---|
| committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2017-08-24 11:07:02 +0200 |
| commit | 80cc54023e198165eca34450e9cc75c9cffcb072 (patch) | |
| tree | 2f482499ddc9e5703aea25b2b493a266238ac57e /lib/igt_core.c | |
| parent | 7552c5af1f43386424085c32bca3cbd0cb0dfe48 (diff) | |
lib/core: Use igt_info instead of printf
igt_info doesn't add anything when printing to stdout, but so looks
the same. But it has the upside of appending the lines also to the igt
crashdump log, where I especially want the backtraces. Atm they're the
only thing that doesn't end up in there, which is a bit confusing.
While at it also convert the other lines - the test summary usually
doesn't make it since the test fails before that, and the version line
tends to scroll off the crashdump.
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
Reviewed-by: Petri Latvala <petri.latvala@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Diffstat (limited to 'lib/igt_core.c')
| -rw-r--r-- | lib/igt_core.c | 20 |
1 files changed, 10 insertions, 10 deletions
diff --git a/lib/igt_core.c b/lib/igt_core.c index 9eb99eda..58d64dc2 100644 --- a/lib/igt_core.c +++ b/lib/igt_core.c @@ -588,9 +588,9 @@ static void print_version(void) uname(&uts); - fprintf(stdout, "IGT-Version: %s-%s (%s) (%s: %s %s)\n", PACKAGE_VERSION, - IGT_GIT_SHA1, TARGET_CPU_PLATFORM, - uts.sysname, uts.release, uts.machine); + igt_info("IGT-Version: %s-%s (%s) (%s: %s %s)\n", PACKAGE_VERSION, + IGT_GIT_SHA1, TARGET_CPU_PLATFORM, + uts.sysname, uts.release, uts.machine); } static void print_usage(const char *help_str, bool output_on_stderr) @@ -1028,10 +1028,10 @@ static void exit_subtest(const char *result) struct timespec now; gettime(&now); - printf("%sSubtest %s: %s (%.3fs)%s\n", - (!__igt_plain_output) ? "\x1b[1m" : "", - in_subtest, result, time_elapsed(&subtest_time, &now), - (!__igt_plain_output) ? "\x1b[0m" : ""); + igt_info("%sSubtest %s: %s (%.3fs)%s\n", + (!__igt_plain_output) ? "\x1b[1m" : "", + in_subtest, result, time_elapsed(&subtest_time, &now), + (!__igt_plain_output) ? "\x1b[0m" : ""); fflush(stdout); in_subtest = NULL; @@ -1216,7 +1216,7 @@ static void print_backtrace(void) unw_context_t uc; int stack_num = 0; - printf("Stack trace:\n"); + igt_info("Stack trace:\n"); unw_getcontext(&uc); unw_init_local(&cursor, &uc); @@ -1227,8 +1227,8 @@ static void print_backtrace(void) if (unw_get_proc_name(&cursor, name, 255, &off) < 0) strcpy(name, "<unknown>"); - printf(" #%d [%s+0x%x]\n", stack_num++, name, - (unsigned int) off); + igt_info(" #%d [%s+0x%x]\n", stack_num++, name, + (unsigned int) off); } } |