block/rnbd: Kill rnbd_clt_destroy_default_group
authorGuoqing Jiang <guoqing.jiang@cloud.ionos.com>
Mon, 19 Apr 2021 07:37:09 +0000 (09:37 +0200)
committerJens Axboe <axboe@kernel.dk>
Tue, 20 Apr 2021 14:59:04 +0000 (08:59 -0600)
No need to have it since we can call sysfs_remove_group in the
rnbd_clt_destroy_sysfs_files.

Then rnbd_clt_destroy_sysfs_files is paired with it's counterpart
rnbd_clt_create_sysfs_files.

Signed-off-by: Guoqing Jiang <guoqing.jiang@ionos.com>
Reviewed-by: Danil Kipnis <danil.kipnis@ionos.com>
Signed-off-by: Gioh Kim <gi-oh.kim@ionos.com>
Signed-off-by: Jack Wang <jinpu.wang@ionos.com>
Reviewed-by: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com>
Link: https://lore.kernel.org/r/20210419073722.15351-7-gi-oh.kim@ionos.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
drivers/block/rnbd/rnbd-clt-sysfs.c
drivers/block/rnbd/rnbd-clt.c
drivers/block/rnbd/rnbd-clt.h

index d4aa6bf..58c2cc0 100644 (file)
@@ -639,13 +639,9 @@ cls_destroy:
        return err;
 }
 
-void rnbd_clt_destroy_default_group(void)
-{
-       sysfs_remove_group(&rnbd_dev->kobj, &default_attr_group);
-}
-
 void rnbd_clt_destroy_sysfs_files(void)
 {
+       sysfs_remove_group(&rnbd_dev->kobj, &default_attr_group);
        kobject_del(rnbd_devs_kobj);
        kobject_put(rnbd_devs_kobj);
        device_destroy(rnbd_dev_class, MKDEV(0, 0));
index f864f06..4e687ec 100644 (file)
@@ -1675,7 +1675,6 @@ static void rnbd_destroy_sessions(void)
        struct rnbd_clt_dev *dev, *tn;
 
        /* Firstly forbid access through sysfs interface */
-       rnbd_clt_destroy_default_group();
        rnbd_clt_destroy_sysfs_files();
 
        /*
index 537d499..714d426 100644 (file)
@@ -159,7 +159,6 @@ int rnbd_clt_resize_disk(struct rnbd_clt_dev *dev, size_t newsize);
 int rnbd_clt_create_sysfs_files(void);
 
 void rnbd_clt_destroy_sysfs_files(void);
-void rnbd_clt_destroy_default_group(void);
 
 void rnbd_clt_remove_dev_symlink(struct rnbd_clt_dev *dev);