mbox series

[v1,0/5] Fixes for 6.8 PR1

Message ID 20240123000604.1211486-1-irogers@google.com
Headers
Series Fixes for 6.8 PR1 |

Message

Ian Rogers Jan. 23, 2024, 12:05 a.m. UTC 
  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

 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(-)

Comments

Namhyung Kim Jan. 23, 2024, 11:59 p.m. UTC | #1 
Hi Ian,

On Mon, Jan 22, 2024 at 4:06 PM Ian Rogers <irogers@google.com> 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@kernel.org>

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
>