rnbd-clt: make rnbd_clt_change_capacity return void
authorGuoqing Jiang <guoqing.jiang@linux.dev>
Wed, 6 Jul 2022 13:31:52 +0000 (21:31 +0800)
committerJens Axboe <axboe@kernel.dk>
Tue, 2 Aug 2022 23:14:45 +0000 (17:14 -0600)
No need to checking the return value, make it return void.

Acked-by: Jack Wang <jinpu.wang@ionos.com>
Signed-off-by: Guoqing Jiang <guoqing.jiang@linux.dev>
Link: https://lore.kernel.org/r/20220706133152.12058-9-guoqing.jiang@linux.dev
Signed-off-by: Jens Axboe <axboe@kernel.dk>
drivers/block/rnbd/rnbd-clt.c

index 58e5ffc..04da33a 100644 (file)
@@ -68,11 +68,11 @@ static inline bool rnbd_clt_get_dev(struct rnbd_clt_dev *dev)
        return refcount_inc_not_zero(&dev->refcount);
 }
 
-static int rnbd_clt_change_capacity(struct rnbd_clt_dev *dev,
+static void rnbd_clt_change_capacity(struct rnbd_clt_dev *dev,
                                    sector_t new_nsectors)
 {
        if (get_capacity(dev->gd) == new_nsectors)
-               return 0;
+               return;
 
        /*
         * If the size changed, we need to revalidate it
@@ -80,7 +80,6 @@ static int rnbd_clt_change_capacity(struct rnbd_clt_dev *dev,
        rnbd_clt_info(dev, "Device size changed from %llu to %llu sectors\n",
                      get_capacity(dev->gd), new_nsectors);
        set_capacity_and_notify(dev->gd, new_nsectors);
-       return 0;
 }
 
 static int process_msg_open_rsp(struct rnbd_clt_dev *dev,
@@ -127,7 +126,7 @@ int rnbd_clt_resize_disk(struct rnbd_clt_dev *dev, sector_t newsize)
                ret = -ENOENT;
                goto out;
        }
-       ret = rnbd_clt_change_capacity(dev, newsize);
+       rnbd_clt_change_capacity(dev, newsize);
 
 out:
        mutex_unlock(&dev->lock);