block: move holder_dir from disk to part0
[safe/jmp/linux-2.6] / fs / partitions / check.c
index 722e12e..f0f6049 100644 (file)
@@ -18,6 +18,7 @@
 #include <linux/fs.h>
 #include <linux/kmod.h>
 #include <linux/ctype.h>
+#include <linux/genhd.h>
 
 #include "check.h"
 
@@ -119,22 +120,26 @@ static int (*check_part[])(struct parsed_partitions *, struct block_device *) =
  * a pointer to that same buffer (for convenience).
  */
 
-char *disk_name(struct gendisk *hd, int part, char *buf)
+char *disk_name(struct gendisk *hd, int partno, char *buf)
 {
-       if (!part)
+       if (!partno)
                snprintf(buf, BDEVNAME_SIZE, "%s", hd->disk_name);
        else if (isdigit(hd->disk_name[strlen(hd->disk_name)-1]))
-               snprintf(buf, BDEVNAME_SIZE, "%sp%d", hd->disk_name, part);
+               snprintf(buf, BDEVNAME_SIZE, "%sp%d", hd->disk_name, partno);
        else
-               snprintf(buf, BDEVNAME_SIZE, "%s%d", hd->disk_name, part);
+               snprintf(buf, BDEVNAME_SIZE, "%s%d", hd->disk_name, partno);
 
        return buf;
 }
 
 const char *bdevname(struct block_device *bdev, char *buf)
 {
-       int part = MINOR(bdev->bd_dev) - bdev->bd_disk->first_minor;
-       return disk_name(bdev->bd_disk, part, buf);
+       int partno = 0;
+
+       if (bdev->bd_part)
+               partno = bdev->bd_part->partno;
+
+       return disk_name(bdev->bd_disk, partno, buf);
 }
 
 EXPORT_SYMBOL(bdevname);
@@ -168,7 +173,7 @@ check_partition(struct gendisk *hd, struct block_device *bdev)
        if (isdigit(state->name[strlen(state->name)-1]))
                sprintf(state->name, "p");
 
-       state->limit = hd->minors;
+       state->limit = disk_max_parts(hd);
        i = res = err = 0;
        while (!res && check_part[i]) {
                memset(&state->parts, 0, sizeof(state->parts));
@@ -195,96 +200,62 @@ check_partition(struct gendisk *hd, struct block_device *bdev)
        return ERR_PTR(res);
 }
 
-/*
- * sysfs bindings for partitions
- */
-
-struct part_attribute {
-       struct attribute attr;
-       ssize_t (*show)(struct hd_struct *,char *);
-       ssize_t (*store)(struct hd_struct *,const char *, size_t);
-};
-
-static ssize_t 
-part_attr_show(struct kobject * kobj, struct attribute * attr, char * page)
-{
-       struct hd_struct * p = container_of(kobj,struct hd_struct,kobj);
-       struct part_attribute * part_attr = container_of(attr,struct part_attribute,attr);
-       ssize_t ret = 0;
-       if (part_attr->show)
-               ret = part_attr->show(p, page);
-       return ret;
-}
-static ssize_t
-part_attr_store(struct kobject * kobj, struct attribute * attr,
-               const char *page, size_t count)
+static ssize_t part_start_show(struct device *dev,
+                              struct device_attribute *attr, char *buf)
 {
-       struct hd_struct * p = container_of(kobj,struct hd_struct,kobj);
-       struct part_attribute * part_attr = container_of(attr,struct part_attribute,attr);
-       ssize_t ret = 0;
+       struct hd_struct *p = dev_to_part(dev);
 
-       if (part_attr->store)
-               ret = part_attr->store(p, page, count);
-       return ret;
+       return sprintf(buf, "%llu\n",(unsigned long long)p->start_sect);
 }
 
