Skip to content

Qgroup cleanups

David Sterba linux-btrfs 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest7 patches 4 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/7btrfs: qgroups: make __del_qgroup_relation staticfs/btrfs/qgroup.c+1 -10 replies
2/7btrfs: ulist: make the finalization function publicfs/btrfs/ulist.c fs/btrfs/ulist.h+2 -12 replies
3/7btrfs: embed extent_changeset::range_changed to the structurefs/btrfs/extent_io.c fs/btrfs/extent_io.h fs/btrfs/qgroup.c+11 -171 reply
4/7btrfs: check quota status earlier and don't do unnecessary freesfs/btrfs/qgroup.c+3 -20 replies
5/7btrfs: remove unnecessary mutex lock in qgroup_account_snapshotfs/btrfs/transaction.c+1 -50 replies
6/7btrfs: qgroups: opencode qgroup_free helperfs/btrfs/qgroup.c+9 -90 replies
7/7btrfs: remove pointless rcu protection from btrfs_qgroup_inheritfs/btrfs/qgroup.c+0 -20 replies

Reviews trailers per patch

1 of 7 patches has a Reviewed-by or Acked-by.

PatchQu Wenruo
1/7
2/7
3/7R-b
4/7
5/7
6/7
7/7

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v1 thread

Cover letter [PATCH 0/7] Qgroup cleanups

I've spotted some cleanup-ready code while going through the file, so here it is. There are no changes to the core qgoups code, just the support and framework code. For 4.11.

David Sterba (7):
btrfs: qgroups: make __del_qgroup_relation static
btrfs: ulist: make the finalization function public
btrfs: embed extent_changeset::range_changed to the structure
btrfs: check quota status earlier and don't do unnecessary frees
btrfs: remove unnecessary mutex lock in qgroup_account_snapshot
btrfs: qgroups: opencode qgroup_free helper
btrfs: remove pointless rcu protection from btrfs_qgroup_inherit

 fs/btrfs/extent_io.c   |  2 +-
 fs/btrfs/extent_io.h   |  2 +-
 fs/btrfs/qgroup.c      | 51 ++++++++++++++++++++++----------------------------
 fs/btrfs/transaction.c |  6 +-----
 fs/btrfs/ulist.c       |  2 +-
 fs/btrfs/ulist.h       |  1 +
 6 files changed, 27 insertions(+), 37 deletions(-)