diff options
author | Tejun Heo <tj@kernel.org> | 2008-08-29 03:01:47 -0400 |
---|---|---|
committer | Jens Axboe <jens.axboe@oracle.com> | 2008-10-09 02:56:07 -0400 |
commit | 548b10eb2959c96cef6fc29fc96e0931eeb53bc5 (patch) | |
tree | 7166bc04336b80a69f87a9add097919b418f4f43 /include | |
parent | 80795aefb76d10c5d698e60c7e7750b5330787da (diff) |
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: Tejun Heo <tj@kernel.org>
Signed-off-by: Jens Axboe <jens.axboe@oracle.com>
Diffstat (limited to 'include')
-rw-r--r-- | include/linux/genhd.h | 13 |
1 files changed, 8 insertions, 5 deletions
diff --git a/include/linux/genhd.h b/include/linux/genhd.h index 1cf828148ec6..ff293ec8b3f7 100644 --- a/include/linux/genhd.h +++ b/include/linux/genhd.h | |||
@@ -16,9 +16,9 @@ | |||
16 | #ifdef CONFIG_BLOCK | 16 | #ifdef CONFIG_BLOCK |
17 | 17 | ||
18 | #define kobj_to_dev(k) container_of((k), struct device, kobj) | 18 | #define kobj_to_dev(k) container_of((k), struct device, kobj) |
19 | #define dev_to_disk(device) container_of((device), struct gendisk, __dev) | 19 | #define dev_to_disk(device) container_of((device), struct gendisk, part0.__dev) |
20 | #define dev_to_part(device) container_of((device), struct hd_struct, __dev) | 20 | #define dev_to_part(device) container_of((device), struct hd_struct, __dev) |
21 | #define disk_to_dev(disk) (&((disk)->__dev)) | 21 | #define disk_to_dev(disk) (&(disk)->part0.__dev) |
22 | #define part_to_dev(part) (&((part)->__dev)) | 22 | #define part_to_dev(part) (&((part)->__dev)) |
23 | 23 | ||
24 | extern struct device_type part_type; | 24 | extern struct device_type part_type; |
@@ -141,7 +141,6 @@ struct gendisk { | |||
141 | 141 | ||
142 | int flags; | 142 | int flags; |
143 | struct device *driverfs_dev; // FIXME: remove | 143 | struct device *driverfs_dev; // FIXME: remove |
144 | struct device __dev; | ||
145 | struct kobject *holder_dir; | 144 | struct kobject *holder_dir; |
146 | struct kobject *slave_dir; | 145 | struct kobject *slave_dir; |
147 | 146 | ||
@@ -164,8 +163,12 @@ struct gendisk { | |||
164 | 163 | ||
165 | static inline struct gendisk *part_to_disk(struct hd_struct *part) | 164 | static inline struct gendisk *part_to_disk(struct hd_struct *part) |
166 | { | 165 | { |
167 | if (likely(part)) | 166 | if (likely(part)) { |
168 | return dev_to_disk(part_to_dev(part)->parent); | 167 | if (part->partno) |
168 | return dev_to_disk(part_to_dev(part)->parent); | ||
169 | else | ||
170 | return dev_to_disk(part_to_dev(part)); | ||
171 | } | ||
169 | return NULL; | 172 | return NULL; |
170 | } | 173 | } |
171 | 174 | ||