Received-SPF: pass (google.com: domain of linux-kernel+bounces-203101-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.48.161 as permitted sender) client-ip=147.75.48.161;
From: Martin Oliveira <martin.oliveira@eideticom.com>
To: linux-rdma@vger.kernel.org,
	linux-kernel@vger.kernel.org,
	linux-pci@vger.kernel.org,
	linux-mm@kvack.org
Cc: Jason Gunthorpe <jgg@ziepe.ca>,
	Leon Romanovsky <leon@kernel.org>,
	Bjorn Helgaas <bhelgaas@google.com>,
	Logan Gunthorpe <logang@deltatee.com>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	Tejun Heo <tj@kernel.org>,
	"Rafael J. Wysocki" <rafael@kernel.org>,
	Andrew Morton <akpm@linux-foundation.org>,
	Mike Marciniszyn <mike.marciniszyn@intel.com>,
	Michael Guralnik <michaelgur@nvidia.com>,
	Martin Oliveira <martin.oliveira@eideticom.com>,
	Dan Williams <dan.j.williams@intel.com>,
	Ard Biesheuvel <ardb@kernel.org>,
	Valentine Sinitsyn <valesini@yandex-team.ru>,
	Lukas Wunner <lukas@wunner.de>
Subject: [PATCH 3/6] PCI/P2PDMA: create VMA without page_mkwrite() operator
Date: Wed,  5 Jun 2024 13:29:31 -0600
Message-Id: <20240605192934.742369-4-martin.oliveira@eideticom.com>
In-Reply-To: <20240605192934.742369-1-martin.oliveira@eideticom.com>
References: <20240605192934.742369-1-martin.oliveira@eideticom.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?us-ascii?Q?2WHdetvNHqzPOiRgdKlU0TnI3fHLhedUykoJBmuhCRWT4lnVOc7+zphNmYk0?=
 =?us-ascii?Q?a7Pf+J/PfeIp6GzMvkedFAEQ9I2YWXccfkxmEWStJk1d0ghwS9wUE2PHVGG4?=
 =?us-ascii?Q?sCEK7MLWVr2SJH8sSndXlZMBdDo3SLbNeegmCRLdiPHRwLnoiImApp1K4MXq?=
 =?us-ascii?Q?WkLNt5WB59sUHE7SCUlEoethGFFOCyG37hcHghTd6v30v5q7n69LFLE8tTt9?=
 =?us-ascii?Q?wdfbS2tf3/v8hRgJ5Wh682hNlFoQzxONE81ygGpxpSKUKSK6NqaM5GR0M3Pm?=
 =?us-ascii?Q?fOIzE0DccthI/qShhXKg4VD1pPCVhIYlzNeUS5Q1kB122tJQa2toYwvSjAxI?=
 =?us-ascii?Q?v/K4H1EmkT53C9yiTOPxEaKNRD97VHJ/RQ6zQTfJJI4pU8ZsrDmfDALSba1/?=
 =?us-ascii?Q?HfyNfxD/dJej8lNEJDCdH8BBs3INAcPZwsoHqsmWvsBXKtfgiF21XMFeasTO?=
 =?us-ascii?Q?789K5zZOLR1+93E1zzh/oWOx6ireG95WS4EvD6Da1OBtkNIUy/Jv/iNUBdWV?=
 =?us-ascii?Q?VP0OdZXlY8WdK26LS+ntmz9bOD+dmrck+WQ8aliTc6plvx8bmbrYtLlCiPDY?=
 =?us-ascii?Q?2VLlbfm2Tb+Y9rHVMiB3sKEjNxtZDyt4J2NHnbmcOSo40E641/qAV+/siDPi?=
 =?us-ascii?Q?zE1M1HGFbZU+v98MynRc2+YQr+iIofn7sW8QFRP5xB/t+RDQ1P8S9ETVG3B/?=
 =?us-ascii?Q?cNsvndNjdHuQ3pQcz24O/mgAncx7W4zjqsPEglHKwhH7mZXjuFiN6lU0patT?=
 =?us-ascii?Q?PaPye5GjmhGZ4t9aHgDFkoihH+9njQR2WIHR3EAVbg9j8i0Gq7hlM1XnibDe?=
 =?us-ascii?Q?0ujLa9t84yVIbQx8CsEBvh4ywlSd8/ckDpSR2lLHUhIAGemxiYYXebpPjJ03?=
 =?us-ascii?Q?X10s468GRZV2L7EaXP6Kk1o9oT2/Hidf4t77h916sCJnXVc2uLn28JWULGVr?=
 =?us-ascii?Q?q5NwMgzkSWsdwcDwm0bnsP43V43hSyFI3MsWdH3cCxI0pcM3d8VG7nUdJ6/z?=
 =?us-ascii?Q?3y6+b4oMJToOQ4hvv6Tc4Zfr14HhEqwUJRDiN/KyDLeBODh5MSOdL0gk5d59?=
 =?us-ascii?Q?sSprnQKE/feVK+HKr+3nqtRD/FFtozCy7PWsFDFH17uB2eZMRtmj9/sf8nCf?=
 =?us-ascii?Q?7lkU/dqsGYWSeH1SNpuJwexk19WGBpJkBl+zrgZkxD7gJihfQEvU29VoLocK?=
 =?us-ascii?Q?rTjkkmGLdo75F9/A748HF5It+fddpwm13wNJgXS4mh/MTRwBKtGuSjukDmjq?=
 =?us-ascii?Q?meaLkdk9pBgUKjZVLh2KBgmEoagdgMaDPv30Lbm3/73i6YtZ0w1refGZWVdZ?=
 =?us-ascii?Q?Vi8ngm5ivibOavAY74sgATghDDHLlKqi1U71O1tiYbqzMO8AO4Fhxz1VT1f8?=
 =?us-ascii?Q?sOVICoGyvY9Mqnba2iCa4MHtPCj77cF5SYQfdHH6zNpIa3sa6XWdnFy6+kIK?=
 =?us-ascii?Q?u+NOUtzqw+4Z/lxCuh7mj/ytryNcd4LViiYGGfkjIML74/IVljckCqF3Aseg?=
 =?us-ascii?Q?zfAh6i+aaYAHywfpwdCTAGtpqpuD/LeS9oaJ+QS0CADv5oB7A+682tN/WA45?=
 =?us-ascii?Q?i9zYnJWsq9GWsP2gLEt2GCuejNe7TP0fBvDalvoG+m7zQdNf8SxCRp9TTAmE?=
 =?us-ascii?Q?ug=3D=3D?=
