summaryrefslogtreecommitdiff
path: root/scripts
diff options
context:
space:
mode:
authorPierre-Eric Pelloux-Prayer <pierre-eric.pelloux-prayer@amd.com>2024-12-06 13:17:45 +0100
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2024-12-27 14:02:11 +0100
commit67291d601f2b032062b1b2f60ffef1b63e10094c (patch)
treef6cfb231d9e9440bfc7b9a7546f50bd60f3b30d6 /scripts
parent5d6f446eca40d5e7ab0f1918b36e113f01cddac3 (diff)
downloadlinux-67291d601f2b032062b1b2f60ffef1b63e10094c.tar.gz
linux-67291d601f2b032062b1b2f60ffef1b63e10094c.tar.bz2
linux-67291d601f2b032062b1b2f60ffef1b63e10094c.zip
drm/amdgpu: don't access invalid sched
[ Upstream commit a93b1020eb9386d7da11608477121b10079c076a ] Since 2320c9e6a768 ("drm/sched: memset() 'job' in drm_sched_job_init()") accessing job->base.sched can produce unexpected results as the initialisation of (*job)->base.sched done in amdgpu_job_alloc is overwritten by the memset. This commit fixes an issue when a CS would fail validation and would be rejected after job->num_ibs is incremented. In this case, amdgpu_ib_free(ring->adev, ...) will be called, which would crash the machine because the ring value is bogus. To fix this, pass a NULL pointer to amdgpu_ib_free(): we can do this because the device is actually not used in this function. The next commit will remove the ring argument completely. Fixes: 2320c9e6a768 ("drm/sched: memset() 'job' in drm_sched_job_init()") Signed-off-by: Pierre-Eric Pelloux-Prayer <pierre-eric.pelloux-prayer@amd.com> Reviewed-by: Alex Deucher <alexander.deucher@amd.com> Reviewed-by: Christian König <christian.koenig@amd.com> Signed-off-by: Alex Deucher <alexander.deucher@amd.com> (cherry picked from commit 2ae520cb12831d264ceb97c61f72c59d33c0dbd7) Signed-off-by: Sasha Levin <sashal@kernel.org>
Diffstat (limited to 'scripts')
0 files changed, 0 insertions, 0 deletions