Re: [PATCH v1 0/5] Fixes for 6.8 PR1

From: Namhyung Kim
Date: Tue Jan 23 2024 - 19:00:21 EST


Hi Ian,

On Mon, Jan 22, 2024 at 4:06 PM Ian Rogers <irogers@xxxxxxxxxx> wrote:
>
> Discovered some testing issues around perf list, perf script and perf
> daemon based on Linux 6.8-rc1. Some of the issues were discovered in
> the context of an Alderlake system.
>
> Ian Rogers (5):
> perf list: Switch error message to pr_err
> perf list: Add output file option
> perf test: Workaround debug output in list test
> perf test: Fix script test for python being disabled
> perf test: Make daemon signal test less racy

Acked-by: Namhyung Kim <namhyung@xxxxxxxxxx>

I only have a nitpick on patch 5.

Thanks,
Namhyung

>
> tools/perf/Documentation/perf-list.txt | 4 +
> tools/perf/builtin-list.c | 211 +++++++++++++++----------
> tools/perf/tests/shell/daemon.sh | 34 ++--
> tools/perf/tests/shell/list.sh | 21 ++-
> tools/perf/tests/shell/script.sh | 3 +-
> tools/perf/util/print-events.c | 2 +-
> 6 files changed, 177 insertions(+), 98 deletions(-)
>
> --
> 2.43.0.429.g432eaa2c6b-goog
>