From 5df69ece6e93cfd4e09b14bf32bd101df6cbde38 Mon Sep 17 00:00:00 2001 From: Lars Ellenberg Date: Tue, 24 Jan 2012 16:49:58 +0100 Subject: [PATCH] drbd: __drbd_make_request() is now void The previous commit causes __drbd_make_request() to always return 0. Change it to void. Signed-off-by: Philipp Reisner Signed-off-by: Lars Ellenberg --- drivers/block/drbd/drbd_int.h | 2 +- drivers/block/drbd/drbd_main.c | 5 ++--- drivers/block/drbd/drbd_req.c | 13 ++++++------- 3 files changed, 9 insertions(+), 11 deletions(-) diff --git a/drivers/block/drbd/drbd_int.h b/drivers/block/drbd/drbd_int.h index 309c121557ae..8536fabbf984 100644 --- a/drivers/block/drbd/drbd_int.h +++ b/drivers/block/drbd/drbd_int.h @@ -1403,7 +1403,7 @@ extern void conn_free_crypto(struct drbd_tconn *tconn); extern int proc_details; /* drbd_req */ -extern int __drbd_make_request(struct drbd_conf *, struct bio *, unsigned long); +extern void __drbd_make_request(struct drbd_conf *, struct bio *, unsigned long); extern int drbd_make_request(struct request_queue *q, struct bio *bio); extern int drbd_read_remote(struct drbd_conf *mdev, struct drbd_request *req); extern int drbd_merge_bvec(struct request_queue *q, struct bvec_merge_data *bvm, struct bio_vec *bvec); diff --git a/drivers/block/drbd/drbd_main.c b/drivers/block/drbd/drbd_main.c index 8c6c48e363cd..bedfeeccd513 100644 --- a/drivers/block/drbd/drbd_main.c +++ b/drivers/block/drbd/drbd_main.c @@ -2236,9 +2236,8 @@ static void do_retry(struct work_struct *ws) /* We are not just doing generic_make_request(), * as we want to keep the start_time information. */ - do { - inc_ap_bio(mdev); - } while(__drbd_make_request(mdev, bio, start_time)); + inc_ap_bio(mdev); + __drbd_make_request(mdev, bio, start_time); } } diff --git a/drivers/block/drbd/drbd_req.c b/drivers/block/drbd/drbd_req.c index d2d61af034ec..6bac415358d7 100644 --- a/drivers/block/drbd/drbd_req.c +++ b/drivers/block/drbd/drbd_req.c @@ -945,7 +945,7 @@ drbd_submit_req_private_bio(struct drbd_request *req) bio_endio(bio, -EIO); } -int __drbd_make_request(struct drbd_conf *mdev, struct bio *bio, unsigned long start_time) +void __drbd_make_request(struct drbd_conf *mdev, struct bio *bio, unsigned long start_time) { const int rw = bio_rw(bio); struct bio_and_error m = { NULL, }; @@ -960,7 +960,7 @@ int __drbd_make_request(struct drbd_conf *mdev, struct bio *bio, unsigned long s * if user cannot handle io errors, that's not our business. */ dev_err(DEV, "could not kmalloc() req\n"); bio_endio(bio, -ENOMEM); - return 0; + return; } req->start_time = start_time; @@ -1039,7 +1039,7 @@ int __drbd_make_request(struct drbd_conf *mdev, struct bio *bio, unsigned long s drbd_submit_req_private_bio(req); /* once we have submitted, we must no longer look at req, * it may already be destroyed. */ - return 0; + return; } else if (no_remote) { nodata: if (__ratelimit(&drbd_ratelimit_state)) @@ -1054,7 +1054,7 @@ out: if (m.bio) complete_master_bio(mdev, &m); - return 0; + return; } int drbd_make_request(struct request_queue *q, struct bio *bio) @@ -1070,9 +1070,8 @@ int drbd_make_request(struct request_queue *q, struct bio *bio) D_ASSERT(bio->bi_size > 0); D_ASSERT(IS_ALIGNED(bio->bi_size, 512)); - do { - inc_ap_bio(mdev); - } while (__drbd_make_request(mdev, bio, start_time)); + inc_ap_bio(mdev); + __drbd_make_request(mdev, bio, start_time); return 0; } -- 2.39.5