-static struct sysfs_ops part_sysfs_ops = {
-       .show   =       part_attr_show,
-       .store  =       part_attr_store,
-};
-
-static ssize_t part_uevent_store(struct hd_struct * p,
-                                const char *page, size_t count)
+ssize_t part_size_show(struct device *dev,
+                      struct device_attribute *attr, char *buf)
 {
-       kobject_uevent(&p->kobj, KOBJ_ADD);
-       return count;
-}
-static ssize_t part_dev_read(struct hd_struct * p, char *page)
-{
-       struct gendisk *disk = container_of(p->kobj.parent,struct gendisk,kobj);
-       dev_t dev = MKDEV(disk->major, disk->first_minor + p->partno); 
-       return print_dev_t(page, dev);
-}
-static ssize_t part_start_read(struct hd_struct * p, char *page)
-{
-       return sprintf(page, "%llu\n",(unsigned long long)p->start_sect);
-}
-static ssize_t part_size_read(struct hd_struct * p, char *page)
-{
-       return sprintf(page, "%llu\n",(unsigned long long)p->nr_sects);
+       struct hd_struct *p = dev_to_part(dev);
+       return sprintf(buf, "%llu\n",(unsigned long long)p->nr_sects);
 }
-static ssize_t part_stat_read(struct hd_struct * p, char *page)
+
+static ssize_t part_stat_show(struct device *dev,
+                             struct device_attribute *attr, char *buf)
 {
-       return sprintf(page, "%8u %8llu %8u %8llu\n",
-                      p->ios[0], (unsigned long long)p->sectors[0],
-                      p->ios[1], (unsigned long long)p->sectors[1]);
+       struct hd_struct *p = dev_to_part(dev);
+       int cpu;
+
+       cpu = disk_stat_lock();
+       part_round_stats(cpu, p);
+       disk_stat_unlock();
+       return sprintf(buf,
+               "%8lu %8lu %8llu %8u "
+               "%8lu %8lu %8llu %8u "
+               "%8u %8u %8u"
+               "\n",
+               part_stat_read(p, ios[READ]),
+               part_stat_read(p, merges[READ]),
+               (unsigned long long)part_stat_read(p, sectors[READ]),
+               jiffies_to_msecs(part_stat_read(p, ticks[READ])),
+               part_stat_read(p, ios[WRITE]),
+               part_stat_read(p, merges[WRITE]),
+               (unsigned long long)part_stat_read(p, sectors[WRITE]),
+               jiffies_to_msecs(part_stat_read(p, ticks[WRITE])),
+               p->in_flight,
+               jiffies_to_msecs(part_stat_read(p, io_ticks)),
+               jiffies_to_msecs(part_stat_read(p, time_in_queue)));
 }
-static struct part_attribute part_attr_uevent = {
-       .attr = {.name = "uevent", .mode = S_IWUSR },
-       .store  = part_uevent_store
-};
-static struct part_attribute part_attr_dev = {
-       .attr = {.name = "dev", .mode = S_IRUGO },
-       .show   = part_dev_read
-};
-static struct part_attribute part_attr_start = {
-       .attr = {.name = "start", .mode = S_IRUGO },
-       .show   = part_start_read
-};
-static struct part_attribute part_attr_size = {
-       .attr = {.name = "size", .mode = S_IRUGO },
-       .show   = part_size_read
-};
-static struct part_attribute part_attr_stat = {
-       .attr = {.name = "stat", .mode = S_IRUGO },
-       .show   = part_stat_read
-};
 
 #ifdef CONFIG_FAIL_MAKE_REQUEST
+static ssize_t part_fail_show(struct device *dev,
+                             struct device_attribute *attr, char *buf)
+{
+       struct hd_struct *p = dev_to_part(dev);
+
+       return sprintf(buf, "%d\n", p->make_it_fail);
+}
 
