Claude review: drm/i915/display: Fix intel_lpe_audio_irq_handler for PREEMPT-RT

[Claude Code Review Bot <claude-review@example.com>]

Overall Series Review

Subject: drm/i915/display: Fix intel_lpe_audio_irq_handler for PREEMPT-RT
Author: Runyu Xiao <runyu.xiao@seu.edu.cn>
Patches: 4
Reviewed: 2026-06-04T13:25:57.081447

---

This is a single-patch fix (v3) that replaces `generic_handle_irq()` with `generic_handle_irq_safe()` in the LPE audio IRQ forwarding path. The change is correct and well-motivated: on PREEMPT_RT kernels, i915's IRQ handler runs in threaded context (not hard IRQ context), so `generic_handle_irq()` — which requires hard IRQ context — is inappropriate. `generic_handle_irq_safe()` wraps the call with `local_irq_save()`/`local_irq_restore()`, making it safe from any context.

The fix is minimal, one-line, and uses the correct upstream API. The commit message accurately describes the problem and the solution. The current drm-next tree still has `generic_handle_irq()` at line 265, confirming this fix is still needed.

**One process concern:** The Signed-off-by chain has Maarten Lankhorst as From/author with his own Signed-off-by, then Runyu Xiao's Signed-off-by as the submitter. The v3 changelog says this is a rebase of Maarten's original fix. The Reviewed-by tags from Matthew Brost and Uma Shankar are carried forward — this is acceptable if the patch content hasn't materially changed from what they reviewed, which appears to be the case (same one-line change).

**Verdict: Looks good.** This is a straightforward, correct RT fix.

---

---
Generated by Claude Code Patch Reviewer