diff options
| author |   Jan Schmidt <list.btrfs@jan-o-sch.net> | 2013-02-21 15:35:27 +0000 |
|---|---|---|
| committer |   Josef Bacik <jbacik@fusionio.com> | 2013-02-26 11:00:50 -0500 |
| commit | ca60ebfa303ca6d9f3cad5028c10f0a17d571a4c (patch) | |
| tree | c8445152a89596fb1d6895a198327268067b11ae /fs/btrfs | |
| parent | Btrfs: make sure NODATACOW also gets NODATASUM set (diff) | |
| download | linux-dev-ca60ebfa303ca6d9f3cad5028c10f0a17d571a4c.tar.xz linux-dev-ca60ebfa303ca6d9f3cad5028c10f0a17d571a4c.zip | |
Btrfs: fix backref walking race with tree deletions
When a subvolume is removed, we remove the root item from the root tree,
while the tree blocks and backrefs remain for a while. When backref walking
comes across one of those orphan tree blocks, it can find a backref for a
no longer existing root. This is all good, we only must tolerate
__resolve_indirect_ref returning an error and continue with the good refs
found.
Reported-by: Alex Lyakas <alex.btrfs@zadarastorage.com>
Signed-off-by: Jan Schmidt <list.btrfs@jan-o-sch.net>
Signed-off-by: Josef Bacik <jbacik@fusionio.com>
Diffstat (limited to 'fs/btrfs')
| -rw-r--r-- | fs/btrfs/backref.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/fs/btrfs/backref.c b/fs/btrfs/backref.c index 04edf69be875..bd605c87adfd 100644 --- a/fs/btrfs/backref.c +++ b/fs/btrfs/backref.c @@ -352,11 +352,8 @@ static int __resolve_indirect_refs(struct btrfs_fs_info *fs_info, err = __resolve_indirect_ref(fs_info, search_commit_root, time_seq, ref, parents, extent_item_pos); - if (err) { - if (ret == 0) - ret = err; + if (err) continue; - } /* we put the first parent into the ref at hand */ ULIST_ITER_INIT(&uiter); |