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 RAS Fixes Date: Mon, 25 May 2026 17:57:40 +1000 Message-ID: In-Reply-To: <20260523050212.557292-1-raag.jadav@intel.com> References: <20260523050212.557292-1-raag.jadav@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: DRM RAS Fixes Author: Raag Jadav Patches: 5 Reviewed: 2026-05-25T17:57:40.177702 --- This is a clean, well-structured 4-patch series fixing real resource leak b= ugs in the DRM RAS subsystem and xe driver. The core issue is that error pa= ths in the original code leaked sk_buffs, genetlink headers, and per-node r= egistrations/allocations when partial failures occurred during setup. Patches 1-3 fix genuine resource leaks and are tagged with appropriate `Fix= es:` lines. Patch 4 is a cosmetic log prefix change =E2=80=94 the `Fixes:` = tag on it is debatable since it's a convention improvement rather than a bu= g fix. The series applies cleanly, the patches are logically ordered, and the v2 c= hanges (goto-based cleanup, retaining NULL on failure) are good responses t= o review feedback. I have a few minor observations below but no blocking is= sues. --- --- Generated by Claude Code Patch Reviewer