Re: [PATCH 3/3] perf list: Document -v option deduplication feature

[Ian Rogers]

On Tue, Feb 25, 2025 at 8:47 AM James Clark <james.clark@xxxxxxxxxx> wrote:
>
> -v disables deduplication of similarly suffixed PMUs so add it to the
> help and doc strings.
>
> Signed-off-by: James Clark <james.clark@xxxxxxxxxx>

Reviewed-by: Ian Rogers <irogers@xxxxxxxxxx>

Thanks,
Ian


> ---
>  tools/perf/Documentation/perf-list.txt | 2 +-
>  tools/perf/builtin-list.c              | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/tools/perf/Documentation/perf-list.txt b/tools/perf/Documentation/perf-list.txt
> index c3ffd93f94d7..8914f12d2b85 100644
> --- a/tools/perf/Documentation/perf-list.txt
> +++ b/tools/perf/Documentation/perf-list.txt
> @@ -27,7 +27,7 @@ Don't print descriptions.
>
>  -v::
>  --long-desc::
> -Print longer event descriptions.
> +Print longer event descriptions and all similar PMUs with alphanumeric suffixes.
>
>  --debug::
>  Enable debugging output.
> diff --git a/tools/perf/builtin-list.c b/tools/perf/builtin-list.c
> index c19826f218a0..fed482adb039 100644
> --- a/tools/perf/builtin-list.c
> +++ b/tools/perf/builtin-list.c
> @@ -527,7 +527,7 @@ int cmd_list(int argc, const char **argv)
>                 OPT_BOOLEAN('d', "desc", &default_ps.desc,
>                             "Print extra event descriptions. --no-desc to not print."),
>                 OPT_BOOLEAN('v', "long-desc", &default_ps.long_desc,
> -                           "Print longer event descriptions."),
> +                           "Print longer event descriptions and all similar PMUs with alphanumeric suffixes."),
>                 OPT_BOOLEAN(0, "details", &default_ps.detailed,
>                             "Print information on the perf event names and expressions used internally by events."),
>                 OPT_STRING('o', "output", &output_path, "file", "output file name"),
> --
> 2.34.1
>