mbox series

[0/6] btrfs: scrub: finish the switch to scrub_stripe and cleanup the old code

Message ID cover.1680073696.git.wqu@suse.com (mailing list archive)
Headers show
Series btrfs: scrub: finish the switch to scrub_stripe and cleanup the old code | expand

Message

Qu Wenruo March 29, 2023, 7:09 a.m. UTC
This series is based on my previous patchset "v7 btrfs: scrub: use a
more reader friendly code to implement scrub_simple_mirror()", the whole
series can be fetched from github:

  https://github.com/adam900710/linux/tree/scrub_stripe

After the mentioned patchset, only one path, scrub_raid56_parity(), is
still using the old facility.

[PATCH SPLIT]

This series would firstly migrate the only path to the new scrub_stripe
based solution.
The change should only affect RAID56, thus zoned mode should not be affected.

Then the remaining 5 patches to remove all the remaining code.
I have tried my best to split the cleanup, starting from RAID56 specific
code, to writeback, then approach the core scrub_bio facility.

If the split is not preferred, it should still be possible to fold all
the last 5 patches into one big cleanup.

[CODE SIZE REDUCTION]

The number of deletion should explain itself, and even with my previous
patchset included, it is still a big win:

 9 files changed, 1488 insertions(+), 2931 deletions(-)

Still a net reduce of almost 1500 lines.

Qu Wenruo (6):
  btrfs: scrub: use scrub_stripe to implement RAID56 P/Q scrub
  btrfs: scrub: remove scrub_parity structure
  btrfs: scrub: remove the old writeback infrastructure
  btrfs: scrub: remove the old scrub recheck code
  btrfs: scrub: remove scrub_block and scrub_sector structures
  btrfs: scrub: remove scrub_bio structure

 fs/btrfs/block-group.c |   11 -
 fs/btrfs/block-group.h |    8 -
 fs/btrfs/fs.h          |    1 -
 fs/btrfs/scrub.c       | 2740 ++++------------------------------------
 4 files changed, 212 insertions(+), 2548 deletions(-)