Commit 6bdf962fe35a8648d(btrfs-progs: Read qgroup status for qgroup
verify) will read qgroup status, and then use it to skip qgroup
reporting.

However since the rescan_running/inconsistent flags are only 1 bit long,
while qgroup flags & BTRFS_QGROUP_FLAGS returns value longer than 1bit,
it doesn't work.

Fix it by doing double negation on (flags & BTRFS_QGROUP_FLAGS) to
convert it to either 1 or 0.

Signed-off-by: Qu Wenruo <[email protected]>
---
 qgroup-verify.c | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/qgroup-verify.c b/qgroup-verify.c
index 1a0d38c..86dcd6d 100644
--- a/qgroup-verify.c
+++ b/qgroup-verify.c
@@ -711,8 +711,13 @@ static void read_qgroup_status(struct btrfs_path *path,
        status_item = btrfs_item_ptr(path->nodes[0], path->slots[0],
                                     struct btrfs_qgroup_status_item);
        flags = btrfs_qgroup_status_flags(path->nodes[0], status_item);
-       counts->qgroup_inconsist = flags & 
BTRFS_QGROUP_STATUS_FLAG_INCONSISTENT;
-       counts->rescan_running = flags & BTRFS_QGROUP_STATUS_FLAG_RESCAN;
+       /*
+        * Since qgroup_inconsist/rescan_running is just one bit,
+        * assign value directly won't work.
+        */
+       counts->qgroup_inconsist = !!(flags &
+                       BTRFS_QGROUP_STATUS_FLAG_INCONSISTENT);
+       counts->rescan_running = !!(flags & BTRFS_QGROUP_STATUS_FLAG_RESCAN);
 }
 
 static int load_quota_info(struct btrfs_fs_info *info)
-- 
2.8.3



--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to