Re: [PATCH v9 6/7] PCI: document SR-IOV sysfs entries

[Greg KH <greg@kroah.com>]

On Wed, Feb 18, 2009 at 01:01:35AM +0800, Yu Zhao wrote:
> On Wed, Feb 18, 2009 at 02:21:16PM +0800, Greg KH wrote:
> > On Tue, Feb 17, 2009 at 10:43:03PM +0800, Yu Zhao wrote:
> > > On Wed, Feb 18, 2009 at 11:49:10AM +0800, Greg KH wrote:
> > > > On Tue, Feb 17, 2009 at 08:50:21PM +0800, Yu Zhao wrote:
> > > > > I'd like to create a subdirectory under PCI device sysfs directory and
> > > > > put some symbol links into that directory (only symbol links). Should I
> > > > > use `device' or `kobject'? Currently I use `device' and get two extra
> > > > > file/directory (uevent and power) which look like useless for my case
> > > > > because this subdirectory doesn't reflect a real device.
> > > > 
> > > > Neither, just use an attribute group, no new struct device should be
> > > > needed at all.
> > > 
> > > Yes, I was thinking using an attribute group, however, it looks like we
> > > can't put symbol link into a group. Maybe I'm wrong, shed some light,
> > > please?
> > 
> > Why would you want a symbolic link in an attribute group?
> 
> Sorry I didn't make myself clear. I'd like to create a subdirectory (its
> parent is controlled by a `struct device') to only hold symbol links.

Ick.  What would those links be to?  And what are they for?

> Can I use an `attribute group' of the parent `struct device' for this,
> or a child `struct device', or a bare `struct kobject'?

You can't use an attribute group for symlinks.
You should use a struct device, but be careful, you are going to be
creating uevents that I don't think you really want to.

Care to describe what you are trying to do first?

thanks,

greg k-h