From: Theodore Tso <tytso@MIT.EDU>
To: "Daniel Taylor" <Daniel.Taylor@wdc.com>
Cc: <linux-ext4@vger.kernel.org>
Subject: Re: Repost (from LKML): EXT3 FS and 64K blocks error
Date: Fri, 9 Jul 2010 08:03:35 -0400 [thread overview]
Message-ID: <7D7CBABC-08DF-4D5D-9655-D378529882B0@mit.edu> (raw)
In-Reply-To: <469D2D911E4BF043BFC8AD32E8E30F5B24AEE9@wdscexbe07.sc.wdc.com>
On Jul 8, 2010, at 8:10 PM, Daniel Taylor wrote:
> We're getting the following error with an EXT3 file sytem that
> has 64K blocks (2.6.32.12, on a powerpc), although I don't see
> any fixes for later kernels in a diff against 2.6.35-rc3):
>
> ext3_readdir: bad entry in directory #11: rec_len is smaller than minimal -
> offset=0, inode=0, rec_len=0, name_len=0
>
> That message is generated in fs/ext3/dir.c:ext3_check_dir_entry()
> when called from fs/ext3/dir.c:ext3_readdir(), AFAICT.
>
> Did something get missed when EXT3 handling for 64K blocks was
> implemented or when a new feature was added?
>
> FWIW, I do NOT see this on EXT4.
I very much doubt anyone has ever tested 64k blocks on ext3. There were
some extensions made to support 64k blocks with ext4, that had to do
with encoding the directory entry rec_len fields (which is 16 bits and will
overflow when trying to store the value 65536, as you have discovered).
Bottom line, it's not so much that EXT3 handling for 64k blocks was ever
*implemented*, as much as no one really thought about it much when
they set the #define's for maximum block size supported. :-)
64k block sizes hasn't received much testing on ext4 as far as I know, but
there was one developer who noted the dirent encoding problem and
proposed a fix.
Is there a particular reason why you care about this with ext3? Ext4 does
provide a superset of the features in ext3...
-- Ted
next prev parent reply other threads:[~2010-07-09 12:03 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-07-09 0:10 Repost (from LKML): EXT3 FS and 64K blocks error Daniel Taylor
2010-07-09 12:03 ` Theodore Tso [this message]
2010-07-09 22:32 ` Daniel Taylor
2010-07-10 0:26 ` Ted Ts'o
2010-07-10 22:14 ` Andreas Dilger
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=7D7CBABC-08DF-4D5D-9655-D378529882B0@mit.edu \
--to=tytso@mit.edu \
--cc=Daniel.Taylor@wdc.com \
--cc=linux-ext4@vger.kernel.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: 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