Claude review: drm/amdgpu: fix recursive ww_mutex in devcoredump IB dump

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

Overall Series Review

Subject: drm/amdgpu: fix recursive ww_mutex in devcoredump IB dump
Author: Mikhail Gavrilov <mikhail.v.gavrilov@gmail.com>
Patches: 21
Reviewed: 2026-05-25T19:53:47.734865

---

This mbox contains a patch series evolving across 5 versions (v1 → v5), fixing a real lockdep recursive locking splat and potential self-deadlock in the amdgpu devcoredump IB dump path. The latest version (v5) is a well-structured 2-patch series that converts `amdgpu_vm_lock_by_pasid()` to use `drm_exec` (patch 1) and then uses that to lock all BOs in one ww ticket (patch 2), as suggested by Christian König.

**The approach is correct and well-motivated.** The commit messages are thorough, the split is logical, and patch 1/2 already has Christian's Reviewed-by. The series also fixes a pre-existing BO refcount leak on the reserve-failure path.

**One correctness bug exists in v5 2/2:** the variable `r` is not initialized before the `drm_exec_until_all_locked` inner for-loop, which means a stale value could incorrectly break out of the locking loop when all IB mappings are NULL. This needs a fix before merging.

**Review of only the v5 patches follows** (v1–v4 are superseded).

---

---
Generated by Claude Code Patch Reviewer