Re: [RFC PATCH v2 5/5] docs: ABI: Add sysfs documentation interface of hardware prefetch driver

From: Dave Hansen
Date: Thu Nov 04 2021 - 10:55:16 EST


On 11/3/21 10:21 PM, Kohei Tarumizu wrote:
> +What: /sys/devices/system/cpu/cpu*/hwpf/l*/enable
> + /sys/devices/system/cpu/cpu*/hwpf/l*/available_dist
> + /sys/devices/system/cpu/cpu*/hwpf/l*/dist
> + /sys/devices/system/cpu/cpu*/hwpf/l*/reliable

How does this look in practice?

# ls /sys/devices/system/cpu/cpu0/hwpf/
l0
l1
l2
...

?

Dumb question, but why don't we give these things names? If the Intel
one is called "L2 Hardware Prefetcher Disable", couldn't the directory
be "l2_prefetch"?

BTW, your "reliable" is mismatched with the "strong" value in the docs.