Commit f4a00a2c authored by Greg Kroah-Hartman's avatar Greg Kroah-Hartman

Block: Fix whole_disk attribute bug

The "whole_disk" attribute was not properly converted in the block
device conversion earlier, and if the file is read, bad things can
happen.  This patch fixes this, making the attribute an empty one,
preserving the original functionality.

Many thanks to David Miller for finding this, and pointing me in the
proper place within the block code to look.
Acked-by: default avatarDavid S. Miller <davem@davemloft.net>
Cc: Kay Sievers <kay.sievers@vrfy.org>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@suse.de>
parent 0afc2edf
...@@ -319,6 +319,14 @@ void delete_partition(struct gendisk *disk, int part) ...@@ -319,6 +319,14 @@ void delete_partition(struct gendisk *disk, int part)
put_device(&p->dev); put_device(&p->dev);
} }
static ssize_t whole_disk_show(struct device *dev,
struct device_attribute *attr, char *buf)
{
return 0;
}
static DEVICE_ATTR(whole_disk, S_IRUSR | S_IRGRP | S_IROTH,
whole_disk_show, NULL);
void add_partition(struct gendisk *disk, int part, sector_t start, sector_t len, int flags) void add_partition(struct gendisk *disk, int part, sector_t start, sector_t len, int flags)
{ {
struct hd_struct *p; struct hd_struct *p;
...@@ -352,13 +360,8 @@ void add_partition(struct gendisk *disk, int part, sector_t start, sector_t len, ...@@ -352,13 +360,8 @@ void add_partition(struct gendisk *disk, int part, sector_t start, sector_t len,
device_add(&p->dev); device_add(&p->dev);
partition_sysfs_add_subdir(p); partition_sysfs_add_subdir(p);
p->dev.uevent_suppress = 0; p->dev.uevent_suppress = 0;
if (flags & ADDPART_FLAG_WHOLEDISK) { if (flags & ADDPART_FLAG_WHOLEDISK)
static struct attribute addpartattr = { err = device_create_file(&p->dev, &dev_attr_whole_disk);
.name = "whole_disk",
.mode = S_IRUSR | S_IRGRP | S_IROTH,
};
err = sysfs_create_file(&p->dev.kobj, &addpartattr);
}
/* suppress uevent if the disk supresses it */ /* suppress uevent if the disk supresses it */
if (!disk->dev.uevent_suppress) if (!disk->dev.uevent_suppress)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment