Re: [U-Boot] Problems with ext2ls SD

2009-04-25 Thread Wolfgang Denk
Dear Karl,

In message 669754ad0903281214x77796b95x76062f1c428ec...@mail.gmail.com you 
wrote:

 I am simply using fdisk under Ubuntu 8.04 and a 2GB SD card, I create a
 primary partition of type 0x83. When I run dumpe2fs it shows an inode size
 of 256 bytes.

I confirm the problem. I finally found some time to run some tests,
and was able to reproduce the problem.


I even re-tested the old patch that Ryan Chen  posted  long  ago  (in
July  2008),  but  as  my  earlier  tests  indicated,  this  patch is
seriously broken and does not work at all.


So we have to state that ext2 file system support in U-Boot is broken
for recent ext2 versions which use bigger inode sizes.

Best regards,
Viele Grüße,

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: w...@denx.de
The most difficult thing in the world is to know how to  do  a  thing
and to watch someone else doing it wrong, without commenting.
-- T.H. White
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] Problems with ext2ls SD

2009-03-31 Thread Wolfgang Denk
Dear DVM,

In message 22756563.p...@talk.nabble.com you wrote:
 
 Further debugging reveals UBoot uses a fixed length for inodes (128 bye). The
 inodes on my 2GB SD card are 256 bytes There was a patch that fizzled out
 back in January looking at this
...
 This seems to improve matters for me BUT I dont have any other hardware to
 test on and I am no ext2fs guru. Does anyone else have any ideas?

If you read my reply to that e-mail you read what I'm replying to you:

|Hm I tried the current code without your patch but I could not
|find any error case.
|
|But with your patch applied, I got this:
|
|= ext2ls ide 0:4 bin
| ** ext2fs_devread() read outside partition sector 536870912
|Failed to mount ext2 filesystem...
|** Bad ext2 partition or disk - ide 0:4 **
|
|
|I tested this on a PowerPC system (big endian). Any chance there might
|be some endianess issues with your patch?

Please provide at least a test image!

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: w...@denx.de
It's not an optical illusion, it just looks like one.   -- Phil White
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] Problems with ext2ls SD

2009-03-28 Thread DVM

Further debugging reveals UBoot uses a fixed length for inodes (128 bye). The
inodes on my 2GB SD card are 256 bytes There was a patch that fizzled out
back in January looking at this

http://www.nabble.com/-U-BootPATCH--Fix-ext2-non-working-td21742132.html#a21742132

The superblock contains the inode length in the function 

static int ext2fs_read_inode()
...

inodes_per_block = EXT2_BLOCK_SIZE (data) /
__le32_to_cpu(sblock-inode_size); 
...

status = ext2fs_devread (((__le32_to_cpu (blkgrp.inode_table_id) + 
blkno)
 LOG2_EXT2_BLOCK_SIZE (data)),
 
__le32_to_cpu(sblock-inode_size) * blkoff,
 sizeof (struct 
ext2_inode),  /*HACK till I fix up struct size*/ 
 (char *) inode); 


This seems to improve matters for me BUT I dont have any other hardware to
test on and I am no ext2fs guru. Does anyone else have any ideas?
-- 
View this message in context: 
http://www.nabble.com/Problems-with-ext2ls---SD-tp22728281p22756563.html
Sent from the Uboot - Users mailing list archive at Nabble.com.

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] Problems with ext2ls SD

2009-03-28 Thread Wolfgang Denk
Dear DVM,

In message 22756563.p...@talk.nabble.com you wrote:
 
 Further debugging reveals UBoot uses a fixed length for inodes (128 bye). The
 inodes on my 2GB SD card are 256 bytes There was a patch that fizzled out
 back in January looking at this

Do you have a file system image that shows the error, i. e. that could
be usedfor testing?

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: w...@denx.de
Well I don't see why I have to make one man  miserable  when  I  can
make so many men happy.  - Ellyn Mustard, about marriage
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot