From patchwork Tue Jun 25 11:44:15 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Pankaj Raghav (Samsung)" X-Patchwork-Id: 13710924 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id A1C21C2BBCA for ; Tue, 25 Jun 2024 11:44:51 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 203DC6B02F0; Tue, 25 Jun 2024 07:44:51 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 1B35B6B02F1; Tue, 25 Jun 2024 07:44:51 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 006406B02F2; Tue, 25 Jun 2024 07:44:50 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0011.hostedemail.com [216.40.44.11]) by kanga.kvack.org (Postfix) with ESMTP id D2FC16B02F0 for ; Tue, 25 Jun 2024 07:44:50 -0400 (EDT) Received: from smtpin30.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay01.hostedemail.com (Postfix) with ESMTP id 9628E1C1920 for ; Tue, 25 Jun 2024 11:44:50 +0000 (UTC) X-FDA: 82269229140.30.298B96E Received: from mout-p-103.mailbox.org (mout-p-103.mailbox.org [80.241.56.161]) by imf17.hostedemail.com (Postfix) with ESMTP id B6F8D4000F for ; Tue, 25 Jun 2024 11:44:48 +0000 (UTC) Authentication-Results: imf17.hostedemail.com; dkim=pass header.d=pankajraghav.com header.s=MBO0001 header.b=Gl9MDuRF; spf=pass (imf17.hostedemail.com: domain of kernel@pankajraghav.com designates 80.241.56.161 as permitted sender) smtp.mailfrom=kernel@pankajraghav.com; dmarc=pass (policy=quarantine) header.from=pankajraghav.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1719315881; a=rsa-sha256; cv=none; b=WcqNk9GhtH1ZDwE3VhycPoNrgEplHxFZQc7sO0ebbEPH4Rm4UQe3gX1YWL/Y/fvSt80gzR KSm3TcSWRsJ+8OP1nGcghB8MciSx3BF0zIVrZlh1HU1jREwcf+BmWvsEBKK5+A/tZp4SJ4 UaWfVMGkf/CIKrVUnfQ4lmCLhLqp+YM= ARC-Authentication-Results: i=1; imf17.hostedemail.com; dkim=pass header.d=pankajraghav.com header.s=MBO0001 header.b=Gl9MDuRF; spf=pass (imf17.hostedemail.com: domain of kernel@pankajraghav.com designates 80.241.56.161 as permitted sender) smtp.mailfrom=kernel@pankajraghav.com; dmarc=pass (policy=quarantine) header.from=pankajraghav.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1719315881; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=B3OUZbSfJC/PeNvlGf0eSPPicsSbDeYTsWtl9oDbqvs=; b=BOaMycVpLg0PnOH/VhOWrk90TP4bT/a99DZxY2J/q5ATx9JxkJYxtc6FOU8gWBHS4O/XqT GCRhRZcpYfsINfe6BoqRZduW7wz56wyHcG3St3ZK19noNn3X7wMNdWaXrUlPFQ7exDGPkm Sf0WZYBz3v4cOQWNxDPg1Ikz9SA+0zs= Received: from smtp2.mailbox.org (smtp2.mailbox.org [10.196.197.2]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by mout-p-103.mailbox.org (Postfix) with ESMTPS id 4W7jg11qg5z9ssB; Tue, 25 Jun 2024 13:44:45 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=pankajraghav.com; s=MBO0001; t=1719315885; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=B3OUZbSfJC/PeNvlGf0eSPPicsSbDeYTsWtl9oDbqvs=; b=Gl9MDuRFaJeRCemR22+awf7zPbIO1zOHLEr3PDpxFJOrDCUo2YJ/Ov7jr3R1FYm7kHzA/i ylPymM9EuDr0WA7A13RxyuRFuX3oFA4VGXswbFQG1UAHgTYyAO6nWwm1Nw+lVoSDPu3Nf2 kURCO/Stb/5fitNOlIbzweXb53CUKT3A13qBhEnV+309WWZ/Oh/mwDEQbvecaDaWecIVb1 4gYZO4fOiTbN5An7f98ORIy5vTSGX5gak03FVnLbMV0eQo4L7J8jJh2XXYuGlO0B+NgLC3 JAS9lG21cK+5PW7LK1XEUwF2I/No07K1phD3V2vT7vmXG7TmqQGwbx0qPfhyzA== From: "Pankaj Raghav (Samsung)" To: david@fromorbit.com, willy@infradead.org, chandan.babu@oracle.com, djwong@kernel.org, brauner@kernel.org, akpm@linux-foundation.org Cc: linux-kernel@vger.kernel.org, yang@os.amperecomputing.com, linux-mm@kvack.org, john.g.garry@oracle.com, linux-fsdevel@vger.kernel.org, hare@suse.de, p.raghav@samsung.com, mcgrof@kernel.org, gost.dev@samsung.com, cl@os.amperecomputing.com, linux-xfs@vger.kernel.org, kernel@pankajraghav.com, hch@lst.de, Zi Yan Subject: [PATCH v8 05/10] filemap: cap PTE range to be created to allowed zero fill in folio_map_range() Date: Tue, 25 Jun 2024 11:44:15 +0000 Message-ID: <20240625114420.719014-6-kernel@pankajraghav.com> In-Reply-To: <20240625114420.719014-1-kernel@pankajraghav.com> References: <20240625114420.719014-1-kernel@pankajraghav.com> MIME-Version: 1.0 X-Stat-Signature: nmhuwez6obj3zjh4fitq9r43qdr3jocy X-Rspamd-Queue-Id: B6F8D4000F X-Rspam-User: X-Rspamd-Server: rspam10 X-HE-Tag: 1719315888-947978 X-HE-Meta: U2FsdGVkX183klk92LzyVv8XRwi/fSjirqYrZfzfRFB2DzLUMvNVjIl8Ar5z5BeHexxtTExCH8aBmcMHV2VAUQ1brImfPw/16hK7zWtkpHLZFOWlr35DovTt2XCXjvaXyq5FXS2JwBomcQa30PFjy6FAz9CexY4EIzsgewWcKQhROLObe/Gy9SLM9HtvKq6OoVC/W/VMOCrxZHM/h8TKmooGGycpR24JFNXrGofebTAt84TRQl24JVuL3MLrihkT1lxCZK75smZLA4lCfIZWKqkBKayHdAmJg180CObCRpZUgdE5hkQBze5RuutKE7Mn2Skxd2RNsaZRsMuTw3XEYAfzvP4neW+n1yciuCzilFV3yDC+YxUUQWwXxpYtLhGIn30t44mpoDArQBLd5Cqur4wLNplMcIFYOzntw/UQf40clwNTg3uYZVE2r4OX/UJBY3OZbWaK5PUGGjSDhv3E4E3BT/26jOGpki5O3NY0XopBa2qP9jp3tnPaDMpMo69h3URoubRaYPQZhmxt7gLQYOXQLkl06O/3n71gc2cfufMpawzfNgbRpL8GTfVyHFvjmM8p0HZLBOKG3CKzVAJ7Z3J/ah4N46k2eVgZ5RumpqgMNImRpiTeTnmGYgELFl4/xYP9F7sNJh+FtIW97jRcyw+VRmlXp63BOhkvvvz/ziVCpTF5P3SuEqLsjrqeciDuOHjSEfFfEw3ZcmhmqV1iYDUwm9byewxpqYsojeTCu12gf/EJXVzbAEdZjPhnjEGaKs9DuTqPF6/K2fkHRikPk76tHi541YE7vs3gLqK0aRfADlAtObHs102KNJB8Wr7AMb+FNi/5hRtHM9YKAr8wUq5ClMMOv0AesQSSXW5dLiZ9/nChIQ6SIU83uvDLJYEdHzvtl6Nfx2kNSf/9Z4kxDtXmt849yK33Y/uSLA4ZwqvmmniTU7hpz53Foez4RgizSrh0qvHH0XtYNhyEnhD 7YK2WzUE KM4tV580y1ZjODjR1Tlb3c+lY837aYCmzUdo7ALuu/zmRQ7+otpVpJEeLNqKoKBKIBZybI8VloZn2EqxUZ9kCZCM5Wmrmp6xpW6SIVZ6qp0cV7+AwWwC1nhDJGePesMlZlHb9ZDTDHpSJYm7OfdSNvu4AePr+tLKxpMpTO2J2p6Mr83K+hIY5qefZ3imXEdNwthpHVUDrG6aU+6fM+QxYZdX1tKT7XAodQIVUwKcqiYVIw5URqpD3FapCUYUaqA9auk3Wjv17FR87HyowTR6hPILdAGv5TC5I0l12g1M6+2Zlbwuy0dTyGRERPUHbizuTZeWDjWFSZAcRhMxgmCoKmKnXH3hREuOY9VDsu73egurzKtQPaXDGqPCDsGnEJqiGxB8AaWB2eRStGX+LOcdOnq8pVg== X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: List-Subscribe: List-Unsubscribe: From: Pankaj Raghav Usually the page cache does not extend beyond the size of the inode, therefore, no PTEs are created for folios that extend beyond the size. But with LBS support, we might extend page cache beyond the size of the inode as we need to guarantee folios of minimum order. While doing a read, do_fault_around() can create PTEs for pages that lie beyond the EOF leading to incorrect error return when accessing a page beyond the mapped file. Cap the PTE range to be created for the page cache up to the end of file(EOF) in filemap_map_pages() so that return error codes are consistent with POSIX[1] for LBS configurations. generic/749(currently in xfstest-dev patches-in-queue branch [0]) has been created to trigger this edge case. This also fixes generic/749 for tmpfs with huge=always on systems with 4k base page size. [0] https://lore.kernel.org/all/20240615002935.1033031-3-mcgrof@kernel.org/ [1](from mmap(2)) SIGBUS Attempted access to a page of the buffer that lies beyond the end of the mapped file. For an explanation of the treatment of the bytes in the page that corresponds to the end of a mapped file that is not a multiple of the page size, see NOTES. Signed-off-by: Luis Chamberlain Signed-off-by: Pankaj Raghav Reviewed-by: Hannes Reinecke Reviewed-by: Matthew Wilcox (Oracle) --- mm/filemap.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/mm/filemap.c b/mm/filemap.c index 8eafbd4a4d0c..56ff1d936aa8 100644 --- a/mm/filemap.c +++ b/mm/filemap.c @@ -3612,7 +3612,7 @@ vm_fault_t filemap_map_pages(struct vm_fault *vmf, struct vm_area_struct *vma = vmf->vma; struct file *file = vma->vm_file; struct address_space *mapping = file->f_mapping; - pgoff_t last_pgoff = start_pgoff; + pgoff_t file_end, last_pgoff = start_pgoff; unsigned long addr; XA_STATE(xas, &mapping->i_pages, start_pgoff); struct folio *folio; @@ -3638,6 +3638,10 @@ vm_fault_t filemap_map_pages(struct vm_fault *vmf, goto out; } + file_end = DIV_ROUND_UP(i_size_read(mapping->host), PAGE_SIZE) - 1; + if (end_pgoff > file_end) + end_pgoff = file_end; + folio_type = mm_counter_file(folio); do { unsigned long end;