-static ssize_t part_fail_store(struct hd_struct * p,
+static ssize_t part_fail_store(struct device *dev,
+                              struct device_attribute *attr,
                               const char *buf, size_t count)
 {
+       struct hd_struct *p = dev_to_part(dev);
        int i;
 
        if (count > 0 && sscanf(buf, "%d", &i) > 0)
@@ -292,209 +263,202 @@ static ssize_t part_fail_store(struct hd_struct * p,
 
        return count;
 }
-static ssize_t part_fail_read(struct hd_struct * p, char *page)
-{
-       return sprintf(page, "%d\n", p->make_it_fail);
-}
-static struct part_attribute part_attr_fail = {
-       .attr = {.name = "make-it-fail", .mode = S_IRUGO | S_IWUSR },
-       .store  = part_fail_store,
-       .show   = part_fail_read
-};
+#endif
 
+static DEVICE_ATTR(start, S_IRUGO, part_start_show, NULL);
+static DEVICE_ATTR(size, S_IRUGO, part_size_show, NULL);
+static DEVICE_ATTR(stat, S_IRUGO, part_stat_show, NULL);
+#ifdef CONFIG_FAIL_MAKE_REQUEST
+static struct device_attribute dev_attr_fail =
+       __ATTR(make-it-fail, S_IRUGO|S_IWUSR, part_fail_show, part_fail_store);
 #endif
 
-static struct attribute * default_attrs[] = {
-       &part_attr_uevent.attr,
-       &part_attr_dev.attr,
-       &part_attr_start.attr,
-       &part_attr_size.attr,
-       &part_attr_stat.attr,
+static struct attribute *part_attrs[] = {
+       &dev_attr_start.attr,
+       &dev_attr_size.attr,
+       &dev_attr_stat.attr,
 #ifdef CONFIG_FAIL_MAKE_REQUEST
-       &part_attr_fail.attr,
+       &dev_attr_fail.attr,
 #endif
-       NULL,
+       NULL
 };
 
-extern struct kset block_subsys;
+static struct attribute_group part_attr_group = {
+       .attrs = part_attrs,
+};
 
-static void part_release(struct kobject *kobj)
+static struct attribute_group *part_attr_groups[] = {
+       &part_attr_group,
+       NULL
+};
+
+static void part_release(struct device *dev)
 {
-       struct hd_struct * p = container_of(kobj,struct hd_struct,kobj);
+       struct hd_struct *p = dev_to_part(dev);
+       free_part_stats(p);
        kfree(p);
 }
 
-struct kobj_type ktype_part = {
+struct device_type part_type = {
+       .name           = "partition",
+       .groups         = part_attr_groups,
        .release        = part_release,
-       .default_attrs  = default_attrs,
-       .sysfs_ops      = &part_sysfs_ops,
 };
 
-static inline void partition_sysfs_add_subdir(struct hd_struct *p)
+static void delete_partition_rcu_cb(struct rcu_head *head)
 {
-       struct kobject *k;
+       struct hd_struct *part = container_of(head, struct hd_struct, rcu_head);
 
-       k = kobject_get(&p->kobj);
-       p->holder_dir = kobject_add_dir(k, "holders");
-       kobject_put(k);
+       part->start_sect = 0;
+       part->nr_sects = 0;
+       part_stat_set_all(part, 0);
+       put_device(part_to_dev(part));
 }
 
-static inline void disk_sysfs_add_subdirs(struct gendisk *disk)
+void delete_partition(struct gendisk *disk, int partno)
 {
-       struct kobject *k;
+       struct hd_struct *part;
+
+       part = disk->__part[partno];
+       if (!part)
+               return;
+
+       blk_free_devt(part_devt(part));
+       rcu_assign_pointer(disk->__part[partno], NULL);
+       kobject_put(part->holder_dir);
+       device_del(part_to_dev(part));
 
-       k = kobject_get(&disk->kobj);
-       disk->holder_dir = kobject_add_dir(k, "holders");
-       disk->slave_dir = kobject_add_dir(k, "slaves");
-       kobject_put(k);
+       call_rcu(&part->rcu_head, delete_partition_rcu_cb);
 }
 
-void delete_partition(struct gendisk *disk, int part)
+static ssize_t whole_disk_show(struct device *dev,
+                              struct device_attribute *attr, char *buf)
 {
-       struct hd_struct *p = disk->part[part-1];
-       if (!p)
-               return;
-       if (!p->nr_sects)
-               return;
-       disk->part[part-1] = NULL;
-       p->start_sect = 0;
-       p->nr_sects = 0;
-       p->ios[0] = p->ios[1] = 0;
-       p->sectors[0] = p->sectors[1] = 0;
-       sysfs_remove_link(&p->kobj, "subsystem");
-       kobject_unregister(p->holder_dir);
-       kobject_uevent(&p->kobj, KOBJ_REMOVE);
-       kobject_del(&p->kobj);
-       kobject_put(&p->kobj);
+       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)
+int add_partition(struct gendisk *disk, int partno,
+                 sector_t start, sector_t len, int flags)
 {
        struct hd_struct *p;
+       dev_t devt = MKDEV(0, 0);
+       struct device *ddev = disk_to_dev(disk);
+       struct device *pdev;
+       const char *dname;
+       int err;
+
+       if (disk->__part[partno])
+               return -EBUSY;
 
        p = kzalloc(sizeof(*p), GFP_KERNEL);
        if (!p)
-               return;
-       
+               return -ENOMEM;
+
+       if (!init_part_stats(p)) {
+               err = -ENOMEM;
+               goto out_free;
+       }
+       pdev = part_to_dev(p);
+
        p->start_sect = start;
        p->nr_sects = len;
-       p->partno = part;
-       p->policy = disk->policy;
+       p->partno = partno;
+       p->policy = get_disk_ro(disk);
 
-       if (isdigit(disk->kobj.k_name[strlen(disk->kobj.k_name)-1]))
-               kobject_set_name(&p->kobj, "%sp%d",
-                                kobject_name(&disk->kobj), part);
+       dname = dev_name(ddev);
+       if (isdigit(dname[strlen(dname) - 1]))
+               snprintf(pdev->bus_id, BUS_ID_SIZE, "%sp%d", dname, partno);
        else
-               kobject_set_name(&p->kobj, "%s%d",
-                                kobject_name(&disk->kobj),part);
-       p->kobj.parent = &disk->kobj;
-       p->kobj.ktype = &ktype_part;
-       kobject_init(&p->kobj);
-       kobject_add(&p->kobj);
-       if (!disk->part_uevent_suppress)
-               kobject_uevent(&p->kobj, KOBJ_ADD);
-       sysfs_create_link(&p->kobj, &block_subsys.kobj, "subsystem");
-       if (flags & ADDPART_FLAG_WHOLEDISK) {
-               static struct attribute addpartattr = {
-                       .name = "whole_disk",
-                       .mode = S_IRUSR | S_IRGRP | S_IROTH,
-               };
-
-               sysfs_create_file(&p->kobj, &addpartattr);
-       }
-       partition_sysfs_add_subdir(p);
-       disk->part[part-1] = p;
-}
+               snprintf(pdev->bus_id, BUS_ID_SIZE, "%s%d", dname, partno);
 
-static char *make_block_name(struct gendisk *disk)
-{
-       char *name;
-       static char *block_str = "block:";
-       int size;
-       char *s;
+       device_initialize(pdev);
+       pdev->class = &block_class;
+       pdev->type = &part_type;
+       pdev->parent = ddev;
 
-       size = strlen(block_str) + strlen(disk->disk_name) + 1;
-       name = kmalloc(size, GFP_KERNEL);
-       if (!name)
-               return NULL;
-       strcpy(name, block_str);
-       strcat(name, disk->disk_name);
-       /* ewww... some of these buggers have / in name... */
-       s = strchr(name, '/');
-       if (s)
-               *s = '!';
-       return name;
-}
-
-static int disk_sysfs_symlinks(struct gendisk *disk)
-{
-       struct device *target = get_device(disk->driverfs_dev);
-       int err;
-       char *disk_name = NULL;
+       err = blk_alloc_devt(p, &devt);
+       if (err)
+               goto out_free;
+       pdev->devt = devt;
 
-       if (target) {
-               disk_name = make_block_name(disk);
-               if (!disk_name) {
-                       err = -ENOMEM;
-                       goto err_out;
-               }
+       /* delay uevent until 'holders' subdir is created */
+       pdev->uevent_suppress = 1;
+       err = device_add(pdev);
+       if (err)
+               goto out_put;
 
-               err = sysfs_create_link(&disk->kobj, &target->kobj, "device");
-               if (err)
-                       goto err_out_disk_name;
+       err = -ENOMEM;
+       p->holder_dir = kobject_create_and_add("holders", &pdev->kobj);
+       if (!p->holder_dir)
+               goto out_del;
 
-               err = sysfs_create_link(&target->kobj, &disk->kobj, disk_name);
+       pdev->uevent_suppress = 0;
+       if (flags & ADDPART_FLAG_WHOLEDISK) {
+               err = device_create_file(pdev, &dev_attr_whole_disk);
                if (err)
-                       goto err_out_dev_link;
+                       goto out_del;
        }
 
-       err = sysfs_create_link(&disk->kobj, &block_subsys.kobj,
-                               "subsystem");
-       if (err)
-               goto err_out_disk_name_lnk;
+       /* everything is up and running, commence */
+       INIT_RCU_HEAD(&p->rcu_head);
+       rcu_assign_pointer(disk->__part[partno], p);
 
-       kfree(disk_name);
+       /* suppress uevent if the disk supresses it */
+       if (!ddev->uevent_suppress)
+               kobject_uevent(&pdev->kobj, KOBJ_ADD);
 
        return 0;
 
-err_out_disk_name_lnk:
-       if (target) {
-               sysfs_remove_link(&target->kobj, disk_name);
-err_out_dev_link:
-               sysfs_remove_link(&disk->kobj, "device");
-err_out_disk_name:
-               kfree(disk_name);
-err_out:
-               put_device(target);
-       }
+out_free:
+       kfree(p);
+       return err;
+out_del:
+       kobject_put(p->holder_dir);
+       device_del(pdev);
+out_put:
+       put_device(pdev);
+       blk_free_devt(devt);
        return err;
 }
 
 /* Not exported, helper to add_disk(). */
 void register_disk(struct gendisk *disk)
 {
+       struct device *ddev = disk_to_dev(disk);
        struct block_device *bdev;
+       struct disk_part_iter piter;
+       struct hd_struct *part;
        char *s;
-       int i;
-       struct hd_struct *p;
        int err;
 
-       kobject_set_name(&disk->kobj, "%s", disk->disk_name);
-       /* ewww... some of these buggers have / in name... */
-       s = strchr(disk->kobj.k_name, '/');
+       ddev->parent = disk->driverfs_dev;
+
+       strlcpy(ddev->bus_id, disk->disk_name, BUS_ID_SIZE);
+       /* ewww... some of these buggers have / in the name... */
+       s = strchr(ddev->bus_id, '/');
        if (s)
                *s = '!';
-       if ((err = kobject_add(&disk->kobj)))
+
+       /* delay uevents, until we scanned partition table */
+       ddev->uevent_suppress = 1;
+
+       if (device_add(ddev))
                return;
-       err = disk_sysfs_symlinks(disk);
+#ifndef CONFIG_SYSFS_DEPRECATED
+       err = sysfs_create_link(block_depr, &ddev->kobj,
+                               kobject_name(&ddev->kobj));
        if (err) {
-               kobject_del(&disk->kobj);
+               device_del(ddev);
                return;
        }
-       disk_sysfs_add_subdirs(disk);
+#endif
+       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->minors == 1)
+       if (!disk_partitionable(disk))
                goto exit;
 
        /* No such device (e.g., media were just removed) */
@@ -505,30 +469,28 @@ void register_disk(struct gendisk *disk)
        if (!bdev)
                goto exit;
 
-       /* scan partition table, but suppress uevents */
        bdev->bd_invalidated = 1;
-       disk->part_uevent_suppress = 1;
        err = blkdev_get(bdev, FMODE_READ, 0);
-       disk->part_uevent_suppress = 0;
        if (err < 0)
                goto exit;
        blkdev_put(bdev);
 
 exit:
-       /* announce disk after possible partitions are already created */
-       kobject_uevent(&disk->kobj, KOBJ_ADD);
+       /* announce disk after possible partitions are created */
+       ddev->uevent_suppress = 0;
+       kobject_uevent(&ddev->kobj, KOBJ_ADD);
 
        /* announce possible partitions */
-       for (i = 1; i < disk->minors; i++) {
-               p = disk->part[i-1];
-               if (!p || !p->nr_sects)
-                       continue;
-               kobject_uevent(&p->kobj, KOBJ_ADD);
-       }
+       disk_part_iter_init(&piter, disk, 0);
+       while ((part = disk_part_iter_next(&piter)))
+               kobject_uevent(&part_to_dev(part)->kobj, KOBJ_ADD);
+       disk_part_iter_exit(&piter);
 }
 
 int rescan_partitions(struct gendisk *disk, struct block_device *bdev)
 {
+       struct disk_part_iter piter;
+       struct hd_struct *part;
        struct parsed_partitions *state;
        int p, res;
 
@@ -538,24 +500,38 @@ int rescan_partitions(struct gendisk *disk, struct block_device *bdev)
        if (res)
                return res;
        bdev->bd_invalidated = 0;
-       for (p = 1; p < disk->minors; p++)
-               delete_partition(disk, p);
+
+       disk_part_iter_init(&piter, disk, DISK_PITER_INCL_EMPTY);
+       while ((part = disk_part_iter_next(&piter)))
+               delete_partition(disk, part->partno);
+       disk_part_iter_exit(&piter);
+
        if (disk->fops->revalidate_disk)
                disk->fops->revalidate_disk(disk);
        if (!get_capacity(disk) || !(state = check_partition(disk, bdev)))
                return 0;
        if (IS_ERR(state))      /* I/O error reading the partition table */
                return -EIO;
+
+       /* tell userspace that the media / partition table may have changed */
+       kobject_uevent(&disk_to_dev(disk)->kobj, KOBJ_CHANGE);
+
        for (p = 1; p < state->limit; p++) {
                sector_t size = state->parts[p].size;
                sector_t from = state->parts[p].from;
                if (!size)
                        continue;
                if (from + size > get_capacity(disk)) {
-                       printk(" %s: p%d exceeds device capacity\n",
+                       printk(KERN_WARNING
+                               "%s: p%d exceeds device capacity\n",
                                disk->disk_name, p);
                }
-               add_partition(disk, p, from, size, state->parts[p].flags);
+               res = add_partition(disk, p, from, size, state->parts[p].flags);
+               if (res) {
+                       printk(KERN_ERR " %s: p%d could not be added: %d\n",
+                               disk->disk_name, p, -res);
+                       continue;
+               }
 #ifdef CONFIG_BLK_DEV_MD
                if (state->parts[p].flags & ADDPART_FLAG_RAID)
                        md_autodetect_dev(bdev->bd_dev+p);
@@ -588,33 +564,30 @@ EXPORT_SYMBOL(read_dev_sector);
 
 void del_gendisk(struct gendisk *disk)
 {
-       int p;
+       struct disk_part_iter piter;
+       struct hd_struct *part;
 
        /* invalidate stuff */
-       for (p = disk->minors - 1; p > 0; p--) {
-               invalidate_partition(disk, p);
-               delete_partition(disk, p);
+       disk_part_iter_init(&piter, disk,
+                            DISK_PITER_INCL_EMPTY | DISK_PITER_REVERSE);
+       while ((part = disk_part_iter_next(&piter))) {
+               invalidate_partition(disk, part->partno);
+               delete_partition(disk, part->partno);
        }
+       disk_part_iter_exit(&piter);
+
        invalidate_partition(disk, 0);
-       disk->capacity = 0;
+       set_capacity(disk, 0);
        disk->flags &= ~GENHD_FL_UP;
        unlink_gendisk(disk);
        disk_stat_set_all(disk, 0);
        disk->stamp = 0;
 
-       kobject_uevent(&disk->kobj, KOBJ_REMOVE);
-       kobject_unregister(disk->holder_dir);
-       kobject_unregister(disk->slave_dir);
-       if (disk->driverfs_dev) {
-               char *disk_name = make_block_name(disk);
-               sysfs_remove_link(&disk->kobj, "device");
-               if (disk_name) {
-                       sysfs_remove_link(&disk->driverfs_dev->kobj, disk_name);
-                       kfree(disk_name);
-               }
-               put_device(disk->driverfs_dev);
-               disk->driverfs_dev = NULL;
-       }
-       sysfs_remove_link(&disk->kobj, "subsystem");
-       kobject_del(&disk->kobj);
+       kobject_put(disk->part0.holder_dir);
+       kobject_put(disk->slave_dir);
+       disk->driverfs_dev = NULL;
+#ifndef CONFIG_SYSFS_DEPRECATED
+       sysfs_remove_link(block_depr, dev_name(disk_to_dev(disk)));
+#endif
+       device_del(disk_to_dev(disk));
 }