From: Johannes Thumshirn <johannes.thumsh...@wdc.com>

The drbd code only adds a single page to a newly created bio. So use
__bio_add_page() to add the page which is guaranteed to succeed in this
case.

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

Signed-off-by: Johannes Thumshirn <johannes.thumsh...@wdc.com>
Reviewed-by: Damien Le Moal <damien.lem...@opensource.wdc.com>
---
 drivers/block/drbd/drbd_bitmap.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/drivers/block/drbd/drbd_bitmap.c b/drivers/block/drbd/drbd_bitmap.c
index 289876ffbc31..050154eb963d 100644
--- a/drivers/block/drbd/drbd_bitmap.c
+++ b/drivers/block/drbd/drbd_bitmap.c
@@ -1043,9 +1043,7 @@ static void bm_page_io_async(struct drbd_bm_aio_ctx *ctx, 
int page_nr) __must_ho
        bio = bio_alloc_bioset(device->ldev->md_bdev, 1, op, GFP_NOIO,
                        &drbd_md_io_bio_set);
        bio->bi_iter.bi_sector = on_disk_sector;
-       /* bio_add_page of a single page to an empty bio will always succeed,
-        * according to api.  Do we want to assert that? */
-       bio_add_page(bio, page, len, 0);
+       __bio_add_page(bio, page, len, 0);
        bio->bi_private = ctx;
        bio->bi_end_io = drbd_bm_endio;
 
-- 
2.39.2

Reply via email to