Commit 28b20135 authored by Linus Torvalds's avatar Linus Torvalds

Merge branch 'for-linus-4.11' of git://git.kernel.org/pub/scm/linux/kernel/git/mason/linux-btrfs

Pull btrfs fix from Chris Mason:
 "We have one more fix for btrfs.

  This gets rid of a new WARN_ON from rc1 that ended up making more
  noise than we really want. The larger fix for the underflow got
  delayed a bit and it's better for now to put it under
  CONFIG_BTRFS_DEBUG"

* 'for-linus-4.11' of git://git.kernel.org/pub/scm/linux/kernel/git/mason/linux-btrfs:
  btrfs: qgroup: move noisy underflow warning to debugging build
parents 8b5d11e4 338bd52f
...@@ -1042,9 +1042,12 @@ static void report_reserved_underflow(struct btrfs_fs_info *fs_info, ...@@ -1042,9 +1042,12 @@ static void report_reserved_underflow(struct btrfs_fs_info *fs_info,
struct btrfs_qgroup *qgroup, struct btrfs_qgroup *qgroup,
u64 num_bytes) u64 num_bytes)
{ {
btrfs_warn(fs_info, #ifdef CONFIG_BTRFS_DEBUG
WARN_ON(qgroup->reserved < num_bytes);
btrfs_debug(fs_info,
"qgroup %llu reserved space underflow, have: %llu, to free: %llu", "qgroup %llu reserved space underflow, have: %llu, to free: %llu",
qgroup->qgroupid, qgroup->reserved, num_bytes); qgroup->qgroupid, qgroup->reserved, num_bytes);
#endif
qgroup->reserved = 0; qgroup->reserved = 0;
} }
/* /*
...@@ -1075,7 +1078,7 @@ static int __qgroup_excl_accounting(struct btrfs_fs_info *fs_info, ...@@ -1075,7 +1078,7 @@ static int __qgroup_excl_accounting(struct btrfs_fs_info *fs_info,
qgroup->excl += sign * num_bytes; qgroup->excl += sign * num_bytes;
qgroup->excl_cmpr += sign * num_bytes; qgroup->excl_cmpr += sign * num_bytes;
if (sign > 0) { if (sign > 0) {
if (WARN_ON(qgroup->reserved < num_bytes)) if (qgroup->reserved < num_bytes)
report_reserved_underflow(fs_info, qgroup, num_bytes); report_reserved_underflow(fs_info, qgroup, num_bytes);
else else
qgroup->reserved -= num_bytes; qgroup->reserved -= num_bytes;
...@@ -1100,7 +1103,7 @@ static int __qgroup_excl_accounting(struct btrfs_fs_info *fs_info, ...@@ -1100,7 +1103,7 @@ static int __qgroup_excl_accounting(struct btrfs_fs_info *fs_info,
WARN_ON(sign < 0 && qgroup->excl < num_bytes); WARN_ON(sign < 0 && qgroup->excl < num_bytes);
qgroup->excl += sign * num_bytes; qgroup->excl += sign * num_bytes;
if (sign > 0) { if (sign > 0) {
if (WARN_ON(qgroup->reserved < num_bytes)) if (qgroup->reserved < num_bytes)
report_reserved_underflow(fs_info, qgroup, report_reserved_underflow(fs_info, qgroup,
num_bytes); num_bytes);
else else
...@@ -2469,7 +2472,7 @@ void btrfs_qgroup_free_refroot(struct btrfs_fs_info *fs_info, ...@@ -2469,7 +2472,7 @@ void btrfs_qgroup_free_refroot(struct btrfs_fs_info *fs_info,
qg = unode_aux_to_qgroup(unode); qg = unode_aux_to_qgroup(unode);
if (WARN_ON(qg->reserved < num_bytes)) if (qg->reserved < num_bytes)
report_reserved_underflow(fs_info, qg, num_bytes); report_reserved_underflow(fs_info, qg, num_bytes);
else else
qg->reserved -= num_bytes; qg->reserved -= num_bytes;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment