From mboxrd@z Thu Jan 1 00:00:00 1970 From: Claude Code Review Bot To: dri-devel-reviews@example.com Subject: Claude review: drm/exec: drm_exec polishing Date: Wed, 01 Apr 2026 07:52:56 +1000 Message-ID: In-Reply-To: <20260331092023.81616-1-thomas.hellstrom@linux.intel.com> References: <20260331092023.81616-1-thomas.hellstrom@linux.intel.com> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit MIME-Version: 1.0 Overall Series Review Subject: drm/exec: drm_exec polishing Author: =?UTF-8?q?Thomas=20Hellstr=C3=B6m?= Patches: 21 Reviewed: 2026-04-01T07:52:56.997519 --- This is a well-motivated cleanup series that hides `drm_exec` internal state from drivers, preparing the ground for future refactoring of `drm_exec` into a WW transaction abstraction. The changes are logically ordered and individually small. There is one **compile-breaking bug** in patch 5 (the amdkfd changes pass `exec` instead of `&exec` to the new accessor). There are also a few minor concerns noted below, but overall the direction is sound. --- --- Generated by Claude Code Patch Reviewer