On 3/11/2026 1:24 PM, Falkowski, Maciej wrote: > Reviewed-by: Maciej Falkowski <[email protected]>
Thank you! Applied to drm-misc-next. > > On 3/10/2026 1:07 PM, Karol Wachowski wrote: >> Replace direct import_attach test with drm_gem_is_imported() >> in ivpu_bo_bind(). >> >> Replace kzalloc(sizeof(*bo), GFP_KERNEL) with kzalloc_obj() >> in ivpu_gem_create_object(). >> >> Remove unnecessary cast to bool in ivpu_dbg_bo(). >> >> No functional changes. >> >> Signed-off-by: Karol Wachowski <[email protected]> >> --- >> drivers/accel/ivpu/ivpu_gem.c | 8 ++++---- >> 1 file changed, 4 insertions(+), 4 deletions(-) >> >> diff --git a/drivers/accel/ivpu/ivpu_gem.c b/drivers/accel/ivpu/ >> ivpu_gem.c >> index 8009965286e0..4f2005a8d496 100644 >> --- a/drivers/accel/ivpu/ivpu_gem.c >> +++ b/drivers/accel/ivpu/ivpu_gem.c >> @@ -1,6 +1,6 @@ >> // SPDX-License-Identifier: GPL-2.0-only >> /* >> - * Copyright (C) 2020-2023 Intel Corporation >> + * Copyright (C) 2020-2026 Intel Corporation >> */ >> #include <linux/dma-buf.h> >> @@ -31,7 +31,7 @@ static inline void ivpu_dbg_bo(struct ivpu_device >> *vdev, struct ivpu_bo *bo, con >> "%6s: bo %8p size %9zu ctx %d vpu_addr %9llx pages %d sgt >> %d mmu_mapped %d wc %d imported %d\n", >> action, bo, ivpu_bo_size(bo), bo->ctx_id, bo->vpu_addr, >> (bool)bo->base.pages, (bool)bo->base.sgt, bo->mmu_mapped, >> bo->base.map_wc, >> - (bool)drm_gem_is_imported(&bo->base.base)); >> + drm_gem_is_imported(&bo->base.base)); >> } >> static inline int ivpu_bo_lock(struct ivpu_bo *bo) >> @@ -81,7 +81,7 @@ int __must_check ivpu_bo_bind(struct ivpu_bo *bo) >> ivpu_dbg_bo(vdev, bo, "bind"); >> - if (bo->base.base.import_attach) >> + if (drm_gem_is_imported(&bo->base.base)) >> sgt = ivpu_bo_map_attachment(vdev, bo); >> else >> sgt = drm_gem_shmem_get_pages_sgt(&bo->base); >> @@ -195,7 +195,7 @@ struct drm_gem_object >> *ivpu_gem_create_object(struct drm_device *dev, size_t siz >> if (size == 0 || !PAGE_ALIGNED(size)) >> return ERR_PTR(-EINVAL); >> - bo = kzalloc(sizeof(*bo), GFP_KERNEL); >> + bo = kzalloc_obj(*bo); >> if (!bo) >> return ERR_PTR(-ENOMEM); >>
