]> git.ipfire.org Git - thirdparty/linux.git/commitdiff
drm/msm: Allow CAP_PERFMON for setting SYSPROF
authorRob Clark <robin.clark@oss.qualcomm.com>
Tue, 26 May 2026 14:50:36 +0000 (07:50 -0700)
committerRob Clark <robin.clark@oss.qualcomm.com>
Fri, 29 May 2026 14:07:27 +0000 (07:07 -0700)
Use perfmon_capable() which checks both CAP_SYS_ADMIN and CAP_PERFMON.
This matches what i915 and xe do, and seems more appropriate.

Signed-off-by: Rob Clark <robin.clark@oss.qualcomm.com>
Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>
Reviewed-by: Anna Maniscalco <anna.maniscalco2000@gmail.com>
Reviewed-by: Akhil P Oommen <akhilpo@oss.qualcomm.com>
Patchwork: https://patchwork.freedesktop.org/patch/728198/
Message-ID: <20260526145137.160554-3-robin.clark@oss.qualcomm.com>

drivers/gpu/drm/msm/adreno/adreno_gpu.c

index 6a0877e5374c77240dca3cd9b53c8d951cc2e284..69918975f7112d8e8214fd99c7c45075537a7517 100644 (file)
@@ -500,7 +500,7 @@ int adreno_set_param(struct msm_gpu *gpu, struct msm_context *ctx,
                return 0;
        }
        case MSM_PARAM_SYSPROF:
-               if (!capable(CAP_SYS_ADMIN))
+               if (!perfmon_capable())
                        return UERR(EPERM, drm, "invalid permissions");
                return msm_context_set_sysprof(ctx, gpu, value);
        case MSM_PARAM_EN_VM_BIND: