Re: [PATCH] drm/amdgpu: race issue when jobs on 2 ring timeout

2021-01-21 Thread Andrey Grodzovsky
Looks good to me Reviewed-by: Andrey Grodzovsky Andrey On 1/21/21 5:21 AM, Horace Chen wrote: Fix a racing issue when jobs on 2 rings timeout simultaneously. If 2 rings timed out at the same time, the amdgpu_device_gpu_recover will be reentered. Then the adev->gmc.xgmi.head will be grabbed

Re: [PATCH] drm/amdgpu: race issue when jobs on 2 ring timeout

2021-01-20 Thread Andrey Grodzovsky
On 1/20/21 9:12 AM, Horace Chen wrote: Fix a racing issue when jobs on 2 rings timeout simultaneously. If 2 rings timed out at the same time, the amdgpu_device_gpu_recover will be reentered. Then the adev->gmc.xgmi.head will be grabbed by 2 local linked list, which may cause wild pointer