Index: b/fs/ext3/namei.c =================================================================== --- a/fs/ext3/namei.c +++ b/fs/ext3/namei.c @@ -115,22 +115,13 @@ struct dx_entry * hash version mod 4 should never be 0. Sincerely, the paranoia department. */ -struct dx_root +struct dx_root_info { - struct fake_dirent dot; - char dot_name[4]; - struct fake_dirent dotdot; - char dotdot_name[4]; - struct dx_root_info - { - __le32 reserved_zero; - u8 hash_version; - u8 info_length; /* 8 */ - u8 indirect_levels; - u8 unused_flags; - } - info; - struct dx_entry entries[0]; + __le32 reserved_zero; + u8 hash_version; + u8 info_length; /* 8 */ + u8 indirect_levels; + u8 unused_flags; }; struct dx_node @@ -174,6 +165,7 @@ typedef enum { #define LDP_FLAGS_RANGE 0x07 #ifdef CONFIG_EXT3_INDEX +struct dx_root_info * dx_get_dx_info(struct ext3_dir_entry_2 *de); static inline unsigned dx_get_block (struct dx_entry *entry); static void dx_set_block (struct dx_entry *entry, unsigned value); static inline unsigned dx_get_hash (struct dx_entry *entry); @@ -210,6 +202,16 @@ static int ext3_dx_add_entry(handle_t *h * Future: use high four bits of block for coalesce-on-delete flags * Mask them off for now. */ +struct dx_root_info * dx_get_dx_info(struct ext3_dir_entry_2 *de) +{ + /* get dotdot first */ + de = (struct ext3_dir_entry_2 *)((char *)de + le16_to_cpu(de->rec_len)); + + /* dx root info is after dotdot entry */ + de = (struct ext3_dir_entry_2 *)((char *)de + EXT3_DIR_REC_LEN(2)); + + return (struct dx_root_info *) de; +} static inline unsigned dx_get_block (struct dx_entry *entry) { @@ -362,7 +364,7 @@ dx_probe(struct dentry *dentry, struct i { unsigned count, indirect; struct dx_entry *at, *entries, *p, *q, *m; - struct dx_root *root; + struct dx_root_info * info; struct buffer_head *bh; struct dx_frame *frame = frame_in; u32 hash; @@ -372,46 +374,45 @@ dx_probe(struct dentry *dentry, struct i dir = dentry->d_parent->d_inode; if (!(bh = ext3_bread (NULL,dir, 0, 0, err))) goto fail; - root = (struct dx_root *) bh->b_data; - if (root->info.hash_version != DX_HASH_TEA && - root->info.hash_version != DX_HASH_HALF_MD4 && - root->info.hash_version != DX_HASH_LEGACY) { + + info = dx_get_dx_info((struct ext3_dir_entry_2*)bh->b_data); + if (info->hash_version != DX_HASH_TEA && + info->hash_version != DX_HASH_HALF_MD4 && + info->hash_version != DX_HASH_LEGACY) { ext3_warning(dir->i_sb, __FUNCTION__, "Unrecognised inode hash code %d for directory " - "#%lu", root->info.hash_version, dir->i_ino); + "#%lu", info->hash_version, dir->i_ino); brelse(bh); *err = ERR_BAD_DX_DIR; goto fail; } - hinfo->hash_version = root->info.hash_version; + hinfo->hash_version = info->hash_version; hinfo->seed = EXT3_SB(dir->i_sb)->s_hash_seed; if (dentry) ext3fs_dirhash(dentry->d_name.name, dentry->d_name.len, hinfo); hash = hinfo->hash; - if (root->info.unused_flags & 1) { + if (info->unused_flags & 1) { ext3_warning(dir->i_sb, __FUNCTION__, "Unimplemented inode hash flags: %#06x", - root->info.unused_flags); + info->unused_flags); brelse(bh); *err = ERR_BAD_DX_DIR; goto fail; } - if ((indirect = root->info.indirect_levels) > 1) { + if ((indirect = info->indirect_levels) > 1) { ext3_warning(dir->i_sb, __FUNCTION__, "Unimplemented inode hash depth: %#06x", - root->info.indirect_levels); + info->indirect_levels); brelse(bh); *err = ERR_BAD_DX_DIR; goto fail; } - entries = (struct dx_entry *) (((char *)&root->info) + - root->info.info_length); + entries = (struct dx_entry *) (((char *)info) + info->info_length); - if (dx_get_limit(entries) != dx_root_limit(dir, - root->info.info_length)) { + if (dx_get_limit(entries) != dx_root_limit(dir, info->info_length)) { ext3_warning(dir->i_sb, __FUNCTION__, "dx entry: limit != root limit"); brelse(bh); @@ -467,10 +469,12 @@ fail: static void dx_release (struct dx_frame *frames) { + struct dx_root_info *info; + if (frames[0].bh == NULL) return; - - if (((struct dx_root *) frames[0].bh->b_data)->info.indirect_levels) + info = dx_get_dx_info((struct ext3_dir_entry_2*)frames[0].bh->b_data); + if (info->indirect_levels) brelse(frames[1].bh); brelse(frames[0].bh); } @@ -1419,17 +1423,16 @@ static int make_indexed_dir(handle_t *ha const char *name = dentry->d_name.name; int namelen = dentry->d_name.len; struct buffer_head *bh2; - struct dx_root *root; struct dx_frame frames[2], *frame; struct dx_entry *entries; - struct ext3_dir_entry_2 *de, *de2; + struct ext3_dir_entry_2 *de, *de2, *dot_de, *dotdot_de; char *data1, *top; unsigned len; int retval; unsigned blocksize; struct dx_hash_info hinfo; u32 block; - struct fake_dirent *fde; + struct dx_root_info *dx_info; blocksize = dir->i_sb->s_blocksize; dxtrace(printk("Creating index\n")); @@ -1439,7 +1442,7 @@ static int make_indexed_dir(handle_t *ha brelse(bh); return retval; } - root = (struct dx_root *) bh->b_data; + dot_de = (struct ext3_dir_entry_2 *) bh->b_data; bh2 = ext3_append (handle, dir, &block, &retval); if (!(bh2)) { @@ -1450,9 +1453,9 @@ static int make_indexed_dir(handle_t *ha data1 = bh2->b_data; /* The 0th block becomes the root, move the dirents out */ - fde = &root->dotdot; - de = (struct ext3_dir_entry_2 *)((char *)fde + le16_to_cpu(fde->rec_len)); - len = ((char *) root) + blocksize - (char *) de; + dotdot_de = (struct ext3_dir_entry_2 *)((char *)dot_de + le16_to_cpu(dot_de->rec_len)); + de = (struct ext3_dir_entry_2 *)((char *)dotdot_de + le16_to_cpu(dotdot_de->rec_len)); + len = ((char *) dot_de) + blocksize - (char *) de; memcpy (data1, de, len); de = (struct ext3_dir_entry_2 *) data1; top = data1 + len; @@ -1460,18 +1463,21 @@ static int make_indexed_dir(handle_t *ha de = de2; de->rec_len = cpu_to_le16(data1 + blocksize - (char *) de); /* Initialize the root; the dot dirents already exist */ - de = (struct ext3_dir_entry_2 *) (&root->dotdot); - de->rec_len = cpu_to_le16(blocksize - EXT3_DIR_REC_LEN(2)); - memset (&root->info, 0, sizeof(root->info)); - root->info.info_length = sizeof(root->info); - root->info.hash_version = EXT3_SB(dir->i_sb)->s_def_hash_version; - entries = root->entries; + dotdot_de->rec_len = cpu_to_le16(blocksize - le16_to_cpu(dot_de->rec_len)); + + /* initialize hashing info */ + dx_info = dx_get_dx_info(dot_de); + memset (dx_info, 0, sizeof(*dx_info)); + dx_info->info_length = sizeof(*dx_info); + dx_info->hash_version = EXT3_SB(dir->i_sb)->s_def_hash_version; + + entries = (void *)dx_info + sizeof(*dx_info); dx_set_block (entries, 1); dx_set_count (entries, 1); - dx_set_limit (entries, dx_root_limit(dir, sizeof(root->info))); + dx_set_limit (entries, dx_root_limit(dir, sizeof(*dx_info))); /* Initialize as for dx_probe */ - hinfo.hash_version = root->info.hash_version; + hinfo.hash_version = dx_info->hash_version; hinfo.seed = EXT3_SB(dir->i_sb)->s_hash_seed; ext3fs_dirhash(name, namelen, &hinfo); frame = frames; @@ -1720,6 +1726,7 @@ static int ext3_dx_add_entry(handle_t *h goto journal_error; brelse (bh2); } else { + struct dx_root_info *info; dxtrace(printk("Creating second level index...\n")); memcpy((char *) entries2, (char *) entries, icount * sizeof(struct dx_entry)); @@ -1728,7 +1735,8 @@ static int ext3_dx_add_entry(handle_t *h /* Set up root */ dx_set_count(entries, 1); dx_set_block(entries + 0, newblock); - ((struct dx_root *) frames[0].bh->b_data)->info.indirect_levels = 1; + info = dx_get_dx_info((struct ext3_dir_entry_2*)frames[0].bh->b_data); + info->indirect_levels = 1; /* Add new access path frame */ frame = frames + 1;