btrfs: qgroup: account shared subtree during snapshot delete
authorMark Fasheh <mfasheh@suse.de>
Thu, 5 Nov 2015 22:38:00 +0000 (14:38 -0800)
committerChris Mason <clm@fb.com>
Wed, 25 Nov 2015 13:27:33 +0000 (05:27 -0800)
commit82bd101b5240d3d1c4078a8017917a40c0dcc514
tree0105605d854d46179bd8805b91994510f5b79365
parent2d9e97761087b46192c18181dfd1e7a930defcfd
btrfs: qgroup: account shared subtree during snapshot delete

Commit 0ed4792 ('btrfs: qgroup: Switch to new extent-oriented qgroup
mechanism.') removed our qgroup accounting during
btrfs_drop_snapshot(). Predictably, this results in qgroup numbers
going bad shortly after a snapshot is removed.

Fix this by adding a dirty extent record when we encounter extents during
our shared subtree walk. This effectively restores the functionality we had
with the original shared subtree walking code in 1152651 (btrfs: qgroup:
account shared subtrees during snapshot delete).

The idea with the original patch (and this one) is that shared subtrees can
get skipped during drop_snapshot. The shared subtree walk then allows us a
chance to visit those extents and add them to the qgroup work for later
processing. This ultimately makes the accounting for drop snapshot work.

The new qgroup code nicely handles all the other extents during the tree
walk via the ref dec/inc functions so we don't have to add actions beyond
what we had originally.

Signed-off-by: Mark Fasheh <mfasheh@suse.de>
Signed-off-by: Chris Mason <clm@fb.com>
fs/btrfs/extent-tree.c
fs/btrfs/qgroup.c