Re: [PATCH 6/7] Documentation/gpu: Add TCC, update TCP in amdgpu glossary

["Christian König" <christian.koenig@amd.com>]

On 4/16/26 22:26, Timur Kristóf wrote:
> These are the L2 and L1 cache on some AMD GPU architectures.
> 
> Signed-off-by: Timur Kristóf <timur.kristof@gmail.com>
> ---
>  Documentation/gpu/amdgpu/amdgpu-glossary.rst | 9 ++++++++-
>  1 file changed, 8 insertions(+), 1 deletion(-)
> 
> diff --git a/Documentation/gpu/amdgpu/amdgpu-glossary.rst b/Documentation/gpu/amdgpu/amdgpu-glossary.rst
> index 033167025fcca..d553dd599c966 100644
> --- a/Documentation/gpu/amdgpu/amdgpu-glossary.rst
> +++ b/Documentation/gpu/amdgpu/amdgpu-glossary.rst
> @@ -233,8 +233,15 @@ we have a dedicated glossary for Display Core at
>      TC
>        Texture Cache
>  
> +    TCC
> +      Texture Cache per Channel - L2 cache attached to the memory channels.
> +      May be used when shader cores are accessing memory.
> +      Despite "Texture" in the name, this is used by any kind of memory access.
> +      TCCs may be mapped to TCPs, depending on the architecture.
> +

Good to have, but maybe put that below TCP. E.g. L1 first and then L2.

Apart from that nit Reviewed-by: Christian König <christian.koenig@amd.com>

Regards,
Christian.

>      TCP (AMDGPU)
> -      Texture Cache per Pipe. Even though the name "Texture" is part of this
> +      Texture Cache per Pipe - L1 cache attached to each CU.
> +      Even though the name "Texture" is part of this
>        acronym, the TCP represents the path to memory shaders; i.e., it is not
>        related to texture. The name is a leftover from older designs where shader
>        stages had different cache designs; it refers to the L1 cache in older