block: create 'slaves' and 'holders' entries for hidden gendisks
authorHannes Reinecke <hare@suse.de>
Thu, 9 Nov 2017 16:57:06 +0000 (17:57 +0100)
committerJens Axboe <axboe@kernel.dk>
Sat, 11 Nov 2017 02:53:25 +0000 (19:53 -0700)
When creating nvme multipath devices we should populate the 'slaves' and
'holders' directorys properly to aid userspace topology detection.

Signed-off-by: Hannes Reinecke <hare@suse.com>
[hch: split from a larger patch]
Reviewed-by: Keith Busch <keith.busch@intel.com>
Reviewed-by: Martin K. Petersen <martin.petersen@oracle.com>
Signed-off-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
block/genhd.c

index 835e907..3de1671 100644 (file)
@@ -585,14 +585,14 @@ static void register_disk(struct device *parent, struct gendisk *disk)
         */
        pm_runtime_set_memalloc_noio(ddev, true);
 
+       disk->part0.holder_dir = kobject_create_and_add("holders", &ddev->kobj);
+       disk->slave_dir = kobject_create_and_add("slaves", &ddev->kobj);
+
        if (disk->flags & GENHD_FL_HIDDEN) {
                dev_set_uevent_suppress(ddev, 0);
                return;
        }
 
-       disk->part0.holder_dir = kobject_create_and_add("holders", &ddev->kobj);
-       disk->slave_dir = kobject_create_and_add("slaves", &ddev->kobj);
-
        /* No minors to use for partitions */
        if (!disk_part_scan_enabled(disk))
                goto exit;
@@ -728,11 +728,11 @@ void del_gendisk(struct gendisk *disk)
                WARN_ON(1);
        }
 
-       if (!(disk->flags & GENHD_FL_HIDDEN)) {
+       if (!(disk->flags & GENHD_FL_HIDDEN))
                blk_unregister_region(disk_devt(disk), disk->minors);
-               kobject_put(disk->part0.holder_dir);
-               kobject_put(disk->slave_dir);
-       }
+
+       kobject_put(disk->part0.holder_dir);
+       kobject_put(disk->slave_dir);
 
        part_stat_set_all(&disk->part0, 0);
        disk->part0.stamp = 0;