On Tue, Oct 18, 2005 at 04:53:36PM -0400, Jeff Garzik wrote:
> Roland Dreier wrote:
> >I just noticed that at some point, struct pci_driver grew a .owner
> >member. However, only a handful of drivers set it:
> >
> > $ grep -r -A10 pci_driver drivers/ | grep owner
> > drivers/block/sx8.c- .owner = THIS_MODULE,
> > drivers/ieee1394/pcilynx.c- .owner = THIS_MODULE,
> > drivers/net/spider_net.c- .owner = THIS_MODULE,
> > drivers/video/imsttfb.c- .owner = THIS_MODULE,
> > drivers/video/kyro/fbdev.c- .owner = THIS_MODULE,
> > drivers/video/tridentfb.c- .owner = THIS_MODULE,
> >
> >Should all drivers be setting .owner = THIS_MODULE? Is this a good
> >kernel janitors task?
>
> In theory its for module refcounting. With so many PCI drivers and so
> few pci_driver::owner users, it makes me wonder how needed it is.
It might in the future be needed for refcounting, I originally added it
when I thought it was needed.
But what it really does today is create the symlink from the driver to
the module that is contained in it, in sysfs. Which is very invaluable
for people who want to know these things (installer programs, etc.)
For example:
$ tree /sys/bus/pci/drivers/uhci_hcd/
/sys/bus/pci/drivers/uhci_hcd/
|-- 0000:00:1d.0 -> ../../../../devices/pci0000:00/0000:00:1d.0
|-- 0000:00:1d.1 -> ../../../../devices/pci0000:00/0000:00:1d.1
|-- 0000:00:1d.2 -> ../../../../devices/pci0000:00/0000:00:1d.2
|-- 0000:00:1d.3 -> ../../../../devices/pci0000:00/0000:00:1d.3
|-- bind
|-- module -> ../../../../module/uhci_hcd
|-- new_id
`-- unbind
That "module" symlink is created only if the .owner field is set.
That's why people are going through and adding it to all of the drivers
in the system.
Hope this helps,
greg k-h
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
