mbox series

[00/13] btrfs-progs: cleanup btrfs_item* accessors

Message ID cover.1645568701.git.josef@toxicpanda.com (mailing list archive)
Headers show
Series btrfs-progs: cleanup btrfs_item* accessors | expand

Message

Josef Bacik Feb. 22, 2022, 10:26 p.m. UTC
Hello,

This is prep work for me adjusting the size of the btrfs_header.  This is a
mirror of the work that was already merged into the kernel.  The only extra work
that I've done here is add some compiler switches that I used to help me catch
all the different users of the helpers to make sure I didn't make any mistakes.
Thanks,

Josef

Josef Bacik (13):
  btrfs-progs: turn on more compiler warnings and use -Wall
  btrfs-progs: store LEAF_DATA_SIZE in the mkfs_config
  btrfs-progs: store BTRFS_LEAF_DATA_SIZE in the fs_info
  btrfs-progs: convert: use cfg->leaf_data_size
  btrfs-progs: reduce usage of __BTRFS_LEAF_DATA_SIZE
  btrfs-progs: btrfs_item_size_nr/btrfs_item_offset_nr everywhere
  btrfs-progs: add btrfs_set_item_*_nr() helpers
  btrfs-progs: change btrfs_file_extent_inline_item_len to take a slot
  btrfs-progs: rename btrfs_item_end_nr to btrfs_item_end
  btrfs-progs: remove the _nr from the item helpers
  btrfs-progs: replace btrfs_item_nr_offset(0)
  btrfs-progs: rework the btrfs_node accessors to match the item
    accessors
  btrfs-progs: make all of the item/key_ptr offset helpers take an eb

 Makefile                    |   3 +
 btrfs-corrupt-block.c       |  17 ++-
 check/main.c                |  90 +++++++--------
 check/mode-common.c         |  12 +-
 check/mode-lowmem.c         |  37 +++---
 check/qgroup-verify.c       |   2 +-
 cmds/inspect-tree-stats.c   |   3 +-
 cmds/rescue-chunk-recover.c |   4 +-
 cmds/restore.c              |   7 +-
 convert/common.c            |  33 +++---
 convert/main.c              |   1 +
 image/main.c                |  31 +++--
 image/sanitize.c            |   4 +-
 kernel-shared/backref.c     |  10 +-
 kernel-shared/ctree.c       | 219 ++++++++++++++++--------------------
 kernel-shared/ctree.h       | 110 ++++++++----------
 kernel-shared/dir-item.c    |  10 +-
 kernel-shared/disk-io.c     |   1 +
 kernel-shared/extent-tree.c |  12 +-
 kernel-shared/file-item.c   |  12 +-
 kernel-shared/inode-item.c  |  14 +--
 kernel-shared/print-tree.c  |  29 +++--
 kernel-shared/root-tree.c   |   2 +-
 kernel-shared/uuid-tree.c   |   4 +-
 kernel-shared/volumes.c     |  10 +-
 mkfs/common.c               |  52 ++++-----
 mkfs/common.h               |   1 +
 mkfs/main.c                 |   1 +
 28 files changed, 333 insertions(+), 398 deletions(-)

Comments

Nikolay Borisov March 9, 2022, 12:46 p.m. UTC | #1
On 23.02.22 г. 0:26 ч., Josef Bacik wrote:
> Hello,
> 
> This is prep work for me adjusting the size of the btrfs_header.  This is a
> mirror of the work that was already merged into the kernel.  The only extra work
> that I've done here is add some compiler switches that I used to help me catch
> all the different users of the helpers to make sure I didn't make any mistakes.
> Thanks,
> 
> Josef
> 
> Josef Bacik (13):
>    btrfs-progs: turn on more compiler warnings and use -Wall
>    btrfs-progs: store LEAF_DATA_SIZE in the mkfs_config
>    btrfs-progs: store BTRFS_LEAF_DATA_SIZE in the fs_info
>    btrfs-progs: convert: use cfg->leaf_data_size
>    btrfs-progs: reduce usage of __BTRFS_LEAF_DATA_SIZE
>    btrfs-progs: btrfs_item_size_nr/btrfs_item_offset_nr everywhere
>    btrfs-progs: add btrfs_set_item_*_nr() helpers
>    btrfs-progs: change btrfs_file_extent_inline_item_len to take a slot
>    btrfs-progs: rename btrfs_item_end_nr to btrfs_item_end
>    btrfs-progs: remove the _nr from the item helpers
>    btrfs-progs: replace btrfs_item_nr_offset(0)
>    btrfs-progs: rework the btrfs_node accessors to match the item
>      accessors
>    btrfs-progs: make all of the item/key_ptr offset helpers take an eb
> 
>   Makefile                    |   3 +
>   btrfs-corrupt-block.c       |  17 ++-
>   check/main.c                |  90 +++++++--------
>   check/mode-common.c         |  12 +-
>   check/mode-lowmem.c         |  37 +++---
>   check/qgroup-verify.c       |   2 +-
>   cmds/inspect-tree-stats.c   |   3 +-
>   cmds/rescue-chunk-recover.c |   4 +-
>   cmds/restore.c              |   7 +-
>   convert/common.c            |  33 +++---
>   convert/main.c              |   1 +
>   image/main.c                |  31 +++--
>   image/sanitize.c            |   4 +-
>   kernel-shared/backref.c     |  10 +-
>   kernel-shared/ctree.c       | 219 ++++++++++++++++--------------------
>   kernel-shared/ctree.h       | 110 ++++++++----------
>   kernel-shared/dir-item.c    |  10 +-
>   kernel-shared/disk-io.c     |   1 +
>   kernel-shared/extent-tree.c |  12 +-
>   kernel-shared/file-item.c   |  12 +-
>   kernel-shared/inode-item.c  |  14 +--
>   kernel-shared/print-tree.c  |  29 +++--
>   kernel-shared/root-tree.c   |   2 +-
>   kernel-shared/uuid-tree.c   |   4 +-
>   kernel-shared/volumes.c     |  10 +-
>   mkfs/common.c               |  52 ++++-----
>   mkfs/common.h               |   1 +
>   mkfs/main.c                 |   1 +
>   28 files changed, 333 insertions(+), 398 deletions(-)
> 

For the whole series :

Reviewed-by: Nikolay Borisov <nborisov@suse.com>