From: OGAWA Hirofumi Signed-off-by: OGAWA Hirofumi Signed-off-by: Andrew Morton --- fs/fat/dir.c | 10 ++-------- fs/vfat/namei.c | 3 +-- include/linux/msdos_fs.h | 11 +++++++++++ 3 files changed, 14 insertions(+), 10 deletions(-) diff -puN fs/fat/dir.c~fat-cleanup-and-optimization-of-checksum fs/fat/dir.c --- 25/fs/fat/dir.c~fat-cleanup-and-optimization-of-checksum Tue Sep 20 15:57:59 2005 +++ 25-akpm/fs/fat/dir.c Tue Sep 20 15:57:59 2005 @@ -263,7 +263,6 @@ parse_record: unsigned char id; unsigned char slot; unsigned char slots; - unsigned char sum; unsigned char alias_checksum; if (!unicode) { @@ -317,9 +316,7 @@ parse_long: goto parse_long; if (IS_FREE(de->name) || (de->attr & ATTR_VOLUME)) continue; - for (sum = 0, i = 0; i < 11; i++) - sum = (((sum&1)<<7)|((sum&0xfe)>>1)) + de->name[i]; - if (sum != alias_checksum) + if (fat_checksum(de->name) != alias_checksum) nr_slots = 0; } @@ -479,7 +476,6 @@ GetNew: unsigned char id; unsigned char slot; unsigned char slots; - unsigned char sum; unsigned char alias_checksum; if (!unicode) { @@ -534,9 +530,7 @@ ParseLong: goto ParseLong; if (IS_FREE(de->name) || (de->attr & ATTR_VOLUME)) goto RecEnd; - for (sum = 0, i = 0; i < 11; i++) - sum = (((sum&1)<<7)|((sum&0xfe)>>1)) + de->name[i]; - if (sum != alias_checksum) + if (fat_checksum(de->name) != alias_checksum) long_slots = 0; } diff -puN fs/vfat/namei.c~fat-cleanup-and-optimization-of-checksum fs/vfat/namei.c --- 25/fs/vfat/namei.c~fat-cleanup-and-optimization-of-checksum Tue Sep 20 15:57:59 2005 +++ 25-akpm/fs/vfat/namei.c Tue Sep 20 15:57:59 2005 @@ -621,8 +621,7 @@ static int vfat_build_slots(struct inode } /* build the entry of long file name */ - for (cksum = i = 0; i < 11; i++) - cksum = (((cksum&1)<<7)|((cksum&0xfe)>>1)) + msdos_name[i]; + cksum = fat_checksum(msdos_name); *nr_slots = usize / 13; for (ps = slots, i = *nr_slots; i > 0; i--, ps++) { diff -puN include/linux/msdos_fs.h~fat-cleanup-and-optimization-of-checksum include/linux/msdos_fs.h --- 25/include/linux/msdos_fs.h~fat-cleanup-and-optimization-of-checksum Tue Sep 20 15:57:59 2005 +++ 25-akpm/include/linux/msdos_fs.h Tue Sep 20 15:57:59 2005 @@ -282,6 +282,17 @@ static inline u8 fat_attr(struct inode * MSDOS_I(inode)->i_attrs; } +static inline unsigned char fat_checksum(const __u8 *name) +{ + unsigned char s = name[0]; + s = (s<<7) + (s>>1) + name[1]; s = (s<<7) + (s>>1) + name[2]; + s = (s<<7) + (s>>1) + name[3]; s = (s<<7) + (s>>1) + name[4]; + s = (s<<7) + (s>>1) + name[5]; s = (s<<7) + (s>>1) + name[6]; + s = (s<<7) + (s>>1) + name[7]; s = (s<<7) + (s>>1) + name[8]; + s = (s<<7) + (s>>1) + name[9]; s = (s<<7) + (s>>1) + name[10]; + return s; +} + static inline sector_t fat_clus_to_blknr(struct msdos_sb_info *sbi, int clus) { return ((sector_t)clus - FAT_START_ENT) * sbi->sec_per_clus _