Re: [PATCH 00/11] Summer argument cleanup

2018-05-31 Thread David Sterba
On Mon, May 28, 2018 at 09:36:39AM +0300, Nikolay Borisov wrote:
> Hello, 
> 
> Here is yet another series removing a bunch of extraneous argument. The 
> series 
> constitutes no functional changes. Some of the function actually have a 
> kernel 
> counterpart (btrfs_lookup_extent_info/clean_tree_block). However the former's 
> userspace signature differs and the patch in this series makes it identical 
> to 
> the kernel one. 
> 
> Nikolay Borisov (11):
>   btrfs-progs: check: Remove root argument from delete_extent_records
>   btrfs-progs: check: Remove root parameter from
> btrfs_fix_block_accounting
>   btrfs-progs: check: Remove root parameter from del_pending_extents
>   btrfs-progs: check: Remove root argument from finish_current_insert
>   btrfs-progs: check: Make update_pinned_extents take btrfs_fs_info
>   btrfs-progs: Remove unused argument from clean_tree_block
>   btrfs-progs: check: Remove unused root argument from
> btrfs_extent_post_op
>   btrfs-progs: Change btrfs_root to btrfs_fs_info argument in
> btrfs_lookup_extent_info
>   btrfs-progs: Remove root argument from btrfs_set_block_flags
>   btrfs-progs: Remove root argument from write_one_cache_group
>   btrfs-progs: Remove fs_info argument from write_ctree_super

Applied, thanks.
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH 00/11] Summer argument cleanup

2018-05-28 Thread Lu Fengqi
On Mon, May 28, 2018 at 09:36:39AM +0300, Nikolay Borisov wrote:
>Hello, 
>
>Here is yet another series removing a bunch of extraneous argument. The series 
>constitutes no functional changes. Some of the function actually have a kernel 
>counterpart (btrfs_lookup_extent_info/clean_tree_block). However the former's 
>userspace signature differs and the patch in this series makes it identical to 
>the kernel one. 

This series looks good to me.

Reviewed-by: Lu Fengqi 

-- 
Thanks,
Lu

>
>Nikolay Borisov (11):
>  btrfs-progs: check: Remove root argument from delete_extent_records
>  btrfs-progs: check: Remove root parameter from
>btrfs_fix_block_accounting
>  btrfs-progs: check: Remove root parameter from del_pending_extents
>  btrfs-progs: check: Remove root argument from finish_current_insert
>  btrfs-progs: check: Make update_pinned_extents take btrfs_fs_info
>  btrfs-progs: Remove unused argument from clean_tree_block
>  btrfs-progs: check: Remove unused root argument from
>btrfs_extent_post_op
>  btrfs-progs: Change btrfs_root to btrfs_fs_info argument in
>btrfs_lookup_extent_info
>  btrfs-progs: Remove root argument from btrfs_set_block_flags
>  btrfs-progs: Remove root argument from write_one_cache_group
>  btrfs-progs: Remove fs_info argument from write_ctree_super
>
> check/main.c|  31 +++---
> check/mode-lowmem.c |   6 +--
> ctree.c |  17 
> ctree.h |  13 +++---
> disk-io.c   |   9 ++--
> disk-io.h   |   6 +--
> extent-tree.c   | 121 
> free-space-tree.c   |   2 +-
> transaction.c   |   2 +-
> 9 files changed, 96 insertions(+), 111 deletions(-)
>
>-- 
>2.7.4
>
>--
>To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
>the body of a message to majord...@vger.kernel.org
>More majordomo info at  http://vger.kernel.org/majordomo-info.html
>
>


--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH 00/11] Summer argument cleanup

2018-05-28 Thread Nikolay Borisov
Hello, 

Here is yet another series removing a bunch of extraneous argument. The series 
constitutes no functional changes. Some of the function actually have a kernel 
counterpart (btrfs_lookup_extent_info/clean_tree_block). However the former's 
userspace signature differs and the patch in this series makes it identical to 
the kernel one. 

Nikolay Borisov (11):
  btrfs-progs: check: Remove root argument from delete_extent_records
  btrfs-progs: check: Remove root parameter from
btrfs_fix_block_accounting
  btrfs-progs: check: Remove root parameter from del_pending_extents
  btrfs-progs: check: Remove root argument from finish_current_insert
  btrfs-progs: check: Make update_pinned_extents take btrfs_fs_info
  btrfs-progs: Remove unused argument from clean_tree_block
  btrfs-progs: check: Remove unused root argument from
btrfs_extent_post_op
  btrfs-progs: Change btrfs_root to btrfs_fs_info argument in
btrfs_lookup_extent_info
  btrfs-progs: Remove root argument from btrfs_set_block_flags
  btrfs-progs: Remove root argument from write_one_cache_group
  btrfs-progs: Remove fs_info argument from write_ctree_super

 check/main.c|  31 +++---
 check/mode-lowmem.c |   6 +--
 ctree.c |  17 
 ctree.h |  13 +++---
 disk-io.c   |   9 ++--
 disk-io.h   |   6 +--
 extent-tree.c   | 121 
 free-space-tree.c   |   2 +-
 transaction.c   |   2 +-
 9 files changed, 96 insertions(+), 111 deletions(-)

-- 
2.7.4

--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html