Kill fs_info::volume_mutex
Get series with b4
b4 am [email protected]Versions newest first · linked by author and title
- v2 Latest16 patches 18 repliesIn review
- v116 patches 30 repliesSuperseded
Changes in v2 from the cover letter
- sanity check in balance resume is only a warning
- read-only check in balance cancel remains and is only moved
- typo fixes
Patches in v2 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/16 | btrfs: squeeze btrfs_dev_replace_continue_on_mount to its callerfs/ | +3 | 0 replies | |
| 2/16 | btrfs: make success path out of btrfs_init_dev_replace_tgtdev more clearfs/ fs/ | +7 | 0 replies | |
| 3/16 | btrfs: export and rename free_devicefs/ fs/ | +13 | 0 replies | |
| 4/16 | btrfs: move btrfs_init_dev_replace_tgtdev to dev-replace.c and make staticfs/ fs/ fs/ | +99 | 0 replies | |
| 5/16 | btrfs: move volume_mutex to callers of btrfs_rm_devicefs/ fs/ | +4 | 0 replies | |
| 6/16 | btrfs: move clearing of EXCL_OP out of __cancel_balancefs/ fs/ | +8 | 2 replies | |
| 7/16 | btrfs: add proper safety check before resuming dev-replacefs/ | +11 | 2 replies | |
| 8/16 | btrfs: add sanity check when resuming balance after mountfs/ | +13 | 1 reply | |
| 9/16 | btrfs: cleanup helpers that reset balance statefs/ fs/ | +17 | 3 replies | |
| 10/16 | btrfs: remove wrong use of volume_mutex from btrfs_dev_replace_startfs/ fs/ | +1 | 0 replies | |
| 11/16 | btrfs: kill btrfs_fs_info::volume_mutexfs/ fs/ fs/ +2 more | +15 | 0 replies | |
| 12/16 | btrfs: track running balance in a simpler wayfs/ fs/ fs/ +1 more | +18 | 6 replies | |
| 13/16 | btrfs: move and comment read-only check in btrfs_cancel_balancefs/ | +8 | 4 replies | |
| 14/16 | btrfs: drop lock parameter from update_ioctl_balance_args and renamefs/ fs/ fs/ | +7 | 1 reply | |
| 15/16 | btrfs: use mutex in btrfs_resume_balance_asyncfs/ | +3 | 1 reply | |
| 16/16 | btrfs: open code set_balance_controlfs/ | +8 | 0 replies |
Reviews trailers per patch
All 16 patches have a Reviewed-by or Acked-by.
| Patch | Anand Jain | Nikolay Borisov |
|---|---|---|
| 1/16 | R-b | |
| 2/16 | R-b | |
| 3/16 | R-b | |
| 4/16 | R-b | |
| 5/16 | R-b | |
| 6/16 | R-b | R-b |
| 7/16 | R-b | R-b |
| 8/16 | R-b | |
| 9/16 | R-b | |
| 10/16 | R-b | |
| 11/16 | R-b | |
| 12/16 | R-b | |
| 13/16 | R-b | |
| 14/16 | R-b | |
| 15/16 | R-b | |
| 16/16 | R-b |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.
Cover letter [PATCH v2 00/16] Kill fs_info::volume_mutex
This series gets rid of the volume mutex because it's redundant. Updated
branch: git://github.com/kdave/btrfs-devel dev/remove-volume-mutex
Changes for v2:
- sanity check in balance resume is only a warning
- read-only check in balance cancel remains and is only moved
- typo fixes
The fstests seem to pass all relevant tests now and qualifies for conditional addition to for-next.
David Sterba (16):
btrfs: squeeze btrfs_dev_replace_continue_on_mount to its caller
btrfs: make success path out of btrfs_init_dev_replace_tgtdev more clear
btrfs: export and rename free_device
btrfs: move btrfs_init_dev_replace_tgtdev to dev-replace.c and make static
btrfs: move volume_mutex to callers of btrfs_rm_device
btrfs: move clearing of EXCL_OP out of __cancel_balance
btrfs: add proper safety check before resuming dev-replace
btrfs: add sanity check when resuming balance after mount
btrfs: cleanup helpers that reset balance state
btrfs: remove wrong use of volume_mutex from btrfs_dev_replace_start
btrfs: kill btrfs_fs_info::volume_mutex
btrfs: track running balance in a simpler way
btrfs: move and comment read-only check in btrfs_cancel_balance
btrfs: drop lock parameter from update_ioctl_balance_args and rename
btrfs: use mutex in btrfs_resume_balance_async
btrfs: open code set_balance_control
fs/btrfs/ctree.h | 9 +- fs/btrfs/dev-replace.c | 135 +++++++++++++++++++++---- fs/btrfs/disk-io.c | 2 - fs/btrfs/extent-tree.c | 2 +- fs/btrfs/ioctl.c | 41 +++----- fs/btrfs/volumes.c | 263 +++++++++++++++---------------------------------- fs/btrfs/volumes.h | 5 +- 7 files changed, 216 insertions(+), 241 deletions(-)
Signature · 1 line
2.16.2