Messages in this thread Patch in this message | ![/](/images/icornerl.gif) | | Subject | [PATCH] RFC: Attributes in /sys/cdev | From | Jonathan Corbet <> | Date | Fri, 19 Sep 2003 17:10:46 -0600 |
| |
I assume that the (empty, currently) entries in /sys/cdev are there for some purpose... for purposes of my own, I've been thinking that it would be nice to be able to find which device numbers got assigned when using the new char dev registration functions. Thus, the following patch, which adds default "dev" and "count" attributes to /sys/cdev entries.
I have no idea whether this follows the original plan for /sys/cdev. There *is* a plan for it, no...?
jon
Jonathan Corbet Executive editor, LWN.net corbet@lwn.net
diff -urN -X dontdiff test5-vanilla/fs/char_dev.c test5/fs/char_dev.c --- test5-vanilla/fs/char_dev.c Mon Sep 8 13:50:01 2003 +++ test5/fs/char_dev.c Sat Sep 20 06:46:24 2003 @@ -342,7 +342,13 @@ int cdev_add(struct cdev *p, dev_t dev, unsigned count) { - int err = kobject_add(&p->kobj); + int err; + + /* Remember for sysfs; maybe caller should set these? */ + p->firstdev = dev; + p->count = count; + + err = kobject_add(&p->kobj); if (err) return err; err = kobj_map(cdev_map, dev, count, NULL, exact_match, exact_lock, p); @@ -383,6 +389,61 @@ } } +/* + * Simple /sys/cdev attribute stuff, added by corbet@lwn.net. This is + * heavily patterned after the /sys/block code. + */ +struct cdev_attribute { + struct attribute attr; + ssize_t (*show)(struct cdev *, char *); +}; + +static ssize_t cdev_dev_read(struct cdev *cd, char *page) +{ + return sprintf(page, "%d:%d\n", MAJOR(cd->firstdev), + MINOR(cd->firstdev)); +} + +static ssize_t cdev_count_read(struct cdev *cd, char *page) +{ + return sprintf(page, "%d\n", cd->count); +} + +static struct cdev_attribute cdev_attr_dev = { + .attr = { .name = "dev", .mode = S_IRUGO }, + .show = cdev_dev_read +}; + +static struct cdev_attribute cdev_attr_count = { + .attr = { .name = "count", .mode = S_IRUGO }, + .show = cdev_count_read +}; + + +static struct attribute *default_cdev_attrs[] = { + &cdev_attr_dev.attr, + &cdev_attr_count.attr, + NULL +}; + +/* + * General show function. + */ +static ssize_t cdev_attr_show(struct kobject *kobj, struct attribute *attr, + char *page) +{ + struct cdev *cd = container_of(kobj, struct cdev, kobj); + struct cdev_attribute *cattr = + container_of(attr, struct cdev_attribute, attr); + if (cattr->show) + return cattr->show(cd, page); + return 0; +} + +static struct sysfs_ops cdev_sysfs_ops = { + .show = cdev_attr_show +}; + static decl_subsys(cdev, NULL, NULL); static void cdev_default_release(struct kobject *kobj) @@ -400,10 +461,14 @@ static struct kobj_type ktype_cdev_default = { .release = cdev_default_release, + .sysfs_ops = &cdev_sysfs_ops, + .default_attrs = default_cdev_attrs, }; static struct kobj_type ktype_cdev_dynamic = { .release = cdev_dynamic_release, + .sysfs_ops = &cdev_sysfs_ops, + .default_attrs = default_cdev_attrs, }; static struct kset kset_dynamic = { diff -urN -X dontdiff test5-vanilla/include/linux/cdev.h test5/include/linux/cdev.h --- test5-vanilla/include/linux/cdev.h Mon Sep 8 13:50:09 2003 +++ test5/include/linux/cdev.h Sat Sep 20 06:12:52 2003 @@ -3,6 +3,8 @@ #ifdef __KERNEL__ struct cdev { + dev_t firstdev; + int count; struct kobject kobj; struct module *owner; struct file_operations *ops; - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| ![\](/images/icornerr.gif) |