From mboxrd@z Thu Jan 1 00:00:00 1970 From: Claude Code Review Bot To: dri-devel-reviews@example.com Subject: Claude review: Add support for clear counter and error event in DRM RAS Date: Thu, 12 Mar 2026 07:06:12 +1000 Message-ID: In-Reply-To: <20260311102913.3387468-6-riana.tauro@intel.com> References: <20260311102913.3387468-6-riana.tauro@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: Add support for clear counter and error event in DRM RAS Author: Riana Tauro Patches: 5 Reviewed: 2026-03-12T07:06:12.645794 --- This series adds two new features to the DRM RAS (Reliability, Availability, Serviceability) netlink framework: a `clear-error-counter` command and an `error-event` multicast notification. The series is well-structured with the DRM core changes preceding the XE driver implementations. The code is generally clean, but there are a few concerns around concurrency/locking in the clear path, lack of node lifetime protection (no RCU/refcount on `xa_load`), and whether the `error_id` variable is actually valid at the point of notification in patch 4. --- Generated by Claude Code Patch Reviewer