[U-Boot] ext2fs.c/ext2fs_mount() fails when inode_size = 256

[Wolfgang Denk]

Dear Bob Furber,

please keep the mailing list on cc:

In message <4A832C44.3060503 at steroidmicros.com> you wrote:
> 
> > You failed to mention some really important fects, like for example
> > which version of U-Boot you are using.
> 
> ftp://ftp.denx.de/pub/u-boot/ --> u-boot-2009.08-rc2.tar.bz2, which 
> appeared to be the most recent relrase.

Indeed.

> > This commit most probably fixes your issue:
> >
> > commit 56fdaadc124a8ef9ec0fd8ff578233ec3b1137be
> > Author: Weirich, Bernhard <Bernhard.Weirich at riedel.net>
> > Date:   Wed Jun 10 14:00:37 2009 +0200
> >
> >     ext2: fix inode size and calculations
> >     
> >     Signed-off-by: unsik Kim <donari75 at gmail.com>
> >     Signed-off-by: Bernhard Weirich <bernhard.weirich at riedel.net>
> >     Signed-off-by: Wolfgang Denk <wd at denx.de>
> >     Tested-by: Wolfgang Denk <wd at denx.de>
> >   
> 
> Forgive me, but I am new at this and I have no idea how to locate 
> "commit 56fdaadc124a8ef9ec0fd8ff578233ec3b1137be".

You can use for example "git show" in your U-Boot git repository.
In worst case, you can search the CHANGELOG.

Anyway - commit 56fdaadc is:

	-> git-describe 56fdaadc
	v2009.08-rc1-1-g56fdaad

so it's the first commit after v2009.08-rc1 and thus included in
v2009.08-rc2.

If this doesn't work for you, I'd be interested in receiving a test
case, i. e. a minimal (as small as possible) file system image that
shows the erroneous behaviour.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
I don't mind criticism. You know me. I've  never  been  one  to  take
offence  at  criticism. No one could say I'm the sort to take offence
at criticism -- Not twice, anyway. Not without blowing bubbles.
                                  - Terry Pratchett, _Witches Abroad_