Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753993AbbKCLTy (ORCPT ); Tue, 3 Nov 2015 06:19:54 -0500 Received: from mail-pa0-f54.google.com ([209.85.220.54]:33262 "EHLO mail-pa0-f54.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752380AbbKCLTs (ORCPT ); Tue, 3 Nov 2015 06:19:48 -0500 Date: Tue, 3 Nov 2015 20:19:36 +0900 From: Namhyung Kim To: Wang Nan Cc: acme@kernel.org, lizefan@huawei.com, pi3orama@163.com, linux-kernel@vger.kernel.org, Arnaldo Carvalho de Melo Subject: Re: [PATCH 1/5] perf test: Keep test result clean if '-v' not set Message-ID: <20151103111936.GC5733@danjae.kornet> References: <1446547486-229499-1-git-send-email-wangnan0@huawei.com> <1446547486-229499-2-git-send-email-wangnan0@huawei.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <1446547486-229499-2-git-send-email-wangnan0@huawei.com> User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 4931 Lines: 152 On Tue, Nov 03, 2015 at 10:44:42AM +0000, Wang Nan wrote: > According to [1], 'perf state' should avoid output too much information > if '-v' not set, only 'Ok', 'FAIL' or 'Skip' need to be printed. > > This patch removes sereval stderr output to make 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/20151020134155.GE4400@redhat.com > > Signed-off-by: Wang Nan > Cc: Arnaldo Carvalho de Melo > Link: http://lkml.kernel.org/n/tip-446450727-218554-1-git-send-email-wangnan0@huawei.com Acked-by: Namhyung Kim Thanks, Namhyung > --- > tools/perf/tests/attr.c | 3 +-- > tools/perf/tests/code-reading.c | 8 ++++---- > tools/perf/tests/keep-tracking.c | 4 ++-- > tools/perf/tests/llvm.c | 11 ++++------- > tools/perf/tests/switch-tracking.c | 4 ++-- > 5 files changed, 13 insertions(+), 17 deletions(-) > > diff --git a/tools/perf/tests/attr.c b/tools/perf/tests/attr.c > index 2dfc9ad..638875a 100644 > --- a/tools/perf/tests/attr.c > +++ b/tools/perf/tests/attr.c > @@ -171,6 +171,5 @@ int test__attr(void) > !lstat(path_perf, &st)) > return run_dir(path_dir, path_perf); > > - fprintf(stderr, " (omitted)"); > - return 0; > + return TEST_SKIP; > } > diff --git a/tools/perf/tests/code-reading.c b/tools/perf/tests/code-reading.c > index 49b1959..a767a64 100644 > --- a/tools/perf/tests/code-reading.c > +++ b/tools/perf/tests/code-reading.c > @@ -613,16 +613,16 @@ int test__code_reading(void) > case TEST_CODE_READING_OK: > return 0; > case TEST_CODE_READING_NO_VMLINUX: > - fprintf(stderr, " (no vmlinux)"); > + pr_debug("no vmlinux\n"); > return 0; > case TEST_CODE_READING_NO_KCORE: > - fprintf(stderr, " (no kcore)"); > + pr_debug("no kcore\n"); > return 0; > case TEST_CODE_READING_NO_ACCESS: > - fprintf(stderr, " (no access)"); > + pr_debug("no access\n"); > return 0; > case TEST_CODE_READING_NO_KERNEL_OBJ: > - fprintf(stderr, " (no kernel obj)"); > + pr_debug("no kernel obj\n"); > return 0; > default: > return -1; > 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 @@ int test__keep_tracking(void) > 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; > } > > diff --git a/tools/perf/tests/llvm.c b/tools/perf/tests/llvm.c > index 52d5597..512d362 100644 > --- a/tools/perf/tests/llvm.c > +++ b/tools/perf/tests/llvm.c > @@ -36,7 +36,7 @@ static int test__bpf_parsing(void *obj_buf, size_t obj_buf_sz) > static int test__bpf_parsing(void *obj_buf __maybe_unused, > size_t obj_buf_sz __maybe_unused) > { > - fprintf(stderr, " (skip bpf parsing)"); > + pr_debug("Skip bpf parsing\n"); > return 0; > } > #endif > @@ -55,7 +55,7 @@ int test__llvm(void) > * and clang is not found in $PATH, and this is not perf test -v > */ > if (verbose == 0 && !llvm_param.user_set_param && llvm__search_clang()) { > - fprintf(stderr, " (no clang, try 'perf test -v LLVM')"); > + pr_debug("No clang and no verbosive, skip this test\n"); > return TEST_SKIP; > } > > @@ -86,11 +86,8 @@ int test__llvm(void) > err = llvm__compile_bpf("-", &obj_buf, &obj_buf_sz); > > verbose = old_verbose; > - if (err) { > - if (!verbose) > - fprintf(stderr, " (use -v to see error message)"); > - return -1; > - } > + if (err) > + return TEST_FAIL; > > err = test__bpf_parsing(obj_buf, obj_buf_sz); > free(obj_buf); > diff --git a/tools/perf/tests/switch-tracking.c b/tools/perf/tests/switch-tracking.c > index e698742..a02af50 100644 > --- a/tools/perf/tests/switch-tracking.c > +++ b/tools/perf/tests/switch-tracking.c > @@ -366,7 +366,7 @@ int test__switch_tracking(void) > > /* Third event */ > if (!perf_evlist__can_select_event(evlist, sched_switch)) { > - fprintf(stderr, " (no sched_switch)"); > + pr_debug("No sched_switch\n"); > err = 0; > goto out; > } > @@ -442,7 +442,7 @@ int test__switch_tracking(void) > } > > if (perf_evlist__open(evlist) < 0) { > - fprintf(stderr, " (not supported)"); > + pr_debug("Not supported\n"); > err = 0; > goto out; > } > -- > 1.8.3.4 > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/