diff options
| author | Boris Brezillon <boris.brezillon@collabora.com> | 2026-03-09 13:43:18 +0100 |
|---|---|---|
| committer | Steven Price <steven.price@arm.com> | 2026-03-11 08:50:35 +0000 |
| commit | dfda8e9a282d3affbd64df3483e1bb8a5e0b015d (patch) | |
| tree | ba6b23f0a05be58cb6e02882d2bb69f238e7dc40 | |
| parent | 81e1035e1b9a1fc7c457337eb7869ef640fa374d (diff) | |
drm/panthor: Fix the "done_fence is initialized" detection logic
After commit 541c8f2468b9 ("dma-buf: detach fence ops on signal v3"),
dma_fence::ops == NULL can't be used to check if the fence is initialized.
Use dma_fence_was_initialized() instead.
v2:
- Use dma_fence_was_initialized() instead of open-coding it
Cc: Nicolas Frattaroli <nicolas.frattaroli@collabora.com>
Cc: Tvrtko Ursulin <tvrtko.ursulin@igalia.com>
Cc: Philipp Stanner <phasta@kernel.org>
Cc: Christian König <christian.koenig@amd.com>
Reported-by: Steven Price <steven.price@arm.com>
Reported-by: Nicolas Frattaroli <nicolas.frattaroli@collabora.com>
Fixes: 541c8f2468b9 ("dma-buf: detach fence ops on signal v3")
Signed-off-by: Boris Brezillon <boris.brezillon@collabora.com>
Reviewed-by: Christian König <christian.koenig@amd.com>
Tested-by: Nicolas Frattaroli <nicolas.frattaroli@collabora.com>
Reviewed-by: Steven Price <steven.price@arm.com>
Reviewed-by: Liviu Dudau <liviu.dudau@arm.com>
Signed-off-by: Steven Price <steven.price@arm.com>
Link: https://patch.msgid.link/20260309124318.222902-1-boris.brezillon@collabora.com
| -rw-r--r-- | drivers/gpu/drm/panthor/panthor_sched.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/gpu/drm/panthor/panthor_sched.c b/drivers/gpu/drm/panthor/panthor_sched.c index bd703a2904a1..c15941ebe07a 100644 --- a/drivers/gpu/drm/panthor/panthor_sched.c +++ b/drivers/gpu/drm/panthor/panthor_sched.c @@ -3915,7 +3915,7 @@ static void job_release(struct kref *ref) if (job->base.s_fence) drm_sched_job_cleanup(&job->base); - if (job->done_fence && job->done_fence->ops) + if (dma_fence_was_initialized(job->done_fence)) dma_fence_put(job->done_fence); else dma_fence_free(job->done_fence); |