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/msm: Allow CAP_PERFMON for setting SYSPROF Date: Tue, 05 May 2026 08:06:01 +1000 Message-ID: In-Reply-To: <20260504190751.61052-3-robin.clark@oss.qualcomm.com> References: <20260504190751.61052-1-robin.clark@oss.qualcomm.com> <20260504190751.61052-3-robin.clark@oss.qualcomm.com> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit MIME-Version: 1.0 Patch Review One-liner changing `capable(CAP_SYS_ADMIN)` to `perfmon_capable()` in `adreno_set_param()`: ```c - if (!capable(CAP_SYS_ADMIN)) + if (!perfmon_capable()) ``` Good change, aligns with i915/xe precedent. Already has R-b from Dmitry. --- Generated by Claude Code Patch Reviewer