Subject: Re: [PATCH] Add ability to register class interfaces for network class

From: "Teras Timo (EXT-YomiGroup/Helsinki)" <[email-masked]>
Date: Wed, 27 Oct 2004 21:38:04 +0300

On Wed, Oct 27, 2004 at 08:59:06AM -0700, ext Stephen Hemminger wrote:
> The Ethernet bridge code creates attribute groups in several places
> without additional interfaces.  How hard is it to do:
> 
> int br_sysfs_addbr(struct net_device *dev)
> {
> 	struct kobject *brobj = &dev->class_dev.kobj;
> 	struct net_bridge *br = netdev_priv(dev);
> 	int err;
> 
> 	err = sysfs_create_group(brobj, &bridge_group);
> 	if (err) {
> 		pr_info("%s: can't create group %s/%s\n",
> 			__FUNCTION__, dev->name, bridge_group.name);
> 		goto out1;
> 	}

This way the problem is that I have to know which devices I will
add the attributes. But the point is to add attributes to
all netdevs.

Using this approach I'd have to enumerate all the interfaces every
now and then.

If I have my class interface I get a callback whenever an
interface is created or deleted and I can automatically add the
attribute to all netdevs.

Cheers,
  Timo