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/v3d: Using spin_lock_irqsave instead of spin_lock in the RT kernel Date: Thu, 05 Mar 2026 13:49:50 +1000 Message-ID: In-Reply-To: <20260304031724.246404-1-xiaolei.wang@windriver.com> References: <20260304031724.246404-1-xiaolei.wang@windriver.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/v3d: Using spin_lock_irqsave instead of spin_lock in the RT kernel Author: Xiaolei Wang Patches: 2 Reviewed: 2026-03-05T13:49:50.617735 --- This is a single patch attempting to fix an RT kernel warning in v3d's `v3d_job_update_stats()` by replacing the `spin_lock()` + manual `local_irq_save()`/`preempt_disable()` pattern with `spin_lock_irqsave()`. While the motivation is valid (the current code causes a sleeping-in-atomic warning on PREEMPT_RT), the patch has a **correctness bug** and is **incomplete**. --- Generated by Claude Code Patch Reviewer