drbd: remove unused argument from drbd_request_prepare and __drbd_make_request
authorGuoqing Jiang <guoqing.jiang@cloud.ionos.com>
Thu, 21 Jan 2021 14:21:50 +0000 (15:21 +0100)
committerJens Axboe <axboe@kernel.dk>
Tue, 26 Jan 2021 20:11:34 +0000 (13:11 -0700)
We can remove start_jif since it is not used by drbd_request_prepare,
then remove it from __drbd_make_request further.

Cc: Philipp Reisner <philipp.reisner@linbit.com>
Cc: Lars Ellenberg <lars.ellenberg@linbit.com>
Cc: drbd-dev@lists.linbit.com
Signed-off-by: Guoqing Jiang <guoqing.jiang@cloud.ionos.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
drivers/block/drbd/drbd_int.h
drivers/block/drbd/drbd_main.c
drivers/block/drbd/drbd_req.c

index b2c93a29c251fdbe1ed11a3f412afcef0cb03089..de59f72d49cc575a5e994e2b70a5d8adb2aa46d4 100644 (file)
@@ -1449,7 +1449,7 @@ extern void conn_free_crypto(struct drbd_connection *connection);
 
 /* drbd_req */
 extern void do_submit(struct work_struct *ws);
-extern void __drbd_make_request(struct drbd_device *, struct bio *, unsigned long);
+extern void __drbd_make_request(struct drbd_device *, struct bio *);
 extern blk_qc_t drbd_submit_bio(struct bio *bio);
 extern int drbd_read_remote(struct drbd_device *device, struct drbd_request *req);
 extern int is_valid_ar_handle(struct drbd_request *, sector_t);
index 1c8c18b2a25f33fc571f10e6eb7dfed21318e407..7e5fcce812e1aa8246efe13b00e85544b5b58722 100644 (file)
@@ -2288,7 +2288,6 @@ static void do_retry(struct work_struct *ws)
        list_for_each_entry_safe(req, tmp, &writes, tl_requests) {
                struct drbd_device *device = req->device;
                struct bio *bio = req->master_bio;
-               unsigned long start_jif = req->start_jif;
                bool expected;
 
                expected =
@@ -2323,7 +2322,7 @@ static void do_retry(struct work_struct *ws)
                /* We are not just doing submit_bio_noacct(),
                 * as we want to keep the start_time information. */
                inc_ap_bio(device);
-               __drbd_make_request(device, bio, start_jif);
+               __drbd_make_request(device, bio);
        }
 }
 
index ea0f31ab334361e654a61f9992ed7e76a955aebe..ee785f2bdf79c59d5106e0facac5e76af54a45d3 100644 (file)
@@ -1188,7 +1188,7 @@ static void drbd_queue_write(struct drbd_device *device, struct drbd_request *re
  * Returns ERR_PTR(-ENOMEM) if we cannot allocate a drbd_request.
  */
 static struct drbd_request *
-drbd_request_prepare(struct drbd_device *device, struct bio *bio, unsigned long start_jif)
+drbd_request_prepare(struct drbd_device *device, struct bio *bio)
 {
        const int rw = bio_data_dir(bio);
        struct drbd_request *req;
@@ -1416,9 +1416,9 @@ out:
                complete_master_bio(device, &m);
 }
 
-void __drbd_make_request(struct drbd_device *device, struct bio *bio, unsigned long start_jif)
+void __drbd_make_request(struct drbd_device *device, struct bio *bio)
 {
-       struct drbd_request *req = drbd_request_prepare(device, bio, start_jif);
+       struct drbd_request *req = drbd_request_prepare(device, bio);
        if (IS_ERR_OR_NULL(req))
                return;
        drbd_send_and_submit(device, req);
@@ -1596,19 +1596,16 @@ void do_submit(struct work_struct *ws)
 blk_qc_t drbd_submit_bio(struct bio *bio)
 {
        struct drbd_device *device = bio->bi_bdev->bd_disk->private_data;
-       unsigned long start_jif;
 
        blk_queue_split(&bio);
 
-       start_jif = jiffies;
-
        /*
         * what we "blindly" assume:
         */
        D_ASSERT(device, IS_ALIGNED(bio->bi_iter.bi_size, 512));
 
        inc_ap_bio(device);
-       __drbd_make_request(device, bio, start_jif);
+       __drbd_make_request(device, bio);
        return BLK_QC_T_NONE;
 }