From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752421AbdJCQp2 (ORCPT ); Tue, 3 Oct 2017 12:45:28 -0400 Received: from terminus.zytor.com ([65.50.211.136]:35745 "EHLO terminus.zytor.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751921AbdJCQp0 (ORCPT ); Tue, 3 Oct 2017 12:45:26 -0400 Date: Tue, 3 Oct 2017 09:44:14 -0700 From: tip-bot for Thomas Richter Message-ID: Cc: tglx@linutronix.de, jolsa@kernel.org, tmricht@linux.vnet.ibm.com, acme@redhat.com, brueckner@linux.vnet.ibm.com, hpa@zytor.com, heiko.carstens@de.ibm.com, linux-kernel@vger.kernel.org, schwidefsky@de.ibm.com, mingo@kernel.org Reply-To: mingo@kernel.org, heiko.carstens@de.ibm.com, hpa@zytor.com, linux-kernel@vger.kernel.org, schwidefsky@de.ibm.com, jolsa@kernel.org, tglx@linutronix.de, brueckner@linux.vnet.ibm.com, acme@redhat.com, tmricht@linux.vnet.ibm.com To: linux-tip-commits@vger.kernel.org Subject: [tip:perf/core] perf test attr: Fix ignored test case result Git-Commit-ID: 22905582f6dd4bbd0c370fe5732c607452010c04 X-Mailer: tip-git-log-daemon Robot-ID: Robot-Unsubscribe: Contact to get blacklisted from these emails MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Content-Type: text/plain; charset=UTF-8 Content-Disposition: inline Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Commit-ID: 22905582f6dd4bbd0c370fe5732c607452010c04 Gitweb: https://git.kernel.org/tip/22905582f6dd4bbd0c370fe5732c607452010c04 Author: Thomas Richter AuthorDate: Wed, 13 Sep 2017 10:12:09 +0200 Committer: Arnaldo Carvalho de Melo CommitDate: Mon, 2 Oct 2017 14:00:57 -0300 perf test attr: Fix ignored test case result Command perf test -v 16 (Setup struct perf_event_attr test) always reports success even if the test case fails. It works correctly if you also specify -F (for don't fork). root@s35lp76 perf]# ./perf test -v 16 15: Setup struct perf_event_attr : --- start --- running './tests/attr/test-record-no-delay' [ perf record: Woken up 1 times to write data ] [ perf record: Captured and wrote 0.002 MB /tmp/tmp4E1h7R/perf.data (1 samples) ] expected task=0, got 1 expected precise_ip=0, got 3 expected wakeup_events=1, got 0 FAILED './tests/attr/test-record-no-delay' - match failure test child finished with 0 ---- end ---- Setup struct perf_event_attr: Ok The reason for the wrong error reporting is the return value of the system() library call. It is called in run_dir() file tests/attr.c and returns the exit status, in above case 0xff00. This value is given as parameter to the exit() function which can only handle values 0-0xff. The child process terminates with exit value of 0 and the parent does not detect any error. This patch corrects the error reporting and prints the correct test result. Signed-off-by: Thomas-Mich Richter Acked-by: Jiri Olsa Cc: Heiko Carstens Cc: Hendrik Brueckner Cc: Martin Schwidefsky Cc: Thomas-Mich Richter LPU-Reference: 20170913081209.39570-2-tmricht@linux.vnet.ibm.com Link: http://lkml.kernel.org/n/tip-rdube6rfcjsr1nzue72c7lqn@git.kernel.org Signed-off-by: Arnaldo Carvalho de Melo --- tools/perf/tests/attr.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/perf/tests/attr.c b/tools/perf/tests/attr.c index c9aafed..25ede44 100644 --- a/tools/perf/tests/attr.c +++ b/tools/perf/tests/attr.c @@ -166,7 +166,7 @@ static int run_dir(const char *d, const char *perf) snprintf(cmd, 3*PATH_MAX, PYTHON " %s/attr.py -d %s/attr/ -p %s %.*s", d, d, perf, vcnt, v); - return system(cmd); + return system(cmd) ? TEST_FAIL : TEST_OK; } int test__attr(struct test *test __maybe_unused, int subtest __maybe_unused)