Re: [PATCH] drm/vc4: Test for imported buffers with drm_gem_is_imported()

["Maíra Canal" <mcanal@igalia.com>]

Hi Thomas,

On 27/02/26 10:31, Thomas Zimmermann wrote:
> Instead of testing import_attach for imported GEM buffers, invoke
> drm_gem_is_imported() to do the test. The test itself does not change.
> 
> Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de>

Reviewed-by: Maíra Canal <mcanal@igalia.com>

Best regards,
- Maíra

> Cc: Maxime Ripard <mripard@kernel.org>
> Cc: Dave Stevenson <dave.stevenson@raspberrypi.com>
> Cc: "Maíra Canal" <mcanal@igalia.com>
> Cc: Raspberry Pi Kernel Maintenance <kernel-list@raspberrypi.com>
> ---
>   drivers/gpu/drm/vc4/vc4_bo.c  | 2 +-
>   drivers/gpu/drm/vc4/vc4_gem.c | 2 +-
>   2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/gpu/drm/vc4/vc4_bo.c b/drivers/gpu/drm/vc4/vc4_bo.c
> index 1f93bc5a3d02..f45ba47b4ba8 100644
> --- a/drivers/gpu/drm/vc4/vc4_bo.c
> +++ b/drivers/gpu/drm/vc4/vc4_bo.c
> @@ -556,7 +556,7 @@ static void vc4_free_object(struct drm_gem_object *gem_bo)
>   	mutex_lock(&vc4->bo_lock);
>   	/* If the object references someone else's memory, we can't cache it.
>   	 */
> -	if (gem_bo->import_attach) {
> +	if (drm_gem_is_imported(gem_bo)) {
>   		vc4_bo_destroy(bo);
>   		goto out;
>   	}
> diff --git a/drivers/gpu/drm/vc4/vc4_gem.c b/drivers/gpu/drm/vc4/vc4_gem.c
> index ad8cbd727b80..2d3df5e621c1 100644
> --- a/drivers/gpu/drm/vc4/vc4_gem.c
> +++ b/drivers/gpu/drm/vc4/vc4_gem.c
> @@ -1250,7 +1250,7 @@ int vc4_gem_madvise_ioctl(struct drm_device *dev, void *data,
>   	/* Not sure it's safe to purge imported BOs. Let's just assume it's
>   	 * not until proven otherwise.
>   	 */
> -	if (gem_obj->import_attach) {
> +	if (drm_gem_is_imported(gem_obj)) {
>   		DRM_DEBUG("madvise not supported on imported BOs\n");
>   		ret = -EINVAL;
>   		goto out_put_gem;