On Tue, Aug 26, 2025 at 5:41 AM Jesse.Zhang <jesse.zh...@amd.com> wrote:
>
> The minimum firmware versions required for user queue functionality
> have been increased to address an issue where the queue privilege
> state was lost during queue connect operations.
>
> The problem occurred because the privilege state was being restored
> to its initial value at the beginning of the function, overwriting
> the state that was properly set during the queue connect case.
>
> This commit updates the minimum version requirements:
> - ME firmware from 2390 to 2420
> - PFP firmware from 2530 to 2580
> - MEC firmware from 2600 to 2650
> - MES firmware remains at 120
>
> These updated firmware versions contain the necessary fixes to
> properly maintain queue privilege state throughout connect operations.
>
> Fixes: d84f90e2bcec ("drm/amdgpu: Add fw minimum version check for usermode
> queue")
>
> Signed-off-by: Jesse Zhang <jesse.zh...@amd.com>
Acked-by: Alex Deucher <alexander.deuc...@amd.com>
> ---
> drivers/gpu/drm/amd/amdgpu/gfx_v11_0.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v11_0.c
> b/drivers/gpu/drm/amd/amdgpu/gfx_v11_0.c
> index 456ba758fa94..c85de8c8f6f5 100644
> --- a/drivers/gpu/drm/amd/amdgpu/gfx_v11_0.c
> +++ b/drivers/gpu/drm/amd/amdgpu/gfx_v11_0.c
> @@ -1612,9 +1612,9 @@ static int gfx_v11_0_sw_init(struct amdgpu_ip_block
> *ip_block)
> case IP_VERSION(11, 0, 2):
> case IP_VERSION(11, 0, 3):
> if (!adev->gfx.disable_uq &&
> - adev->gfx.me_fw_version >= 2390 &&
> - adev->gfx.pfp_fw_version >= 2530 &&
> - adev->gfx.mec_fw_version >= 2600 &&
> + adev->gfx.me_fw_version >= 2420 &&
> + adev->gfx.pfp_fw_version >= 2580 &&
> + adev->gfx.mec_fw_version >= 2650 &&
> adev->mes.fw_version[0] >= 120) {
> adev->userq_funcs[AMDGPU_HW_IP_GFX] =
> &userq_mes_funcs;
> adev->userq_funcs[AMDGPU_HW_IP_COMPUTE] =
> &userq_mes_funcs;
> --
> 2.49.0
>
