From patchwork Tue Jun 25 09:06:42 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Gavin Shan X-Patchwork-Id: 13710785 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 C99F6C30653 for ; Tue, 25 Jun 2024 09:07:24 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 44E846B00F5; Tue, 25 Jun 2024 05:07:24 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 3FDC36B0109; Tue, 25 Jun 2024 05:07:24 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 2C5AE6B0103; Tue, 25 Jun 2024 05:07:24 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0015.hostedemail.com [216.40.44.15]) by kanga.kvack.org (Postfix) with ESMTP id 0EB266B02F1 for ; Tue, 25 Jun 2024 05:07:24 -0400 (EDT) Received: from smtpin11.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay02.hostedemail.com (Postfix) with ESMTP id A4A96120889 for ; Tue, 25 Jun 2024 09:07:23 +0000 (UTC) X-FDA: 82268832366.11.E989C44 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) by imf11.hostedemail.com (Postfix) with ESMTP id EC91E40008 for ; Tue, 25 Jun 2024 09:07:21 +0000 (UTC) Authentication-Results: imf11.hostedemail.com; dkim=pass header.d=redhat.com header.s=mimecast20190719 header.b=c869Lkjz; spf=pass (imf11.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=1719306427; 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: references:dkim-signature; bh=HhE9Ca9CX/1k2IL1D3853ePqCbKiPcGhrbBvznh+l8I=; b=jWR7PUgpQsdI3OdUJLBXn1FUZnNZVW6Zgl1747zF7BTtmseCdoEHudFSIy/oGLysXdW1pE 5HYJoDmQnNTKf2T6z1t6OdKLAmmh0UAAEXDoKFnUnvxLpZszOqPLMxnCzvVB0PUZDRTtVg 3TXC5x+JYU0Nr/6iIRIaFH1aFg2zEDs= ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1719306427; a=rsa-sha256; cv=none; b=YcyUgq8ICTTQWPjquZtGTp55UNEGtrEBUtokegDweuVWpKLJvb3UNzuqtwdhMS5paT3YTW 7WnkU3I5IL1jT089eOUCujxYVssrEhnlWrcaAr6i0x4CuNQN7vgkUPsZjTCk3/cQ7uPdS2 YCQHSfIMfAsc3v3ovLavy1GBOOZMWcM= ARC-Authentication-Results: i=1; imf11.hostedemail.com; dkim=pass header.d=redhat.com header.s=mimecast20190719 header.b=c869Lkjz; spf=pass (imf11.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 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1719306441; 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; bh=HhE9Ca9CX/1k2IL1D3853ePqCbKiPcGhrbBvznh+l8I=; b=c869Lkjz86d1BVU9THPIeSEwvtp8p/3PRjsii/T9WjYDv2pf1TEbIYyiG1ujTH0/RZcOQ7 +65o802om3lYdVLExfgWX2kM6LLzE/nQLUk20RmsHwLrOkJdraesii0rhtOrqvX9jSNIWi 1X/VFXIWjwp9cB/2/celojwcTD+7WCk= 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-271-kCbIwANPM0iHMkxZZDZyqw-1; Tue, 25 Jun 2024 05:07:12 -0400 X-MC-Unique: kCbIwANPM0iHMkxZZDZyqw-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 0B4991955E72; Tue, 25 Jun 2024 09:07:11 +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 22ABA1955E82; Tue, 25 Jun 2024 09:07:03 +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 0/4] mm/filemap: Limit page cache size to that supported by xarray Date: Tue, 25 Jun 2024 19:06:42 +1000 Message-ID: <20240625090646.1194644-1-gshan@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 3.0 on 10.30.177.15 X-Rspam-User: X-Rspamd-Server: rspam04 X-Rspamd-Queue-Id: EC91E40008 X-Stat-Signature: 6edfxoncgrstqdaksmdm4e9r9grq8tt6 X-HE-Tag: 1719306441-218021 X-HE-Meta: U2FsdGVkX19aF2ucpuGYh9+UFSd4AYwB2DHeEu4ip7gwZGLVpqvrnrAns8NEwOj9shKAjp2QQe1HcxPAIvveRKLh4PwdFh7qSzexSyL2tn77m8aAbDSoBdORiBnCC4cZexaMG6k1JK8yHx2ZgeVLCtBUSI8s/rGFggnpxTtNOlULLRVUSGx30jXMQ7Pp95wR3a3vJ6Kgi9ctoKM80XsSJaX0ciSlxN9LAG1jW75yMv8WVtFHiY/FB7nLWasnrDT+jHd0r4pneyTUBzcePfB9V5sTg1gaPxmjHoLXsFGhZzvRFEMl1H5RFyS89nOuT9plH5bNrhANfZugjb2tJBVHYLsg3Yp9D558YfM+qFjp4hvZ7FdNPN4HzN9fVdt+8CM8AGbVMpxGJg0M2Q6bf0W+my+xmgN6u8vTxlYS2eg2vL4wqse9gako5QuERFiyUMB0MJ0mFBjfC+yvTuX+NEGC+G0kre/GbZ1J1jjsEvvfpf8iNa3XS88hS9qQY/P6U1MGNlNIoWW/Wflwn7DdCkrkBk+/WW5HZVdJNU+sGp/WmBRio8c+1q8+AcqMaKCgDY5WObYSYPWJnFpJCk+/oH/OjTD0TWiL0D85eKexyZZhtOkWIzLSGfVgCIWZGesrA34dRB1Z5DkYxOPGK/7g9NfWBx8Nxe0BsVA0KyyAHi1GhOTZZMSqBxXK8SvqKaKGbUeXM9evpTwxZ1CJ5rpGgwgtxTnovgjjiKHMqDt9ZpQgatUvDOb2SNmPIqQuMWjaBYBJkIclqis2Pgj4Uq4AEARVCdXs0UdK61X7qwEyJI1qRHL4ZN+Y9uODN1cBMcGnw2Kgj7gzLBsh28xiCF2zpV1ZK/9JdVVUzKidJJw6FzDFTJYubPQOTgXtNCcIWBncXonUVKUKmHZfbPNKymNHbvnNer5qhNve+PZTBJ6pLULuQT0nl87uwJyh6CiOL6Oei2H33rO6B55f8jeig+QXYdl DhzVY3S8 +t7t5RHJCObsYhKIr7jnj3j/ctFZdoNQvGAJzwtAU0m44ih8qQ1aNDb85YLSwXR1hLB8oY+PrxurmhNzg9QMwE1GmewbtENwe6pGEcdPGkmm3X1tFOxQOGmg0Fpm4n1eyulXZpKUWtcRX04xJh307m+8anzekxxgQ6T+Up0GkuMU5Hjpn5X1KJTNl9woMuENhGPLRAWpi3yoSaJJ3aRFXrYEVSSaoSp+t+Dw99a2FKkJehdxfOcgBoM6vJwOBc7YuFihmN8A7kvKTP5tmVYELDS69nRE5dz8bbUHgEdleFI3R6BG+uLRETI9k0DFmdV+BxDuBp/cIH3xsktOo0n8JO5d6bGNUtCzU/vkc9qZQPdnLmR/i+ETTK2iLZ9TsLWjzgxlalLbVnbXGaDcFKSaoGzPE44SXCFHNBUvQGsjZzinEyw/ycK9OVuortEHA+LTGL2ne2k8EgmydqF3fz0E6BypHKg== 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: Currently, xarray can't support arbitrary page cache size. More details can be found from the WARN_ON() statement in xas_split_alloc(). In our test whose code is attached below, we hit the WARN_ON() on ARM64 system where the base page size is 64KB and huge page size is 512MB. The issue was reported long time ago and some discussions on it can be found here [1]. [1] https://www.spinics.net/lists/linux-xfs/msg75404.html In order to fix the issue, we need to adjust MAX_PAGECACHE_ORDER to one supported by xarray and avoid PMD-sized page cache if needed. The code changes are suggested by David Hildenbrand. PATCH[1] adjusts MAX_PAGECACHE_ORDER to that supported by xarray PATCH[2-3] avoids PMD-sized page cache in the synchronous readahead path PATCH[4] avoids PMD-sized page cache for shmem files if needed Test program ============ # cat test.c #define _GNU_SOURCE #include #include #include #include #include #include #include #include #define TEST_XFS_FILENAME "/tmp/data" #define TEST_SHMEM_FILENAME "/dev/shm/data" #define TEST_MEM_SIZE 0x20000000 int main(int argc, char **argv) { const char *filename; int fd = 0; void *buf = (void *)-1, *p; int pgsize = getpagesize(); int ret; if (pgsize != 0x10000) { fprintf(stderr, "64KB base page size is required\n"); return -EPERM; } system("echo force > /sys/kernel/mm/transparent_hugepage/shmem_enabled"); system("rm -fr /tmp/data"); system("rm -fr /dev/shm/data"); system("echo 1 > /proc/sys/vm/drop_caches"); /* Open xfs or shmem file */ filename = TEST_XFS_FILENAME; if (argc > 1 && !strcmp(argv[1], "shmem")) filename = TEST_SHMEM_FILENAME; fd = open(filename, O_CREAT | O_RDWR | O_TRUNC); if (fd < 0) { fprintf(stderr, "Unable to open <%s>\n", filename); return -EIO; } /* Extend file size */ ret = ftruncate(fd, TEST_MEM_SIZE); if (ret) { fprintf(stderr, "Error %d to ftruncate()\n", ret); goto cleanup; } /* Create VMA */ buf = mmap(NULL, TEST_MEM_SIZE, PROT_READ | PROT_WRITE, MAP_SHARED, fd, 0); if (buf == (void *)-1) { fprintf(stderr, "Unable to mmap <%s>\n", filename); goto cleanup; } fprintf(stdout, "mapped buffer at 0x%p\n", buf); ret = madvise(buf, TEST_MEM_SIZE, MADV_HUGEPAGE); if (ret) { fprintf(stderr, "Unable to madvise(MADV_HUGEPAGE)\n"); goto cleanup; } /* Populate VMA */ ret = madvise(buf, TEST_MEM_SIZE, MADV_POPULATE_WRITE); if (ret) { fprintf(stderr, "Error %d to madvise(MADV_POPULATE_WRITE)\n", ret); goto cleanup; } /* Punch the file to enforce xarray split */ ret = fallocate(fd, FALLOC_FL_KEEP_SIZE | FALLOC_FL_PUNCH_HOLE, TEST_MEM_SIZE - pgsize, pgsize); if (ret) fprintf(stderr, "Error %d to fallocate()\n", ret); cleanup: if (buf != (void *)-1) munmap(buf, TEST_MEM_SIZE); if (fd > 0) close(fd); return 0; } # gcc test.c -o test # cat /proc/1/smaps | grep KernelPageSize | head -n 1 KernelPageSize: 64 kB # ./test shmem : ------------[ cut here ]------------ WARNING: CPU: 17 PID: 5253 at lib/xarray.c:1025 xas_split_alloc+0xf8/0x128 Modules linked in: 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 nf_tables rfkill nfnetlink vfat fat virtio_balloon \ drm fuse xfs libcrc32c crct10dif_ce ghash_ce sha2_ce sha256_arm64 \ virtio_net sha1_ce net_failover failover virtio_console virtio_blk \ dimlib virtio_mmio CPU: 17 PID: 5253 Comm: test Kdump: loaded Tainted: G W 6.10.0-rc5-gavin+ #12 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 : ffff80008a92f5b0 x29: ffff80008a92f5b0 x28: ffff80008a92f610 x27: ffff80008a92f728 x26: 0000000000000cc0 x25: 000000000000000d x24: ffff0000cf00c858 x23: ffff80008a92f610 x22: ffffffdfc0600000 x21: 0000000000000000 x20: 0000000000000000 x19: ffffffdfc0600000 x18: 0000000000000000 x17: 0000000000000000 x16: 0000018000000000 x15: 3374004000000000 x14: 0000e00000000000 x13: 0000000000002000 x12: 0000000000000020 x11: 3374000000000000 x10: 3374e1c0ffff6000 x9 : ffffb463a84c681c x8 : 0000000000000003 x7 : 0000000000000000 x6 : ffff00011c976ce0 x5 : ffffb463aa47e378 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 Gavin Shan (4): mm/filemap: Make MAX_PAGECACHE_ORDER acceptable to xarray mm/filemap: Skip to allocate PMD-sized folios if needed mm/readahead: Limit page cache size in page_cache_ra_order() mm/shmem: Disable PMD-sized page cache if needed include/linux/pagemap.h | 11 +++++++++-- mm/filemap.c | 2 +- mm/readahead.c | 8 ++++---- mm/shmem.c | 15 +++++++++++++-- 4 files changed, 27 insertions(+), 9 deletions(-)