Re: [Intel-gfx] [PATCH 2/2] drm/i915: Add __rcu annotation to cursor when iterating client objects

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, 13 Nov 2023, Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxxxxxxxx> wrote:
> From: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx>
>
> __rcu annotation is needed to avoid the sparse warnings such as:
>
>   .../i915_drm_client.c:92:9: sparse: sparse: incompatible types in comparison expression (different address spaces):
>   .../i915_drm_client.c:92:9: sparse:    struct list_head [noderef] __rcu *
>   .../i915_drm_client.c:92:9: sparse:    struct list_head *
>
> Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx>
> Fixes: 968853033d8a ("drm/i915: Implement fdinfo memory stats printing")
> Reported-by: kernel test robot <lkp@xxxxxxxxx>
> Closes: https://lore.kernel.org/oe-kbuild-all/202311110610.h0m6ydI5-lkp@xxxxxxxxx/
> Cc: Andi Shyti <andi.shyti@xxxxxxxxxxxxxxx>
> Cc: Aravind Iddamsetty <aravind.iddamsetty@xxxxxxxxx>

Reviewed-by: Jani Nikula <jani.nikula@xxxxxxxxx>

> ---
>  drivers/gpu/drm/i915/i915_drm_client.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/i915/i915_drm_client.c b/drivers/gpu/drm/i915/i915_drm_client.c
> index be9acfd9410e..fa6852713bee 100644
> --- a/drivers/gpu/drm/i915/i915_drm_client.c
> +++ b/drivers/gpu/drm/i915/i915_drm_client.c
> @@ -78,7 +78,7 @@ static void show_meminfo(struct drm_printer *p, struct drm_file *file)
>  	struct drm_i915_private *i915 = fpriv->i915;
>  	struct drm_i915_gem_object *obj;
>  	struct intel_memory_region *mr;
> -	struct list_head *pos;
> +	struct list_head __rcu *pos;
>  	unsigned int id;
>  
>  	/* Public objects. */

-- 
Jani Nikula, Intel



[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux