Claude review: drm/logicvc: Avoid use-after-free with devm_kzalloc()

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

Overall Series Review

Subject: drm/logicvc: Avoid use-after-free with devm_kzalloc()
Author: Romain Gantois <romain.gantois@bootlin.com>
Patches: 2
Reviewed: 2026-06-04T14:34:10.308240

---

This is a single well-motivated patch converting the logicvc driver from `devm_kzalloc`-based allocation to DRM-managed (`drmm_*`) allocation for planes, CRTCs, encoders, and connectors. The core problem is real and well-understood: `devm`-managed structs can be freed before the DRM core is done with them (device unbind frees `devm` resources, but userspace may still hold DRM file handles), leading to use-after-free. The author confirms this was observed via KASAN on v6.6.

The approach is correct and follows the standard pattern that many other DRM drivers have adopted. The API usage is verified correct against the kernel headers. Error handling simplification is appropriate since `drmm_*` resources are automatically cleaned up on DRM device release.

**Verdict: Patch looks good.** A couple of minor style nits below, nothing blocking.

---

---
Generated by Claude Code Patch Reviewer