RE: [PATCH 2/2] drm/amdgpu: Set queue reset method for GC 12.1

"Zhang, Jesse(Jie)" <[email protected]>
Newsgroups org.freedesktop.lists.amd-gfx
Message-ID <DM4PR12MB51528B0FEBDEBAB288359587E3A32@DM4PR12MB5152.namprd12.prod.outlook.com>
AMD General

> -----Original Message-----
> From: Lin, Amber <[email protected]>
> Sent: Thursday, August 20, 2026 11:25 PM
> To: [email protected]; Liu, Shaoyun <[email protected]>;
> Deucher, Alexander <[email protected]>; Zhang, Jesse(Jie)
> <[email protected]>
> Cc: Lin, Amber <[email protected]>
> Subject: [PATCH 2/2] drm/amdgpu: Set queue reset method for GC 12.1
>
> Explicitely set use_mmio_for_reset=false in gfx_v12_1_sw_init for using MES to
> perform queue/pipe reset
>
> Signed-off-by: Amber Lin <[email protected]>
> ---
>  drivers/gpu/drm/amd/amdgpu/gfx_v12_1.c | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v12_1.c
> b/drivers/gpu/drm/amd/amdgpu/gfx_v12_1.c
> index 9dd7a3335719..97f4da7b7607 100644
> --- a/drivers/gpu/drm/amd/amdgpu/gfx_v12_1.c
> +++ b/drivers/gpu/drm/amd/amdgpu/gfx_v12_1.c
> @@ -1560,6 +1560,9 @@ static int gfx_v12_1_sw_init(struct amdgpu_ip_block
> *ip_block)
>
>       gfx_v12_1_alloc_ip_dump(adev);
>
> +     adev->gfx.me.use_mmio_for_reset = false;
> +     adev->gfx.mec.use_mmio_for_reset = false;
> +

The MES firmware already supports queue/pipeline reset, right? Do I need to check the MES firmware version before switching to MES?
Apart form that, the series is Review-by:Jesse Zhang <[email protected]>

Thanks
Jesse

>       mutex_init(&adev->gfx.mec.reset_mutex);
>
>       return 0;
> --
> 2.43.0
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.