From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from bali.collaboradmins.com (bali.collaboradmins.com [148.251.105.195]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 3D7C29443 for ; Thu, 30 Apr 2026 07:57:42 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=148.251.105.195 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1777535864; cv=none; b=j+vE/LFIQOeiqXwYi6qb+jjiqr/SipT4qK6LpLQZ67cY2mHqwI7iyTwDPmyyqkI48pdhEbf/qUDXpXvDk7V5eGpEH6fWVZoCgTYxfJMW/SapcOke08Z5bsuYGBoHESCGssUm/2er78uUt350wRwkzZ2wjCH76dzhAspQaW/gM7w= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1777535864; c=relaxed/simple; bh=XchW9YUc1tPO3prYku284a4ISpGs3mRYztaK2Z7JK0Y=; h=Date:From:To:Cc:Subject:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=ctXlP8OhVmG1pDX+9IrFY7NAy/ulhAJKhBSdSBIPBd8Wk6HQmvXY9Hxls/ESc3823s6nqF7qWDl4Q0aVB4IKABNhbK71yAyoW0eNlNC32TeA8Vr3ZrkOjC05+yqI63SZ+Nh+E66vWJPOkaorRn4rL4M96yA2q4hDKv4vW7hkvYY= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=collabora.com; spf=pass smtp.mailfrom=collabora.com; dkim=pass (2048-bit key) header.d=collabora.com header.i=@collabora.com header.b=chbuNCcy; arc=none smtp.client-ip=148.251.105.195 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=collabora.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=collabora.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=collabora.com header.i=@collabora.com header.b="chbuNCcy" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=collabora.com; s=mail; t=1777535860; bh=XchW9YUc1tPO3prYku284a4ISpGs3mRYztaK2Z7JK0Y=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=chbuNCcyuadbRUOjl8jsNrd+P8bELhJEnpS+PqIJUfeepqy+PwmE6CcxEHAqtfiQq 6Ly+nhbj9eKRyBHIpmZp78ulr7l9Bz2GKcbRdp0vAsILvEVag0AxhxKnEQzGaNXZEY Ps7UI+NU5fcSltlQkERWG4mywYKMmJwvByPdtpO5JH+YmpZ9bTjPLqfgMcSC7FybJ0 WrFifLWMUNdxdnE+w8Q5xGUM/5UVGmufWMRqCK4nwXoQUO75x7uANtmyy8CBhSwDLA KKpUxHVfOSrHasslD/00K1A4DoHnL/nTnFUczs8ComMBNo8xGIXPEjBFyv1XUFFOtH psrCwuGXOh8CQ== Received: from fedora (unknown [100.64.0.11]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange ECDHE (prime256v1) server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) (Authenticated sender: bbrezillon) by bali.collaboradmins.com (Postfix) with ESMTPSA id ACA8317E13D6; Thu, 30 Apr 2026 09:57:39 +0200 (CEST) Date: Thu, 30 Apr 2026 09:57:34 +0200 From: Boris Brezillon To: =?UTF-8?B?QWRyacOhbg==?= Larumbe Cc: linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, Steven Price , kernel@collabora.com, Liviu Dudau , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie , Simona Vetter , Daniel Almeida , Alice Ryhl Subject: Re: [PATCH v10 5/6] drm/panthor: Support sparse mappings Message-ID: <20260430095734.28bc98cf@fedora> In-Reply-To: <20260429183253.66422-6-adrian.larumbe@collabora.com> References: <20260429183253.66422-1-adrian.larumbe@collabora.com> <20260429183253.66422-6-adrian.larumbe@collabora.com> Organization: Collabora X-Mailer: Claws Mail 4.4.0 (GTK 3.24.52; x86_64-redhat-linux-gnu) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable On Wed, 29 Apr 2026 19:32:17 +0100 Adri=C3=A1n Larumbe wrote: > Allow UM to bind sparsely populated memory regions by cyclically mapping > virtual ranges over a kernel-allocated dummy BO. This alternative is > preferable to the old method of handling sparseness in the UMD, because it > relied on the creation of a buffer object to the same end, despite the fa= ct > Vulkan sparse resources don't need to be backed by a driver BO. >=20 > The choice of backing sparsely-bound regions with a Panhtor BO was made so > as to profit from the existing shrinker reclaim code. That way no special > treatment must be given to the dummy sparse BOs when reclaiming memory, as > would be the case if we had chosen a raw kernel page implementation. >=20 > A new dummy BO is allocated per open file context, because even though the > Vulkan spec mandates that writes into sparsely bound regions must be > discarded, our implementation is still a workaround over the fact Mali CSF > GPUs cannot support this behaviour on the hardware level, so writes still > make it into the backing BO. If we had a global one, then it could be a > venue for information leaks between file contexts, which should never > happen in DRM. >=20 > Reviewed-by: Boris Brezillon > Signed-off-by: Adri=C3=A1n Larumbe > --- > drivers/gpu/drm/panthor/panthor_gem.c | 18 +++ > drivers/gpu/drm/panthor/panthor_gem.h | 2 + > drivers/gpu/drm/panthor/panthor_mmu.c | 159 ++++++++++++++++++++++---- > include/uapi/drm/panthor_drm.h | 12 ++ > 4 files changed, 170 insertions(+), 21 deletions(-) >=20 > diff --git a/drivers/gpu/drm/panthor/panthor_gem.c b/drivers/gpu/drm/pant= hor/panthor_gem.c > index 13295d7a593d..c798ac2963e1 100644 > --- a/drivers/gpu/drm/panthor/panthor_gem.c > +++ b/drivers/gpu/drm/panthor/panthor_gem.c > @@ -1345,6 +1345,24 @@ panthor_kernel_bo_create(struct panthor_device *pt= dev, struct panthor_vm *vm, > return ERR_PTR(ret); > } > =20 > +/** > + * panthor_dummy_bo_create() - Create a Panthor BO meant to back sparse = bindings. > + * @ptdev: Device. > + * > + * Return: A valid pointer in case of success, an ERR_PTR() otherwise. > + */ > +struct panthor_gem_object * > +panthor_dummy_bo_create(struct panthor_device *ptdev) > +{ > + /* Since even when the DRM device's mount point has enabled THP we have= no guarantee > + * that drm_gem_get_pages() will return a single 2MiB PMD, and also we = cannot be sure > + * that the 2MiB won't be reclaimed and re-allocated later on as 4KiB c= hunks, it doesn't > + * make sense to pre-populate this object's page array, nor to fall bac= k on a BO size > + * of 4KiB. Sticking to a dummy object size of 2MiB lets us keep things= simple for now. > + */ > + return panthor_gem_create(&ptdev->base, SZ_2M, DRM_PANTHOR_BO_NO_MMAP, = NULL, 0); > +} > + > static bool can_swap(void) > { > return get_nr_swap_pages() > 0; > diff --git a/drivers/gpu/drm/panthor/panthor_gem.h b/drivers/gpu/drm/pant= hor/panthor_gem.h > index ae0491d0b121..8639c2fa08e6 100644 > --- a/drivers/gpu/drm/panthor/panthor_gem.h > +++ b/drivers/gpu/drm/panthor/panthor_gem.h > @@ -315,6 +315,8 @@ panthor_kernel_bo_create(struct panthor_device *ptdev= , struct panthor_vm *vm, > =20 > void panthor_kernel_bo_destroy(struct panthor_kernel_bo *bo); > =20 > +struct panthor_gem_object *panthor_dummy_bo_create(struct panthor_device= *ptdev); > + > #ifdef CONFIG_DEBUG_FS > void panthor_gem_debugfs_init(struct drm_minor *minor); > #endif > diff --git a/drivers/gpu/drm/panthor/panthor_mmu.c b/drivers/gpu/drm/pant= hor/panthor_mmu.c > index f54a60cd0ec4..9257afd6adc9 100644 > --- a/drivers/gpu/drm/panthor/panthor_mmu.c > +++ b/drivers/gpu/drm/panthor/panthor_mmu.c > @@ -112,6 +112,17 @@ struct panthor_mmu { > struct panthor_vm_pool { > /** @xa: Array used for VM handle tracking. */ > struct xarray xa; > + > + /** > + * @dummy: Dummy object used for sparse mappings > + * > + * Sparse bindings map virtual address ranges onto a dummy > + * BO in a modulo fashion. Even though sparse writes are meant > + * to be discarded and reads undefined, writes are still reflected > + * in the dummy buffer. That means we must keep a dummy object per > + * file context, to avoid data leaks between them. > + */ > + struct panthor_gem_object *dummy; > }; > =20 > /** > @@ -391,6 +402,16 @@ struct panthor_vm { > */ > struct list_head lru_node; > } reclaim; > + > + /** > + * @dummy: Dummy object used for sparse mappings. > + * > + * VM's must keep a reference to the file context-wide dummy BO because > + * they can outlive the file context, which includes the VM pool holding > + * the original dummy BO reference. > + * nit: Drop the extra blank line. > + */ > + struct panthor_gem_object *dummy; > }; > =20 > /** > @@ -1020,6 +1041,30 @@ panthor_vm_map_pages(struct panthor_vm *vm, u64 io= va, int prot, > return 0; > } > =20 > +static int > +panthor_vm_map_sparse(struct panthor_vm *vm, u64 iova, int prot, > + struct sg_table *sgt, u64 size) > +{ > + u64 mapped =3D 0; > + int ret; > + > + while (mapped < size) { > + u64 addr =3D iova + mapped; > + u32 chunk_size =3D min(size - mapped, SZ_2M - (addr & (SZ_2M - 1))); > + > + ret =3D panthor_vm_map_pages(vm, addr, prot, > + sgt, 0, chunk_size); > + if (ret) { > + panthor_vm_unmap_pages(vm, iova, mapped); > + return ret; > + } > + > + mapped +=3D chunk_size; > + } > + > + return 0; > +} > + > static int flags_to_prot(u32 flags) > { > int prot =3D 0; > @@ -1262,6 +1307,7 @@ static int panthor_vm_op_ctx_prealloc_pts(struct pa= nthor_vm_op_ctx *op_ctx) > (DRM_PANTHOR_VM_BIND_OP_MAP_READONLY | \ > DRM_PANTHOR_VM_BIND_OP_MAP_NOEXEC | \ > DRM_PANTHOR_VM_BIND_OP_MAP_UNCACHED | \ > + DRM_PANTHOR_VM_BIND_OP_MAP_SPARSE | \ > DRM_PANTHOR_VM_BIND_OP_TYPE_MASK) > =20 > static int panthor_vm_prepare_map_op_ctx(struct panthor_vm_op_ctx *op_ct= x, > @@ -1269,6 +1315,7 @@ static int panthor_vm_prepare_map_op_ctx(struct pan= thor_vm_op_ctx *op_ctx, > struct panthor_gem_object *bo, > const struct drm_panthor_vm_bind_op *op) > { > + bool is_sparse =3D op->flags & DRM_PANTHOR_VM_BIND_OP_MAP_SPARSE; > struct drm_gpuvm_bo *preallocated_vm_bo; > struct sg_table *sgt =3D NULL; > int ret; > @@ -1280,8 +1327,21 @@ static int panthor_vm_prepare_map_op_ctx(struct pa= nthor_vm_op_ctx *op_ctx, > (op->flags & DRM_PANTHOR_VM_BIND_OP_TYPE_MASK) !=3D DRM_PANTHOR_VM_= BIND_OP_TYPE_MAP) > return -EINVAL; > =20 > - /* Make sure the VA and size are in-bounds. */ > - if (op->size > bo->base.size || op->bo_offset > bo->base.size - op->siz= e) > + /* uAPI mandates sparsely bound regions must not be executable. */ > + if (is_sparse && !(op->flags & DRM_PANTHOR_VM_BIND_OP_MAP_NOEXEC)) > + return -EINVAL; > + > + /* For non-sparse, make sure the VA and size are in-bounds. > + * For sparse, this is not applicable, because the dummy BO is > + * repeatedly mapped over a potentially wider VA range. > + */ > + if (!is_sparse && (op->size > bo->base.size || op->bo_offset > bo->base= .size - op->size)) > + return -EINVAL; > + > + /* For sparse, we don't expect any user BO, the BO we get passed > + * is the dummy BO attached to the VM pool. > + */ > + if (is_sparse && (op->bo_handle || op->bo_offset)) > return -EINVAL; > =20 > /* If the BO has an exclusive VM attached, it can't be mapped to other = VMs. */ > @@ -1543,6 +1603,9 @@ int panthor_vm_pool_create_vm(struct panthor_device= *ptdev, > return ret; > } > =20 > + drm_gem_object_get(&pool->dummy->base); > + vm->dummy =3D pool->dummy; > + > args->user_va_range =3D kernel_va_start; > return id; > } > @@ -1634,6 +1697,7 @@ void panthor_vm_pool_destroy(struct panthor_file *p= file) > xa_for_each(&pfile->vms->xa, i, vm) > panthor_vm_destroy(vm); > =20 > + drm_gem_object_put(&pfile->vms->dummy->base); > xa_destroy(&pfile->vms->xa); > kfree(pfile->vms); > } > @@ -1651,6 +1715,13 @@ int panthor_vm_pool_create(struct panthor_file *pf= ile) > return -ENOMEM; > =20 > xa_init_flags(&pfile->vms->xa, XA_FLAGS_ALLOC1); > + > + pfile->vms->dummy =3D panthor_dummy_bo_create(pfile->ptdev); > + if (IS_ERR(pfile->vms->dummy)) { > + kfree(pfile->vms); > + return PTR_ERR(pfile->vms->dummy); > + } > + > return 0; > } > =20 > @@ -1987,6 +2058,9 @@ static void panthor_vm_free(struct drm_gpuvm *gpuvm) > =20 > free_io_pgtable_ops(vm->pgtbl_ops); > =20 > + if (vm->dummy) > + drm_gem_object_put(&vm->dummy->base); > + > drm_mm_takedown(&vm->mm); > kfree(vm); > } > @@ -2146,7 +2220,23 @@ static void panthor_vma_init(struct panthor_vma *v= ma, u32 flags) > #define PANTHOR_VM_MAP_FLAGS \ > (DRM_PANTHOR_VM_BIND_OP_MAP_READONLY | \ > DRM_PANTHOR_VM_BIND_OP_MAP_NOEXEC | \ > - DRM_PANTHOR_VM_BIND_OP_MAP_UNCACHED) > + DRM_PANTHOR_VM_BIND_OP_MAP_UNCACHED | \ > + DRM_PANTHOR_VM_BIND_OP_MAP_SPARSE) > + > +static int > +panthor_vm_exec_map_op(struct panthor_vm *vm, u32 flags, > + const struct drm_gpuva_op_map *op) > +{ > + struct panthor_gem_object *bo =3D to_panthor_bo(op->gem.obj); > + int prot =3D flags_to_prot(flags); > + > + if (flags & DRM_PANTHOR_VM_BIND_OP_MAP_SPARSE) > + return panthor_vm_map_sparse(vm, op->va.addr, prot, > + bo->dmap.sgt, op->va.range); > + > + return panthor_vm_map_pages(vm, op->va.addr, prot, bo->dmap.sgt, > + op->gem.offset, op->va.range); > +} > =20 > static int panthor_gpuva_sm_step_map(struct drm_gpuva_op *op, void *priv) > { > @@ -2160,9 +2250,7 @@ static int panthor_gpuva_sm_step_map(struct drm_gpu= va_op *op, void *priv) > =20 > panthor_vma_init(vma, op_ctx->flags & PANTHOR_VM_MAP_FLAGS); > =20 > - ret =3D panthor_vm_map_pages(vm, op->map.va.addr, flags_to_prot(vma->fl= ags), > - op_ctx->map.bo->dmap.sgt, op->map.gem.offset, > - op->map.va.range); > + ret =3D panthor_vm_exec_map_op(vm, vma->flags, &op->map); > if (ret) { > panthor_vm_op_ctx_return_vma(op_ctx, vma); > return ret; > @@ -2178,13 +2266,16 @@ static int panthor_gpuva_sm_step_map(struct drm_g= puva_op *op, void *priv) > } > =20 > static bool > -iova_mapped_as_huge_page(struct drm_gpuva_op_map *op, u64 addr) > +iova_mapped_as_huge_page(struct drm_gpuva_op_map *op, u64 addr, bool is_= sparse) > { > struct panthor_gem_object *bo =3D to_panthor_bo(op->gem.obj); > const struct page *pg; > pgoff_t bo_offset; > =20 > - bo_offset =3D addr - op->va.addr + op->gem.offset; > + /* Per-VM Dummy BO in sparse mappings is always 2MiB, so checking the > + * size of the very first page is enough. > + */ > + bo_offset =3D !is_sparse ? addr - op->va.addr + op->gem.offset : 0; > pg =3D bo->backing.pages[bo_offset >> PAGE_SHIFT]; > =20 > return folio_size(page_folio(pg)) >=3D SZ_2M; > @@ -2194,6 +2285,8 @@ static void > unmap_hugepage_align(const struct drm_gpuva_op_remap *op, > u64 *unmap_start, u64 *unmap_range) > { > + struct panthor_vma *unmap_vma =3D container_of(op->unmap->va, struct pa= nthor_vma, base); > + bool is_sparse =3D unmap_vma->flags & DRM_PANTHOR_VM_BIND_OP_MAP_SPARSE; > u64 aligned_unmap_start, aligned_unmap_end, unmap_end; > =20 > unmap_end =3D *unmap_start + *unmap_range; > @@ -2205,7 +2298,7 @@ unmap_hugepage_align(const struct drm_gpuva_op_rema= p *op, > */ > if (op->prev && aligned_unmap_start < *unmap_start && > op->prev->va.addr <=3D aligned_unmap_start && > - iova_mapped_as_huge_page(op->prev, *unmap_start)) { > + (iova_mapped_as_huge_page(op->prev, *unmap_start, is_sparse))) { Actually, I think this could be: =20 (is_sparse || iova_mapped_as_huge_page(op->prev, *unmap_start)) { such that we always end up with sparse mappings starting at offset=3D0 on the dummy GEM, even if those mappings are not 2M aligned (see below for more reasons to keep this thing accurate). > *unmap_range +=3D *unmap_start - aligned_unmap_start; > *unmap_start =3D aligned_unmap_start; > } > @@ -2215,7 +2308,7 @@ unmap_hugepage_align(const struct drm_gpuva_op_rema= p *op, > */ > if (op->next && aligned_unmap_end > unmap_end && > op->next->va.addr + op->next->va.range >=3D aligned_unmap_end && > - iova_mapped_as_huge_page(op->next, unmap_end - 1)) { > + iova_mapped_as_huge_page(op->next, unmap_end - 1, is_sparse)) { > *unmap_range +=3D aligned_unmap_end - unmap_end; > } > } > @@ -2250,15 +2343,27 @@ static int panthor_gpuva_sm_step_remap(struct drm= _gpuva_op *op, > panthor_vm_unmap_pages(vm, unmap_start, unmap_range); > } > =20 > + /* In the following two branches, neither remap::unmap::offset nor rema= p::unmap::keep > + * can be trusted to contain legitimate values in the case of sparse ma= ppings, because > + * the drm_gpuvm core calculates them on the assumption that a VM_BIND = operation's > + * range is always less than the target BO. That doesn't hold in the ca= se of sparse > + * bindings, but we don't care to adjust the BO offset of new VA's spaw= ned by a remap > + * operation because we ignore them altogether when sparse-mapping page= s on a HW level > + * just further below. I think I'd prefer if we were setting gem.offset to zero when we're dealing with a sparse remap, even if it doesn't matter in practice because it will be ignored in case of sparse. > * If we ever wanted to make use of remap::unmap::keep, then this > + * logic would have to be reworked. > + */ > if (op->remap.prev) { > - struct panthor_gem_object *bo =3D to_panthor_bo(op->remap.prev->gem.ob= j); > u64 offset =3D op->remap.prev->gem.offset + unmap_start - op->remap.pr= ev->va.addr; > u64 size =3D op->remap.prev->va.addr + op->remap.prev->va.range - unma= p_start; > + const struct drm_gpuva_op_map map_op =3D { > + .va.addr =3D unmap_start, > + .va.range =3D size, > + .gem.obj =3D op->remap.prev->gem.obj, > + .gem.offset =3D offset, That one doesn't matter much, but for consistency, I'd go: .gem.offset =3D is_sparse ? 0 : offset, and maybe add a WARN_ON() in panthor_vm_exec_map_op() if this is a SPARSE request and the offset is not zero. > + }; > =20 > - if (!unmap_vma->evicted) { > - ret =3D panthor_vm_map_pages(vm, unmap_start, > - flags_to_prot(unmap_vma->flags), > - bo->dmap.sgt, offset, size); > + if (!unmap_vma->evicted && size > 0) { > + ret =3D panthor_vm_exec_map_op(vm, unmap_vma->flags, &map_op); > if (ret) > return ret; > } > @@ -2269,14 +2374,17 @@ static int panthor_gpuva_sm_step_remap(struct drm= _gpuva_op *op, > } > =20 > if (op->remap.next) { > - struct panthor_gem_object *bo =3D to_panthor_bo(op->remap.next->gem.ob= j); > u64 addr =3D op->remap.next->va.addr; > u64 size =3D unmap_start + unmap_range - op->remap.next->va.addr; > + const struct drm_gpuva_op_map map_op =3D { > + .va.addr =3D addr, > + .va.range =3D size, > + .gem.obj =3D op->remap.next->gem.obj, > + .gem.offset =3D op->remap.next->gem.offset, If this is a sparse mapping, the original mapping offset should already be zero, so I guess we can pass it directly, and let panthor_vm_exec_map_op() complain if it's not zero. > + }; > =20 But more importantly, I think we should if (is_sparse) op->remap.next->gem.offset =3D 0; such that the remapped VA appears with a zero offset in the debugfs file exposed by GPUVM. This makes me realize we should prevent panthor_vm_get_bo_for_va() from returning a valid pair for a sparse VA, or at the very least, make it so the offset is masked with (SZ_2M - 1). > - if (!unmap_vma->evicted) { > - ret =3D panthor_vm_map_pages(vm, addr, flags_to_prot(unmap_vma->flags= ), > - bo->dmap.sgt, op->remap.next->gem.offset, > - size); > + if (!unmap_vma->evicted && size > 0) { > + ret =3D panthor_vm_exec_map_op(vm, unmap_vma->flags, &map_op); > if (ret) > return ret; > } > @@ -2835,7 +2943,13 @@ panthor_vm_bind_prepare_op_ctx(struct drm_file *fi= le, > =20 > switch (op->flags & DRM_PANTHOR_VM_BIND_OP_TYPE_MASK) { > case DRM_PANTHOR_VM_BIND_OP_TYPE_MAP: > - gem =3D drm_gem_object_lookup(file, op->bo_handle); > + if (!(op->flags & DRM_PANTHOR_VM_BIND_OP_MAP_SPARSE)) { > + gem =3D drm_gem_object_lookup(file, op->bo_handle); > + } else { > + gem =3D &vm->dummy->base; > + drm_gem_object_get(&vm->dummy->base); > + } > + > ret =3D panthor_vm_prepare_map_op_ctx(op_ctx, vm, > gem ? to_panthor_bo(gem) : NULL, > op); > @@ -3043,6 +3157,9 @@ int panthor_vm_map_bo_range(struct panthor_vm *vm, = struct panthor_gem_object *bo > struct panthor_vm_op_ctx op_ctx; > int ret; > =20 > + if (drm_WARN_ON(&vm->ptdev->base, flags & DRM_PANTHOR_VM_BIND_OP_MAP_SP= ARSE)) > + return -EINVAL; > + > ret =3D panthor_vm_prepare_map_op_ctx(&op_ctx, vm, bo, &op); > if (ret) > return ret; > diff --git a/include/uapi/drm/panthor_drm.h b/include/uapi/drm/panthor_dr= m.h > index 14a93a4ef6ff..1b706d00b0a1 100644 > --- a/include/uapi/drm/panthor_drm.h > +++ b/include/uapi/drm/panthor_drm.h > @@ -614,6 +614,18 @@ enum drm_panthor_vm_bind_op_flags { > */ > DRM_PANTHOR_VM_BIND_OP_MAP_UNCACHED =3D 1 << 2, > =20 > + /** > + * @DRM_PANTHOR_VM_BIND_OP_MAP_SPARSE: Sparsely map a virtual memory ra= nge > + * > + * Only valid with DRM_PANTHOR_VM_BIND_OP_TYPE_MAP. > + * > + * When this flag is set, the whole vm_bind range is mapped over a dumm= y object in a cyclic > + * fashion, and all GPU reads from addresses in the range return undefi= ned values. This flag > + * being set means drm_panthor_vm_bind_op:offset and drm_panthor_vm_bin= d_op::handle must > + * both be set to 0. DRM_PANTHOR_VM_BIND_OP_MAP_NOEXEC must also be set. > + */ > + DRM_PANTHOR_VM_BIND_OP_MAP_SPARSE =3D 1 << 3, > + > /** > * @DRM_PANTHOR_VM_BIND_OP_TYPE_MASK: Mask used to determine the type o= f operation. > */