提交 548b10eb 编写于 作者: T Tejun Heo 提交者: Jens Axboe

block: move __dev from disk to part0

Move disk->__dev to part0->__dev.  This simplifies bdget_disk() and
lookup_devt() and allows common sysfs attributes to be unified.
part_to_disk() is updated to handle part0 -> disk.

Updated to include a fix from Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>,
he writes:

"part0 is a "special" partition and doesn't need to have capacity set - this
fixes regression caused by "block: move __dev from disk to part0" commit."
Signed-off-by: NTejun Heo <tj@kernel.org>
Signed-off-by: NJens Axboe <jens.axboe@oracle.com>
上级 80795aef
...@@ -537,22 +537,15 @@ struct gendisk *get_gendisk(dev_t devt, int *partno) ...@@ -537,22 +537,15 @@ struct gendisk *get_gendisk(dev_t devt, int *partno)
*/ */
extern struct block_device *bdget_disk(struct gendisk *disk, int partno) extern struct block_device *bdget_disk(struct gendisk *disk, int partno)
{ {
dev_t devt = MKDEV(0, 0); struct hd_struct *part;
struct block_device *bdev = NULL;
if (partno == 0) part = disk_get_part(disk, partno);
devt = disk_devt(disk); if (part && (part->nr_sects || partno == 0))
else { bdev = bdget(part_devt(part));
struct hd_struct *part; disk_put_part(part);
part = disk_get_part(disk, partno); return bdev;
if (part && part->nr_sects)
devt = part_devt(part);
disk_put_part(part);
}
if (likely(devt != MKDEV(0, 0)))
return bdget(devt);
return NULL;
} }
EXPORT_SYMBOL(bdget_disk); EXPORT_SYMBOL(bdget_disk);
...@@ -1000,27 +993,18 @@ dev_t blk_lookup_devt(const char *name, int partno) ...@@ -1000,27 +993,18 @@ dev_t blk_lookup_devt(const char *name, int partno)
class_dev_iter_init(&iter, &block_class, NULL, &disk_type); class_dev_iter_init(&iter, &block_class, NULL, &disk_type);
while ((dev = class_dev_iter_next(&iter))) { while ((dev = class_dev_iter_next(&iter))) {
struct gendisk *disk = dev_to_disk(dev); struct gendisk *disk = dev_to_disk(dev);
struct hd_struct *part;
if (strcmp(dev->bus_id, name)) if (strcmp(dev->bus_id, name))
continue; continue;
if (partno < 0 || partno >= disk_max_parts(disk))
continue;
if (partno == 0)
devt = disk_devt(disk);
else {
struct hd_struct *part;
part = disk_get_part(disk, partno);
if (!part || !part->nr_sects) {
disk_put_part(part);
continue;
}
part = disk_get_part(disk, partno);
if (part && (part->nr_sects || partno == 0)) {
devt = part_devt(part); devt = part_devt(part);
disk_put_part(part); disk_put_part(part);
break;
} }
break; disk_put_part(part);
} }
class_dev_iter_exit(&iter); class_dev_iter_exit(&iter);
return devt; return devt;
......
...@@ -16,9 +16,9 @@ ...@@ -16,9 +16,9 @@
#ifdef CONFIG_BLOCK #ifdef CONFIG_BLOCK
#define kobj_to_dev(k) container_of((k), struct device, kobj) #define kobj_to_dev(k) container_of((k), struct device, kobj)
#define dev_to_disk(device) container_of((device), struct gendisk, __dev) #define dev_to_disk(device) container_of((device), struct gendisk, part0.__dev)
#define dev_to_part(device) container_of((device), struct hd_struct, __dev) #define dev_to_part(device) container_of((device), struct hd_struct, __dev)
#define disk_to_dev(disk) (&((disk)->__dev)) #define disk_to_dev(disk) (&(disk)->part0.__dev)
#define part_to_dev(part) (&((part)->__dev)) #define part_to_dev(part) (&((part)->__dev))
extern struct device_type part_type; extern struct device_type part_type;
...@@ -141,7 +141,6 @@ struct gendisk { ...@@ -141,7 +141,6 @@ struct gendisk {
int flags; int flags;
struct device *driverfs_dev; // FIXME: remove struct device *driverfs_dev; // FIXME: remove
struct device __dev;
struct kobject *holder_dir; struct kobject *holder_dir;
struct kobject *slave_dir; struct kobject *slave_dir;
...@@ -164,8 +163,12 @@ struct gendisk { ...@@ -164,8 +163,12 @@ struct gendisk {
static inline struct gendisk *part_to_disk(struct hd_struct *part) static inline struct gendisk *part_to_disk(struct hd_struct *part)
{ {
if (likely(part)) if (likely(part)) {
return dev_to_disk(part_to_dev(part)->parent); if (part->partno)
return dev_to_disk(part_to_dev(part)->parent);
else
return dev_to_disk(part_to_dev(part));
}
return NULL; return NULL;
} }
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册