ext4: let empty_dir handle inline dir
authorTao Ma <boyu.mt@taobao.com>
Mon, 10 Dec 2012 19:06:01 +0000 (14:06 -0500)
committerTheodore Ts'o <tytso@mit.edu>
Mon, 10 Dec 2012 19:06:01 +0000 (14:06 -0500)
empty_dir is used when deleting a dir.  So it should handle inline dir
properly.

Signed-off-by: Tao Ma <boyu.mt@taobao.com>
Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
fs/ext4/inline.c
fs/ext4/namei.c
fs/ext4/xattr.h

index f5e9c0e6d737821eec7e62a07a1ce338a7ab6cb8..e5da458fabad378558c5000e5741522d14012b65 100644 (file)
@@ -1565,6 +1565,96 @@ out:
        return err;
 }
 
+/*
+ * Get the inline dentry at offset.
+ */
+static inline struct ext4_dir_entry_2 *
+ext4_get_inline_entry(struct inode *inode,
+                     struct ext4_iloc *iloc,
+                     unsigned int offset,
+                     void **inline_start,
+                     int *inline_size)
+{
+       void *inline_pos;
+
+       BUG_ON(offset > ext4_get_inline_size(inode));
+
+       if (offset < EXT4_MIN_INLINE_DATA_SIZE) {
+               inline_pos = (void *)ext4_raw_inode(iloc)->i_block;
+               *inline_size = EXT4_MIN_INLINE_DATA_SIZE;
+       } else {
+               inline_pos = ext4_get_inline_xattr_pos(inode, iloc);
+               offset -= EXT4_MIN_INLINE_DATA_SIZE;
+               *inline_size = ext4_get_inline_size(inode) -
+                               EXT4_MIN_INLINE_DATA_SIZE;
+       }
+
+       if (inline_start)
+               *inline_start = inline_pos;
+       return (struct ext4_dir_entry_2 *)(inline_pos + offset);
+}
+
+int empty_inline_dir(struct inode *dir, int *has_inline_data)
+{
+       int err, inline_size;
+       struct ext4_iloc iloc;
+       void *inline_pos;
+       unsigned int offset;
+       struct ext4_dir_entry_2 *de;
+       int ret = 1;
+
+       err = ext4_get_inode_loc(dir, &iloc);
+       if (err) {
+               EXT4_ERROR_INODE(dir, "error %d getting inode %lu block",
+                                err, dir->i_ino);
+               return 1;
+       }
+
+       down_read(&EXT4_I(dir)->xattr_sem);
+       if (!ext4_has_inline_data(dir)) {
+               *has_inline_data = 0;
+               goto out;
+       }
+
+       de = (struct ext4_dir_entry_2 *)ext4_raw_inode(&iloc)->i_block;
+       if (!le32_to_cpu(de->inode)) {
+               ext4_warning(dir->i_sb,
+                            "bad inline directory (dir #%lu) - no `..'",
+                            dir->i_ino);
+               ret = 1;
+               goto out;
+       }
+
+       offset = EXT4_INLINE_DOTDOT_SIZE;
+       while (offset < dir->i_size) {
+               de = ext4_get_inline_entry(dir, &iloc, offset,
+                                          &inline_pos, &inline_size);
+               if (ext4_check_dir_entry(dir, NULL, de,
+                                        iloc.bh, inline_pos,
+                                        inline_size, offset)) {
+                       ext4_warning(dir->i_sb,
+                                    "bad inline directory (dir #%lu) - "
+                                    "inode %u, rec_len %u, name_len %d"
+                                    "inline size %d\n",
+                                    dir->i_ino, le32_to_cpu(de->inode),
+                                    le16_to_cpu(de->rec_len), de->name_len,
+                                    inline_size);
+                       ret = 1;
+                       goto out;
+               }
+               if (le32_to_cpu(de->inode)) {
+                       ret = 0;
+                       goto out;
+               }
+               offset += ext4_rec_len_from_disk(de->rec_len, inline_size);
+       }
+
+out:
+       up_read(&EXT4_I(dir)->xattr_sem);
+       brelse(iloc.bh);
+       return ret;
+}
+
 int ext4_destroy_inline_data(handle_t *handle, struct inode *inode)
 {
        int ret;
index a32228a73df0317f317909f0803fc081716fa5e0..e3e20d0aa299c2e3d31e979063be0333a48b010d 100644 (file)
@@ -2464,6 +2464,14 @@ static int empty_dir(struct inode *inode)
        struct super_block *sb;
        int err = 0;
 
+       if (ext4_has_inline_data(inode)) {
+               int has_inline_data = 1;
+
+               err = empty_inline_dir(inode, &has_inline_data);
+               if (has_inline_data)
+                       return err;
+       }
+
        sb = inode->i_sb;
        if (inode->i_size < EXT4_DIR_REC_LEN(1) + EXT4_DIR_REC_LEN(2) ||
            !(bh = ext4_bread(NULL, inode, 0, 0, &err))) {
index f86e424d75e48ac1846e3df8a2ea78379d4650b2..7747bbcebb33e8b926c3a28bcd6095370bc910b4 100644 (file)
@@ -180,6 +180,7 @@ extern int ext4_delete_inline_entry(handle_t *handle,
                                    struct ext4_dir_entry_2 *de_del,
                                    struct buffer_head *bh,
                                    int *has_inline_data);
+extern int empty_inline_dir(struct inode *dir, int *has_inline_data);
 # else  /* CONFIG_EXT4_FS_XATTR */
 
 static inline int
@@ -381,6 +382,11 @@ static inline int ext4_delete_inline_entry(handle_t *handle,
 {
        return 0;
 }
+
+static inline int empty_inline_dir(struct inode *dir, int *has_inline_data)
+{
+       return 0;
+}
 # endif  /* CONFIG_EXT4_FS_XATTR */
 
 #ifdef CONFIG_EXT4_FS_SECURITY