Linux-Fsdevel Archive on lore.kernel.org help / color / mirror / Atom feed
From: Jan Kara <jack@suse.cz> To: Mikulas Patocka <mpatocka@redhat.com> Cc: Dave Chinner <david@fromorbit.com>, Dan Williams <dan.j.williams@intel.com>, Linus Torvalds <torvalds@linux-foundation.org>, Alexander Viro <viro@zeniv.linux.org.uk>, Andrew Morton <akpm@linux-foundation.org>, Vishal Verma <vishal.l.verma@intel.com>, Dave Jiang <dave.jiang@intel.com>, Ira Weiny <ira.weiny@intel.com>, Matthew Wilcox <willy@infradead.org>, Jan Kara <jack@suse.cz>, Eric Sandeen <esandeen@redhat.com>, Dave Chinner <dchinner@redhat.com>, "Kani, Toshi" <toshi.kani@hpe.com>, "Norton, Scott J" <scott.norton@hpe.com>, "Tadakamadla, Rajesh (DCIG/CDI/HPS Perf)" <rajesh.tadakamadla@hpe.com>, Linux Kernel Mailing List <linux-kernel@vger.kernel.org>, linux-fsdevel <linux-fsdevel@vger.kernel.org>, linux-nvdimm <linux-nvdimm@lists.01.org> Subject: Re: NVFS XFS metadata (was: [PATCH] pmem: export the symbols __copy_user_flushcache and __copy_from_user_flushcache) Date: Wed, 23 Sep 2020 11:57:39 +0200 [thread overview] Message-ID: <20200923095739.GC6719@quack2.suse.cz> (raw) In-Reply-To: <alpine.LRH.2.02.2009220815420.16480@file01.intranet.prod.int.rdu2.redhat.com> On Tue 22-09-20 12:46:05, Mikulas Patocka wrote: > > mapping 2^21 blocks requires a 5 level indirect tree. Which one if going > > to be faster to truncate away - a single record or 2 million individual > > blocks? > > > > IOWs, we can take afford to take an extra cacheline miss or two on a > > tree block search, because we're accessing and managing orders of > > magnitude fewer records in the mapping tree than an indirect block > > tree. > > > > PMEM doesn't change this: extents are more time and space efficient > > at scale for mapping trees than indirect block trees regardless > > of the storage medium in use. > > PMEM doesn't have to be read linearly, so the attempts to allocate large > linear space are not needed. They won't harm but they won't help either. > > That's why NVFS has very simple block allocation alrogithm - it uses a > per-cpu pointer and tries to allocate by a bit scan from this pointer. If > the group is full, it tries a random group with above-average number of > free blocks. I agree with Dave here. People are interested in 2MB or 1GB contiguous allocations for DAX so that files can be mapped at PMD or event PUD levels thus saving a lot of CPU time on page faults and TLB. > EXT4 uses bit scan for allocations and people haven't complained that it's > inefficient, so it is probably OK. Yes, it is more or less OK but once you get to 1TB filesystem size and larger, the number of block groups grows enough that it isn't that great anymore. We are actually considering new allocation schemes for ext4 for this large filesystems... > If you think that the lack of journaling is show-stopper, I can implement > it. But then, I'll have something that has complexity of EXT4 and > performance of EXT4. So that there will no longer be any reason why to use > NVFS over EXT4. Without journaling, it will be faster than EXT4 and it may > attract some users who want good performance and who don't care about GID > and UID being updated atomically, etc. I'd hope that your filesystem offers more performance benefits than just what you can get from a lack of journalling :). ext4 can be configured to run without a journal as well - mkfs.ext4 -O ^has_journal. And yes, it does significantly improve performance for some workloads but you have to have some way to recover from crashes so it's mostly used for scratch filesystems (e.g. in build systems, Google uses this feature a lot for some of their infrastructure as well). Honza -- Jan Kara <jack@suse.com> SUSE Labs, CR
next prev parent reply other threads:[~2020-09-23 9:57 UTC|newest] Thread overview: 39+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-09-15 12:34 [RFC] nvfs: a filesystem for persistent memory Mikulas Patocka 2020-09-15 13:00 ` Matthew Wilcox 2020-09-15 13:24 ` Mikulas Patocka 2020-09-22 10:04 ` Ritesh Harjani 2020-09-15 15:16 ` Dan Williams 2020-09-15 16:58 ` Mikulas Patocka 2020-09-15 17:38 ` Mikulas Patocka 2020-09-16 10:57 ` [PATCH] pmem: export the symbols __copy_user_flushcache and __copy_from_user_flushcache Mikulas Patocka 2020-09-16 16:21 ` Dan Williams 2020-09-16 17:24 ` Mikulas Patocka 2020-09-16 17:40 ` Dan Williams 2020-09-16 18:06 ` Mikulas Patocka 2020-09-21 16:20 ` NVFS XFS metadata (was: [PATCH] pmem: export the symbols __copy_user_flushcache and __copy_from_user_flushcache) Mikulas Patocka 2020-09-22 5:03 ` Dave Chinner 2020-09-22 16:46 ` Mikulas Patocka 2020-09-22 17:25 ` Matthew Wilcox 2020-09-24 15:00 ` Mikulas Patocka 2020-09-28 15:22 ` Mikulas Patocka 2020-09-23 2:45 ` Dave Chinner 2020-09-23 9:20 ` A bug in ext4 with big directories (was: NVFS XFS metadata) Mikulas Patocka 2020-09-23 9:44 ` Jan Kara 2020-09-23 12:46 ` Mikulas Patocka 2020-09-23 20:20 ` Andreas Dilger 2020-09-23 17:19 ` NVFS XFS metadata (was: [PATCH] pmem: export the symbols __copy_user_flushcache and __copy_from_user_flushcache) Mikulas Patocka 2020-09-23 9:57 ` Jan Kara [this message] 2020-09-23 13:11 ` Mikulas Patocka 2020-09-23 15:04 ` Matthew Wilcox 2020-09-22 12:28 ` Matthew Wilcox 2020-09-22 12:39 ` Mikulas Patocka 2020-09-16 18:56 ` [PATCH] pmem: fix __copy_user_flushcache Mikulas Patocka 2020-09-18 1:53 ` Dan Williams 2020-09-18 12:25 ` the "read" syscall sees partial effects of the "write" syscall Mikulas Patocka 2020-09-18 13:13 ` Jan Kara 2020-09-18 18:02 ` Linus Torvalds 2020-09-20 23:41 ` Dave Chinner 2020-09-17 6:50 ` [PATCH] pmem: export the symbols __copy_user_flushcache and __copy_from_user_flushcache Christoph Hellwig 2020-09-21 16:19 ` [RFC] nvfs: a filesystem for persistent memory Mikulas Patocka 2020-09-21 16:29 ` Dan Williams 2020-09-22 15:43 ` Ira Weiny
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20200923095739.GC6719@quack2.suse.cz \ --to=jack@suse.cz \ --cc=akpm@linux-foundation.org \ --cc=dan.j.williams@intel.com \ --cc=dave.jiang@intel.com \ --cc=david@fromorbit.com \ --cc=dchinner@redhat.com \ --cc=esandeen@redhat.com \ --cc=ira.weiny@intel.com \ --cc=linux-fsdevel@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-nvdimm@lists.01.org \ --cc=mpatocka@redhat.com \ --cc=rajesh.tadakamadla@hpe.com \ --cc=scott.norton@hpe.com \ --cc=torvalds@linux-foundation.org \ --cc=toshi.kani@hpe.com \ --cc=viro@zeniv.linux.org.uk \ --cc=vishal.l.verma@intel.com \ --cc=willy@infradead.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).