From patchwork Fri Nov 8 04:12:57 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Baolin Wang X-Patchwork-Id: 13867521 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 217C0D5E124 for ; Fri, 8 Nov 2024 04:13:38 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 7E3598D0001; Thu, 7 Nov 2024 23:13:32 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 793FC6B00B6; Thu, 7 Nov 2024 23:13:32 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 682EC8D0001; Thu, 7 Nov 2024 23:13:32 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0017.hostedemail.com [216.40.44.17]) by kanga.kvack.org (Postfix) with ESMTP id 45CF46B00B4 for ; Thu, 7 Nov 2024 23:13:32 -0500 (EST) Received: from smtpin13.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay09.hostedemail.com (Postfix) with ESMTP id 023CB808A0 for ; Fri, 8 Nov 2024 04:13:31 +0000 (UTC) X-FDA: 82761606816.13.E37B460 Received: from out30-97.freemail.mail.aliyun.com (out30-97.freemail.mail.aliyun.com [115.124.30.97]) by imf17.hostedemail.com (Postfix) with ESMTP id ED51540007 for ; Fri, 8 Nov 2024 04:13:01 +0000 (UTC) Authentication-Results: imf17.hostedemail.com; dkim=pass header.d=linux.alibaba.com header.s=default header.b="RQhA4ZW/"; dmarc=pass (policy=none) header.from=linux.alibaba.com; spf=pass (imf17.hostedemail.com: domain of baolin.wang@linux.alibaba.com designates 115.124.30.97 as permitted sender) smtp.mailfrom=baolin.wang@linux.alibaba.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1731039040; 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=kKtwOTm/io/miazqQHXaa70Oq2bCu/U2TTLxQP5qiDM=; b=6Buq9nFdYkxkFkhIMVthwROW7FNrh1rpf35qU7U5kRSXphZAjX6F6Yg4iZu67KloyWLdek STv9KGXpjF24UmsEfX8t4qMEA7Aa5P5gLFf7kwhNLEqSKv12Xq2xZ433JslcqQrK9gylmG vCeGTxR4kVpLda2cNSFNl9PwIxNHf9M= ARC-Authentication-Results: i=1; imf17.hostedemail.com; dkim=pass header.d=linux.alibaba.com header.s=default header.b="RQhA4ZW/"; dmarc=pass (policy=none) header.from=linux.alibaba.com; spf=pass (imf17.hostedemail.com: domain of baolin.wang@linux.alibaba.com designates 115.124.30.97 as permitted sender) smtp.mailfrom=baolin.wang@linux.alibaba.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1731039040; a=rsa-sha256; cv=none; b=pdypK5dkJ0eetCPtGozl0+UnxiDBZC/U8VtY3PFcvozp0nM+/V9MoL9YVHELJQRdAe2xwd XaT3EufvpaAUvurZesCzevYmR3Ano3lzSgSiElzy3axVPgHIbfyUVoQQzbmsOJHIjKMxqO wqtGHohFHXr01ijWm9xDFqzvOVhapBI= DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.alibaba.com; s=default; t=1731039207; h=From:To:Subject:Date:Message-Id:MIME-Version; bh=kKtwOTm/io/miazqQHXaa70Oq2bCu/U2TTLxQP5qiDM=; b=RQhA4ZW/CqyHM20Pm0dsvDGk0WoRKfNOoFL30lfD0TMTlqZdur4/MxZqrycQn/ONM0/HvX/t94CKxjaOGodXbQlQYTOvGzIsBoWlbGu/EkU2vchNY9rPjJ2vDLjoREEJt58sZcY1mhYppwSctG0q5icNhD/JfjkkGGKy5YNUM38= Received: from localhost(mailfrom:baolin.wang@linux.alibaba.com fp:SMTPD_---0WIxRHWc_1731039205 cluster:ay36) by smtp.aliyun-inc.com; Fri, 08 Nov 2024 12:13:25 +0800 From: Baolin Wang To: akpm@linux-foundation.org, hughd@google.com Cc: willy@infradead.org, david@redhat.com, wangkefeng.wang@huawei.com, 21cnbao@gmail.com, ryan.roberts@arm.com, ioworker0@gmail.com, da.gomez@samsung.com, baolin.wang@linux.alibaba.com, linux-mm@kvack.org, linux-kernel@vger.kernel.org Subject: [PATCH 3/4] mm: shmem: add large folio support for tmpfs Date: Fri, 8 Nov 2024 12:12:57 +0800 Message-Id: X-Mailer: git-send-email 2.39.3 In-Reply-To: References: MIME-Version: 1.0 X-Rspamd-Server: rspam04 X-Rspamd-Queue-Id: ED51540007 X-Stat-Signature: fsrr68qi5i9baupgzmw6pgunnskreih4 X-Rspam-User: X-HE-Tag: 1731039181-966583 X-HE-Meta: U2FsdGVkX19gVP7AMeuFBOsbXUbYIStW1VGZUgLS7Ukgbw/eEI0X/SyXGroZVE9PsfJG/R5Xt2wSmyF7zFm/V7WMymc3AWTcXgB3kwQuAQnFwnTArMp2GT1dw1L6Fem/CPsl+4lDUeRWLdq2VIWuCuY2z/hiWhz2VfyyVUwb2d0v0WYrLMcpHBn2Px8afJGAY+vY5M9YnbybL2Irx+FtSKFsB7VWtZta+O8YEppeirBCuj4oeHIYLdwd4uRXT2WnImdVphR0Zx28XV0hJ/ZTEEwSvM+T2bD58v5JE3gaNJRbW1umhnjYdpGr/jbCylABqxOqScIcIGCI/EeiIAVsDy6kDRRx1K39BeglkNnzkYnRXUFfBmeOnfIHCEQqLrDLlr0tYMVcRq/ZX2bPFp4DVumndHXtCbCLsHOnfOSjl+11mbOm7iGv70/b6BBN8gp/KTq/I0gn7vBQnvBRPrCXeaQnATbUo78gfIXv86TSoahDLwVpqt/I+iihwQ2gx1nY7MZ1e2KUAcov0S2vDN6HmYt1DryEMiim0jYGv3MabEmpccOhmywSJ4lWNan4ufEBZFtFQQbc/DB1BEZMzTj1GtKu/aCfIuAUJBHMy/LcZMoOVgnUbruWCMx4N36zEVbLU3iBFDaoI9W+NNFKVEoN0k/S83SkZD760pPZP2jt9yMp1rfrA+VRYo/uWIg3QSsT02FfHxBvEb2LolRL1CQImc8/4ql/CZNBPY0Px4+DOishDiPmDHg+W1hng4JjYaUNkVmL1QWobi07RiS9fOqAZEY7eD6qySJeC960JrGlkFr/5topFXGxrEKbUdqZlA9GmvM22y9sa62aReym8IH8QbBKTi2sR2AOJ4AiRtKiJUB1Be7l0fNdHrYAqh1KQiylIheU8b6sgANuJJnWzAIyVmcNezN+Fl+ZGnrc6MNjb/8zJA7GMe1F0V9aXTPAirkgVrbyOFp+lOctg747/NK kFajNOz0 JKV4dN8rdCJ4ZYFIVPxU5v9WTsdzHtdATOyDQx45hJutVK7J7EtdeQmsINqyc8mMFXikAswo9q8qVlbFwJGxXeC1HEMG8dkfhubEtau6yUYyNbZogMAyfTeHE8AGnKoKx8ISM5erDWNOc9HJJG8T/XkjseOZmA1P5I1ZC7iDCso1DrW89uiM8xi740/VZkLI9veEh9RoGC0ZlQBeWpI/ymclSZXjdqwDGrxSWOoW5lYuc2vpp7c/OWyW5pETaITNs9omt 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: Add large folio support for tmpfs write and fallocate paths matching the same high order preference mechanism used in the iomap buffered IO path as used in __filemap_get_folio(). Add shmem_mapping_size_orders() to get a hint for the orders of the folio based on the file size which takes care of the mapping requirements. Traditionally, tmpfs only supported PMD-sized huge folios. However nowadays with other file systems supporting any sized large folios, and extending anonymous to support mTHP, we should not restrict tmpfs to allocating only PMD-sized huge folios, making it more special. Instead, we should allow tmpfs can allocate any sized large folios. Considering that tmpfs already has the 'huge=' option to control the huge folios allocation, we can extend the 'huge=' option to allow any sized huge folios. The semantics of the 'huge=' mount option are: huge=never: no any sized huge folios huge=always: any sized huge folios huge=within_size: like 'always' but respect the i_size huge=advise: like 'always' if requested with fadvise()/madvise() Note: for tmpfs mmap() faults, due to the lack of a write size hint, still allocate the PMD-sized huge folios if huge=always/within_size/advise is set. Moreover, the 'deny' and 'force' testing options controlled by '/sys/kernel/mm/transparent_hugepage/shmem_enabled', still retain the same semantics. The 'deny' can disable any sized large folios for tmpfs, while the 'force' can enable PMD sized large folios for tmpfs. Co-developed-by: Daniel Gomez Signed-off-by: Daniel Gomez Signed-off-by: Baolin Wang --- mm/shmem.c | 91 +++++++++++++++++++++++++++++++++++++++++++++--------- 1 file changed, 77 insertions(+), 14 deletions(-) diff --git a/mm/shmem.c b/mm/shmem.c index 361da46c4bd5..98503a93a404 100644 --- a/mm/shmem.c +++ b/mm/shmem.c @@ -549,10 +549,50 @@ static bool shmem_confirm_swap(struct address_space *mapping, static int shmem_huge __read_mostly = SHMEM_HUGE_NEVER; +/** + * shmem_mapping_size_orders - Get allowable folio orders for the given file size. + * @mapping: Target address_space. + * @index: The page index. + * @size: The suggested size of the folio to create. + * + * This returns a high order for folios (when supported) based on the file size + * which the mapping currently allows at the given index. The index is relevant + * due to alignment considerations the mapping might have. The returned order + * may be less than the size passed. + * + * Return: The orders. + */ +static inline unsigned int +shmem_mapping_size_orders(struct address_space *mapping, pgoff_t index, loff_t write_end) +{ + unsigned int order; + size_t size; + + if (!mapping_large_folio_support(mapping) || !write_end) + return 0; + + /* Calculate the write size based on the write_end */ + size = write_end - (index << PAGE_SHIFT); + order = filemap_get_order(size); + if (!order) + return 0; + + /* If we're not aligned, allocate a smaller folio */ + if (index & ((1UL << order) - 1)) + order = __ffs(index); + + order = min_t(size_t, order, MAX_PAGECACHE_ORDER); + return order > 0 ? BIT(order + 1) - 1 : 0; +} + static unsigned int shmem_huge_global_enabled(struct inode *inode, pgoff_t index, loff_t write_end, bool shmem_huge_force, + struct vm_area_struct *vma, unsigned long vm_flags) { + unsigned long within_size_orders; + unsigned int order; + pgoff_t aligned_index; loff_t i_size; if (HPAGE_PMD_ORDER > MAX_PAGECACHE_ORDER) @@ -564,15 +604,41 @@ static unsigned int shmem_huge_global_enabled(struct inode *inode, pgoff_t index if (shmem_huge_force || shmem_huge == SHMEM_HUGE_FORCE) return BIT(HPAGE_PMD_ORDER); + /* + * The huge order allocation for anon shmem is controlled through + * the mTHP interface, so we still use PMD-sized huge order to + * check whether global control is enabled. + * + * For tmpfs mmap()'s huge order, we still use PMD-sized order to + * allocate huge pages due to lack of a write size hint. + * + * Otherwise, tmpfs will allow getting a highest order hint based on + * the size of write and fallocate paths, then will try each allowable + * huge orders. + */ switch (SHMEM_SB(inode->i_sb)->huge) { case SHMEM_HUGE_ALWAYS: - return BIT(HPAGE_PMD_ORDER); - case SHMEM_HUGE_WITHIN_SIZE: - index = round_up(index + 1, HPAGE_PMD_NR); - i_size = max(write_end, i_size_read(inode)); - i_size = round_up(i_size, PAGE_SIZE); - if (i_size >> PAGE_SHIFT >= index) + if (vma) return BIT(HPAGE_PMD_ORDER); + + return shmem_mapping_size_orders(inode->i_mapping, index, write_end); + case SHMEM_HUGE_WITHIN_SIZE: + if (vma) + within_size_orders = BIT(HPAGE_PMD_ORDER); + else + within_size_orders = shmem_mapping_size_orders(inode->i_mapping, + index, write_end); + + order = highest_order(within_size_orders); + while (within_size_orders) { + aligned_index = round_up(index + 1, 1 << order); + i_size = max(write_end, i_size_read(inode)); + i_size = round_up(i_size, PAGE_SIZE); + if (i_size >> PAGE_SHIFT >= aligned_index) + return within_size_orders; + + order = next_order(&within_size_orders, order); + } fallthrough; case SHMEM_HUGE_ADVISE: if (vm_flags & VM_HUGEPAGE) @@ -776,6 +842,7 @@ static unsigned long shmem_unused_huge_shrink(struct shmem_sb_info *sbinfo, static unsigned int shmem_huge_global_enabled(struct inode *inode, pgoff_t index, loff_t write_end, bool shmem_huge_force, + struct vm_area_struct *vma, unsigned long vm_flags) { return 0; @@ -1174,7 +1241,7 @@ static int shmem_getattr(struct mnt_idmap *idmap, inode_unlock_shared(inode); #ifdef CONFIG_TRANSPARENT_HUGEPAGE - if (shmem_huge_global_enabled(inode, 0, 0, false, 0) == + if (shmem_huge_global_enabled(inode, 0, 0, false, NULL, 0) == BIT(HPAGE_PMD_ORDER)) stat->blksize = HPAGE_PMD_SIZE; #endif @@ -1693,14 +1760,10 @@ unsigned long shmem_allowable_huge_orders(struct inode *inode, return 0; global_orders = shmem_huge_global_enabled(inode, index, write_end, - shmem_huge_force, vm_flags); - if (!vma || !vma_is_anon_shmem(vma)) { - /* - * For tmpfs, we now only support PMD sized THP if huge page - * is enabled, otherwise fallback to order 0. - */ + shmem_huge_force, vma, vm_flags); + /* Tmpfs huge pages allocation? */ + if (!vma || !vma_is_anon_shmem(vma)) return global_orders; - } /* * Following the 'deny' semantics of the top level, force the huge