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 clear-error-counter command to drm_ras Date: Sun, 12 Apr 2026 11:34:39 +1000 Message-ID: In-Reply-To: <20260409073318.2909379-4-riana.tauro@intel.com> References: <20260409073318.2909379-4-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 clear-error-counter command to drm_ras Author: Riana Tauro Patches: 8 Reviewed: 2026-04-12T11:34:39.897653 --- This is a clean, well-structured 2-patch series that adds a `clear-error-counter` netlink command to the DRM RAS subsystem and implements it in the XE driver. The new command follows the established patterns from the existing `get-error-counter` command closely, making it easy to review. The YAML spec, UAPI, core framework, and driver callback are all properly wired up. The series has already received Reviewed-by from Jakub Kicinski (patch 1, netlink aspects) and Raag Jadav (both patches). I have one functional concern about the error code when a driver doesn't implement the callback, and a few minor observations. --- --- Generated by Claude Code Patch Reviewer