X-OriginatorOrg: eideticom.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 7aca9095-b192-4d61-0f68-08dc8595e840
X-MS-Exchange-CrossTenant-AuthSource: MW3PR19MB4250.namprd19.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 05 Jun 2024 19:30:07.3996
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 3490cd4b-0360-4377-abb1-15f8c5af8fc2
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: OzI08ObFfg51Y3IXXgHpZqt/XXVhthmYxvHT1F7jB2yJpfPynR1mwxgKejqnhAv6sTCZApUMR4BY/M4n/C2peOfYW2NObOJYyfcagRraxY0=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH8PR19MB7093

The P2PDMA code does not need (or want) a page_mkwrite() operator on its
VMA.

Furthermore, having the page_mkwrite() operator causes
writable_file_mapping_allowed() to fail due to
vma_needs_dirty_tracking() on the gup flow, which is a pre-requisite for
enabling P2PDMA with FOLL_LONGTERM use cases.

Co-developed-by: Logan Gunthorpe <logang@deltatee.com>
Signed-off-by: Logan Gunthorpe <logang@deltatee.com>
Signed-off-by: Martin Oliveira <martin.oliveira@eideticom.com>
---
 drivers/pci/p2pdma.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/pci/p2pdma.c b/drivers/pci/p2pdma.c
index 4f47a13cb500..ac07053abfea 100644
--- a/drivers/pci/p2pdma.c
+++ b/drivers/pci/p2pdma.c
@@ -171,6 +171,7 @@ static struct bin_attribute p2pmem_alloc_attr = {
 	 * to be very large.
 	 */
 	.size = SZ_1T,
+	.mmap_allocates = true,
 };
 
 static struct attribute *p2pmem_attrs[] = {
-- 
2.34.1