kset: convert block_subsys to use kset_create

Dynamically create the kset instead of declaring it statically.  We also
rename block_subsys to block_kset to catch all users of this symbol
with a build error instead of an easy-to-ignore build warning.

Cc: Kay Sievers <kay.sievers@vrfy.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This commit is contained in:
Greg Kroah-Hartman 2007-11-06 10:36:58 -08:00
parent c60b717879
commit 830d3cfb16
2 changed files with 19 additions and 21 deletions

View file

@ -17,7 +17,8 @@
#include <linux/buffer_head.h> #include <linux/buffer_head.h>
#include <linux/mutex.h> #include <linux/mutex.h>
struct kset block_subsys; struct kset *block_kset;
static struct kset_uevent_ops block_uevent_ops;
static DEFINE_MUTEX(block_subsys_lock); static DEFINE_MUTEX(block_subsys_lock);
/* /*
@ -221,7 +222,7 @@ void __init printk_all_partitions(void)
mutex_lock(&block_subsys_lock); mutex_lock(&block_subsys_lock);
/* For each block device... */ /* For each block device... */
list_for_each_entry(sgp, &block_subsys.list, kobj.entry) { list_for_each_entry(sgp, &block_kset->list, kobj.entry) {
char buf[BDEVNAME_SIZE]; char buf[BDEVNAME_SIZE];
/* /*
* Don't show empty devices or things that have been surpressed * Don't show empty devices or things that have been surpressed
@ -270,7 +271,7 @@ static void *part_start(struct seq_file *part, loff_t *pos)
loff_t l = *pos; loff_t l = *pos;
mutex_lock(&block_subsys_lock); mutex_lock(&block_subsys_lock);
list_for_each(p, &block_subsys.list) list_for_each(p, &block_kset->list)
if (!l--) if (!l--)
return list_entry(p, struct gendisk, kobj.entry); return list_entry(p, struct gendisk, kobj.entry);
return NULL; return NULL;
@ -280,7 +281,7 @@ static void *part_next(struct seq_file *part, void *v, loff_t *pos)
{ {
struct list_head *p = ((struct gendisk *)v)->kobj.entry.next; struct list_head *p = ((struct gendisk *)v)->kobj.entry.next;
++*pos; ++*pos;
return p==&block_subsys.list ? NULL : return p==&block_kset->list ? NULL :
list_entry(p, struct gendisk, kobj.entry); list_entry(p, struct gendisk, kobj.entry);
} }
@ -295,7 +296,7 @@ static int show_partition(struct seq_file *part, void *v)
int n; int n;
char buf[BDEVNAME_SIZE]; char buf[BDEVNAME_SIZE];
if (&sgp->kobj.entry == block_subsys.list.next) if (&sgp->kobj.entry == block_kset->list.next)
seq_puts(part, "major minor #blocks name\n\n"); seq_puts(part, "major minor #blocks name\n\n");
/* Don't show non-partitionable removeable devices or empty devices */ /* Don't show non-partitionable removeable devices or empty devices */
@ -345,15 +346,14 @@ static struct kobject *base_probe(dev_t dev, int *part, void *data)
static int __init genhd_device_init(void) static int __init genhd_device_init(void)
{ {
int err;
bdev_map = kobj_map_init(base_probe, &block_subsys_lock); bdev_map = kobj_map_init(base_probe, &block_subsys_lock);
blk_dev_init(); blk_dev_init();
err = subsystem_register(&block_subsys); block_kset = kset_create_and_add("block", &block_uevent_ops, NULL);
if (err < 0) if (!block_kset) {
printk(KERN_WARNING "%s: subsystem_register error: %d\n", printk(KERN_WARNING "%s: kset_create error\n", __FUNCTION__);
__FUNCTION__, err); return -ENOMEM;
return err; }
return 0;
} }
subsys_initcall(genhd_device_init); subsys_initcall(genhd_device_init);
@ -584,8 +584,6 @@ static struct kset_uevent_ops block_uevent_ops = {
.uevent = block_uevent, .uevent = block_uevent,
}; };
decl_subsys(block, &block_uevent_ops);
/* /*
* aggregate disk stat collector. Uses the same stats that the sysfs * aggregate disk stat collector. Uses the same stats that the sysfs
* entries do, above, but makes them available through one seq_file. * entries do, above, but makes them available through one seq_file.
@ -603,7 +601,7 @@ static void *diskstats_start(struct seq_file *part, loff_t *pos)
struct list_head *p; struct list_head *p;
mutex_lock(&block_subsys_lock); mutex_lock(&block_subsys_lock);
list_for_each(p, &block_subsys.list) list_for_each(p, &block_kset->list)
if (!k--) if (!k--)
return list_entry(p, struct gendisk, kobj.entry); return list_entry(p, struct gendisk, kobj.entry);
return NULL; return NULL;
@ -613,7 +611,7 @@ static void *diskstats_next(struct seq_file *part, void *v, loff_t *pos)
{ {
struct list_head *p = ((struct gendisk *)v)->kobj.entry.next; struct list_head *p = ((struct gendisk *)v)->kobj.entry.next;
++*pos; ++*pos;
return p==&block_subsys.list ? NULL : return p==&block_kset->list ? NULL :
list_entry(p, struct gendisk, kobj.entry); list_entry(p, struct gendisk, kobj.entry);
} }
@ -629,7 +627,7 @@ static int diskstats_show(struct seq_file *s, void *v)
int n = 0; int n = 0;
/* /*
if (&sgp->kobj.entry == block_subsys.kset.list.next) if (&sgp->kobj.entry == block_kset->list.next)
seq_puts(s, "major minor name" seq_puts(s, "major minor name"
" rio rmerge rsect ruse wio wmerge " " rio rmerge rsect ruse wio wmerge "
"wsect wuse running use aveq" "wsect wuse running use aveq"
@ -721,7 +719,7 @@ struct gendisk *alloc_disk_node(int minors, int node_id)
} }
} }
disk->minors = minors; disk->minors = minors;
disk->kobj.kset = &block_subsys; disk->kobj.kset = block_kset;
disk->kobj.ktype = &ktype_block; disk->kobj.ktype = &ktype_block;
kobject_init(&disk->kobj); kobject_init(&disk->kobj);
rand_initialize_disk(disk); rand_initialize_disk(disk);

View file

@ -316,7 +316,7 @@ static struct attribute * default_attrs[] = {
NULL, NULL,
}; };
extern struct kset block_subsys; extern struct kset *block_kset;
static void part_release(struct kobject *kobj) static void part_release(struct kobject *kobj)
{ {
@ -393,7 +393,7 @@ void add_partition(struct gendisk *disk, int part, sector_t start, sector_t len,
kobject_add(&p->kobj); kobject_add(&p->kobj);
if (!disk->part_uevent_suppress) if (!disk->part_uevent_suppress)
kobject_uevent(&p->kobj, KOBJ_ADD); kobject_uevent(&p->kobj, KOBJ_ADD);
sysfs_create_link(&p->kobj, &block_subsys.kobj, "subsystem"); sysfs_create_link(&p->kobj, &block_kset->kobj, "subsystem");
if (flags & ADDPART_FLAG_WHOLEDISK) { if (flags & ADDPART_FLAG_WHOLEDISK) {
static struct attribute addpartattr = { static struct attribute addpartattr = {
.name = "whole_disk", .name = "whole_disk",
@ -448,7 +448,7 @@ static int disk_sysfs_symlinks(struct gendisk *disk)
goto err_out_dev_link; goto err_out_dev_link;
} }
err = sysfs_create_link(&disk->kobj, &block_subsys.kobj, err = sysfs_create_link(&disk->kobj, &block_kset->kobj,
"subsystem"); "subsystem");
if (err) if (err)
goto err_out_disk_name_lnk; goto err_out_disk_name_lnk;