btrfs: use __bio_add_page to add single a page in rbio_add_io_sector
authorJohannes Thumshirn <johannes.thumshirn@wdc.com>
Thu, 30 Mar 2023 10:43:51 +0000 (03:43 -0700)
committerDavid Sterba <dsterba@suse.com>
Mon, 17 Apr 2023 16:01:20 +0000 (18:01 +0200)
The btrfs raid56 sector submission code uses bio_add_page() to add a
page to a newly created bio. bio_add_page() can fail, but the return
value is never checked.

Use __bio_add_page() as adding a single page to a newly created bio is
guaranteed to succeed.

This brings us a step closer to marking bio_add_page() as __must_check.

Reviewed-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
Signed-off-by: Johannes Thumshirn <johannes.thumshirn@wdc.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
fs/btrfs/raid56.c

index 6cbbaa6..f4651b6 100644 (file)
@@ -1102,7 +1102,7 @@ static int rbio_add_io_sector(struct btrfs_raid_bio *rbio,
        bio->bi_iter.bi_sector = disk_start >> 9;
        bio->bi_private = rbio;
 
-       bio_add_page(bio, sector->page, sectorsize, sector->pgoff);
+       __bio_add_page(bio, sector->page, sectorsize, sector->pgoff);
        bio_list_add(bio_list, bio);
        return 0;
 }