Re: [PATCH v4 01/18] Documentation, ABI: Add a document entry for cache id

2016-10-17 Thread Thomas Gleixner
On Fri, 14 Oct 2016, Fenghua Yu wrote:

> From: Fenghua Yu 
> 
> Add an ABI document entry for /sys/devices/system/cpu/cpu*/cache/index*/id.
> 
> Signed-off-by: Fenghua Yu 
> Signed-off-by: Tony Luck 

The SOB chain here is bogus 

> ---
>  Documentation/ABI/testing/sysfs-devices-system-cpu | 16 
>  1 file changed, 16 insertions(+)
> 
> diff --git a/Documentation/ABI/testing/sysfs-devices-system-cpu 
> b/Documentation/ABI/testing/sysfs-devices-system-cpu
> index 4987417..b1c3d69 100644
> --- a/Documentation/ABI/testing/sysfs-devices-system-cpu
> +++ b/Documentation/ABI/testing/sysfs-devices-system-cpu
> @@ -272,6 +272,22 @@ Description: Parameters for the CPU cache attributes
>the modified cache line is written to main
>memory only when it is replaced
>  
> +
> +What:/sys/devices/system/cpu/cpu*/cache/index*/id
> +Date:September 2016
> +Contact: Linux kernel mailing list 
> +Description: Cache id
> +
> + The id provides a unique name for a specific instance of

s/name/number/

> + a cache of a particular type. E.g. there may be a level
> + 3 unified cache on each socket in a server and we may
> + assign them ids 0, 1, 2, ...
> +
> + Note that id value may not be contiguous. E.g. level 1

Note, that the id values can be non-contiguous.

> + caches typically exist per core, but there may not be a
> + power of two cores on a socket, so these caches may be
> + numbered 0, 1, 2, 3, 4, 5, 8, 9, 10, ...
> +

Thanks,

tglx


Re: [PATCH v4 01/18] Documentation, ABI: Add a document entry for cache id

2016-10-17 Thread Thomas Gleixner
On Fri, 14 Oct 2016, Fenghua Yu wrote:

> From: Fenghua Yu 
> 
> Add an ABI document entry for /sys/devices/system/cpu/cpu*/cache/index*/id.
> 
> Signed-off-by: Fenghua Yu 
> Signed-off-by: Tony Luck 

The SOB chain here is bogus 

> ---
>  Documentation/ABI/testing/sysfs-devices-system-cpu | 16 
>  1 file changed, 16 insertions(+)
> 
> diff --git a/Documentation/ABI/testing/sysfs-devices-system-cpu 
> b/Documentation/ABI/testing/sysfs-devices-system-cpu
> index 4987417..b1c3d69 100644
> --- a/Documentation/ABI/testing/sysfs-devices-system-cpu
> +++ b/Documentation/ABI/testing/sysfs-devices-system-cpu
> @@ -272,6 +272,22 @@ Description: Parameters for the CPU cache attributes
>the modified cache line is written to main
>memory only when it is replaced
>  
> +
> +What:/sys/devices/system/cpu/cpu*/cache/index*/id
> +Date:September 2016
> +Contact: Linux kernel mailing list 
> +Description: Cache id
> +
> + The id provides a unique name for a specific instance of

s/name/number/

> + a cache of a particular type. E.g. there may be a level
> + 3 unified cache on each socket in a server and we may
> + assign them ids 0, 1, 2, ...
> +
> + Note that id value may not be contiguous. E.g. level 1

Note, that the id values can be non-contiguous.

> + caches typically exist per core, but there may not be a
> + power of two cores on a socket, so these caches may be
> + numbered 0, 1, 2, 3, 4, 5, 8, 9, 10, ...
> +

Thanks,

tglx