Re: [PATCH 00/21] btrfs: first batch of zoned cleanups

From: Josef Bacik <josef@toxicpanda.com>
To: Johannes Thumshirn <johannes.thumshirn@wdc.com>
Cc: David Sterba <dsterba@suse.com>,
	linux-btrfs@vger.kernel.org, Nikolay Borisov <nborisov@suse.com>,
	Naohiro Aota <Naohiro.Aota@wdc.com>
Subject: Re: [PATCH 00/21] btrfs: first batch of zoned cleanups
Date: Wed, 24 Nov 2021 11:34:45 -0500	[thread overview]
Message-ID: <YZ5ppVSi6DAuw/A2@localhost.localdomain> (raw)
In-Reply-To: <cover.1637745470.git.johannes.thumshirn@wdc.com>

On Wed, Nov 24, 2021 at 01:30:26AM -0800, Johannes Thumshirn wrote:
> Here's a first batch of cleanups for the zoend code. It reduces the number of
> calls to btrfs_is_zoned() outside of zoned.[ch] from 46 on misc-next to 33.
> 
> As I had to create a scrub.h file, I also moved the scrub related function
> prototypes from ctree.h to scrub.h.
> 
> Johannes Thumshirn (21):
>   btrfs: zoned: encapsulate inode locking for zoned relocation
>   btrfs: zoned: simplify btrfs_check_meta_write_pointer
>   btrfs: zoned: sink zone check into btrfs_repair_one_zone
>   btrfs: zoned: it's pointless to check for REQ_OP_ZONE_APPEND and
>     btrfs_is_zoned
>   btrfs: zoned: move compatible fs flags check to zoned code
>   btrfs: zoned: move mark_block_group_to_copy to zoned code
>   btrfs: zoned: move btrfs_finish_block_group_to_copy to zoned code
>   btrfs: zoned: move is_block_group_to_copy to zoned code
>   btrfs: zoned: skip zoned check if block_group is marked as copy
>   btrfs: move struct scrub_ctx to scrub.h
>   btrfs: zoned: move fill_writer_pointer_gap to zoned code
>   btrfs: zoned: sync_write_pointer_for_zoned to zoned code
>   btrfs: make scrub_submit and scrub_wr_submit non-static
>   btrfs: zoned: move sync_replace_for_zoned to zoned code
>   btrfs: zoned: move finish_extent_writes_for_zoned to zoned code
>   btrfs: move btrfs_scrub_dev() definition to scrub.h
>   btrfs: move btrfs_scrub_pause() definition to scrub.h
>   btrfs: move btrfs_scrub_continue() definition to scrub.h
>   btrfs: move btrfs_scrub_cancel() definition to scrub.h
>   btrfs: move btrfs_scrub_cancel_dev() definition to scrub.h
>   btrfs: move btrfs_scrub_progress() definition to scrub.h

These last six could have been a single patch but no sense in redoing it now.
Fix up that random return and you can add

Reviewed-by: Josef Bacik <josef@toxicpanda.com>

to the series, thanks,

Josef