aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorWang Shilong <wangsl.fnst@cn.fujitsu.com>2014-01-23 00:47:49 -0500
committerChris Mason <clm@fb.com>2014-01-29 10:06:23 -0500
commit95def2ede1a9dd12b164932eaf5fefb67aefc41c (patch)
treeb9af144d22220cbbc08ce544ad0f0ea83a2e1537
parent538f72cdf03cad1c21c551ea542c8ce7d9fa2d81 (diff)
Btrfs: fix to catch all errors when resolving indirect ref
We can only tolerate ENOENT here, for other errors, we should return directly. Signed-off-by: Wang Shilong <wangsl.fnst@cn.fujitsu.com> Signed-off-by: Josef Bacik <jbacik@fb.com> Signed-off-by: Chris Mason <clm@fb.com>
-rw-r--r--fs/btrfs/backref.c12
1 files changed, 9 insertions, 3 deletions
diff --git a/fs/btrfs/backref.c b/fs/btrfs/backref.c
index 10ae5700ab1e..55ffcf44b909 100644
--- a/fs/btrfs/backref.c
+++ b/fs/btrfs/backref.c
@@ -388,10 +388,16 @@ static int __resolve_indirect_refs(struct btrfs_fs_info *fs_info,
388 continue; 388 continue;
389 err = __resolve_indirect_ref(fs_info, path, time_seq, ref, 389 err = __resolve_indirect_ref(fs_info, path, time_seq, ref,
390 parents, extent_item_pos); 390 parents, extent_item_pos);
391 if (err == -ENOMEM) 391 /*
392 goto out; 392 * we can only tolerate ENOENT,otherwise,we should catch error
393 if (err) 393 * and return directly.
394 */
395 if (err == -ENOENT) {
394 continue; 396 continue;
397 } else if (err) {
398 ret = err;
399 goto out;
400 }
395 401
396 /* we put the first parent into the ref at hand */ 402 /* we put the first parent into the ref at hand */
397 ULIST_ITER_INIT(&uiter); 403 ULIST_ITER_INIT(&uiter);