[SRU][S][PATCH 8/9] drm/i915: Consider RCU read section as atomic.
Benjamin Wheeler
benjamin.wheeler at canonical.com
Thu Sep 24 12:06:32 UTC 2026
From: Sebastian Andrzej Siewior <bigeasy at linutronix.de>
BugLink: https://bugs.launchpad.net/bugs/2168432
Locking and/ or running inside interrupt handler will not lead to an
atomic section on PREEMPT_RT. The RCU read section needs to be
considered because all locks, which become sleeping locks on
PREEMPT_RT, start a RCU read section. Scheduling/ sleeping while within
a RCU read section is invalid.
Check for also for RCU read section in stop_timeout() to determine if it
is safe to sleep.
Reviewed-by: Rodrigo Vivi <rodrigo.vivi at intel.com>
Link: https://lore.kernel.org/r/20250714153954.629393-9-bigeasy@linutronix.de
Signed-off-by: Sebastian Andrzej Siewior <bigeasy at linutronix.de>
Signed-off-by: Benjamin Wheeler <benjamin.wheeler at canonical.com>
---
drivers/gpu/drm/i915/gt/intel_engine_cs.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/i915/gt/intel_engine_cs.c b/drivers/gpu/drm/i915/gt/intel_engine_cs.c
index c0fd349a4600..9dd9665128ca 100644
--- a/drivers/gpu/drm/i915/gt/intel_engine_cs.c
+++ b/drivers/gpu/drm/i915/gt/intel_engine_cs.c
@@ -1607,7 +1607,7 @@ u64 intel_engine_get_last_batch_head(const struct intel_engine_cs *engine)
static unsigned long stop_timeout(const struct intel_engine_cs *engine)
{
- if (in_atomic() || irqs_disabled()) /* inside atomic preempt-reset? */
+ if (in_atomic() || irqs_disabled() || rcu_preempt_depth()) /* inside atomic preempt-reset? */
return 0;
/*
--
2.43.0
More information about the kernel-team
mailing list