[PATCH 0/3 v3] Expose the 'perf test -w' workloads functionality

Arnaldo Carvalho de Melo posted 3 patches 1 month, 1 week ago
tools/perf/Documentation/perf-test.txt | 17 ++++++++++++++++
tools/perf/tests/builtin-test.c        | 27 ++++++++++++++++++++++----
2 files changed, 40 insertions(+), 4 deletions(-)
[PATCH 0/3 v3] Expose the 'perf test -w' workloads functionality
Posted by Arnaldo Carvalho de Melo 1 month, 1 week ago
From: Arnaldo Carvalho de Melo <acme@redhat.com>

Hi,

        I noticed that the 'perf test -w' code wasn't well exposed, so
add a '--list-workloads' option to list the existing workloads and document it
in the 'perf test' man page.

- Arnaldo

v3:

- Fixup references to --list-workloads in a patch subject (Namhyumg)

- Fixup references to --list-workloads in the 'perf test' man page (James)

- Add an entry for --list-workloads in the 'perf test' man page.

v2:

- Use --list-workloads, as suggested by James Clark.

- Add info in the man page about the parameters the built-in workloads
  accept.

Arnaldo Carvalho de Melo (3):
  perf test: Introduce workloads__for_each()
  perf test: Introduce --list-workloads to list the available workloads
  perf test: Document the -w/--workload option

 tools/perf/Documentation/perf-test.txt | 17 ++++++++++++++++
 tools/perf/tests/builtin-test.c        | 27 ++++++++++++++++++++++----
 2 files changed, 40 insertions(+), 4 deletions(-)

-- 
2.46.2
Re: [PATCH 0/3 v3] Expose the 'perf test -w' workloads functionality
Posted by Namhyung Kim 1 month ago
On Sat, 19 Oct 2024 23:18:39 -0300, Arnaldo Carvalho de Melo wrote:

> From: Arnaldo Carvalho de Melo <acme@redhat.com>
> 
> Hi,
> 
>         I noticed that the 'perf test -w' code wasn't well exposed, so
> add a '--list-workloads' option to list the existing workloads and document it
> in the 'perf test' man page.
> 
> [...]

Applied to perf-tools-next, thanks!

Best regards,
Namhyung
Re: [PATCH 0/3 v3] Expose the 'perf test -w' workloads functionality
Posted by James Clark 1 month ago

On 20/10/2024 3:18 am, Arnaldo Carvalho de Melo wrote:
> From: Arnaldo Carvalho de Melo <acme@redhat.com>
> 
> Hi,
> 
>          I noticed that the 'perf test -w' code wasn't well exposed, so
> add a '--list-workloads' option to list the existing workloads and document it
> in the 'perf test' man page.
> 
> - Arnaldo
> 
> v3:
> 
> - Fixup references to --list-workloads in a patch subject (Namhyumg)
> 
> - Fixup references to --list-workloads in the 'perf test' man page (James)
> 
> - Add an entry for --list-workloads in the 'perf test' man page.
> 
> v2:
> 
> - Use --list-workloads, as suggested by James Clark.
> 
> - Add info in the man page about the parameters the built-in workloads
>    accept.
> 
> Arnaldo Carvalho de Melo (3):
>    perf test: Introduce workloads__for_each()
>    perf test: Introduce --list-workloads to list the available workloads
>    perf test: Document the -w/--workload option
> 
>   tools/perf/Documentation/perf-test.txt | 17 ++++++++++++++++
>   tools/perf/tests/builtin-test.c        | 27 ++++++++++++++++++++++----
>   2 files changed, 40 insertions(+), 4 deletions(-)
> 

Reviewed-by: James Clark <james.clark@linaro.org>