In btrfs_remove_qgroup(), each qgroup relation is removed by calling
__del_qgroup_relation(). However, __del_qgroup_relation() returns 1
if deletion of qgroup relation causes inconsistency and current code
exits immediately in that case.
Therefore if there are several qgroup relations and removing first
relation causes inconsistency, remaining items will not be removed.
Fix this by continuing to remove items if return value of
__del_qgroup_relation() is 1.
Signed-off-by: Misono Tomohiro <misono.tomohiro@xxxxxxxxxxxxxx>
---
fs/btrfs/qgroup.c | 15 ++++++++++++---
1 file changed, 12 insertions(+), 3 deletions(-)
diff --git a/fs/btrfs/qgroup.c b/fs/btrfs/qgroup.c
index 2ba29f0609d9..f18284253e77 100644
--- a/fs/btrfs/qgroup.c
+++ b/fs/btrfs/qgroup.c
@@ -1428,12 +1428,21 @@ int btrfs_remove_qgroup(struct btrfs_trans_handle *trans, u64 qgroupid)
goto out;
while (!list_empty(&qgroup->groups)) {
+ int ret2;
+
list = list_first_entry(&qgroup->groups,
struct btrfs_qgroup_list, next_group);
- ret = __del_qgroup_relation(trans, qgroupid,
+ ret2 = __del_qgroup_relation(trans, qgroupid,
list->group->qgroupid);
- if (ret)
- goto out;
+ if (ret2) {
+ ret = ret2;
+ /*
+ * __del_qgroup_relation() returns 1 if qgroup becomes
+ * inconsistent. Continue to remove items in that case.
+ */
+ if (ret != 1)
+ goto out;
+ }
}
spin_lock(&fs_info->qgroup_lock);
--
2.14.4
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
