Skip to content

Commit

Permalink
block: create the bdi link earlier in device_add_disk
Browse files Browse the repository at this point in the history
This will simplify error handling going forward.

Signed-off-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Hannes Reinecke <hare@suse.de>
Link: https://lore.kernel.org/r/20210818144542.19305-5-hch@lst.de
Signed-off-by: Jens Axboe <axboe@kernel.dk>
  • Loading branch information
Christoph Hellwig authored and Jens Axboe committed Aug 23, 2021
1 parent 8235b5c commit 9d5ee67
Showing 1 changed file with 6 additions and 7 deletions.
13 changes: 6 additions & 7 deletions block/genhd.c
Original file line number Diff line number Diff line change
Expand Up @@ -518,8 +518,13 @@ void device_add_disk(struct device *parent, struct gendisk *disk,
disk->major, disk->first_minor);
WARN_ON(ret);
bdi_set_owner(disk->bdi, ddev);
bdev_add(disk->part0, ddev->devt);
if (disk->bdi->dev) {
ret = sysfs_create_link(&ddev->kobj,
&disk->bdi->dev->kobj, "bdi");
WARN_ON(ret);
}

bdev_add(disk->part0, ddev->devt);
disk_scan_partitions(disk);

/*
Expand All @@ -528,12 +533,6 @@ void device_add_disk(struct device *parent, struct gendisk *disk,
*/
dev_set_uevent_suppress(ddev, 0);
disk_uevent(disk, KOBJ_ADD);

if (disk->bdi->dev) {
ret = sysfs_create_link(&ddev->kobj,
&disk->bdi->dev->kobj, "bdi");
WARN_ON(ret);
}
}

blk_register_queue(disk);
Expand Down

0 comments on commit 9d5ee67

Please sign in to comment.