Re: [PATCH 00/12] Cleanups of blk_status_t use

[Qu Wenruo <quwenruo.btrfs@gmx.com>]

在 2025/4/24 01:27, David Sterba 写道:
> Use the blk_status_t only for interaction with block layer API and
> remove it from our functions. Do some naming unifications and minor
> cleanups.

Reviewed-by: Qu Wenruo <wqu@suse.com>

Although I'm wondering if it's even possible to require a forced 
compiler warning when using blk_status_t as other types and vice verse?

Thanks,
Qu

> 
> David Sterba (12):
>    btrfs: drop redundant local variable in raid_wait_write_end_io()
>    btrfs: change return type of btrfs_lookup_bio_sums() to int
>    btrfs: change return type of btrfs_csum_one_bio() to int
>    btrfs: change return type of btree_csum_one_bio() to int
>    btrfs: change return type of btrfs_bio_csum() to int
>    btrfs: rename ret to status in btrfs_submit_chunk()
>    btrfs: rename error to ret in btrfs_submit_chunk()
>    btrfs: simplify reading bio status in end_compressed_writeback()
>    btrfs: rename ret to status in btrfs_submit_compressed_read()
>    btrfs: rename ret2 to ret in btrfs_submit_compressed_read()
>    btrfs: change return type of btrfs_alloc_dummy_sum() to int
>    btrfs: raid56: rename parameter err to status in endio helpers
> 
>   fs/btrfs/bio.c         | 33 ++++++++++++++++++---------------
>   fs/btrfs/compression.c | 22 +++++++++++-----------
>   fs/btrfs/disk-io.c     | 16 ++++++++--------
>   fs/btrfs/disk-io.h     |  2 +-
>   fs/btrfs/file-item.c   | 20 ++++++++++----------
>   fs/btrfs/file-item.h   |  6 +++---
>   fs/btrfs/raid56.c      | 13 ++++++-------
>   7 files changed, 57 insertions(+), 55 deletions(-)
>