Commit 189c466f authored May 11, 2015 by He Kuang Committed by Arnaldo Carvalho de Melo May 12, 2015

perf tests: Fix to get negative exit codes



WEXITSTATUS consists of the least significant 8 bits of the status
argument, so we should convert the value to signed char if we have valid
negative exit codes. And the return value of test->func() contains
negative values:

  enum {
          TEST_OK   =  0,
          TEST_FAIL = -1,
          TEST_SKIP = -2,
  };

Before this patch:

  $ perf test -v 1
  ...
  test child finished with 254
  ---- end ----
  vmlinux symtab matches kallsyms: FAILED!

After this patch:

  $ perf test -v 1
  ...
  test child finished with -2
  ---- end ----
  vmlinux symtab matches kallsyms: Skip

Signed-off-by: He Kuang <hekuang@huawei.com>
Acked-by: Jiri Olsa <jolsa@kernel.org>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: Wang Nan <wangnan0@huawei.com>
Link: http://lkml.kernel.org/r/1431347316-30401-1-git-send-email-hekuang@huawei.com


Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>

parent 7d5eaba9

tools/perf/tests/builtin-test.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -219,7 +219,7 @@ static int run_test(struct test *test)
		wait(&status);

		if (WIFEXITED(status)) {
		err = WEXITSTATUS(status);
		err = (signed char)WEXITSTATUS(status);
		pr_debug("test child finished with %d\n", err);
		} else if (WIFSIGNALED(status)) {
		err = -1;