Claude review: drm/i915/selftests: Fix build after dma-fence locking rework

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

Overall Series Review

Subject: drm/i915/selftests: Fix build after dma-fence locking rework
Author: Matthew Brost <matthew.brost@intel.com>
Patches: 7
Reviewed: 2026-02-24T09:54:10.645046

---

This is a single-patch build fix for the i915_active selftest, needed after a dma-fence locking rework that inlined the spinlock into `struct dma_fence` and introduced a `dma_fence_spinlock()` accessor. The patch mechanically replaces two instances of direct `fence->lock` access with the new accessor. The change is minimal, correct, and already has Christian König's Reviewed-by.

The mailing list discussion following the patch is more interesting than the patch itself. Christian raised a pre-existing question about the use of `spin_lock_irq()` vs `spin_lock_irqsave()` in `active_flush()`, and a lockdep splat was reported in `dma_fence_chain_enable_signaling()` that Christian identified as a pre-existing bug newly exposed by the locking rework. Neither issue is introduced by this patch.

---
Generated by Claude Code Patch Reviewer