rnbd-clt: kill read_only from struct rnbd_clt_dev
authorGuoqing Jiang <guoqing.jiang@linux.dev>
Wed, 6 Jul 2022 13:31:47 +0000 (21:31 +0800)
committerJens Axboe <axboe@kernel.dk>
Tue, 2 Aug 2022 23:14:45 +0000 (17:14 -0600)
The member is not needed since we can call get_disk_ro to achieve the
same goal.

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-4-guoqing.jiang@linux.dev
Signed-off-by: Jens Axboe <axboe@kernel.dk>
drivers/block/rnbd/rnbd-clt.c
drivers/block/rnbd/rnbd-clt.h

index 9e0521a..e01d37e 100644 (file)
@@ -949,7 +949,7 @@ static int rnbd_client_open(struct block_device *block_device, fmode_t mode)
 {
        struct rnbd_clt_dev *dev = block_device->bd_disk->private_data;
 
-       if (dev->read_only && (mode & FMODE_WRITE))
+       if (get_disk_ro(dev->gd) && (mode & FMODE_WRITE))
                return -EPERM;
 
        if (dev->dev_state == DEV_STATE_UNMAPPED ||
@@ -1402,12 +1402,8 @@ static int rnbd_clt_setup_gen_disk(struct rnbd_clt_dev *dev, int idx)
 
        set_capacity(dev->gd, dev->nsectors);
 
-       if (dev->access_mode == RNBD_ACCESS_RO) {
-               dev->read_only = true;
+       if (dev->access_mode == RNBD_ACCESS_RO)
                set_disk_ro(dev->gd, true);
-       } else {
-               dev->read_only = false;
-       }
 
        /*
         * Network device does not need rotational
index 2e2e8c4..26fb91d 100644 (file)
@@ -117,7 +117,6 @@ struct rnbd_clt_dev {
        char                    *pathname;
        enum rnbd_access_mode   access_mode;
        u32                     nr_poll_queues;
-       bool                    read_only;
        bool                    wc;
        bool                    fua;
        u32                     max_hw_sectors;