[v2,0/3] Ordered extent flushing refactor
mbox series

Message ID 20190507071924.17643-1-nborisov@suse.com
Headers show
Series
  • Ordered extent flushing refactor
Related show

Message

Nikolay Borisov May 7, 2019, 7:19 a.m. UTC
Here is v2 of factoring out common code when flushing ordered extent. The main
change in this version is the switch from inode to btrfs_inode for  function 
interfaces as per David's feedback. 

Nikolay Borisov (3):
  btrfs: Implement btrfs_lock_and_flush_ordered_range
  btrfs: Use newly introduced btrfs_lock_and_flush_ordered_range
  btrfs: Always use a cached extent_state in
    btrfs_lock_and_flush_ordered_range

 fs/btrfs/extent_io.c    | 29 ++++----------------------
 fs/btrfs/file.c         | 14 ++-----------
 fs/btrfs/inode.c        | 17 ++--------------
 fs/btrfs/ordered-data.c | 45 +++++++++++++++++++++++++++++++++++++++++
 fs/btrfs/ordered-data.h |  4 ++++
 5 files changed, 57 insertions(+), 52 deletions(-)

Comments

Josef Bacik May 7, 2019, 5:39 p.m. UTC | #1
On Tue, May 07, 2019 at 10:19:21AM +0300, Nikolay Borisov wrote:
> Here is v2 of factoring out common code when flushing ordered extent. The main
> change in this version is the switch from inode to btrfs_inode for  function 
> interfaces as per David's feedback. 
> 
> Nikolay Borisov (3):
>   btrfs: Implement btrfs_lock_and_flush_ordered_range
>   btrfs: Use newly introduced btrfs_lock_and_flush_ordered_range
>   btrfs: Always use a cached extent_state in
>     btrfs_lock_and_flush_ordered_range
> 

You can add

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

for the whole series.  Thanks,

Josef
David Sterba May 10, 2019, 12:05 p.m. UTC | #2
On Tue, May 07, 2019 at 10:19:21AM +0300, Nikolay Borisov wrote:
> Here is v2 of factoring out common code when flushing ordered extent. The main
> change in this version is the switch from inode to btrfs_inode for  function 
> interfaces as per David's feedback. 
> 
> Nikolay Borisov (3):
>   btrfs: Implement btrfs_lock_and_flush_ordered_range
>   btrfs: Use newly introduced btrfs_lock_and_flush_ordered_range
>   btrfs: Always use a cached extent_state in
>     btrfs_lock_and_flush_ordered_range

1-3 added to 5.3 queue, thanks.