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: Add sysprof accessors Date: Sat, 16 May 2026 10:50:09 +1000 Message-ID: In-Reply-To: <20260514134052.361771-8-robin.clark@oss.qualcomm.com> References: <20260514134052.361771-1-robin.clark@oss.qualcomm.com> <20260514134052.361771-8-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 Good refactor splitting sysprof into two concerns: `msm_gpu_sysprof_no_perfcntr_zap()` (disable counter clearing on context switch) and `msm_gpu_sysprof_no_ifpc()` (disable IFPC). For now they are the same, but patch 16 splits them. Note: this patch also adds `.sysprof_setup = a6xx_gmu_sysprof_setup` to `a8xx_gpu_funcs`, which was previously missing. This is a bug fix bundled into a refactor - might be worth calling out in the commit message. --- Generated by Claude Code Patch Reviewer