[AMD Official Use Only - AMD Internal Distribution Only]

Yes , I can confirm that  from 0x52 , the  mes have the set_hw_resource_1 API 
support on gfx_11.  Actually , I don't see a official release on 0x51 from mes 
side , maybe  it's an internal testing  version.

Regards
Shaoyun.liu

-----Original Message-----
From: Alex Deucher <[email protected]>
Sent: Thursday, January 29, 2026 3:13 PM
To: Limonciello, Mario <[email protected]>; Liu, Shaoyun 
<[email protected]>
Cc: [email protected]; [email protected]
Subject: Re: [PATCH] drm/amd: Set minimum version for set_hw_resource_1 on 
gfx11 to 0x52

On Thu, Jan 29, 2026 at 2:57 PM Mario Limonciello <[email protected]> 
wrote:
>
> commit f81cd793119e7 ("drm/amd/amdgpu: Fix MES init sequence") caused
> a dependency on new enough MES firmware to use amdgpu.  This was fixed
> on most gfx11 and gfx12 hardware with commit 0180e0a5dd5c6
> ("drm/amdgpu/mes: add compatibility checks for set_hw_resource_1"),
> but this left out that GC 11.0.4 had breakage at MES 0x51.
>
> Bump the requirement to 0x52 instead.

Do we know that 0x52 actually has it?  @Shaoyun Liu can you verify?
Assuming that is correct,
Reviewed-by: Alex Deucher <[email protected]>

>
> Reported-by: [email protected]
> Closes: https://gitlab.freedesktop.org/drm/amd/-/issues/4576
> Fixes: f81cd793119e7 ("drm/amd/amdgpu: Fix MES init sequence")
> Signed-off-by: Mario Limonciello <[email protected]>
> ---
>  drivers/gpu/drm/amd/amdgpu/mes_v11_0.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/mes_v11_0.c
> b/drivers/gpu/drm/amd/amdgpu/mes_v11_0.c
> index 81bf9385d55a0..09ebb13ca5e81 100644
> --- a/drivers/gpu/drm/amd/amdgpu/mes_v11_0.c
> +++ b/drivers/gpu/drm/amd/amdgpu/mes_v11_0.c
> @@ -1673,7 +1673,7 @@ static int mes_v11_0_hw_init(struct amdgpu_ip_block 
> *ip_block)
>         if (r)
>                 goto failure;
>
> -       if ((adev->mes.sched_version & AMDGPU_MES_VERSION_MASK) >= 0x50) {
> +       if ((adev->mes.sched_version & AMDGPU_MES_VERSION_MASK) >=
> + 0x52) {
>                 r = mes_v11_0_set_hw_resources_1(&adev->mes);
>                 if (r) {
>                         DRM_ERROR("failed
> mes_v11_0_set_hw_resources_1, r=%d\n", r);
> --
> 2.52.0
>

Reply via email to