From: Kay Sievers <kay.sievers@vrfy.org>
To: linux-hotplug@vger.kernel.org
Subject: Re: The DRIVER rule doesn't seem to work
Date: Thu, 03 Aug 2006 18:12:46 +0000 [thread overview]
Message-ID: <1154628766.28034.20.camel@pim.off.vrfy.org> (raw)
In-Reply-To: <44CCAD5F.9010407@communitymesh.com>
On Thu, 2006-08-03 at 10:45 +0100, Anthony Wright wrote:
> Kay Sievers wrote:
> >>>> What I believe is happening is that DRIVER is looking at the device's
> >>>> ancestor's drivers (parent, grandparent etc, but not the device itself),
> >>>> and returning their driver if they have one.
> >>>>
> >>> Sure, it looks a the device itself, but also walks up the parents and
> >>> finds a DRIVER value there.
> >>>
> >> I had a good look at this, and from what I can tell it only looks at the
> >> device's parents, not the device itself. I compiled and ran udev with
> >> debug to see what was going on (and have lots of logs if you're
> >> interested), but the short version is that while udev makes a DRIVER
> >> test for the device, it looks like the driver field hasn't been loaded
> >> for the device, only for the device's parents. From my logs, it seems
> >> that to load values for the device the function
> >> sysfs_device_set_values() is called in isolation, while for the parents
> >> sysfs_device_get() is called which calls sysfs_device_set_values(), but
> >> also sets the driver field.
> >>
> > If you ask for a class device, it will never be set, cause class devices
> > don't have a driver, only parent devices which are created by a bus
> I don't think they are class devices, but I can't be 100% sure as I'm
> still learning how all the device handling works.
>
> The path to the devices and their driver symbolic links are:
>
> Path Driver Symlink
> a) /sys/devices/pci0000:00/0000:00:1d.1 uhci_hcd
> b) /sys/devices/pci0000:00/0000:00:1d.1/usb2 usb
> c) /sys/devices/pci0000:00/0000:00:1d.1/usb2/2-0:1.0 hub
> d) /sys/devices/pci0000:00/0000:00:1d.1/usb2/2-1 usb
> e) /sys/devices/pci0000:00/0000:00:1d.1/usb2/2-1/2-1:1.0 NONE
> f) /sys/devices/pci0000:00/0000:00:1d.1/usb2/2-1/2-1:1.1 NONE
> g) /sys/devices/pci0000:00/0000:00:1d.1/usb2/2-1/2-1:1.2 NONE
>
> and there are a lot of references to the bus sub-directories in the logs.
>
> The rule DRIVER="?*" for each of these devices according to match_key
> matches the following matching values:
>
> Matching value
> a) MATCHING FAILS
> b) uhci_hcd
> c) usb
> d) usb
> e) usb
> f) usb
> g) usb
>
> This is consistant with DRIVER matching against a device's parents only,
> not the device itself. If DRIVER also matched against the device I
> believe the matching values should be as follows.
>
> Matching value
> a) uhci_hcd
> b) usb
> c) hub
> d) usb
> e) usb
> f) usb
> g) usb
Hmm, that seems to work for me, for the device we receive the event for:
DRIVER="uhci_hcd", PROGRAM="/bin/true usb"
$ ls -l /sys/devices/pci0000:00/0000:00:1d.1/driver
/sys/devices/pci0000:00/0000:00:1d.1/driver -> ../../../bus/pci/drivers/uhci_hcd
$ udevtest /devices/pci0000:00/0000:00:1d.1
main: looking at device '/devices/pci0000:00/0000:00:1d.1' from subsystem 'pci'
run_program: '/bin/true usb'
...
Kay
-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys -- and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CIDÞVDEV
_______________________________________________
Linux-hotplug-devel mailing list http://linux-hotplug.sourceforge.net
Linux-hotplug-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-hotplug-devel
next prev parent reply other threads:[~2006-08-03 18:12 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-07-30 12:58 The DRIVER rule doesn't seem to work Anthony Wright
2006-07-30 14:20 ` Kay Sievers
2006-08-03 8:40 ` Anthony Wright
2006-08-03 8:44 ` Kay Sievers
2006-08-03 9:45 ` Anthony Wright
2006-08-03 18:12 ` Kay Sievers [this message]
2006-08-11 21:59 ` Anthony Wright
2006-08-13 3:23 ` Kay Sievers
2006-08-14 7:29 ` Anthony Wright
2006-08-14 10:59 ` Kay Sievers
2006-08-23 0:05 ` Kay Sievers
2006-08-23 0:16 ` Kay Sievers
2006-08-24 14:34 ` Anthony Wright
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1154628766.28034.20.camel@pim.off.vrfy.org \
--to=kay.sievers@vrfy.org \
--cc=linux-hotplug@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).