]> git.itanic.dy.fi Git - linux-stable/commit
drm/amdgpu: drop redundant sched job cleanup when cs is aborted
authorGuchun Chen <guchun.chen@amd.com>
Wed, 26 Apr 2023 01:46:54 +0000 (09:46 +0800)
committerAlex Deucher <alexander.deucher@amd.com>
Thu, 4 May 2023 03:10:02 +0000 (23:10 -0400)
commit1253685f0d3eb3eab0bfc4bf15ab341a5f3da0c8
tree3627f524598e21b20f09e1c2bd8ad803c24a0d7c
parent682439fffad9fa9a38d37dd1b1318e9374232213
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>
Cc: stable@vger.kernel.org
drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c