[PATCH] drm/amdkfd: remove gfx 12 trap handler page size cap

[Belanger, David]

[Public]

Instead of removing the check entirely, I think it should be changed to:
BUILD_BUG_ON(sizeof(cwsr_trap_gfx12_hex) > KFD_CWSR_TMA_OFFSET);
Like the pre-GFX11 cases.   A cwsr trap handler too large will overwrite the TMA area.

Regards,
David B.

> -----Original Message-----
> From: Kim, Jonathan <Jonathan.Kim at amd.com>
> Sent: Tuesday, November 5, 2024 12:44 PM
> To: amd-gfx at lists.freedesktop.org
> Cc: Belanger, David <David.Belanger at amd.com>; Kim, Jonathan
> <Jonathan.Kim at amd.com>
> Subject: [PATCH] drm/amdkfd: remove gfx 12 trap handler page size cap
>
> GFX 12 does not require a page size cap for the trap handler because it does not
> require a CWSR work around like GFX 11 did.
>
> Signed-off-by: Jonathan Kim <jonathan.kim at amd.com>
> ---
>  drivers/gpu/drm/amd/amdkfd/kfd_device.c | 1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_device.c
> b/drivers/gpu/drm/amd/amdkfd/kfd_device.c
> index 348925254bff..4705ebb07ba0 100644
> --- a/drivers/gpu/drm/amd/amdkfd/kfd_device.c
> +++ b/drivers/gpu/drm/amd/amdkfd/kfd_device.c
> @@ -534,7 +534,6 @@ static void kfd_cwsr_init(struct kfd_dev *kfd)
>                       kfd->cwsr_isa = cwsr_trap_gfx11_hex;
>                       kfd->cwsr_isa_size = sizeof(cwsr_trap_gfx11_hex);
>               } else {
> -                     BUILD_BUG_ON(sizeof(cwsr_trap_gfx12_hex) >
> PAGE_SIZE);
>                       kfd->cwsr_isa = cwsr_trap_gfx12_hex;
>                       kfd->cwsr_isa_size = sizeof(cwsr_trap_gfx12_hex);
>               }
> --
> 2.34.1