Re: [PATCH] accel/ivpu: Test for imported buffers with drm_gem_is_imported()

[Thomas Zimmermann <tzimmermann@suse.de>]

Hi

Am 09.03.26 um 10:27 schrieb Karol Wachowski:
> Instead of testing import_attach for imported GEM buffers, invoke
> drm_gem_is_imported() to do the test. The test itself does not change.

Thanks a lot!

>
> Suggested-by: Thomas Zimmermann <tzimmermann@suse.de>
> Signed-off-by: Karol Wachowski <karol.wachowski@linux.intel.com>

Reviewed-by: Thomas Zimmermann <tzimmermann@suse.de>

> ---
>   drivers/accel/ivpu/ivpu_gem.c | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/accel/ivpu/ivpu_gem.c b/drivers/accel/ivpu/ivpu_gem.c
> index 98b9ce26962b..8009965286e0 100644
> --- a/drivers/accel/ivpu/ivpu_gem.c
> +++ b/drivers/accel/ivpu/ivpu_gem.c
> @@ -48,7 +48,7 @@ static struct sg_table *ivpu_bo_map_attachment(struct ivpu_device *vdev, struct
>   {
>   	struct sg_table *sgt;
>   
> -	drm_WARN_ON(&vdev->drm, !bo->base.base.import_attach);
> +	drm_WARN_ON(&vdev->drm, !drm_gem_is_imported(&bo->base.base));
>   
>   	ivpu_bo_lock(bo);
>   
> @@ -157,7 +157,7 @@ static void ivpu_bo_unbind_locked(struct ivpu_bo *bo)
>   	}
>   
>   	if (bo->base.sgt) {
> -		if (bo->base.base.import_attach) {
> +		if (drm_gem_is_imported(&bo->base.base)) {
>   			dma_buf_unmap_attachment(bo->base.base.import_attach,
>   						 bo->base.sgt, DMA_BIDIRECTIONAL);
>   		} else {

-- 
--
Thomas Zimmermann
Graphics Driver Developer
SUSE Software Solutions Germany GmbH
Frankenstr. 146, 90461 Nürnberg, Germany, www.suse.com
GF: Jochen Jaser, Andrew McDonald, Werner Knoblich, (HRB 36809, AG Nürnberg)