[PATCH] drm/amdgpu/gfx12: Add Cleaner Shader Support for GFX12.0 GPUs

[Alex Deucher]

On Mon, Jan 13, 2025 at 12:32 PM Srinivasan Shanmugam
<srinivasan.shanmugam at amd.com> wrote:
>
> This commit enables the cleaner shader feature for GFX12.0 and GFX12.0.1
> GPUs. The cleaner shader is important for clearing GPU resources such as
> Local Data Share (LDS), Vector General Purpose Registers (VGPRs), and
> Scalar General Purpose Registers (SGPRs) between workloads.
>
> - This feature ensures that GPU resources are reset between workloads,
>   preventing data leaks and ensuring accurate computation.
>
> By enabling the cleaner shader, this update enhances the security and
> reliability of GPU operations on GFX12.0 hardware.
>
> 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>

Reviewed-by: Alex Deucher <alexander.deucher at amd.com>

> ---
>  drivers/gpu/drm/amd/amdgpu/gfx_v12_0.c | 8 ++++++++
>  1 file changed, 8 insertions(+)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v12_0.c b/drivers/gpu/drm/amd/amdgpu/gfx_v12_0.c
> index 69941442f00b..fc75af08a7fc 100644
> --- a/drivers/gpu/drm/amd/amdgpu/gfx_v12_0.c
> +++ b/drivers/gpu/drm/amd/amdgpu/gfx_v12_0.c
> @@ -1387,6 +1387,14 @@ static int gfx_v12_0_sw_init(struct amdgpu_ip_block *ip_block)
>         }
>
>         switch (amdgpu_ip_version(adev, GC_HWIP, 0)) {
> +       case IP_VERSION(12, 0, 0):
> +       case IP_VERSION(12, 0, 1):
> +               if (adev->gfx.me_fw_version  >= 2480 &&
> +                   adev->gfx.pfp_fw_version >= 2530 &&
> +                   adev->gfx.mec_fw_version >= 2680 &&
> +                   adev->mes.fw_version[0] >= 100)
> +                       adev->gfx.enable_cleaner_shader = true;
> +               break;
>         default:
>                 adev->gfx.enable_cleaner_shader = false;
>                 break;
> --
> 2.34.1
>