Re: [PATCH v2] drm/xe/uapi: update used tracking kernel-doc

["Yadav, Sanjay Kumar" <sanjay.kumar.yadav@intel.com>]

On 30-01-2026 18:21, Matthew Auld wrote:
> In commit 4d0b035fd6da ("drm/xe/uapi: loosen used tracking restriction")
> we dropped the CAP_PERMON restriction but missed updating the
> corresponding kernel-doc. Fix that.
>
> v2 (Sanjay):
>    - Don't drop the note around the extra cpu_visible_used expectations.
>
> Reported-by: Ulisses Furquim <ulisses.furquim@intel.com>
> Fixes: 4d0b035fd6da ("drm/xe/uapi: loosen used tracking restriction")
> Signed-off-by: Matthew Auld <matthew.auld@intel.com>
> Cc: Sanjay Yadav <sanjay.kumar.yadav@intel.com>

Reviewed-by: Sanjay Yadav <sanjay.kumar.yadav@intel.com>

LGTM

> ---
>   include/uapi/drm/xe_drm.h | 8 +-------
>   1 file changed, 1 insertion(+), 7 deletions(-)
>
> diff --git a/include/uapi/drm/xe_drm.h b/include/uapi/drm/xe_drm.h
> index 077e66a682e2..c9e70f78e723 100644
> --- a/include/uapi/drm/xe_drm.h
> +++ b/include/uapi/drm/xe_drm.h
> @@ -335,10 +335,6 @@ struct drm_xe_mem_region {
>   	__u64 total_size;
>   	/**
>   	 * @used: Estimate of the memory used in bytes for this region.
> -	 *
> -	 * Requires CAP_PERFMON or CAP_SYS_ADMIN to get reliable
> -	 * accounting.  Without this the value here will always equal
> -	 * zero.
>   	 */
>   	__u64 used;
>   	/**
> @@ -363,9 +359,7 @@ struct drm_xe_mem_region {
>   	 * @cpu_visible_used: Estimate of CPU visible memory used, in
>   	 * bytes.
>   	 *
> -	 * Requires CAP_PERFMON or CAP_SYS_ADMIN to get reliable
> -	 * accounting. Without this the value here will always equal
> -	 * zero.  Note this is only currently tracked for
> +	 * Note this is only currently tracked for
>   	 * DRM_XE_MEM_REGION_CLASS_VRAM regions (for other types the value
>   	 * here will always be zero).
>   	 */