• Chengdong Li's avatar
    perf test tsc: Fix error message when not supported · 290fa68b
    Chengdong Li authored
    By default `perf test tsc` does not return the error message when the
    child process detected kernel does not support it. Instead, the child
    process prints an error message to stderr, unfortunately stderr is
    redirected to /dev/null when verbose <= 0.
    
    This patch does:
    
    - return TEST_SKIP to the parent process instead of TEST_OK when
      perf_read_tsc_conversion() is not supported.
    
    - Add a new subtest of testing if TSC is supported on current
      architecture by moving exist code to a separate function.
      It avoids two places in test__perf_time_to_tsc() that return
      TEST_SKIP by doing this.
    
    - Extend the test suite definition to contain above two subtests.
      Current test_suite and test_case structs do not support printing skip
      reason when the number of subtest less than 1. To print skip reason, it
      is necessary to extend current test suite definition.
    Reviewed-by: default avatarAdrian Hunter <adrian.hunter@intel.com>
    Signed-off-by: default avatarChengdong Li <chengdongli@tencent.com>
    Cc: Alexander Shishkin <alexander.shishkin@linux.intel.com>
    Cc: Andi Kleen <ak@linux.intel.com>
    Cc: Ingo Molnar <mingo@redhat.com>
    Cc: Jiri Olsa <jolsa@kernel.org>
    Cc: Mark Rutland <mark.rutland@arm.com>
    Cc: Namhyung Kim <namhyung@kernel.org>
    Cc: Peter Zijlstra <peterz@infradead.org>
    Cc: likexu@tencent.com
    Link: https://lore.kernel.org/r/20220408084748.43707-1-chengdongli@tencent.comSigned-off-by: default avatarArnaldo Carvalho de Melo <acme@redhat.com>
    290fa68b
perf-time-to-tsc.c 5.4 KB