| commit | 597bdeb4ab7396c43935eded15f82e3e100b3ff3 | [log] [tgz] |
|---|---|---|
| author | Wang Nan <[email protected]> | Tue Nov 03 10:44:42 2015 +0000 |
| committer | Arnaldo Carvalho de Melo <[email protected]> | Tue Nov 03 11:45:40 2015 -0300 |
| tree | 289c61adebac5e14fde781cb31764409a514d6bf | |
| parent | bebd23a2ed31d47e7dd746d3b125068aa2c42d85 [diff] [blame] |
perf test: Keep test result clean if '-v' not set According to [1], 'perf test' should avoid output too much information if '-v' is not set, only 'Ok', 'FAIL' or 'Skip' need to be printed. This patch removes several messages sent directly to stderr to make the output clean. Before this patch: # perf test dummy 23: Test using a dummy software event to keep tracking : (not supported) Ok After this patch: # perf test dummy 23: Test using a dummy software event to keep tracking : Skip [1] http://lkml.kernel.org/r/[email protected] Signed-off-by: Wang Nan <[email protected]> Acked-by: Namhyung Kim <[email protected]> Tested-by: Arnaldo Carvalho de Melo <[email protected]> Cc: Zefan Li <[email protected]> Cc: [email protected] Link: http://lkml.kernel.org/r/[email protected] Signed-off-by: Arnaldo Carvalho de Melo <[email protected]>
diff --git a/tools/perf/tests/keep-tracking.c b/tools/perf/tests/keep-tracking.c index 4d4b983..a2e2269 100644 --- a/tools/perf/tests/keep-tracking.c +++ b/tools/perf/tests/keep-tracking.c
@@ -90,8 +90,8 @@ evsel->attr.enable_on_exec = 0; if (perf_evlist__open(evlist) < 0) { - fprintf(stderr, " (not supported)"); - err = 0; + pr_debug("Unable to open dummy and cycles event\n"); + err = TEST_SKIP; goto out_err; }