From: Theodore Tso <tytso@mit.edu>
To: Toshiyuki Okajima <toshi.okajima@jp.fujitsu.com>
Cc: Andreas Dilger <adilger@sun.com>, linux-ext4@vger.kernel.org
Subject: Re: [PATCH][BUG] ext4: dx_map_entry cannot support over 64KB block size
Date: Sun, 21 Jun 2009 22:47:57 -0400 [thread overview]
Message-ID: <20090622024757.GB26079@mit.edu> (raw)
In-Reply-To: <4A3EE269.3020704@jp.fujitsu.com>
On Mon, Jun 22, 2009 at 10:46:17AM +0900, Toshiyuki Okajima wrote:
> I have been reviewing the logic around dir_index(fs/ext4/dir.c
> fs/ext4/namei.c) for the sake of performance and quality
> improvement. Then I found this bug. Sorry, therefore it is not a
> result of the real test.
I've started doing some testing on the e2fsprogs side, and fixed some
problems in for the upcoming e2fsprogs 1.41.7, but both the kernel and
e2fsprogs currently don't officially support a blocksize larger than
64k. At least e2fsprogs in the git tree will mke2fs that passes
e2fsck for a 128k block filesystem; this is *not* yet true for 256k
block filesystems (but it requires some commits in the 64k block
numbers patches which I don't want to merge into the maint branch).
Are you aware of any patches that enable an IA64 system to support a
page size greater than 64kB? (And whether any customer would actually
want to use them, given the downsides of very large page size?) The
issue is that Linux doesn't support filesystem block sizes > than the
page size. In any case, I don't mind adding patches that attempt to
make it better to support large block sizes; I don't really want to
claim that we support it until we can actually fully test for that
feature, though. Is it your intent to actually try to provide support
for this at some point? I'll help you if you are, but you'll have to
do the testing, since I don't have access to an IA64 platform that
might be able to support these sorts of large pages.
Thanks,
- Ted
next prev parent reply other threads:[~2009-06-22 2:48 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-06-05 7:50 [PATCH][BUG] ext4: dx_map_entry cannot support over 64KB block size Toshiyuki Okajima
2009-06-05 21:20 ` Andreas Dilger
2009-06-08 7:30 ` Toshiyuki Okajima
2009-06-21 3:57 ` Theodore Tso
2009-06-22 1:46 ` Toshiyuki Okajima
2009-06-22 2:47 ` Theodore Tso [this message]
2009-06-23 3:32 ` Toshiyuki Okajima
2009-06-23 15:28 ` Theodore Tso
2009-06-08 13:31 ` Theodore Tso
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=20090622024757.GB26079@mit.edu \
--to=tytso@mit.edu \
--cc=adilger@sun.com \
--cc=linux-ext4@vger.kernel.org \
--cc=toshi.okajima@jp.fujitsu.com \
/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: link
Be 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).