From nobody Sat Jun 13 11:23:26 2026 Received: from sender4-pp-f112.zoho.com (sender4-pp-f112.zoho.com [136.143.188.112]) (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 3DD7E347536 for ; Thu, 7 May 2026 21:50:10 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=pass smtp.client-ip=136.143.188.112 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778190611; cv=pass; b=oJtM4GtIMzM09YPAKtAWvuOSz33lwiIsggVBxxZSekzGIFVflGB5ghBRrRUi8uLkB3MGbgnHhJOXzhnfPv3uIMzTcQw3uTb9TjYkus6ZHaNdDrZY20fxkuVvCbgJwCsfy6feeAHK/WWL9xDgwxDPIzbSRBrewQkuSLryKMj3fbU= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778190611; c=relaxed/simple; bh=3YAMIS4FUapbjt/AjxXz37J8XtdAOhbo3GyXODAk2ns=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=hC6Xloqxian9FcWJ7N2WjbWQ0gWGiERS1C0l3nU0W9ccQtRRWvd75MqhdfnvjLrH9YVmXeWcuAyFJEX4MfKwDRpnhzwX2Ny2Vg9iIJBJOlfIQZXkwjLVmjhdp63IMPEbV8VEhzBltJxwgjtZUnRYQW7tRmGn7ANkUCMOFubdVww= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=collabora.com; spf=pass smtp.mailfrom=collabora.com; dkim=pass (1024-bit key) header.d=collabora.com header.i=adrian.larumbe@collabora.com header.b=CsbA6jXk; arc=pass smtp.client-ip=136.143.188.112 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 (1024-bit key) header.d=collabora.com header.i=adrian.larumbe@collabora.com header.b="CsbA6jXk" ARC-Seal: i=1; a=rsa-sha256; t=1778190593; cv=none; d=zohomail.com; s=zohoarc; b=RkpU5a0blkHD1o62DEU4Ua9BVAKNmbpdPmcSinkYbmAkkvosOCvs8qGhzWqoRGHWj5ei1UNnhdaOQZHi559x/DZIp8ysP2BU18zJTysIAsxTCJJst1Hr6ZFMUCtvfSBvDfkdN0uWA75is7S/CHadzi43bytpECrWnDSWy1M20Gg= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1778190593; h=Content-Type:Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:In-Reply-To:MIME-Version:Message-ID:References:Subject:Subject:To:To:Message-Id:Reply-To; bh=Yn+lNA4MFHmNERMplaWuzXxuMrhoQvQYCwIda7hIGOQ=; b=K961gvbrswS0JCRv4VJcnPc/Py+Q6b1mPD2J3LeREdvZSFKr+8RRpMJCF6pAMdFfkQZRHoLJViV2TM0xmq9OHra+qsqLBsAC/WLjPryrGtJbc8eEjVL5QIDY/S0uHH8QSqRugO06HOaVdPG/fyqWgHp5ZiP0LWDEwP3YttAVHsE= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass header.i=collabora.com; spf=pass smtp.mailfrom=adrian.larumbe@collabora.com; dmarc=pass header.from= DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; t=1778190593; s=zohomail; d=collabora.com; i=adrian.larumbe@collabora.com; h=From:From:To:To:Cc:Cc:Subject:Subject:Date:Date:Message-ID:In-Reply-To:References:MIME-Version:Content-Type:Content-Transfer-Encoding:Message-Id:Reply-To; bh=Yn+lNA4MFHmNERMplaWuzXxuMrhoQvQYCwIda7hIGOQ=; b=CsbA6jXk1l6yNuHEDJjq5ahkwKtjlOn1QIeWD0MWvMtAbOA7irREbEdxXzFJlh3U MowfZVVBNBgGlk/+i6pnNBSustZMZBA15gtmHVv4ry8LptbdBBmKvwY9I+0RoUklsDz zqvHx0i3WFRG/CNzhbkFApFmBa+cuFTmcntu3ngo= Received: by mx.zohomail.com with SMTPS id 1778190593102994.9072727823724; Thu, 7 May 2026 14:49:53 -0700 (PDT) From: =?UTF-8?q?Adri=C3=A1n=20Larumbe?= To: linux-kernel@vger.kernel.org Cc: dri-devel@lists.freedesktop.org, Steven Price , Boris Brezillon , kernel@collabora.com, =?UTF-8?q?Adri=C3=A1n=20Larumbe?= , Liviu Dudau , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie , Simona Vetter , Daniel Almeida , Alice Ryhl Subject: [PATCH v11 1/6] drm/panthor: Expose GPU page sizes to UM Date: Thu, 7 May 2026 22:49:02 +0100 Message-ID: <20260507214939.2852489-2-adrian.larumbe@collabora.com> X-Mailer: git-send-email 2.53.0 In-Reply-To: <20260507214939.2852489-1-adrian.larumbe@collabora.com> References: <20260507214939.2852489-1-adrian.larumbe@collabora.com> 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 In future commits that will implement repeated mappings, only repeat values multiple of GPU page sizes will be tolerated. That means these values must be made known to UM. Do it through a queriable GPU info value. Reviewed-by: Steven Price Reviewed-by: Boris Brezillon Signed-off-by: Adri=C3=A1n Larumbe --- drivers/gpu/drm/panthor/panthor_device.h | 3 +++ drivers/gpu/drm/panthor/panthor_drv.c | 8 ++++++++ drivers/gpu/drm/panthor/panthor_mmu.c | 9 ++++++++- include/uapi/drm/panthor_drm.h | 13 +++++++++++++ 4 files changed, 32 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/panthor/panthor_device.h b/drivers/gpu/drm/pan= thor/panthor_device.h index 5cba272f9b4d..d856a4fe1d61 100644 --- a/drivers/gpu/drm/panthor/panthor_device.h +++ b/drivers/gpu/drm/panthor/panthor_device.h @@ -158,6 +158,9 @@ struct panthor_device { /** @csif_info: Command stream interface information. */ struct drm_panthor_csif_info csif_info; =20 + /** @mmu_info: MMU info */ + struct drm_panthor_mmu_info mmu_info; + /** @hw: GPU-specific data. */ struct panthor_hw *hw; =20 diff --git a/drivers/gpu/drm/panthor/panthor_drv.c b/drivers/gpu/drm/pantho= r/panthor_drv.c index 73fc983dc9b4..beca75b34293 100644 --- a/drivers/gpu/drm/panthor/panthor_drv.c +++ b/drivers/gpu/drm/panthor/panthor_drv.c @@ -175,6 +175,7 @@ panthor_get_uobj_array(const struct drm_panthor_obj_arr= ay *in, u32 min_stride, _Generic(_obj_name, \ PANTHOR_UOBJ_DECL(struct drm_panthor_gpu_info, tiler_present), \ PANTHOR_UOBJ_DECL(struct drm_panthor_csif_info, pad), \ + PANTHOR_UOBJ_DECL(struct drm_panthor_mmu_info, page_size_bitmap), \ PANTHOR_UOBJ_DECL(struct drm_panthor_timestamp_info, current_timestamp)= , \ PANTHOR_UOBJ_DECL(struct drm_panthor_group_priorities_info, pad), \ PANTHOR_UOBJ_DECL(struct drm_panthor_sync_op, timeline_value), \ @@ -954,6 +955,10 @@ static int panthor_ioctl_dev_query(struct drm_device *= ddev, void *data, struct d args->size =3D sizeof(priorities_info); return 0; =20 + case DRM_PANTHOR_DEV_QUERY_MMU_INFO: + args->size =3D sizeof(ptdev->mmu_info); + return 0; + default: return -EINVAL; } @@ -984,6 +989,9 @@ static int panthor_ioctl_dev_query(struct drm_device *d= dev, void *data, struct d panthor_query_group_priorities_info(file, &priorities_info); return PANTHOR_UOBJ_SET(args->pointer, args->size, priorities_info); =20 + case DRM_PANTHOR_DEV_QUERY_MMU_INFO: + return PANTHOR_UOBJ_SET(args->pointer, args->size, ptdev->mmu_info); + default: return -EINVAL; } diff --git a/drivers/gpu/drm/panthor/panthor_mmu.c b/drivers/gpu/drm/pantho= r/panthor_mmu.c index fc930ee158a5..9b526b61d96d 100644 --- a/drivers/gpu/drm/panthor/panthor_mmu.c +++ b/drivers/gpu/drm/panthor/panthor_mmu.c @@ -2768,7 +2768,7 @@ panthor_vm_create(struct panthor_device *ptdev, bool = for_mcu, refcount_set(&vm->as.active_cnt, 0); =20 pgtbl_cfg =3D (struct io_pgtable_cfg) { - .pgsize_bitmap =3D SZ_4K | SZ_2M, + .pgsize_bitmap =3D ptdev->mmu_info.page_size_bitmap, .ias =3D va_bits, .oas =3D pa_bits, .coherent_walk =3D ptdev->coherent, @@ -3213,6 +3213,11 @@ static void panthor_mmu_release_wq(struct drm_device= *ddev, void *res) destroy_workqueue(res); } =20 +static void panthor_mmu_info_init(struct panthor_device *ptdev) +{ + ptdev->mmu_info.page_size_bitmap =3D SZ_4K | SZ_2M; +} + /** * panthor_mmu_init() - Initialize the MMU logic. * @ptdev: Device. @@ -3225,6 +3230,8 @@ int panthor_mmu_init(struct panthor_device *ptdev) struct panthor_mmu *mmu; int ret, irq; =20 + panthor_mmu_info_init(ptdev); + mmu =3D drmm_kzalloc(&ptdev->base, sizeof(*mmu), GFP_KERNEL); if (!mmu) return -ENOMEM; diff --git a/include/uapi/drm/panthor_drm.h b/include/uapi/drm/panthor_drm.h index 0e455d91e77d..b462752c793d 100644 --- a/include/uapi/drm/panthor_drm.h +++ b/include/uapi/drm/panthor_drm.h @@ -253,6 +253,9 @@ enum drm_panthor_dev_query_type { * @DRM_PANTHOR_DEV_QUERY_GROUP_PRIORITIES_INFO: Query allowed group prio= rities information. */ DRM_PANTHOR_DEV_QUERY_GROUP_PRIORITIES_INFO, + + /** @DRM_PANTHOR_DEV_QUERY_MMU_INFO: Query MMU information. */ + DRM_PANTHOR_DEV_QUERY_MMU_INFO, }; =20 /** @@ -487,6 +490,16 @@ struct drm_panthor_timestamp_info { __u64 cpu_timestamp_nsec; }; =20 +/** + * struct drm_panthor_mmu_info - MMU information + * + * Structure grouping all queryable information relating to the MMU. + */ +struct drm_panthor_mmu_info { + /** @page_size_bitmap: Allowed page sizes */ + __u64 page_size_bitmap; +}; + /** * struct drm_panthor_group_priorities_info - Group priorities information * --=20 2.53.0 From nobody Sat Jun 13 11:23:26 2026 Received: from sender4-pp-f112.zoho.com (sender4-pp-f112.zoho.com [136.143.188.112]) (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 29D0934D3B9 for ; Thu, 7 May 2026 21:50:12 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=pass smtp.client-ip=136.143.188.112 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778190613; cv=pass; b=NDCKgm1FPXioy9To96Z2FHt+ZRJDmB3IZPMPv5s/Tbr+31GAW1xsBGA30FJRUW6N5Cc07hiUfURgYSZjAzvsXt+lg+Yly/lDifgg8VuHin7KD35EXBOJuiXqQWmb+icIzL1ZxPYiKi/eDwcV/QyewiHQ/iPXDYZsIgmTQhldNMA= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778190613; c=relaxed/simple; bh=fIRERsnpk7YVLUQx4qf9QFK4wmJqaWgHkyLUDnzUTP8=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=boKt0cxzRe9c5oEKik3p1OH9oOu02iuwOuhBsObCLX1LwzTQ8Rx0zYLmc2R94vCibrA+ofp5uj6Y4Pnp+2whB2iBWpRXhTUTI7D0dhdkoBfM2k3BHJ1zrIM+zLhyW+OO0HHLHif9gOnbld2V02eQlRwlrzeT/h8yoQ+I4rTnVP8= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=collabora.com; spf=pass smtp.mailfrom=collabora.com; dkim=pass (1024-bit key) header.d=collabora.com header.i=adrian.larumbe@collabora.com header.b=ZLkSIoHZ; arc=pass smtp.client-ip=136.143.188.112 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 (1024-bit key) header.d=collabora.com header.i=adrian.larumbe@collabora.com header.b="ZLkSIoHZ" ARC-Seal: i=1; a=rsa-sha256; t=1778190597; cv=none; d=zohomail.com; s=zohoarc; b=Y1UgryMgjZWceW+IBggL2oiPUOEWvK8S/n3CqI+DuyQ9xkFIcOBHJyeCxz3mKfGwCZP1FWMnrTsMyegnCZ8BIvHRRsAefnrcVISmPXKBJmuikU6TPlwIluUwlacX0MHGGWA+vYVBzmx5hT+mywdzddokM9wDgCOUwznvFoQ0R5s= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1778190597; h=Content-Type:Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:In-Reply-To:MIME-Version:Message-ID:References:Subject:Subject:To:To:Message-Id:Reply-To; bh=TDrcB9ymQKzDhvp0iWYAQfC2xlKOmF7TeTcK83C1p1Q=; b=VbnizLmGP4uHHH3sxftmhGIBKMfyjeXtGK+jIz91OOgqCSzGKRINnCy83zUEV1BQARIabvYVGqUPHTKNpNzPAPOaiz+exQeS72FXDuU8xKW8vJt3bxGlaCIbxuEnB9mAGzs5ua4Ke27W+4FJPjcRNe5GVxHd9M5AvK1pvchb/vI= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass header.i=collabora.com; spf=pass smtp.mailfrom=adrian.larumbe@collabora.com; dmarc=pass header.from= DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; t=1778190597; s=zohomail; d=collabora.com; i=adrian.larumbe@collabora.com; h=From:From:To:To:Cc:Cc:Subject:Subject:Date:Date:Message-ID:In-Reply-To:References:MIME-Version:Content-Type:Content-Transfer-Encoding:Message-Id:Reply-To; bh=TDrcB9ymQKzDhvp0iWYAQfC2xlKOmF7TeTcK83C1p1Q=; b=ZLkSIoHZn8CTTXh51wEPMFSXOdNU80cV2umgpNQwASoafobKv0UJNJ6r6W9peYqP GtNRYsS9YDOsrLhzz4elH7Nm1cix7A4Ipp0jJBSiZGuwebNzbnIO6ar+BoF4LcnXjLA 9zVpqWsge6rf639fVHFRSoo5Q6jCxyaPYqj8v8so= Received: by mx.zohomail.com with SMTPS id 1778190596246294.4669231868851; Thu, 7 May 2026 14:49:56 -0700 (PDT) From: =?UTF-8?q?Adri=C3=A1n=20Larumbe?= To: linux-kernel@vger.kernel.org Cc: dri-devel@lists.freedesktop.org, Steven Price , Boris Brezillon , kernel@collabora.com, =?UTF-8?q?Adri=C3=A1n=20Larumbe?= , Liviu Dudau , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie , Simona Vetter Subject: [PATCH v11 2/6] drm/panthor: Pass vm_bind_op to vm_prepare_map_op_ctx Date: Thu, 7 May 2026 22:49:03 +0100 Message-ID: <20260507214939.2852489-3-adrian.larumbe@collabora.com> X-Mailer: git-send-email 2.53.0 In-Reply-To: <20260507214939.2852489-1-adrian.larumbe@collabora.com> References: <20260507214939.2852489-1-adrian.larumbe@collabora.com> 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 Instead of passing its constituent elements, pass the whole struct to simplify the function prototype. Reviewed-by: Steven Price Reviewed-by: Boris Brezillon Signed-off-by: Adri=C3=A1n Larumbe --- drivers/gpu/drm/panthor/panthor_mmu.c | 27 ++++++++++++++------------- 1 file changed, 14 insertions(+), 13 deletions(-) diff --git a/drivers/gpu/drm/panthor/panthor_mmu.c b/drivers/gpu/drm/pantho= r/panthor_mmu.c index 9b526b61d96d..d35e65cab2b1 100644 --- a/drivers/gpu/drm/panthor/panthor_mmu.c +++ b/drivers/gpu/drm/panthor/panthor_mmu.c @@ -1275,9 +1275,7 @@ static int panthor_vm_op_ctx_prealloc_pts(struct pant= hor_vm_op_ctx *op_ctx) static int panthor_vm_prepare_map_op_ctx(struct panthor_vm_op_ctx *op_ctx, struct panthor_vm *vm, struct panthor_gem_object *bo, - u64 offset, - u64 size, u64 va, - u32 flags) + const struct drm_panthor_vm_bind_op *op) { struct drm_gpuvm_bo *preallocated_vm_bo; struct sg_table *sgt =3D NULL; @@ -1286,12 +1284,12 @@ static int panthor_vm_prepare_map_op_ctx(struct pan= thor_vm_op_ctx *op_ctx, if (!bo) return -EINVAL; =20 - if ((flags & ~PANTHOR_VM_BIND_OP_MAP_FLAGS) || - (flags & DRM_PANTHOR_VM_BIND_OP_TYPE_MASK) !=3D DRM_PANTHOR_VM_BIND_O= P_TYPE_MAP) + if ((op->flags & ~PANTHOR_VM_BIND_OP_MAP_FLAGS) || + (op->flags & DRM_PANTHOR_VM_BIND_OP_TYPE_MASK) !=3D DRM_PANTHOR_VM_BI= ND_OP_TYPE_MAP) return -EINVAL; =20 /* Make sure the VA and size are in-bounds. */ - if (size > bo->base.size || offset > bo->base.size - size) + if (op->size > bo->base.size || op->bo_offset > bo->base.size - op->size) return -EINVAL; =20 /* If the BO has an exclusive VM attached, it can't be mapped to other VM= s. */ @@ -1299,7 +1297,7 @@ static int panthor_vm_prepare_map_op_ctx(struct panth= or_vm_op_ctx *op_ctx, bo->exclusive_vm_root_gem !=3D panthor_vm_root_gem(vm)) return -EINVAL; =20 - panthor_vm_init_op_ctx(op_ctx, size, va, flags); + panthor_vm_init_op_ctx(op_ctx, op->size, op->va, op->flags); =20 ret =3D panthor_vm_op_ctx_prealloc_vmas(op_ctx); if (ret) @@ -1328,7 +1326,7 @@ static int panthor_vm_prepare_map_op_ctx(struct panth= or_vm_op_ctx *op_ctx, } =20 op_ctx->map.vm_bo =3D drm_gpuvm_bo_obtain_prealloc(preallocated_vm_bo); - op_ctx->map.bo_offset =3D offset; + op_ctx->map.bo_offset =3D op->bo_offset; =20 ret =3D panthor_vm_op_ctx_prealloc_pts(op_ctx); if (ret) @@ -2848,10 +2846,7 @@ panthor_vm_bind_prepare_op_ctx(struct drm_file *file, gem =3D drm_gem_object_lookup(file, op->bo_handle); ret =3D panthor_vm_prepare_map_op_ctx(op_ctx, vm, gem ? to_panthor_bo(gem) : NULL, - op->bo_offset, - op->size, - op->va, - op->flags); + op); drm_gem_object_put(gem); return ret; =20 @@ -3047,10 +3042,16 @@ int panthor_vm_bind_exec_sync_op(struct drm_file *f= ile, int panthor_vm_map_bo_range(struct panthor_vm *vm, struct panthor_gem_obje= ct *bo, u64 offset, u64 size, u64 va, u32 flags) { + struct drm_panthor_vm_bind_op op =3D { + .bo_offset =3D offset, + .size =3D size, + .va =3D va, + .flags =3D flags, + }; struct panthor_vm_op_ctx op_ctx; int ret; =20 - ret =3D panthor_vm_prepare_map_op_ctx(&op_ctx, vm, bo, offset, size, va, = flags); + ret =3D panthor_vm_prepare_map_op_ctx(&op_ctx, vm, bo, &op); if (ret) return ret; =20 --=20 2.53.0 From nobody Sat Jun 13 11:23:26 2026 Received: from sender4-pp-f112.zoho.com (sender4-pp-f112.zoho.com [136.143.188.112]) (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 211E234D929 for ; Thu, 7 May 2026 21:50:18 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=pass smtp.client-ip=136.143.188.112 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778190620; cv=pass; b=GtkQZyMVS9XZuZh0XykysWFobULtkwW/OEiijM7+wIn2I58NfeHQOnVlX6EALuzXyo4OuuLCL+b5h+HfMf7iFPLp3HIZ07idm34h9xqfodJeW7wXhOrOcNARh9oVVtZ1RP89IgCSVBx8AcZkw8+BOA4ArSHrMhyGq/oVlRyCEzk= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778190620; c=relaxed/simple; bh=aJEQEOv1AwHlqR9vhWPtvf47jiLzbFDbg66Z45jzSAw=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=q5NQOtOkiNpxVvsIzo7ERf3ezbjhRd2p/YVQ8AXVbUJLE20iDeJfLsUAjHa2hSyFBUqeDG0v29Eb20kOnhIG54X9+ivl9/LYg2Q9QHGJQSeCcTwT9HwB6EdADf5YqFEqFKXSSmTdMJbLWfNryHfO3v4TG65OkYzprzHh1/gXrVI= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=collabora.com; spf=pass smtp.mailfrom=collabora.com; dkim=pass (1024-bit key) header.d=collabora.com header.i=adrian.larumbe@collabora.com header.b=IjtJD8jW; arc=pass smtp.client-ip=136.143.188.112 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 (1024-bit key) header.d=collabora.com header.i=adrian.larumbe@collabora.com header.b="IjtJD8jW" ARC-Seal: i=1; a=rsa-sha256; t=1778190601; cv=none; d=zohomail.com; s=zohoarc; b=Z5q+qjgwbXoPI416blWQMoX1KueK+OdamMaubRv0+6wd4rc026LouOPc1LS+cnVovOJ3/+IvhZiukG0RZtLlxuY9Mc6Ur/jcTRwLz0swbp9YmjqiDUvr2CmY2Q44N8gk189h23QcndUm6A+iR3VrqEVuaJISlt/fOVUR1on2pKI= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1778190601; h=Content-Type:Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:In-Reply-To:MIME-Version:Message-ID:References:Subject:Subject:To:To:Message-Id:Reply-To; bh=do/tIM3Ojb1Vv1appOYmHiLTKOw9/ngQpz7qtd/LP/Q=; b=gfRyoCkiM9o330PQcX4hYpX/lp/XzQL5/g/kdUlrrgp+fnNaDFx/TrQc8Fs2+Fn+kuLmUFvOR4oqtzxFmDuA3BiyGQc3jXOCx0nIdCEDlTVPAo1Zk9yM/k6pjtQdMUj+HDccdYs7fxfN3vqC2VG+Avzuu3pPc9r+j9kVRmIv330= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass header.i=collabora.com; spf=pass smtp.mailfrom=adrian.larumbe@collabora.com; dmarc=pass header.from= DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; t=1778190601; s=zohomail; d=collabora.com; i=adrian.larumbe@collabora.com; h=From:From:To:To:Cc:Cc:Subject:Subject:Date:Date:Message-ID:In-Reply-To:References:MIME-Version:Content-Type:Content-Transfer-Encoding:Message-Id:Reply-To; bh=do/tIM3Ojb1Vv1appOYmHiLTKOw9/ngQpz7qtd/LP/Q=; b=IjtJD8jWJXDWaTz2jM0J09jGDt4ishoWixLR4rm15soOdGZM2UlIgPSzl1PfKX9x PBuFtz17NHLZxzyeKqYa8i1MAVifCtHVe/aaYgRlQFQ5VTdSKEb4onKXgDyIh0NoQMp M0/a3i0kv1i4HsJfWDAm/utKV1OXINHFhgO/8iBk= Received: by mx.zohomail.com with SMTPS id 1778190600125432.8407328057764; Thu, 7 May 2026 14:50:00 -0700 (PDT) From: =?UTF-8?q?Adri=C3=A1n=20Larumbe?= To: linux-kernel@vger.kernel.org Cc: dri-devel@lists.freedesktop.org, Steven Price , Boris Brezillon , kernel@collabora.com, =?UTF-8?q?Adri=C3=A1n=20Larumbe?= , Liviu Dudau , Daniel Almeida , Alice Ryhl , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie , Simona Vetter Subject: [PATCH v11 3/6] drm/panthor: Delete spurious whitespace from uAPI header Date: Thu, 7 May 2026 22:49:04 +0100 Message-ID: <20260507214939.2852489-4-adrian.larumbe@collabora.com> X-Mailer: git-send-email 2.53.0 In-Reply-To: <20260507214939.2852489-1-adrian.larumbe@collabora.com> References: <20260507214939.2852489-1-adrian.larumbe@collabora.com> 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 There's no extra blank line after the last member of any other uAPI structures, so delete it. Reviewed-by: Steven Price Reviewed-by: Boris Brezillon Signed-off-by: Adri=C3=A1n Larumbe --- include/uapi/drm/panthor_drm.h | 1 - 1 file changed, 1 deletion(-) diff --git a/include/uapi/drm/panthor_drm.h b/include/uapi/drm/panthor_drm.h index b462752c793d..14a93a4ef6ff 100644 --- a/include/uapi/drm/panthor_drm.h +++ b/include/uapi/drm/panthor_drm.h @@ -677,7 +677,6 @@ struct drm_panthor_vm_bind_op { * This array shall not be empty for sync-only operations. */ struct drm_panthor_obj_array syncs; - }; =20 /** --=20 2.53.0 From nobody Sat Jun 13 11:23:26 2026 Received: from sender4-pp-f112.zoho.com (sender4-pp-f112.zoho.com [136.143.188.112]) (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 71B3534D382 for ; Thu, 7 May 2026 21:50:17 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=pass smtp.client-ip=136.143.188.112 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778190618; cv=pass; b=iUr5jtBn1QQr6ViVqXz6r94su6oU7o1EF/mpMm7lOSuiEAbfbzydQYUiH6HCbn8F6tXVFkGnlPryfqrg44grfVm5grOB5R4It0mnW2Qs0AL40+Lo5GDR8NBhBoh7Q2p3TUhvmJp0Iwm7pQgJxJS0kjiFML76LSBNGdVfgaRQ8iw= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778190618; c=relaxed/simple; bh=ulJMlEh2MVlFxkFgBOyAm4Yesi0pQoJ/jQeVyeWgbGI=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=fj5r68Nvpql6BH8sfNrWqISxq56RsrfZyBcczpMlSs42NHxDX622AOs9r4irsmmRXyoH50B0jOTx0FsishQ+bVeEjGksl77B8TsTRgebNUDQ1ZO8Gwy7HgxByEjepzJTtie+Qc5iaZmBVA+caLUrmfkOzeUAByeEXGZ/Bped0SU= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=collabora.com; spf=pass smtp.mailfrom=collabora.com; dkim=pass (1024-bit key) header.d=collabora.com header.i=adrian.larumbe@collabora.com header.b=QU6sbjjY; arc=pass smtp.client-ip=136.143.188.112 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 (1024-bit key) header.d=collabora.com header.i=adrian.larumbe@collabora.com header.b="QU6sbjjY" ARC-Seal: i=1; a=rsa-sha256; t=1778190604; cv=none; d=zohomail.com; s=zohoarc; b=aOKDn5aCdfjabMj5SaqiJk2zQ9YnN2jahS3YrykoXbrFLAF6A+nhKzbyJrsufRNEMrFzDjhZ0qeVSu5bijlrbLedUD1dfuoEVKnFfo2cX78BboLw64UFtqMNEW3OelBNUwxv7xKJcXuQqV9f06s9OjMKGmaXDf4UZ+xCbJjUPHQ= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1778190604; h=Content-Type:Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:In-Reply-To:MIME-Version:Message-ID:References:Subject:Subject:To:To:Message-Id:Reply-To; bh=pPkjyQ7Aco3dEdYJdTItQJYvI7c1hnPfmf9scHUrQh8=; b=K8fPVyEXAK4AmSePAng4xk6hUEIoUI39IW4IgXWTPSiuH4sdmL35uEKWk4UoHvolyTZ99gddaEwZlT15yRAHXasOIBXbCTgEC1BdP7Q87kZzMUOdYxiTFqY0kutCJVN7wZsIkhN1/xo6pKmxVemnXp/ung8hUS3IHlwNqBP2PoI= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass header.i=collabora.com; spf=pass smtp.mailfrom=adrian.larumbe@collabora.com; dmarc=pass header.from= DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; t=1778190603; s=zohomail; d=collabora.com; i=adrian.larumbe@collabora.com; h=From:From:To:To:Cc:Cc:Subject:Subject:Date:Date:Message-ID:In-Reply-To:References:MIME-Version:Content-Type:Content-Transfer-Encoding:Message-Id:Reply-To; bh=pPkjyQ7Aco3dEdYJdTItQJYvI7c1hnPfmf9scHUrQh8=; b=QU6sbjjY4uVnhdtQhRVon8WJkqhmWUxKD8qGb/lxmIJtdb5uv3Owts7R2ve1vQpr GawSK1Qko40YEufiA0q3ROqRbHPJr4nb7vTjNCX7bRzH6jkxWuR9leg0wfOgpAx74RM 6ucDwUPokPdy/Qy4UAY8Znp0kVGR64wf/DVVdvKo= Received: by mx.zohomail.com with SMTPS id 1778190603106499.29029932971673; Thu, 7 May 2026 14:50:03 -0700 (PDT) From: =?UTF-8?q?Adri=C3=A1n=20Larumbe?= To: linux-kernel@vger.kernel.org Cc: dri-devel@lists.freedesktop.org, Steven Price , Boris Brezillon , kernel@collabora.com, =?UTF-8?q?Adri=C3=A1n=20Larumbe?= , Liviu Dudau , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie , Simona Vetter Subject: [PATCH v11 4/6] drm/panthor: Remove unused operation context field Date: Thu, 7 May 2026 22:49:05 +0100 Message-ID: <20260507214939.2852489-5-adrian.larumbe@collabora.com> X-Mailer: git-send-email 2.53.0 In-Reply-To: <20260507214939.2852489-1-adrian.larumbe@collabora.com> References: <20260507214939.2852489-1-adrian.larumbe@collabora.com> 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 A Panthor BO's sgtable is now retrieved from its dmap field. Reviewed-by: Steven Price Reviewed-by: Boris Brezillon Signed-off-by: Adri=C3=A1n Larumbe --- drivers/gpu/drm/panthor/panthor_mmu.c | 8 -------- 1 file changed, 8 deletions(-) diff --git a/drivers/gpu/drm/panthor/panthor_mmu.c b/drivers/gpu/drm/pantho= r/panthor_mmu.c index d35e65cab2b1..f54a60cd0ec4 100644 --- a/drivers/gpu/drm/panthor/panthor_mmu.c +++ b/drivers/gpu/drm/panthor/panthor_mmu.c @@ -195,14 +195,6 @@ struct panthor_vm_op_ctx { /** @map.bo_offset: Offset in the buffer object. */ u64 bo_offset; =20 - /** - * @map.sgt: sg-table pointing to pages backing the GEM object. - * - * This is gathered at job creation time, such that we don't have - * to allocate in ::run_job(). - */ - struct sg_table *sgt; - /** @map.bo: the BO being mapped. */ struct panthor_gem_object *bo; } map; --=20 2.53.0 From nobody Sat Jun 13 11:23:26 2026 Received: from sender4-pp-f112.zoho.com (sender4-pp-f112.zoho.com [136.143.188.112]) (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 8A56B3431F5 for ; Thu, 7 May 2026 21:50:22 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=pass smtp.client-ip=136.143.188.112 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778190626; cv=pass; b=kAKOCJ99SXNXRGaI+MJnLjqg+66OTBdxRN5urUQ6ID64OfzxhZQlkq7Xui1QmhshAUTMsjW+pUmxVOvE/mXgTkas7KQ+aDt+jFWKXlJoa7ooZpixNh+dBAZ646kAEwaCM8jc7hT0RR7NLWiQ4WlAwH0sEpYx0u4IZFzJP8J3A78= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778190626; c=relaxed/simple; bh=26iQzbchrjHER7/r158K+y5kBNalOzk7B+vOWmZBOME=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=ogIe0+b4jVgj/w2zsBMSuzmX+aUp2JcN7ypthDEiGcMmjeAC84AEB4FZAGeXjbmUp2qgfYcqcOifpdbemk59hSBWwvs87X07+QCqzvV44EivIxx1CnbDVDl5RZNrqQiNUPah/6QcHybkochPNThg7bolXADiP8LPizmrZdShzr4= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=collabora.com; spf=pass smtp.mailfrom=collabora.com; dkim=pass (1024-bit key) header.d=collabora.com header.i=adrian.larumbe@collabora.com header.b=lNj3NCWv; arc=pass smtp.client-ip=136.143.188.112 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 (1024-bit key) header.d=collabora.com header.i=adrian.larumbe@collabora.com header.b="lNj3NCWv" ARC-Seal: i=1; a=rsa-sha256; t=1778190607; cv=none; d=zohomail.com; s=zohoarc; b=AHbbg4E6q40UxpcQ5BMDv/VvRIzOP765YX/BIxeA3WR7TNfq081z0nao3UEl4lpJbR4VU3VD0+YnFhOZNkNYbBm6J1BE/IIos/D6CMhTRh8G2ShqTWqhh0QET4yzuicq1/xN+G/6LE5J+xFKOUDzBJcA0FT9TRJUiQ96zMTMp9U= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1778190607; h=Content-Type:Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:In-Reply-To:MIME-Version:Message-ID:References:Subject:Subject:To:To:Message-Id:Reply-To; bh=gbrl28ut/aunJo9imgsgKTpqYwK7qNliujjU3OJOY6c=; b=cGGcoF7i/xwmLA4E4oqKxffDHbSdehGQxxrkxl8KuZ9DRD+CYiZ3gEISrivrSn3wJGegwVbUyk/DkdVeUUVUFicTMuKoiYc4lFTi3sjRHOwFbMHP/yRgk0quvaxnxlSDqRggMWYYxNurobhRk/DcEJ963kO1OyXeM0CFWDqzD1U= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass header.i=collabora.com; spf=pass smtp.mailfrom=adrian.larumbe@collabora.com; dmarc=pass header.from= DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; t=1778190607; s=zohomail; d=collabora.com; i=adrian.larumbe@collabora.com; h=From:From:To:To:Cc:Cc:Subject:Subject:Date:Date:Message-ID:In-Reply-To:References:MIME-Version:Content-Type:Content-Transfer-Encoding:Message-Id:Reply-To; bh=gbrl28ut/aunJo9imgsgKTpqYwK7qNliujjU3OJOY6c=; b=lNj3NCWvXWr8iEw55oWFfSxlUlREyIJCDjb3gqqo/oY2bVUFC+M7QITwNvhpcNqu EM071qqueul1yz8ZhyT8Z/eYbbhPpUMWkZKqmcqQTGKZL7WlWTzBiZ3xnTckT+rtGNJ qFTz9Fs/HcMLBoJ//Xg8/K261h7woEA17yPAWJhM= Received: by mx.zohomail.com with SMTPS id 1778190606800243.22851647535128; Thu, 7 May 2026 14:50:06 -0700 (PDT) From: =?UTF-8?q?Adri=C3=A1n=20Larumbe?= To: linux-kernel@vger.kernel.org Cc: dri-devel@lists.freedesktop.org, Steven Price , Boris Brezillon , kernel@collabora.com, =?UTF-8?q?Adri=C3=A1n=20Larumbe?= , Liviu Dudau , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie , Simona Vetter , Daniel Almeida , Alice Ryhl Subject: [PATCH v11 5/6] drm/panthor: Support sparse mappings Date: Thu, 7 May 2026 22:49:06 +0100 Message-ID: <20260507214939.2852489-6-adrian.larumbe@collabora.com> X-Mailer: git-send-email 2.53.0 In-Reply-To: <20260507214939.2852489-1-adrian.larumbe@collabora.com> References: <20260507214939.2852489-1-adrian.larumbe@collabora.com> 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 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 fact Vulkan sparse resources don't need to be backed by a driver BO. The choice of backing sparsely-bound regions with a Panthor 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. 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. As a side note, care was put to adjust dummy BO offsets for sparse mappings so that all addresses in the new VA are mapped aligned against it. 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 | 179 +++++++++++++++++++++++--- include/uapi/drm/panthor_drm.h | 12 ++ 4 files changed, 190 insertions(+), 21 deletions(-) diff --git a/drivers/gpu/drm/panthor/panthor_gem.c b/drivers/gpu/drm/pantho= r/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 *ptde= v, struct panthor_vm *vm, return ERR_PTR(ret); } =20 +/** + * panthor_dummy_bo_create() - Create a Panthor BO meant to back sparse bi= ndings. + * @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 n= o guarantee + * that drm_gem_get_pages() will return a single 2MiB PMD, and also we ca= nnot be sure + * that the 2MiB won't be reclaimed and re-allocated later on as 4KiB chu= nks, it doesn't + * make sense to pre-populate this object's page array, nor to fall back = on a BO size + * of 4KiB. Sticking to a dummy object size of 2MiB lets us keep things s= imple for now. + */ + return panthor_gem_create(&ptdev->base, SZ_2M, DRM_PANTHOR_BO_NO_MMAP, NU= LL, 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/pantho= r/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/pantho= r/panthor_mmu.c index f54a60cd0ec4..f0425dd70d2c 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,15 @@ 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. + */ + struct panthor_gem_object *dummy; }; =20 /** @@ -1020,6 +1040,30 @@ panthor_vm_map_pages(struct panthor_vm *vm, u64 iova= , 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, + addr % SZ_2M, 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 +1306,7 @@ static int panthor_vm_op_ctx_prealloc_pts(struct pant= hor_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_ctx, @@ -1269,6 +1314,7 @@ static int panthor_vm_prepare_map_op_ctx(struct panth= or_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 +1326,21 @@ static int panthor_vm_prepare_map_op_ctx(struct pant= hor_vm_op_ctx *op_ctx, (op->flags & DRM_PANTHOR_VM_BIND_OP_TYPE_MASK) !=3D DRM_PANTHOR_VM_BI= ND_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->size) + /* 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.s= ize - 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 VM= s. */ @@ -1430,7 +1489,9 @@ panthor_vm_get_bo_for_va(struct panthor_vm *vm, u64 v= a, u64 *bo_offset) if (vma && vma->base.gem.obj) { drm_gem_object_get(vma->base.gem.obj); bo =3D to_panthor_bo(vma->base.gem.obj); - *bo_offset =3D vma->base.gem.offset + (va - vma->base.va.addr); + *bo_offset =3D !(vma->flags & DRM_PANTHOR_VM_BIND_OP_MAP_SPARSE) ? + vma->base.gem.offset + (va - vma->base.va.addr) : + va & (SZ_2M - 1); } mutex_unlock(&vm->op_lock); =20 @@ -1543,6 +1604,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 +1698,8 @@ void panthor_vm_pool_destroy(struct panthor_file *pfi= le) xa_for_each(&pfile->vms->xa, i, vm) panthor_vm_destroy(vm); =20 + if (pfile->vms->dummy) + drm_gem_object_put(&pfile->vms->dummy->base); xa_destroy(&pfile->vms->xa); kfree(pfile->vms); } @@ -1646,12 +1712,28 @@ void panthor_vm_pool_destroy(struct panthor_file *p= file) */ int panthor_vm_pool_create(struct panthor_file *pfile) { + struct panthor_gem_object *dummy; + int ret; + pfile->vms =3D kzalloc_obj(*pfile->vms); if (!pfile->vms) return -ENOMEM; =20 xa_init_flags(&pfile->vms->xa, XA_FLAGS_ALLOC1); + + dummy =3D panthor_dummy_bo_create(pfile->ptdev); + if (IS_ERR(dummy)) { + ret =3D PTR_ERR(dummy); + goto err_destroy_vm_pool; + } + + pfile->vms->dummy =3D dummy; + return 0; + +err_destroy_vm_pool: + panthor_vm_pool_destroy(pfile); + return ret; } =20 /* dummy TLB ops, the real TLB flush happens in panthor_vm_flush_range() */ @@ -1987,6 +2069,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 +2231,30 @@ static void panthor_vma_init(struct panthor_vma *vma= , 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 void +panthor_fix_sparse_map_offset(struct drm_gpuva_op_map *op, u32 flags) +{ + if (op && (flags & DRM_PANTHOR_VM_BIND_OP_MAP_SPARSE)) + op->gem.offset =3D op->va.addr & (SZ_2M - 1); +} + +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) { @@ -2159,10 +2267,9 @@ static int panthor_gpuva_sm_step_map(struct drm_gpuv= a_op *op, void *priv) return -EINVAL; =20 panthor_vma_init(vma, op_ctx->flags & PANTHOR_VM_MAP_FLAGS); + panthor_fix_sparse_map_offset(&op->map, vma->flags); =20 - ret =3D panthor_vm_map_pages(vm, op->map.va.addr, flags_to_prot(vma->flag= s), - 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; @@ -2194,6 +2301,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 pant= hor_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; @@ -2201,11 +2310,15 @@ unmap_hugepage_align(const struct drm_gpuva_op_rema= p *op, aligned_unmap_end =3D ALIGN(unmap_end, SZ_2M); =20 /* If we're dealing with a huge page, make sure the unmap region is - * aligned on the start of the page. + * aligned on the start of the page. If the unmapped VMA stands for + * a sparse mapping, always assume the backing storage is a THP, since + * the overhead of unmapping 2MiB worth of 4KiB pages and remapping + * some of them is offset by the logic of working out whether it's + * the opposite case right below. This also holds true for op->next. */ 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)) { + (is_sparse || iova_mapped_as_huge_page(op->prev, *unmap_start))) { *unmap_range +=3D *unmap_start - aligned_unmap_start; *unmap_start =3D aligned_unmap_start; } @@ -2215,7 +2328,7 @@ unmap_hugepage_align(const struct drm_gpuva_op_remap = *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)) { + (is_sparse || iova_mapped_as_huge_page(op->next, unmap_end - 1))) { *unmap_range +=3D aligned_unmap_end - unmap_end; } } @@ -2232,6 +2345,11 @@ static int panthor_gpuva_sm_step_remap(struct drm_gp= uva_op *op, =20 drm_gpuva_op_remap_to_unmap_range(&op->remap, &unmap_start, &unmap_range); =20 + /* op->remap.prev's BO offset is always the same as the unmap va's, but + * that of op->remap.next must be adjusted so as to remain < SZ_2M + */ + panthor_fix_sparse_map_offset(op->remap.next, unmap_vma->flags); + /* * ARM IOMMU page table management code disallows partial unmaps of huge = pages, * so when a partial unmap is requested, we must first unmap the entire h= uge @@ -2251,14 +2369,19 @@ static int panthor_gpuva_sm_step_remap(struct drm_g= puva_op *op, } =20 if (op->remap.prev) { - struct panthor_gem_object *bo =3D to_panthor_bo(op->remap.prev->gem.obj); u64 offset =3D op->remap.prev->gem.offset + unmap_start - op->remap.prev= ->va.addr; u64 size =3D op->remap.prev->va.addr + op->remap.prev->va.range - unmap_= start; =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) { + 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, + }; + panthor_fix_sparse_map_offset(&map_op, unmap_vma->flags); + + ret =3D panthor_vm_exec_map_op(vm, unmap_vma->flags, &map_op); if (ret) return ret; } @@ -2269,14 +2392,19 @@ static int panthor_gpuva_sm_step_remap(struct drm_g= puva_op *op, } =20 if (op->remap.next) { - struct panthor_gem_object *bo =3D to_panthor_bo(op->remap.next->gem.obj); u64 addr =3D op->remap.next->va.addr; u64 size =3D unmap_start + unmap_range - op->remap.next->va.addr; =20 - 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) { + 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, + }; + panthor_fix_sparse_map_offset(&map_op, unmap_vma->flags); + + ret =3D panthor_vm_exec_map_op(vm, unmap_vma->flags, &map_op); if (ret) return ret; } @@ -2835,7 +2963,13 @@ panthor_vm_bind_prepare_op_ctx(struct drm_file *file, =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 +3177,9 @@ int panthor_vm_map_bo_range(struct panthor_vm *vm, st= ruct 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_SPAR= SE)) + 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_drm.h index 14a93a4ef6ff..a2ff0f4ec691 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 range + * + * Only valid with DRM_PANTHOR_VM_BIND_OP_TYPE_MAP. + * + * When this flag is set, the whole vm_bind range is mapped over a dummy = object in a cyclic + * fashion, and all GPU reads from addresses in the range return undefine= d values. This flag + * being set means drm_panthor_vm_bind_op::bo_offset and drm_panthor_vm_b= ind_op::bo_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 of = operation. */ --=20 2.53.0 From nobody Sat Jun 13 11:23:26 2026 Received: from sender4-pp-f112.zoho.com (sender4-pp-f112.zoho.com [136.143.188.112]) (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 A7BDC31E82F for ; Thu, 7 May 2026 21:50:26 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=pass smtp.client-ip=136.143.188.112 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778190627; cv=pass; b=FE4wyA80Gcrba74LgnGyf2PDRFZRPyBHqmglBMrlCqwhmfgr7XlTXkRYZ2WDL9QKWi2ME9hRFbhu92UW200kj5f3292lRU2b8sTIRQKRXjQ0IcRYus4Xr3W749W6mcPtBKeG+EWDnPxVesbRGc8rwQXDiK+IpUKqfuimpVxkyyY= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778190627; c=relaxed/simple; bh=3Gf9V2u1R1lZL5ItZCYmlkh38Y2n4lRMxZyfnhJPoQY=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=mA0rfG6PSq3LzhfkQiwX/pNCvsnFiS45wsUeC+BdQSndVrUVcGZeu2/dK8Wp8LK8Q7T1Fyu+UUugGBHQnYXV5REtiA5HKVHayNwQos5oLJ09vMdUztDbQsE/Gt6ibt5ZUsqjrSbocyHSBn06qUunRSfYBCAFvrvGuz4ulFY7WbU= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=collabora.com; spf=pass smtp.mailfrom=collabora.com; dkim=pass (1024-bit key) header.d=collabora.com header.i=adrian.larumbe@collabora.com header.b=MQ2o752T; arc=pass smtp.client-ip=136.143.188.112 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 (1024-bit key) header.d=collabora.com header.i=adrian.larumbe@collabora.com header.b="MQ2o752T" ARC-Seal: i=1; a=rsa-sha256; t=1778190611; cv=none; d=zohomail.com; s=zohoarc; b=FjdOf0jkxzABNhEJEIJv/4Txdk8gkNe7Q5KvzcViHR2TEm8iB98zq5G6Bw5ZZhJ5bZyFKOEXMt5zr/WOi4bN48ANrnTaqLRekzvn9sq7mZQHmrM8pRten9qAx6SyME7CC/oIXEDnrCJiGq5cU6Rznip4NLzJjdbUV5DodEK32X0= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1778190611; h=Content-Type:Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:In-Reply-To:MIME-Version:Message-ID:References:Subject:Subject:To:To:Message-Id:Reply-To; bh=cLsPoqXdNgnJpPLfc54fcRrQummLcdcVrXH6Ve1sFN0=; b=gizQizqj17QayR/9Ca0vi3PvbEJWf5clA7RGZFHlJVbx1FMgwVSxUeC3YqJiMuGshHIPoPt4NsOiMA/+6XPJ2wxAGvsNxLq7d1oYYetxWfnMSiKeKAbVNbw8ji7+R/N7/Pw+HmZC93zU1UUHrCSQPa0hu0ZLyt6haF2U+BMRr9Q= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass header.i=collabora.com; spf=pass smtp.mailfrom=adrian.larumbe@collabora.com; dmarc=pass header.from= DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; t=1778190611; s=zohomail; d=collabora.com; i=adrian.larumbe@collabora.com; h=From:From:To:To:Cc:Cc:Subject:Subject:Date:Date:Message-ID:In-Reply-To:References:MIME-Version:Content-Type:Content-Transfer-Encoding:Message-Id:Reply-To; bh=cLsPoqXdNgnJpPLfc54fcRrQummLcdcVrXH6Ve1sFN0=; b=MQ2o752TEJtJcVTgo2KRvTnjcI5oOXpaTdoM2UjQLNy/Ze/jvvwMAKUo/B8sbcDv N4uHF6RSrsku3aZ6v2LrD4C8qvOwq8LgB2IpaGrx4elSHFGBAreu/JOrMp8DcYXWe07 68DyvEPX9UFMY/BLqJISDc7Hc6JyPYrj/ifT96Wc= Received: by mx.zohomail.com with SMTPS id 1778190609763546.4197834228718; Thu, 7 May 2026 14:50:09 -0700 (PDT) From: =?UTF-8?q?Adri=C3=A1n=20Larumbe?= To: linux-kernel@vger.kernel.org Cc: dri-devel@lists.freedesktop.org, Steven Price , Boris Brezillon , kernel@collabora.com, =?UTF-8?q?Adri=C3=A1n=20Larumbe?= , Liviu Dudau , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie , Simona Vetter Subject: [PATCH v11 6/6] drm/panthor: Bump the driver version to 1.9 Date: Thu, 7 May 2026 22:49:07 +0100 Message-ID: <20260507214939.2852489-7-adrian.larumbe@collabora.com> X-Mailer: git-send-email 2.53.0 In-Reply-To: <20260507214939.2852489-1-adrian.larumbe@collabora.com> References: <20260507214939.2852489-1-adrian.larumbe@collabora.com> 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 Bump the driver version to reflect the new DRM_PANTHOR_DEV_QUERY_MMU_INFO query type and the DRM_PANTHOR_VM_BIND_OP_MAP_SPARSE flag. Reviewed-by: Boris Brezillon Reviewed-by: Steven Price Signed-off-by: Adri=C3=A1n Larumbe --- drivers/gpu/drm/panthor/panthor_drv.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/panthor/panthor_drv.c b/drivers/gpu/drm/pantho= r/panthor_drv.c index beca75b34293..d0070a28a634 100644 --- a/drivers/gpu/drm/panthor/panthor_drv.c +++ b/drivers/gpu/drm/panthor/panthor_drv.c @@ -1787,6 +1787,8 @@ static void panthor_debugfs_init(struct drm_minor *mi= nor) * - adds DRM_IOCTL_PANTHOR_BO_QUERY_INFO ioctl * - adds drm_panthor_gpu_info::selected_coherency * - 1.8 - extends DEV_QUERY_TIMESTAMP_INFO with flags + * - 1.9 - adds DRM_PANTHOR_DEV_QUERY_MMU_INFO query + * - adds DRM_PANTHOR_VM_BIND_OP_MAP_SPARSE flag */ static const struct drm_driver panthor_drm_driver =3D { .driver_features =3D DRIVER_RENDER | DRIVER_GEM | DRIVER_SYNCOBJ | @@ -1800,7 +1802,7 @@ static const struct drm_driver panthor_drm_driver =3D= { .name =3D "panthor", .desc =3D "Panthor DRM driver", .major =3D 1, - .minor =3D 8, + .minor =3D 9, =20 .gem_prime_import_sg_table =3D panthor_gem_prime_import_sg_table, .gem_prime_import =3D panthor_gem_prime_import, --=20 2.53.0