Re: [PATCH 0/3] Bool return cleanups

[Naohiro Aota <Naohiro.Aota@wdc.com>]

On Thu Apr 24, 2025 at 1:53 AM JST, David Sterba wrote:
> We have old code that uses int in place of bool, unify it to bool.  The
> rest are collateral fixups.
>
> David Sterba (3):
>   btrfs: trivial conversion to return bool instead of int
>   btrfs: switch int dev_replace_is_ongoing variables/parameters to bool
>   btrfs: reformat comments in acls_after_inode_item()

Other than some trivial nits on the patch 3, for the series

Reviewed-by: Naohiro Aota <naohiro.aota@wdc.com>

>
>  fs/btrfs/block-group.c   |  12 ++---
>  fs/btrfs/defrag.c        |   8 +--
>  fs/btrfs/delayed-inode.c |   8 +--
>  fs/btrfs/dev-replace.c   |   8 +--
>  fs/btrfs/dev-replace.h   |   2 +-
>  fs/btrfs/extent-tree.c   |  10 ++--
>  fs/btrfs/file.c          |  33 ++++++-------
>  fs/btrfs/inode.c         |  51 +++++++++++--------
>  fs/btrfs/ioctl.c         |  10 ++--
>  fs/btrfs/locking.c       |   8 +--
>  fs/btrfs/locking.h       |   2 +-
>  fs/btrfs/super.c         |   6 +--
>  fs/btrfs/transaction.c   |   8 +--
>  fs/btrfs/volumes.c       | 104 ++++++++++++++++++---------------------
>  14 files changed, 136 insertions(+), 134 deletions(-)