btrfs-progs: check: Enhance leaf traversal function to handle missing inode item
authorQu Wenruo <quwenruo@cn.fujitsu.com>
Wed, 21 Sep 2016 02:37:27 +0000 (10:37 +0800)
committerDavid Sterba <dsterba@suse.com>
Wed, 14 Dec 2016 14:06:34 +0000 (15:06 +0100)
The leaf traversal function in lowmem mode will skip to the first inode
item of leaf and begin to check the inode.

That's designed to avoid checking overlapping part of a leaf.

But that will cause problem in fsck/010 test case, as in that case inode
item of the first inode(256) is missing.
So above traversal will check from inode 257 and found nothing wrong.

The fix is done in 2 part:
1) Manually check the first inode
   To avoid case like fsck/010

2) Check inode if ino changes from the first ino of a leaf
   To avoid missing inode_item problem.

Signed-off-by: Qu Wenruo <quwenruo@cn.fujitsu.com>
Signed-off-by: David Sterba <dsterba@suse.com>
cmds-check.c

index 14ab7df..22d3a66 100644 (file)
@@ -1875,6 +1875,7 @@ static int process_one_leaf_v2(struct btrfs_root *root, struct btrfs_path *path,
        struct btrfs_key key;
        u64 cur_bytenr;
        u32 nritems;
+       u64 first_ino = 0;
        int root_level = btrfs_header_level(root->node);
        int i;
        int ret = 0; /* Final return value */
@@ -1882,11 +1883,14 @@ static int process_one_leaf_v2(struct btrfs_root *root, struct btrfs_path *path,
 
        cur_bytenr = cur->start;
 
-       /* skip to first inode item in this leaf */
+       /* skip to first inode item or the first inode number change */
        nritems = btrfs_header_nritems(cur);
        for (i = 0; i < nritems; i++) {
                btrfs_item_key_to_cpu(cur, &key, i);
-               if (key.type == BTRFS_INODE_ITEM_KEY)
+               if (i == 0)
+                       first_ino = key.objectid;
+               if (key.type == BTRFS_INODE_ITEM_KEY ||
+                   (first_ino && first_ino != key.objectid))
                        break;
        }
        if (i == nritems) {
@@ -4928,6 +4932,34 @@ out:
        return err;
 }
 
+static int check_fs_first_inode(struct btrfs_root *root, unsigned int ext_ref)
+{
+       struct btrfs_path *path;
+       struct btrfs_key key;
+       int err = 0;
+       int ret;
+
+       path = btrfs_alloc_path();
+       key.objectid = 256;
+       key.type = BTRFS_INODE_ITEM_KEY;
+       key.offset = 0;
+
+       ret = btrfs_search_slot(NULL, root, &key, path, 0, 0);
+       if (ret < 0)
+               return ret;
+       if (ret > 0) {
+               ret = 0;
+               err |= INODE_ITEM_MISSING;
+       }
+
+       err |= check_inode_item(root, path, ext_ref);
+       err &= ~LAST_ITEM;
+       if (err && !ret)
+               ret = -EIO;
+       btrfs_free_path(path);
+       return ret;
+}
+
 /*
  * Iterate all item on the tree and call check_inode_item() to check.
  *
@@ -4945,6 +4977,16 @@ static int check_fs_root_v2(struct btrfs_root *root, unsigned int ext_ref)
        int ret, wret;
        int level;
 
+       /*
+        * We need to manually check the first inode item(256)
+        * As the following traversal function will only start from
+        * the first inode item in the leaf, if inode item(256) is missing
+        * we will just skip it forever.
+        */
+       ret = check_fs_first_inode(root, ext_ref);
+       if (ret < 0)
+               return ret;
+
        path = btrfs_alloc_path();
        if (!path)
                return -ENOMEM;