Hi Scott, For sure, the modification will increase code size, but I think it is reasonable. 1. Make U-Boot could support the EXT2 dynamic version that ext2_inode_size beyond 128bytes. According to Linux kernel source code, now EXT2 filesystem could support two revision: EXT2_GOOD_OLD_REV EXT2_DYNAMIC_REV ext2_inode_size be fixed as 128bytes in the first revision while it could beyond 128bytes in another revision. In the past, the first revision is the default option for mkfs.ext2 command. But now there are some OSes like Mandriva Linux 2008.1 for STLS want to use new revision option to format disk. I have tested it and our current U-Boot couldn't support it. So I think it is time to do it.
2. Make EXT2 read more fast. I have tested our U-Boot to access EXT2 filesystem on USB disk. It's quite slow and it takes about 200 seconds to read a file that its size is 8MB. Our USB host controller is NEC upd720101 OHCI host controller and operation mode is 'bulk'. I think its width is enough. I have analyzed our ext2fs.c file and found it only read 4KB(512B/block * 8blocks, I format EXT2 filesystem with default options) once time. One time is a USB timing slot - 100ms. So it is so slow. Now I mainly modified ext2fs_read_block() function to detect how many seriate blocks and read them in one time. The modification fasten read opertion a lot especially big file. Best Regards, Ryan Chen -----Original Message----- From: Scott Wood [mailto:[EMAIL PROTECTED] Sent: Wednesday, July 30, 2008 4:11 AM To: Ryan CHEN Cc: u-boot-users@lists.sourceforge.net; Antonio Maria BORNEO Subject: Re: [U-Boot-Users] PATCH: Update U-Boot EXT2 Filesystem to support dynamic inode size and optimize ext2 read function to fasten read speed! On Tue, Jul 29, 2008 at 04:48:51PM +0800, Ryan CHEN wrote: > Description: > The patch updates /fs/ext2/ext2fs.c file. There are two aims: > 1. Make U-Boot could support the EXT2 dynamic version that ext2_inode_size beyond 128bytes. > One new feature be involved: CFG_EXT2_SUPPORT_DYNAMIC_REV > Refer to: linux-2.6.24.3 source code. > 2. Make EXT2 read more fast. > One new feature be involved: CFG_OPTIMIZE_EXT2_READ Does it significantly increase code size, or is there any other reason why these need to be conditionalized? -Scott ------------------------------------------------------------------------- This SF.Net email is sponsored by the Moblin Your Move Developer's challenge Build the coolest Linux based applications with Moblin SDK & win great prizes Grand prize is a trip for two to an Open Source event anywhere in the world http://moblin-contest.org/redirect.php?banner_id=100&url=/ _______________________________________________ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users