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

[Bob Furber]

Wolfgang Denk wrote:

Thanks Wolfgang. You were right. I had used u-boot-2009.08-rc1 which 
still had the 128 byte inode size hard-coded in 
...\u-boot-2009.08-rc1\fs\ext2\ext2fs.c. Upgrading to u-boot-2009.08-rc2 
now allows me to boot from 2GB SD cards formatted with Ubuntu-2.6.27.

Thanks,

Bob Furber

> please keep the mailing list on cc:
>   

My carelessness. I am new to this list and accustomed to simply click 
"Reply".

> 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
>
>