btrfs-progs: Replace usage of list_for_each with list_for_each_entry
[platform/upstream/btrfs-progs.git] / cmds-check.c
index 03ff89a..5fc0ea9 100644 (file)
@@ -1635,8 +1635,9 @@ static int process_dir_item(struct extent_buffer *eb,
                                          namebuf, len, filetype,
                                          key->type, error);
                } else {
-                       fprintf(stderr, "invalid location in dir item %u\n",
-                               location.type);
+                       fprintf(stderr,
+                               "unknown location type %d in DIR_ITEM[%llu %llu]\n",
+                               location.type, key->objectid, key->offset);
                        add_inode_backref(inode_cache, BTRFS_MULTIPLE_OBJECTIDS,
                                          key->objectid, key->offset, namebuf,
                                          len, filetype, key->type, error);
@@ -2631,8 +2632,7 @@ static int repair_tree_block_ref(struct btrfs_trans_handle *trans,
                }
                btrfs_mark_buffer_dirty(eb);
                printf("Added an extent item [%llu %u]\n", bytenr, node_size);
-               btrfs_update_block_group(trans, extent_root, bytenr, node_size,
-                                        1, 0);
+               btrfs_update_block_group(extent_root, bytenr, node_size, 1, 0);
 
                nrefs->refs[level] = 0;
                nrefs->full_backref[level] =
@@ -5873,8 +5873,9 @@ static int check_file_extent(struct btrfs_root *root, struct btrfs_key *fkey,
                if (!repair || ret) {
                        err |= FILE_EXTENT_ERROR;
                        error(
-               "root %llu EXTENT_DATA[%llu %llu] interrupt, should start at %llu",
-                       root->objectid, fkey->objectid, fkey->offset, *end);
+"root %llu EXTENT_DATA[%llu %llu] gap exists, expected: EXTENT_DATA[%llu %llu]",
+                               root->objectid, fkey->objectid, fkey->offset,
+                               fkey->objectid, *end);
                }
        }
 
