Claude review: drm/vmwgfx: Never destroy STDU zero's Screen Target

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

Overall Series Review

Subject: drm/vmwgfx: Never destroy STDU zero's Screen Target
Author: Ian Forbes <ian.forbes@broadcom.com>
Patches: 1
Reviewed: 2026-05-16T10:41:44.665104

---

This is a single-patch fix for the VMware SVGA Screen Target Display Unit (STDU) driver. The patch addresses a real usability problem: when a VM display goes to sleep and the CRTC is disabled, the Screen Target for display unit 0 gets destroyed, which causes some VMware clients to fail to create a window (since they query existing screen target info to determine window size). Without a window, the user can't provide input to wake the VM.

The fix is simple and narrowly scoped — it adds an early return in `vmw_stdu_crtc_atomic_disable()` for unit 0, preventing the Screen Target from being destroyed. The approach is reasonable and consistent with how the driver already handles the "inactive but enabled" case (the existing early return at lines 446-449).

**Verdict: Looks correct and well-motivated. One minor concern worth discussing below.**

---
Generated by Claude Code Patch Reviewer