diff options
| author | André Almeida <[email protected]> | 2025-07-04 03:06:29 +0000 |
|---|---|---|
| committer | André Almeida <[email protected]> | 2025-07-10 21:41:21 +0000 |
| commit | 667efb341917bde19f5d7517b65defcdaed67c9e (patch) | |
| tree | 32b90d4dc9a6129807071e06856bb7ede4f61b26 /drivers/gpu/drm/lima/lima_sched.c | |
| parent | drm/doc: Fix grammar for "Task information" (diff) | |
| download | kernel-667efb341917bde19f5d7517b65defcdaed67c9e.tar.gz kernel-667efb341917bde19f5d7517b65defcdaed67c9e.zip | |
drm/amdgpu: Fix lifetime of struct amdgpu_task_info after ring reset
When a ring reset happens, amdgpu calls drm_dev_wedged_event() using
struct amdgpu_task_info *ti as one of the arguments. After using *ti, a
call to amdgpu_vm_put_task_info(ti) is required to correctly track its
lifetime.
However, it's called from a place that the ring reset path never reaches
due to a goto after drm_dev_wedged_event() is called. Move
amdgpu_vm_put_task_info() bellow the exit label to make sure that it's
called regardless of the code path.
amdgpu_vm_put_task_info() can only accept a valid address or NULL as
argument, so initialise *ti to make sure we can call this function if
*ti isn't used.
Fixes: a72002cb181f ("drm/amdgpu: Make use of drm_wedge_task_info")
Reported-by: Dave Airlie <[email protected]>
Closes: https://lore.kernel.org/dri-devel/CAPM=9tz0rQP8VZWKWyuF8kUMqRScxqoa6aVdwWw9=5yYxyYQ2Q@mail.gmail.com/
Reviewed-by: Christian König <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: André Almeida <[email protected]>
Diffstat (limited to 'drivers/gpu/drm/lima/lima_sched.c')
0 files changed, 0 insertions, 0 deletions
