Re: [PATCH -v1.1] x86/topology: Document cpu_llc_id

[Yazen Ghannam <yazen.ghannam@amd.com>]

On Thu, Nov 17, 2016 at 10:45:57AM +0100, Borislav Petkov wrote:
> It means different things on Intel and AMD so write it down so that
> there's no confusion.
> 
> Signed-off-by: Borislav Petkov <bp@suse.de>
> Cc: Peter Zijlstra <peterz@infradead.org>
> Cc: Thomas Gleixner <tglx@linutronix.de>
> Cc: Yazen Ghannam <yazen.ghannam@amd.com>
> ---
>  Documentation/x86/topology.txt | 9 +++++++++
>  1 file changed, 9 insertions(+)
> 
> diff --git a/Documentation/x86/topology.txt b/Documentation/x86/topology.txt
> index 06afac252f5b..f3e9d7e9ed6c 100644
> --- a/Documentation/x86/topology.txt
> +++ b/Documentation/x86/topology.txt
> @@ -63,6 +63,15 @@ The topology of a system is described in the units of:
>      The maximum possible number of packages in the system. Helpful for per
>      package facilities to preallocate per package information.
>  
> +  - cpu_llc_id:
> +
> +    A per-CPU variable containing:
> +    - On Intel, the first APIC ID of the list of CPUs sharing the Last Level
> +    Cache
> +
> +    - On AMD, the Node ID or Core Complex ID containing the Last Level
> +    Cache. In general, it is a number identifying an LLC uniquely on the
> +    system.
>  
>  * Cores:
> 

Look good to me.

Thanks,
Yazen