Hi Luben,

As far as I know of this is only for gc 11.0.3. Mes self test is also skipped 
in mes late init for this version of IP.

Best Regards,
Yubiao Wang

-----Original Message-----
From: Tuikov, Luben <[email protected]> 
Sent: Wednesday, October 19, 2022 11:53 AM
To: Wang, YuBiao <[email protected]>; [email protected]
Cc: Grodzovsky, Andrey <[email protected]>; Quan, Evan 
<[email protected]>; Chen, Horace <[email protected]>; Koenig, Christian 
<[email protected]>; Deucher, Alexander <[email protected]>; 
Xiao, Jack <[email protected]>; Zhang, Hawking <[email protected]>; Liu, 
Monk <[email protected]>; Xu, Feifei <[email protected]>; Wang, Yang(Kevin) 
<[email protected]>
Subject: Re: [PATCH] drm/amdgpu: skip mes self test for gc 11.0.3 in recover

On 2022-10-18 23:45, YuBiao Wang wrote:
> Temporary disable mes self teset for gc 11.0.3 during gpu_recovery.
> 

Is this "temporary" as in "we'll revert this commit later", or is it 
"temporary" as in the code execution itself?

> Signed-off-by: YuBiao Wang <[email protected]>
> ---
>  drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c 
> b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
> index e0445e8cc342..5b8362727226 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
> @@ -5381,7 +5381,7 @@ int amdgpu_device_gpu_recover(struct amdgpu_device 
> *adev,
>                       drm_sched_start(&ring->sched, 
> !tmp_adev->asic_reset_res);
>               }
>  
> -             if (adev->enable_mes)
> +             if (adev->enable_mes && adev->ip_versions[GC_HWIP][0] != 
> +IP_VERSION(11, 0, 3))
>                       amdgpu_mes_self_test(tmp_adev);

Is this just for this version of the IP or this and any newer versions?

Regards,
Luben

>  
>               if (!drm_drv_uses_atomic_modeset(adev_to_drm(tmp_adev)) && 
> !job_signaled) {

Reply via email to