diff options
| author |   Jeff Mahoney <jeffm@suse.com> | 2020-02-11 15:25:37 +0800 |
|---|---|---|
| committer |   David Sterba <dsterba@suse.com> | 2020-02-19 00:35:54 +0100 |
| commit | 81f7eb00ff5bb8326e82503a32809421d14abb8a (patch) | |
| tree | 6c2ddb770d4bcdbd85db0881d040db3bbd21eb7b /fs/btrfs/qgroup.c | |
| parent | btrfs: don't set path->leave_spinning for truncate (diff) | |
| download | linux-dev-81f7eb00ff5bb8326e82503a32809421d14abb8a.tar.xz linux-dev-81f7eb00ff5bb8326e82503a32809421d14abb8a.zip | |
btrfs: destroy qgroup extent records on transaction abort
We clean up the delayed references when we abort a transaction but we
leave the pending qgroup extent records behind, leaking memory.
This patch destroys the extent records when we destroy the delayed refs
and makes sure ensure they're gone before releasing the transaction.
Fixes: 3368d001ba5d ("btrfs: qgroup: Record possible quota-related extent for qgroup.")
CC: stable@vger.kernel.org # 4.4+
Reviewed-by: Josef Bacik <josef@toxicpanda.com>
Signed-off-by: Jeff Mahoney <jeffm@suse.com>
[ Rebased to latest upstream, remove to_qgroup() helper, use
rbtree_postorder_for_each_entry_safe() wrapper ]
Signed-off-by: Qu Wenruo <wqu@suse.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/btrfs/qgroup.c')
| -rw-r--r-- | fs/btrfs/qgroup.c | 13 |
1 files changed, 13 insertions, 0 deletions
diff --git a/fs/btrfs/qgroup.c b/fs/btrfs/qgroup.c index 98d9a50352d6..ff1870ff3474 100644 --- a/fs/btrfs/qgroup.c +++ b/fs/btrfs/qgroup.c @@ -4002,3 +4002,16 @@ out: } return ret; } + +void btrfs_qgroup_destroy_extent_records(struct btrfs_transaction *trans) +{ + struct btrfs_qgroup_extent_record *entry; + struct btrfs_qgroup_extent_record *next; + struct rb_root *root; + + root = &trans->delayed_refs.dirty_extent_root; + rbtree_postorder_for_each_entry_safe(entry, next, root, node) { + ulist_free(entry->old_roots); + kfree(entry); + } +} |