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

James Clark posted 3 patches 11 months, 2 weeks ago
There is a newer version of this series
[PATCH 3/3] perf list: Document -v option deduplication feature
Posted by James Clark 11 months, 2 weeks ago
-v disables deduplication of similarly suffixed PMUs so add it to the
help and doc strings.

Signed-off-by: James Clark <james.clark@linaro.org>
---
 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
Re: [PATCH 3/3] perf list: Document -v option deduplication feature
Posted by Ian Rogers 11 months, 2 weeks ago
On Tue, Feb 25, 2025 at 8:47 AM James Clark <james.clark@linaro.org> wrote:
>
> -v disables deduplication of similarly suffixed PMUs so add it to the
> help and doc strings.
>
> Signed-off-by: James Clark <james.clark@linaro.org>

Reviewed-by: Ian Rogers <irogers@google.com>

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
>