On Wed, Jun 11, 2008 at 08:02:32AM +0000, Holger Kiehl wrote:Hi Holger, You didn't say exactly which version of the kernel/ext4 you were testing, but a recent change which we made to ext4, but which hasn't been made to ext3 yet is that barrier support has been enabled to improve filesystem safety; unfortunately this does imply with it a slight performance slowdown, which would be more pronounced on benchmarks with small filesystems. So when you mount the filesystem for ext3 and ext4 for benchmarking purposes, you should consistently use a mount options of barrier=1 or barrier=0. With ext4, you can use the mount option "barrier=1,journal_async_commit" which should ameliorate part of the performance decrease. The reason why it is not yet the default is it requires support from e2fsprogs that has not been released except in development git branches; but as long as you don't require running e2fsck on uncleanly shutdown systems (probably not necessary if you are just benchmarking), you can use journal_async_commit in good health. Another change which might help out the bonnie benchmark, but which again requires the latest version of e2fsprogs is to create the filesystem with flex_bg filesystem feature. In fact, for convenience's sake, if you are using the latest development version of e2fsprogs, you can just use the command "mke2fs -t ext4dev /dev/hdXX" and it will set up the filesystem with the correct filesystem features for ext4. (The "ext4dev" sets the test_fs feature, and is basically there so it's clear we are still trying to finish up ext4 support.) As Aneesh has mentioned, there were some bugs in version of ext4's delalloc caused by insufficient testing of the ext4 patch queue some changes to our locking strategy went in. That's been fixed in the latest patch queue, and we're in the process of cleaning up delalloc before merging it into the mainline kernel. (When you tested ext4 two years ago, none of this was yet in mainline, so it's not a matter of things delalloc "disappearing", but rather that we've been slow getting to the point where it was ready for merging. - Ted --
| Mariusz Kozlowski | [PATCH 01] kmalloc + memset conversion co kzalloc |
| Greg Kroah-Hartman | [PATCH 001/196] Chinese: Add the known_regression URI to the HOWTO |
| Vladislav Bolkhovitin | Re: Integration of SCST in the mainstream Linux kernel |
| Jeremy Allison | Re: [RFC] Heads up on sys_fallocate() |
git: | |
| Gerrit Renker | [PATCH 0/37] dccp: Feature negotiation - last call for comments |
| Natalie Protasevich | [BUG] New Kernel Bugs |
| David Miller | Re: [GIT]: Networking |
| Jeff Garzik | Re: [bug?] tg3: Failed to load firmware "tigon/tg3_tso.bin" |
