[PATCH] drm/amdgpu/gfx10: Enable cleaner shader for GFX10.1.1/10.1.2 GPUs

[Marek Olšák]

Does this commit really enable it though? Or is it just for sysfs?

Marek

On Fri, Jan 24, 2025 at 1:42 AM Srinivasan Shanmugam <
srinivasan.shanmugam at amd.com> wrote:

> Enable the cleaner shader for GFX10.1.1/10.1.2 GPUs to provide data
> isolation between GPU workloads. The cleaner shader is responsible for
> clearing the Local Data Store (LDS), Vector General Purpose Registers
> (VGPRs), and Scalar General Purpose Registers (SGPRs), which helps
> prevent data leakage and ensures accurate computation results.
>
> This update extends cleaner shader support to GFX10.1.1/10.1.2 GPUs,
> previously available for GFX10.1.10. It enhances security by clearing
> GPU memory between processes and maintains a consistent GPU state across
> KGD and KFD workloads.
>
> Cc: Christian König <christian.koenig at amd.com>
> Cc: Alex Deucher <alexander.deucher at amd.com>
> Signed-off-by: Srinivasan Shanmugam <srinivasan.shanmugam at amd.com>
> ---
>  drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c
> b/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c
> index 1878c83ff7e3..938f7d60a0ee 100644
> --- a/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c
> +++ b/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c
> @@ -4795,6 +4795,8 @@ static int gfx_v10_0_sw_init(struct amdgpu_ip_block
> *ip_block)
>         }
>         switch (amdgpu_ip_version(adev, GC_HWIP, 0)) {
>         case IP_VERSION(10, 1, 10):
> +       case IP_VERSION(10, 1, 1):
> +       case IP_VERSION(10, 1, 2):
>                 adev->gfx.cleaner_shader_ptr =
> gfx_10_1_10_cleaner_shader_hex;
>                 adev->gfx.cleaner_shader_size =
> sizeof(gfx_10_1_10_cleaner_shader_hex);
>                 if (adev->gfx.me_fw_version >= 101 &&
> --
> 2.34.1
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/amd-gfx/attachments/20250124/4d62dd67/attachment.htm>