From patchwork Tue Jun 25 09:06:46 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Gavin Shan X-Patchwork-Id: 13710789 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 BA18AC2BBCA for ; Tue, 25 Jun 2024 09:07:47 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 44DED6B0202; Tue, 25 Jun 2024 05:07:47 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 3FFBD6B0203; Tue, 25 Jun 2024 05:07:47 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 2C70F6B0204; Tue, 25 Jun 2024 05:07:47 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0016.hostedemail.com [216.40.44.16]) by kanga.kvack.org (Postfix) with ESMTP id 0EC276B0202 for ; Tue, 25 Jun 2024 05:07:47 -0400 (EDT) Received: from smtpin08.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay02.hostedemail.com (Postfix) with ESMTP id B5C48120889 for ; Tue, 25 Jun 2024 09:07:46 +0000 (UTC) X-FDA: 82268833332.08.4A465CA Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) by imf29.hostedemail.com (Postfix) with ESMTP id 0689A120016 for ; Tue, 25 Jun 2024 09:07:44 +0000 (UTC) Authentication-Results: imf29.hostedemail.com; dkim=pass header.d=redhat.com header.s=mimecast20190719 header.b=GV6qY0rY; spf=pass (imf29.hostedemail.com: domain of gshan@redhat.com designates 170.10.129.124 as permitted sender) smtp.mailfrom=gshan@redhat.com; dmarc=pass (policy=none) header.from=redhat.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1719306453; 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-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=CGvYW01hrriXIF+o1pxrfngY25C2k8nRiGAXWHuPddk=; b=WFASLoNZ4nBFs3GxtoPa0ivc4xrIJemMLm7txgdUlllbK7bwUocA5hPOwhPWWfYouJM2Ah TVdX4JYIyonyz9rPxNl1vIDC0gzTse/ybnXreA0/BcsiWq/QiLChyTluhFQinZUornZprG 3TynlDyAIvMs99MYDgENiGmWh1eKQ6Y= ARC-Authentication-Results: i=1; imf29.hostedemail.com; dkim=pass header.d=redhat.com header.s=mimecast20190719 header.b=GV6qY0rY; spf=pass (imf29.hostedemail.com: domain of gshan@redhat.com designates 170.10.129.124 as permitted sender) smtp.mailfrom=gshan@redhat.com; dmarc=pass (policy=none) header.from=redhat.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1719306453; a=rsa-sha256; cv=none; b=cv5pEywy9x/W8sAoPossEqZnYZbgU/hUgs+Pl7pTgDCN4eoeGWiLPT08YX3blEMtph3bfj BntyF2eF08Qo8u/LNPcwVvO+YhJfeSDGNrRbbmWwhC7e4UevtzI7WdtJ6prE4ZKFytVUP3 PTckdn1odA+q3OSugjqrsiqgV2BjHfE= DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1719306464; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=CGvYW01hrriXIF+o1pxrfngY25C2k8nRiGAXWHuPddk=; b=GV6qY0rYm+835gmBgxQZ6I9Rs+zD2iP5qZdWokWdDaUM5bAmwiSAlU9JqJKUITkQdTd9Bq FJw6cyYBVEhQNKleWJFbFmJOv5+BRl55F8dQIZWkzfWZLKOa9qXu6Ynt1EW9YFDLiCUAkT 5IRdxvggZB1v99O6yiFMg1AK6z3LhYI= Received: from mx-prod-mc-02.mail-002.prod.us-west-2.aws.redhat.com (ec2-54-186-198-63.us-west-2.compute.amazonaws.com [54.186.198.63]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-647-6AGImXKrPlyqhXgDXD6VTQ-1; Tue, 25 Jun 2024 05:07:41 -0400 X-MC-Unique: 6AGImXKrPlyqhXgDXD6VTQ-1 Received: from mx-prod-int-02.mail-002.prod.us-west-2.aws.redhat.com (mx-prod-int-02.mail-002.prod.us-west-2.aws.redhat.com [10.30.177.15]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by mx-prod-mc-02.mail-002.prod.us-west-2.aws.redhat.com (Postfix) with ESMTPS id 36A681955DCE; Tue, 25 Jun 2024 09:07:40 +0000 (UTC) Received: from gshan-thinkpadx1nanogen2.remote.csb (unknown [10.67.24.180]) by mx-prod-int-02.mail-002.prod.us-west-2.aws.redhat.com (Postfix) with ESMTPS id 282E41956087; Tue, 25 Jun 2024 09:07:33 +0000 (UTC) From: Gavin Shan To: linux-mm@kvack.org Cc: linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org, david@redhat.com, djwong@kernel.org, willy@infradead.org, akpm@linux-foundation.org, hughd@google.com, torvalds@linux-foundation.org, zhenyzha@redhat.com, shan.gavin@gmail.com Subject: [PATCH 4/4] mm/shmem: Disable PMD-sized page cache if needed Date: Tue, 25 Jun 2024 19:06:46 +1000 Message-ID: <20240625090646.1194644-5-gshan@redhat.com> In-Reply-To: <20240625090646.1194644-1-gshan@redhat.com> References: <20240625090646.1194644-1-gshan@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 3.0 on 10.30.177.15 X-Stat-Signature: k8ugkq9ogtma1wsquyes9xpqg4u8ds8e X-Rspam-User: X-Rspamd-Queue-Id: 0689A120016 X-Rspamd-Server: rspam02 X-HE-Tag: 1719306464-641401 X-HE-Meta: U2FsdGVkX18Q+6MyXdWDI1lneOVsut1scJ/bPsndgw5itp1rV5UrVXX0rlca8t7wqz3LIdCS1Hn6rVapRWYZn2l+ci2ULQnr3ppx1ScWKVE/Flkg28n8G8N+FfkRGLdLRRK+nsszSXtFOkjqINa7eX63mOZWlIhmtu6h3OLIG36mY0pLFiWWZrKr+fMtJ5iZdcVmVfRVfprw6pbia34pVSb+nZ5zItlWE/ZZ+/2ii24iqJfh1LgrsFVaxWraVmJg3dpVHUN+GnxDiNgT2KOEs/Tt0X/y6rcUtmYE8ULtuJtji4RWdVSOEP5cJkZ/qfAZ5fkmnHivkr/t4SXcUP6MRAQ6JhkLKVOcO55vsNmpPbQPy8CadkqDs5n8MwN6c8e+XZma/ghjKG2ArwGA4oUMN3L75NBnsamOlaz9cvs+lDSFEfFC/wrGDhPV++BYZ6SLrK8TgD2EpVAnGsF3pCI/OzAXUKJ5RwzR0ORasW+NkSA+RBdwbBBt4kZLpS84FxkQz4NhUjGPH2ahcr3HUGzrmrqvpUsOXrjmSlLImlD4W3gqDjuzh+QFaLI9eqfhiI8TmZTQzucdgejOOgV75/g/BzFBhteJsHT+6zYJIrzGV5ZUn2DMU33QT/vRfyNTnlVxlBSoQxhMZ6u+l58uUwtW+K6+BRRgU1QSr6sCfSXh6L9b5LJRv5uEp0IReRCncQH8vg9pFPpjDxW6WTXHFSd9KRJNcWNTnD1Z40Ctww8CogE74R7Tp5J+ObaieOjdi4hCejte0++pLchppEUdTZSjLgNmdWFd83xK0QtnjRN6E5FAezzOpfTA1TGgDOeoQ4RTDhzr2zeM67lWfz6c4Hjwvyy/yq0U+8GYwkw4yTI8ZP92gm0GGhjwGobZxFDoREnDeKWrwMQeyWqBSl8zdlA4B9oUBIOW8GFFE/WFUe89rbVEymOmwZfzWWwle+dGtuNG3QmgNms8DSaGUBWrZLK g7sQzfr9 aVxwx57o2ETrcO5X+wxed4AYsDQGemkdJ6ImF8uj+kGQ3YiJWktipdKLKIQ0JJaRsXZgL/8p6kiYf8w4n+mQjJdkfjBkcvvJNiZ3of23oK0L0n9FZFVw8wOow8SwA5TlRxYuBCtar1NMvzDhWSgtaIBCsfI965cG3nwm6dTr/P5yWigOV7Y+6AJAm761xCrTcXL5eIoI4898tQHlLJDXxYdDhYCmIFs1hLgISTaSfXF6SSHbwHirTiSm3SBRYO/6A9diZeh0t1S1wKix2dwYnU4JICtgGodTG09Qul9FcQsX1JTXlta5omRGj+u4OqOxm53MFzL9JtHwHUoi7bSzunIQ8e0yiNoZSZ1bMd95k0BGpRBlEqFAVbaM9siOJQgcL6+Lt 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: For shmem files, it's possible that PMD-sized page cache can't be supported by xarray. For example, 512MB page cache on ARM64 when the base page size is 64KB can't be supported by xarray. It leads to errors as the following messages indicate when this sort of xarray entry is split. WARNING: CPU: 34 PID: 7578 at lib/xarray.c:1025 xas_split_alloc+0xf8/0x128 Modules linked in: binfmt_misc nft_fib_inet nft_fib_ipv4 nft_fib_ipv6 \ nft_fib nft_reject_inet nf_reject_ipv4 nf_reject_ipv6 nft_reject \ nft_ct nft_chain_nat nf_nat nf_conntrack nf_defrag_ipv6 nf_defrag_ipv4 \ ip_set rfkill nf_tables nfnetlink vfat fat virtio_balloon drm fuse xfs \ libcrc32c crct10dif_ce ghash_ce sha2_ce sha256_arm64 sha1_ce virtio_net \ net_failover virtio_console virtio_blk failover dimlib virtio_mmio CPU: 34 PID: 7578 Comm: test Kdump: loaded Tainted: G W 6.10.0-rc5-gavin+ #9 Hardware name: QEMU KVM Virtual Machine, BIOS edk2-20240524-1.el9 05/24/2024 pstate: 83400005 (Nzcv daif +PAN -UAO +TCO +DIT -SSBS BTYPE=--) pc : xas_split_alloc+0xf8/0x128 lr : split_huge_page_to_list_to_order+0x1c4/0x720 sp : ffff8000882af5f0 x29: ffff8000882af5f0 x28: ffff8000882af650 x27: ffff8000882af768 x26: 0000000000000cc0 x25: 000000000000000d x24: ffff00010625b858 x23: ffff8000882af650 x22: ffffffdfc0900000 x21: 0000000000000000 x20: 0000000000000000 x19: ffffffdfc0900000 x18: 0000000000000000 x17: 0000000000000000 x16: 0000018000000000 x15: 52f8004000000000 x14: 0000e00000000000 x13: 0000000000002000 x12: 0000000000000020 x11: 52f8000000000000 x10: 52f8e1c0ffff6000 x9 : ffffbeb9619a681c x8 : 0000000000000003 x7 : 0000000000000000 x6 : ffff00010b02ddb0 x5 : ffffbeb96395e378 x4 : 0000000000000000 x3 : 0000000000000cc0 x2 : 000000000000000d x1 : 000000000000000c x0 : 0000000000000000 Call trace: xas_split_alloc+0xf8/0x128 split_huge_page_to_list_to_order+0x1c4/0x720 truncate_inode_partial_folio+0xdc/0x160 shmem_undo_range+0x2bc/0x6a8 shmem_fallocate+0x134/0x430 vfs_fallocate+0x124/0x2e8 ksys_fallocate+0x4c/0xa0 __arm64_sys_fallocate+0x24/0x38 invoke_syscall.constprop.0+0x7c/0xd8 do_el0_svc+0xb4/0xd0 el0_svc+0x44/0x1d8 el0t_64_sync_handler+0x134/0x150 el0t_64_sync+0x17c/0x180 Fix it by disabling PMD-sized page cache when HPAGE_PMD_ORDER is larger than MAX_PAGECACHE_ORDER. Suggested-by: David Hildenbrand Signed-off-by: Gavin Shan Acked-by: David Hildenbrand --- mm/shmem.c | 15 +++++++++++++-- 1 file changed, 13 insertions(+), 2 deletions(-) diff --git a/mm/shmem.c b/mm/shmem.c index a8b181a63402..5453875e3810 100644 --- a/mm/shmem.c +++ b/mm/shmem.c @@ -541,8 +541,9 @@ static bool shmem_confirm_swap(struct address_space *mapping, static int shmem_huge __read_mostly = SHMEM_HUGE_NEVER; -bool shmem_is_huge(struct inode *inode, pgoff_t index, bool shmem_huge_force, - struct mm_struct *mm, unsigned long vm_flags) +static bool __shmem_is_huge(struct inode *inode, pgoff_t index, + bool shmem_huge_force, struct mm_struct *mm, + unsigned long vm_flags) { loff_t i_size; @@ -573,6 +574,16 @@ bool shmem_is_huge(struct inode *inode, pgoff_t index, bool shmem_huge_force, } } +bool shmem_is_huge(struct inode *inode, pgoff_t index, + bool shmem_huge_force, struct mm_struct *mm, + unsigned long vm_flags) +{ + if (!__shmem_is_huge(inode, index, shmem_huge_force, mm, vm_flags)) + return false; + + return HPAGE_PMD_ORDER <= MAX_PAGECACHE_ORDER; +} + #if defined(CONFIG_SYSFS) static int shmem_parse_huge(const char *str) {