Claude review: drm/panthor: Avoid potential UAF due to memory reclaim

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

Overall Series Review

Subject: drm/panthor: Avoid potential UAF due to memory reclaim
Author: Akash Goel <akash.goel@arm.com>
Patches: 1
Reviewed: 2026-04-12T09:28:38.331168

---

This is a single-patch fix for a real use-after-free vulnerability in the panthor GPU driver's shrinker path. The bug was introduced by `fb42964e2a76 ("drm/panthor: Add a GEM shrinker")`. The scenario is:

1. Shrinker evicts a BO, invalidating all GPU mappings and setting `vm_bo->evicted = true`.
2. A new GPU mapping is subsequently created for that BO (adding a VA to the same vm_bo).
3. The BO goes back onto a reclaim list due to the new mapping.
4. Shrinker fires again — but the old code's `if (vm_bo->evicted) continue` skips the entire vm_bo, **including** the VA walk that would tear down the new mapping.
5. Backing pages are freed while still GPU-mapped → UAF.

The fix is correct and well-targeted: it removes the early `continue` and instead only gates the `drm_gpuvm_bo_evict()` call on the evicted flag, while always running the VA invalidation loop.

**Verdict: The patch is correct and fixes a real vulnerability. One minor comment accuracy issue noted below.**

---

---
Generated by Claude Code Patch Reviewer