@@ -8365,7 +8366,13 @@ static int process_extent_item(struct btrfs_root *root,
        if (item_size < sizeof(*ei)) {
 #ifdef BTRFS_COMPAT_EXTENT_TREE_V0
                struct btrfs_extent_item_v0 *ei0;
-               BUG_ON(item_size != sizeof(*ei0));
+               if (item_size != sizeof(*ei0)) {
+                       error(
+       "invalid extent item format: ITEM[%llu %u %llu] leaf: %llu slot: %d",
+                               key.objectid, key.type, key.offset,
+                               btrfs_header_bytenr(eb), slot);
+                       BUG();
+               }
                ei0 = btrfs_item_ptr(eb, slot, struct btrfs_extent_item_v0);
                refs = btrfs_extent_refs_v0(eb, ei0);
 #else
@@ -9660,7 +9667,7 @@ static int delete_extent_records(struct btrfs_trans_handle *trans,
                        u64 bytes = (found_key.type == BTRFS_EXTENT_ITEM_KEY) ?
                                found_key.offset : root->fs_info->nodesize;
 
-                       ret = btrfs_update_block_group(trans, root, bytenr,
+                       ret = btrfs_update_block_group(root, bytenr,
                                                       bytes, 0, 0);
                        if (ret)
                                break;
@@ -9739,7 +9746,7 @@ static int record_extent(struct btrfs_trans_handle *trans,
                }
 
                btrfs_mark_buffer_dirty(leaf);
-               ret = btrfs_update_block_group(trans, extent_root, rec->start,
+               ret = btrfs_update_block_group(extent_root, rec->start,
                                               rec->max_size, 1, 0);
                if (ret)
                        goto fail;
@@ -11700,16 +11707,12 @@ static int check_tree_block_ref(struct btrfs_root *root,
        u32 nodesize = root->fs_info->nodesize;
        u32 item_size;
        u64 offset;
-       int tree_reloc_root = 0;
        int found_ref = 0;
        int err = 0;
        int ret;
        int strict = 1;
        int parent = 0;
 
-       if (root->root_key.objectid == BTRFS_TREE_RELOC_OBJECTID &&
-           btrfs_header_bytenr(root->node) == bytenr)
-               tree_reloc_root = 1;
        btrfs_init_path(&path);
        key.objectid = bytenr;
        if (btrfs_fs_incompat(root->fs_info, SKINNY_METADATA))
@@ -11817,8 +11820,12 @@ static int check_tree_block_ref(struct btrfs_root *root,
                        /*
                         * Backref of tree reloc root points to itself, no need
                         * to check backref any more.
+                        *
+                        * This may be an error of loop backref, but extent tree
+                        * checker should have already handled it.
+                        * Here we only need to avoid infinite iteration.
                         */
-                       if (tree_reloc_root) {
+                       if (offset == bytenr) {
                                found_ref = 1;
                        } else {
                                /*
@@ -11850,6 +11857,30 @@ static int check_tree_block_ref(struct btrfs_root *root,
                if (!ret)
                        found_ref = 1;
        }
+       /*
+        * Finally check SHARED BLOCK REF, any found will be good
+        * Here we're not doing comprehensive extent backref checking,
+        * only need to ensure there is some extent referring to this
+        * tree block.
+        */
+       if (!found_ref) {
+               btrfs_release_path(&path);
+               key.objectid = bytenr;
+               key.type = BTRFS_SHARED_BLOCK_REF_KEY;
+               key.offset = (u64)-1;
+
+               ret = btrfs_search_slot(NULL, extent_root, &key, &path, 0, 0);
+               if (ret < 0) {
+                       err |= BACKREF_MISSING;
+                       goto out;
+               }
+               ret = btrfs_previous_extent_item(extent_root, &path, bytenr);
+               if (ret) {
+                       err |= BACKREF_MISSING;
+                       goto out;
+               }
+               found_ref = 1;
+       }
        if (!found_ref)
                err |= BACKREF_MISSING;
 out:
@@ -11948,7 +11979,7 @@ static int repair_extent_data_item(struct btrfs_trans_handle *trans,
                btrfs_set_extent_flags(eb, ei, BTRFS_EXTENT_FLAG_DATA);
 
                btrfs_mark_buffer_dirty(eb);
-               ret = btrfs_update_block_group(trans, extent_root, disk_bytenr,
+               ret = btrfs_update_block_group(extent_root, disk_bytenr,
                                               num_bytes, 1, 0);
                btrfs_release_path(&path);
        }
@@ -12005,11 +12036,11 @@ static int check_extent_data_item(struct btrfs_root *root,
        u64 disk_num_bytes;
        u64 extent_num_bytes;
        u64 extent_flags;
+       u64 offset;
        u32 item_size;
        unsigned long end;
        unsigned long ptr;
        int type;
-       u64 ref_root;
        int found_dbackref = 0;
        int slot = pathp->slots[0];
        int err = 0;
@@ -12027,6 +12058,7 @@ static int check_extent_data_item(struct btrfs_root *root,
        disk_bytenr = btrfs_file_extent_disk_bytenr(eb, fi);
        disk_num_bytes = btrfs_file_extent_disk_num_bytes(eb, fi);
        extent_num_bytes = btrfs_file_extent_num_bytes(eb, fi);
+       offset = btrfs_file_extent_offset(eb, fi);
 
        /* Check unaligned disk_num_bytes and num_bytes */
        if (!IS_ALIGNED(disk_num_bytes, root->fs_info->sectorsize)) {
@@ -12081,6 +12113,11 @@ static int check_extent_data_item(struct btrfs_root *root,
        strict = should_check_extent_strictly(root, nrefs, -1);
 
        while (ptr < end) {
+               u64 ref_root;
+               u64 ref_objectid;
+               u64 ref_offset;
+               bool match = false;
+
                iref = (struct btrfs_extent_inline_ref *)ptr;
                type = btrfs_extent_inline_ref_type(leaf, iref);
                dref = (struct btrfs_extent_data_ref *)(&iref->offset);
@@ -12092,9 +12129,15 @@ static int check_extent_data_item(struct btrfs_root *root,
                }
                if (type == BTRFS_EXTENT_DATA_REF_KEY) {
                        ref_root = btrfs_extent_data_ref_root(leaf, dref);
-                       if (ref_root == root->objectid)
+                       ref_objectid = btrfs_extent_data_ref_objectid(leaf, dref);
+                       ref_offset = btrfs_extent_data_ref_offset(leaf, dref);
+
+                       if (ref_objectid == fi_key.objectid &&
+                           ref_offset == fi_key.offset - offset)
+                               match = true;
+                       if (ref_root == root->objectid && match)
                                found_dbackref = 1;
-                       else if (!strict && owner == ref_root)
+                       else if (!strict && owner == ref_root && match)
                                found_dbackref = 1;
                } else if (type == BTRFS_SHARED_DATA_REF_KEY) {
                        found_dbackref = !check_tree_block_ref(root, NULL,
@@ -12114,7 +12157,7 @@ static int check_extent_data_item(struct btrfs_root *root,
                dbref_key.objectid = btrfs_file_extent_disk_bytenr(eb, fi);
                dbref_key.type = BTRFS_EXTENT_DATA_REF_KEY;
                dbref_key.offset = hash_extent_data_ref(root->objectid,
-                               fi_key.objectid, fi_key.offset);
+                               fi_key.objectid, fi_key.offset - offset);
 
                ret = btrfs_search_slot(NULL, root->fs_info->extent_root,
                                        &dbref_key, &path, 0, 0);
@@ -12482,11 +12525,17 @@ static int check_extent_data_backref(struct btrfs_fs_info *fs_info,
                 * Except normal disk bytenr and disk num bytes, we still
                 * need to do extra check on dbackref offset as
                 * dbackref offset = file_offset - file_extent_offset
+                *
+                * Also, we must check the leaf owner.
+                * In case of shared tree blocks (snapshots) we can inherit
+                * leaves from source snapshot.
+                * In that case, reference from source snapshot should not
+                * count.
                 */
                if (btrfs_file_extent_disk_bytenr(leaf, fi) == bytenr &&
                    btrfs_file_extent_disk_num_bytes(leaf, fi) == len &&
                    (u64)(key.offset - btrfs_file_extent_offset(leaf, fi)) ==
-                   offset)
+                   offset && btrfs_header_owner(leaf) == root_id)
                        found_count++;
 
 next: