From mboxrd@z Thu Jan 1 00:00:00 1970 From: Claude Code Review Bot To: dri-devel-reviews@example.com Subject: Claude review: Virtio-GPU S4 support Date: Wed, 27 May 2026 14:17:08 +1000 Message-ID: In-Reply-To: <20260526192814.179673-1-dongwon.kim@intel.com> References: <20260526192814.179673-1-dongwon.kim@intel.com> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Overall Series Review Subject: Virtio-GPU S4 support Author: dongwon.kim@intel.com Patches: 4 Reviewed: 2026-05-27T14:17:08.027796 --- This v9 series adds S4 (hibernation) support for the virtio-gpu driver acro= ss 3 patches: freeze/restore hooks, object tracking/restoration, and a PM n= otifier. The architecture is sound =E2=80=94 PM_HIBERNATION_PREPARE sets a = flag, freeze unrefs all tracked objects to clean the host side, and restore= re-submits them to a freshly started QEMU. The series has been through ext= ensive review already (v9), and the core design is solid. There are a few issues worth raising: missing error recovery in the freeze = path (leaving DRM suspended on timeout), an unnecessary include, use of `in= t` instead of `bool` for the `no_cb` parameter, and VRAM objects silently n= ot being restored. None are showstoppers, but the error recovery gap in fre= eze is worth fixing. --- --- Generated by Claude Code Patch Reviewer