Re: [PATCH v2 0/3] btrfs: optimize the argument list for submit_extent_page()

[David Sterba <dsterba@suse.cz>]

On Tue, Sep 13, 2022 at 01:31:11PM +0800, Qu Wenruo wrote:
> [Changelog]
> v2:
> - Add a patch to remove stale argument from the comments of
>   submit_extent_page()
> 
> - Update the comment of submit_extent_page() to reflect the new arugment
>   list
> 
> The argument list of submit_extent_page() is already a little long.
> 
> Although we have things like page, pg_len, pg_off which can not be saved
> anyway, we can still improve the situation by:
> 
> - Update the stale comment of submit_extent_page()
>   Done by the first patch.
> 
> - Make sure @page, @pg_len, @pg_off are always batched together
>   Just like bio_add_page().
> 
>   This is done by the second page, just switching the position between
>   @page and @disk_bytenr.
> 
> - Move @end_io_func arugment into btrfs_bio_ctrl structure.
> 
> Qu Wenruo (3):
>   btrfs: update the comment for submit_extent_page()
>   btrfs: switch the page and disk_bytenr argument position for
>     submit_extent_page()
>   btrfs: move end_io_func argument to btrfs_bio_ctrl structure

Added to misc-next, thanks.