]> git.ipfire.org Git - thirdparty/kernel/stable.git/commit
drm/radeon: delete radeon_fence_process in is_signaled, no deadlock
authorRobert McClinton <rbmccav@gmail.com>
Mon, 2 Feb 2026 07:58:55 +0000 (15:58 +0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 6 Feb 2026 15:44:24 +0000 (16:44 +0100)
commitd40a72d7e3bad4dfb311ef078f5a57362f088c7f
tree22cf26b0dcc61ec291a0eb3b381b7849408068ac
parent451c72f5e7cf5d339a6410a635cee0825687c3dc
drm/radeon: delete radeon_fence_process in is_signaled, no deadlock

[ Upstream commit 9eb00b5f5697bd56baa3222c7a1426fa15bacfb5 ]

Delete the attempt to progress the queue when checking if fence is
signaled. This avoids deadlock.

dma-fence_ops::signaled can be called with the fence lock in unknown
state. For radeon, the fence lock is also the wait queue lock. This can
cause a self deadlock when signaled() tries to make forward progress on
the wait queue. But advancing the queue is unneeded because incorrectly
returning false from signaled() is perfectly acceptable.

Link: https://github.com/brave/brave-browser/issues/49182
Closes: https://gitlab.freedesktop.org/drm/amd/-/issues/4641
Cc: Alex Deucher <alexander.deucher@amd.com>
Signed-off-by: Robert McClinton <rbmccav@gmail.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
(cherry picked from commit 527ba26e50ec2ca2be9c7c82f3ad42998a75d0db)
Cc: stable@vger.kernel.org
[ Minor conflict resolved. ]
Signed-off-by: Li hongliang <1468888505@139.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/gpu/drm/radeon/radeon_fence.c