On Wed, Mar 04, 2026 at 08:04:09PM +0800, Andreas Gruenbacher wrote:
> Instead of setting bio->bi_status to BLK_STS_IOERR and calling
> bio_endio(bio), use the shorthand bio_io_error(bio).
>
> Created with Coccinelle using the following semantic patch:
>
> @@
> struct bio *bio;
> @@
> - bio->bi_status = BLK_STS_IOERR;
> - bio_endio(bio);
> + bio_io_error(bio);
>
> Signed-off-by: Andreas Gruenbacher <[email protected]>
> Reviewed-by: Christoph Hellwig <[email protected]>
> ---
> block/fops.c | 3 +--
> drivers/block/drbd/drbd_int.h | 3 +--
> drivers/md/bcache/bcache.h | 3 +--
> drivers/md/bcache/request.c | 6 ++----
> drivers/md/dm-mpath.c | 3 +--
> drivers/md/dm-writecache.c | 3 +--
> fs/f2fs/segment.c | 3 +--
> 7 files changed, 8 insertions(+), 16 deletions(-)
>
[snipped]
>
> diff --git a/drivers/md/bcache/bcache.h b/drivers/md/bcache/bcache.h
> index ec9ff9715081..e0c9d9eef0a0 100644
> --- a/drivers/md/bcache/bcache.h
> +++ b/drivers/md/bcache/bcache.h
> @@ -947,8 +947,7 @@ static inline void closure_bio_submit(struct cache_set *c,
> {
> closure_get(cl);
> if (unlikely(test_bit(CACHE_SET_IO_DISABLE, &c->flags))) {
> - bio->bi_status = BLK_STS_IOERR;
> - bio_endio(bio);
> + bio_io_error(bio);
> return;
> }
> submit_bio_noacct(bio);
> diff --git a/drivers/md/bcache/request.c b/drivers/md/bcache/request.c
> index 3fa3b13a410f..0f6fa0a2920b 100644
> --- a/drivers/md/bcache/request.c
> +++ b/drivers/md/bcache/request.c
> @@ -1178,8 +1178,7 @@ void cached_dev_submit_bio(struct bio *bio)
>
> if (unlikely((d->c && test_bit(CACHE_SET_IO_DISABLE, &d->c->flags)) ||
> dc->io_disable)) {
> - bio->bi_status = BLK_STS_IOERR;
> - bio_endio(bio);
> + bio_io_error(bio);
> return;
> }
>
> @@ -1283,8 +1282,7 @@ void flash_dev_submit_bio(struct bio *bio)
> struct bcache_device *d = bio->bi_bdev->bd_disk->private_data;
>
> if (unlikely(d->c && test_bit(CACHE_SET_IO_DISABLE, &d->c->flags))) {
> - bio->bi_status = BLK_STS_IOERR;
> - bio_endio(bio);
> + bio_io_error(bio);
> return;
> }
>
For bcache part, I feel current hard code is explict and clear.
The bio_io_error() wrapper is not so directly understood. This is just
my opinion, not a strong objection.
Thanks.
Coly Li
[snipped]
