[v2,0/4] Parameter cleanup

Message

David Sterba Oct. 20, 2022, 4:59 p.m. UTC

A few more cases where value passed by parameter can be used directly.

v2:
- move related code from patch 2 to patch 1

David Sterba (4):
  btrfs: sink gfp_t parameter to btrfs_backref_iter_alloc
  btrfs: sink gfp_t parameter to btrfs_qgroup_trace_extent
  btrfs: switch GFP_NOFS to GFP_KERNEL in scrub_setup_recheck_block
  btrfs: sink gfp_t parameter to alloc_scrub_sector

 fs/btrfs/backref.c    |  5 ++---
 fs/btrfs/backref.h    |  3 +--
 fs/btrfs/qgroup.c     | 17 +++++++----------
 fs/btrfs/qgroup.h     |  2 +-
 fs/btrfs/relocation.c |  2 +-
 fs/btrfs/scrub.c      | 14 +++++++-------
 fs/btrfs/tree-log.c   |  3 +--
 7 files changed, 20 insertions(+), 26 deletions(-)

Comments

Anand Jain Oct. 21, 2022, 2:42 a.m. UTC | #1

On 21/10/2022 00:59, David Sterba wrote:
> A few more cases where value passed by parameter can be used directly.
> 
> v2:
> - move related code from patch 2 to patch 1
> 
> David Sterba (4):
>    btrfs: sink gfp_t parameter to btrfs_backref_iter_alloc
>    btrfs: sink gfp_t parameter to btrfs_qgroup_trace_extent
>    btrfs: switch GFP_NOFS to GFP_KERNEL in scrub_setup_recheck_block
>    btrfs: sink gfp_t parameter to alloc_scrub_sector

For the series:
Reviewed-by: Anand Jain <[e-mail]>

> 
>   fs/btrfs/backref.c    |  5 ++---
>   fs/btrfs/backref.h    |  3 +--
>   fs/btrfs/qgroup.c     | 17 +++++++----------
>   fs/btrfs/qgroup.h     |  2 +-
>   fs/btrfs/relocation.c |  2 +-
>   fs/btrfs/scrub.c      | 14 +++++++-------
>   fs/btrfs/tree-log.c   |  3 +--
>   7 files changed, 20 insertions(+), 26 deletions(-)
>

Johannes Thumshirn Oct. 21, 2022, 8:07 a.m. UTC | #2

For the series,
Reviewed-by: Johannes Thumshirn <[e-mail]>