On 10/27/2025 4:09 PM, Maciej Falkowski wrote:
> Rework of imported buffers introduced in the commit
> e0c0891cd63b ("accel/ivpu: Rework bind/unbind of imported buffers")
> switched the logic of imported buffers by dma mapping/unmaping
> them just as the regular buffers.
>
> The commit didn't include removal of skipping dma unmap of imported
> buffers which results in them being mapped without unmapping.
>
> Fixes: e0c0891cd63b ("accel/ivpu: Rework bind/unbind of imported buffers")
> Signed-off-by: Maciej Falkowski <[email protected]>
> ---
> drivers/accel/ivpu/ivpu_gem.c | 3 ---
> 1 file changed, 3 deletions(-)
>
> diff --git a/drivers/accel/ivpu/ivpu_gem.c b/drivers/accel/ivpu/ivpu_gem.c
> index e7277e02840a..2a159535421a 100644
> --- a/drivers/accel/ivpu/ivpu_gem.c
> +++ b/drivers/accel/ivpu/ivpu_gem.c
> @@ -158,9 +158,6 @@ static void ivpu_bo_unbind_locked(struct ivpu_bo *bo)
> bo->ctx = NULL;
> }
>
> - if (drm_gem_is_imported(&bo->base.base))
> - return;
> -
> if (bo->base.sgt) {
> if (bo->base.base.import_attach) {
> dma_buf_unmap_attachment(bo->base.base.import_attach,
Reviewed-by: Karol Wachowski <[email protected]>
