Re: [PATCH v2] docs: perf: Fujitsu: Fix htmldocs build warnings and errors

[Randy Dunlap]

Hi,

On 9/25/25 5:39 PM, Gopi Krishna Menon wrote:
> Running "make htmldocs" generates the following build errors and
> warnings for fujitsu_uncore_pmu.rst:
>
> Documentation/admin-guide/perf/fujitsu_uncore_pmu.rst:20: ERROR: Unexpected indentation.
> Documentation/admin-guide/perf/fujitsu_uncore_pmu.rst:23: WARNING: Block quote ends without a blank line; unexpected unindent.
> Documentation/admin-guide/perf/fujitsu_uncore_pmu.rst:28: ERROR: Unexpected indentation.
> Documentation/admin-guide/perf/fujitsu_uncore_pmu.rst:29: WARNING: Block quote ends without a blank line; unexpected unindent.
> Documentation/admin-guide/perf/fujitsu_uncore_pmu.rst:81: ERROR: Unexpected indentation.
> Documentation/admin-guide/perf/fujitsu_uncore_pmu.rst:82: WARNING: Block quote ends without a blank line; unexpected unindent.
>
> Add blank line before bullet lists and block quotes to fix build
> errors and resolve warnings.

Yes, this is better. No warnings, but some of the output
html file still isn't formatted as expected or desired.
See comments below.

> Signed-off-by: Gopi Krishna Menon <krishnagopi487@xxxxxxxxx>
> ---
> Changelog:
>
> Changes since v1:
>    - Remove formatting changes unrelated to the fix.
>    
> Suggested-by: Shuah Khan <skhan@xxxxxxxxxxxxxxxxxxx>
>
>   Documentation/admin-guide/perf/fujitsu_uncore_pmu.rst | 5 +++++
>   1 file changed, 5 insertions(+)
>
> diff --git a/Documentation/admin-guide/perf/fujitsu_uncore_pmu.rst b/Documentation/admin-guide/perf/fujitsu_uncore_pmu.rst
> index 46595b788d3a..4edf26f8cfe6 100644
> --- a/Documentation/admin-guide/perf/fujitsu_uncore_pmu.rst
> +++ b/Documentation/admin-guide/perf/fujitsu_uncore_pmu.rst
> @@ -15,15 +15,19 @@ The driver provides a description of its available events and configuration
>   options in sysfs, see /sys/bus/event_sources/devices/mac_iod<iod>_mac<mac>_ch<ch>/
>   and /sys/bus/event_sources/devices/pci_iod<iod>_pci<pci>/.
>   This driver exports:
> +
>   - formats, used by perf user space and other tools to configure events
>   - events, used by perf user space and other tools to create events
>     symbolically, e.g.:
> +
>       perf stat -a -e mac_iod0_mac0_ch0/event=0x21/ ls
>       perf stat -a -e pci_iod0_pci0/event=0x24/ ls

This produces:

events, used by perf user space and other tools to create events 
symbolically, e.g.:

perf stat -a -e mac_iod0_mac0_ch0/event=0x21/ ls perf stat -a -e 
pci_iod0_pci0/event=0x24/ ls

(That last line is all on one line in my browser.)

Please try using "e.g.::" before those 2 perf lines.
And then check the output in a browser.

> +
>   - cpumask, used by perf user space and other tools to know on which CPUs
>     to open the events
>   
>   This driver supports the following events for MAC:
> +
>   - cycles
>     This event counts MAC cycles at MAC frequency.
>   - read-count
> @@ -77,6 +81,7 @@ Examples for use with perf::
>     perf stat -e mac_iod0_mac0_ch0/ea-mac/ ls
>   
>   And, this driver supports the following events for PCI:
> +
>   - pci-port0-cycles
>     This event counts PCI cycles at PCI frequency in port0.
>   - pci-port0-read-count

Thanks.
--
~Randy