From mboxrd@z Thu Jan 1 00:00:00 1970 From: Claude Code Review Bot To: dri-devel-reviews@example.com Subject: Claude review: drm/imagination: Drain interrupts before suspending the GPU Date: Wed, 11 Mar 2026 13:25:23 +1000 Message-ID: In-Reply-To: <20260310-drain-irqs-before-suspend-v1-0-bf4f9ed68e75@imgtec.com> References: <20260310-drain-irqs-before-suspend-v1-0-bf4f9ed68e75@imgtec.com> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit MIME-Version: 1.0 Overall Series Review Subject: drm/imagination: Drain interrupts before suspending the GPU Author: Alessio Belle Patches: 3 Reviewed: 2026-03-11T13:25:23.778248 --- This is a well-structured 2-patch series fixing a real crash where the GPU threaded IRQ handler can race with runtime PM suspend, accessing GPU registers after the GPU is powered off. The crash signature (SError on ARM64 in `pvr_riscv_irq_pending`) clearly demonstrates the problem. The fix is sound: patch 1 adds `synchronize_irq()` as a minimal stable-backportable fix, and patch 2 strengthens it to `disable_irq()` for a more robust approach. The series is correctly split for stable backporting purposes. --- Generated by Claude Code Patch Reviewer