Re: [PATCH v2] arm64: perf: Drop the example the userspace counter section points at

[sashiko-bot@kernel.org]

Thank you for your contribution! Sashiko AI review found 1 potential
issue to consider:
- [Low] docs: arm64: perf: commit message includes v1 changelog and
  repetitive text
--

commit 4f292c888b9c6465b91000fd55cd46c1bc198a65
Author: TAKANO Mitsuhiro <takano32@gmail.com>

> arm64: perf: Drop the example the userspace counter section points at
>
> Documentation/arch/arm64/perf.rst points at
> tools/perf/arch/arm64/tests/user-events.c, in the text and in the link
> target, but the file does not exist: no file of that name is in
> tools/perf/arch/arm64/tests/, and no commit in the history leading to this
> base has added one.
>
> The section already describes how to use the libperf helpers and points at
> tools/lib/perf/tests/test-evsel.c as the example, so the sentence that
> sends the reader elsewhere is redundant. Remove it and the link target
> rather than pointing at another test, and move the instruction for running
> the example into the How-to section, next to the reference it belongs to.
>
> v1 pointed the reference at tools/perf/tests/mmap-basic.c. The review said
> that file is a confusing example and that the section above already covers
> the how-to, which is what this version does.
>
> Signed-off-by: TAKANO Mitsuhiro <takano32@gmail.com>

[Severity: Low]
This isn't a bug, but the v1 review history in the third paragraph belongs
below the --- separator line rather than in the permanent commit log, per
Documentation/process/submitting-patches.rst.

Also, the first paragraph repeats that
tools/perf/arch/arm64/tests/user-events.c is missing in three different
ways ("the file does not exist", "no file of that name is in ...", and
"no commit in the history ... has added one"), which could be trimmed for
conciseness.

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20261111023550.1971639-1-takano32@gmail.com?part=1