lkml.org 
[lkml]   [2017]   [Jun]   [2]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCHv2] perf test: Disable breakpoint signal tests for powerpc
Em Thu, Jun 01, 2017 at 10:54:50PM +0200, Jiri Olsa escreveu:
>
> + if (t->is_supported && !t->is_supported()) {
> + pr_debug("%2d: %-*s: Disabled\n", i, width, t->desc);

This isn't clear, I'll change this to leave the pr_debug up to the
->is_supported() code (that will receive the struct test pointer too),
that can then say something more clear like what you put in that
comment.

This way, ppc hackers running 'perf test' will see that longstanding
TODO entry ;-)

- Arnaldo

> + continue;
> + }
> +
> pr_info("%2d: %-*s:", i, width, t->desc);
>
> if (intlist__find(skiplist, i)) {
> diff --git a/tools/perf/tests/tests.h b/tools/perf/tests/tests.h
> index 631859629403..577363809c9b 100644
> --- a/tools/perf/tests/tests.h
> +++ b/tools/perf/tests/tests.h
> @@ -34,6 +34,7 @@ struct test {
> int (*get_nr)(void);
> const char *(*get_desc)(int subtest);
> } subtest;
> + bool (*is_supported)(void);
> };
>
> /* Tests */
> @@ -99,6 +100,8 @@ const char *test__clang_subtest_get_desc(int subtest);
> int test__clang_subtest_get_nr(void);
> int test__unit_number__scnprint(int subtest);
>
> +bool test__bp_signal_is_supported(void);
> +
> #if defined(__arm__) || defined(__aarch64__)
> #ifdef HAVE_DWARF_UNWIND_SUPPORT
> struct thread;
> --
> 2.9.4

\
 
 \ /
  Last update: 2017-06-05 00:13    [W:0.393 / U:0.160 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site