diff options
| author | Guchun Chen <guchun.chen@amd.com> | 2023-04-26 04:46:54 +0300 |
|---|---|---|
| committer | Alex Deucher <alexander.deucher@amd.com> | 2023-06-09 16:29:58 +0300 |
| commit | 967a66396e4668ef314e06a0391e67aa1cf6d058 (patch) | |
| tree | 1609f90a43c97159e0c5bb42b638d2881811a250 /drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.c | |
| parent | 9e690184586bfb88efa176cdf912414f6c53519c (diff) | |
| download | linux-967a66396e4668ef314e06a0391e67aa1cf6d058.tar.xz | |
drm/amdgpu: drop redundant sched job cleanup when cs is aborted
Once command submission failed due to userptr invalidation in
amdgpu_cs_submit, legacy code will perform cleanup of scheduler
job. However, it's not needed at all, as former commit has integrated
job cleanup stuff into amdgpu_job_free. Otherwise, because of double
free, a NULL pointer dereference will occur in such scenario.
Bug: https://gitlab.freedesktop.org/drm/amd/-/issues/2457
Fixes: f7d66fb2ea43 ("drm/amdgpu: cleanup scheduler job initialization v2")
Signed-off-by: Guchun Chen <guchun.chen@amd.com>
Reviewed-by: Christian König <christian.koenig@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Diffstat (limited to 'drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.c')
0 files changed, 0 insertions, 0 deletions