On Tue, July 24, 2007 10:20, Kay Sievers wrote: > On 7/24/07, Greg KH <[EMAIL PROTECTED]> wrote: >> On Tue, Jul 24, 2007 at 10:03:14AM +0200, Cornelia Huck wrote: >> > On Tue, 24 Jul 2007 00:25:40 -0700, >> > Greg KH <[EMAIL PROTECTED]> wrote: >> > >> > > On Tue, Jul 24, 2007 at 07:39:38AM +0100, Simon Arlott wrote: >> > > > The following commit appears to break some of my udev rules (I don't >> > > > have the time to finish the bisect right now, but there's only four >> > > > changes showing in "git bisect visualize" - this one is tagged >> > > > bisect/bad, and the other three are docs/docs/unrelated). >> > > > >> > > > Neither of these symlinks get created by udev on kernels marked bad >> > > > (see bisect log below): >> > > > >> > > > ACTION=="add", \ >> > > > KERNEL=="event*", \ >> > > > SUBSYSTEM=="input", \ >> > > > SYSFS{description}=="i8042 KBD port", \ >> > > > NAME="input/%k", \ >> > > > SYMLINK="input/i8042-kbd", \ >> > > > MODE="0640", \ >> > > > GROUP="event" >> > > > >> > > > ACTION=="add", \ >> > > > KERNEL=="event*", \ >> > > > SUBSYSTEM=="input", \ >> > > > SYSFS{manufacturer}=="Logitech", \ >> > > > SYSFS{product}=="USB-PS/2 Optical Mouse", \ >> > > > NAME="input/%k", \ >> > > > SYMLINK="input/logitech-mouse", \ >> > > > MODE="0640", \ >> > > > GROUP="event" > > Simon, please run: > udevinfo --attribute-walk --path=<devpath> > for the mouse on the working and the non-working kernel.
On the working kernel: looking at device '/devices/pci0000:00/0000:00:0c.0/usb2/2-2/2-2:1.0': KERNEL=="2-2:1.0" SUBSYSTEM=="usb" DRIVER=="usbhid" ATTR{modalias}=="usb:v046DpC044d2710dc00dsc00dp00ic03isc01ip02" ATTR{bInterfaceProtocol}=="02" ATTR{bInterfaceSubClass}=="01" ATTR{bInterfaceClass}=="03" ATTR{bNumEndpoints}=="01" ATTR{bAlternateSetting}==" 0" ATTR{bInterfaceNumber}=="00" looking at parent device '/devices/pci0000:00/0000:00:0c.0/usb2/2-2': KERNELS=="2-2" SUBSYSTEMS=="usb" DRIVERS=="usb" ATTRS{product}=="USB-PS/2 Optical Mouse" ATTRS{manufacturer}=="Logitech" ATTRS{quirks}=="0x0" ATTRS{maxchild}=="0" ATTRS{version}==" 2.00" ATTRS{devnum}=="3" ATTRS{busnum}=="2" ATTRS{speed}=="1.5" ATTRS{bMaxPacketSize0}=="8" ATTRS{bNumConfigurations}=="1" ATTRS{bDeviceProtocol}=="00" ATTRS{bDeviceSubClass}=="00" ATTRS{bDeviceClass}=="00" ATTRS{bcdDevice}=="2710" ATTRS{idProduct}=="c044" ATTRS{idVendor}=="046d" ATTRS{bMaxPower}==" 98mA" ATTRS{bmAttributes}=="a0" ATTRS{bConfigurationValue}=="1" ATTRS{bNumInterfaces}==" 1" ATTRS{configuration}=="" ATTRS{dev}=="189:130" looking at parent device '/devices/pci0000:00/0000:00:0c.0/usb2': KERNELS=="usb2" SUBSYSTEMS=="usb" DRIVERS=="usb" ATTRS{serial}=="0000:00:0c.0" ATTRS{product}=="UHCI Host Controller" ATTRS{manufacturer}=="Linux 2.6.22-git uhci_hcd" ATTRS{quirks}=="0x0" ATTRS{maxchild}=="2" ATTRS{version}==" 1.10" ATTRS{devnum}=="1" ATTRS{busnum}=="2" ATTRS{speed}=="12" ATTRS{bMaxPacketSize0}=="64" ATTRS{bNumConfigurations}=="1" ATTRS{bDeviceProtocol}=="00" ATTRS{bDeviceSubClass}=="00" ATTRS{bDeviceClass}=="09" ATTRS{bcdDevice}=="0206" ATTRS{idProduct}=="0000" ATTRS{idVendor}=="0000" ATTRS{bMaxPower}==" 0mA" ATTRS{bmAttributes}=="e0" ATTRS{bConfigurationValue}=="1" ATTRS{bNumInterfaces}==" 1" ATTRS{configuration}=="" ATTRS{dev}=="189:128" looking at parent device '/devices/pci0000:00/0000:00:0c.0': KERNELS=="0000:00:0c.0" SUBSYSTEMS=="pci" DRIVERS=="uhci_hcd" ATTRS{msi_bus}=="" ATTRS{broken_parity_status}=="0" ATTRS{enable}=="1" ATTRS{modalias}=="pci:v00001106d00003038sv00001106sd00003038bc0Csc03i00" ATTRS{local_cpus}=="1" ATTRS{irq}=="11" ATTRS{class}=="0x0c0300" ATTRS{subsystem_device}=="0x3038" ATTRS{subsystem_vendor}=="0x1106" ATTRS{device}=="0x3038" ATTRS{vendor}=="0x1106" looking at parent device '/devices/pci0000:00': KERNELS=="pci0000:00" SUBSYSTEMS=="" DRIVERS=="" ATTRS{uevent}=="" I can't run it on the non-working kernel until this evening... > And make sure you have only one rule for input devices or use > SYMLINK+="", otherwise it overwrites any earlier rule. And if you have > an earlier rule which already names the device, this one will be > ignored because it has NAME= in it. These rules are before the other event* rules. >> > > Ugh, I thought this was all fixed up properly :( >> > >> > I thought this as well :( >> > >> > But I'm a bit confused: The patch in git has >> > >> > + /* only bus-device parents get a "device"-link */ >> > + if (dev->parent && dev->parent->bus) { >> > + error = sysfs_create_link(&dev->kobj, &dev->parent->kobj, >> > + "device"); >> > >> > and >> > >> > - if (parent) { >> > - sysfs_create_link(&dev->kobj, &dev->parent->kobj, >> > - "device"); >> > >> > which really look like two different things. (My original patch didn't >> > have the check for the parent's bus.) Don't know what happened here :( >> >> Ugh, this might be a merge issue with Kay's block layer device work that >> was in my tree, but I had to merge by hand around this area. >> >> > (Simon: Do the links reappear if you change >> > if (dev->parent && dev->parent->bus) >> > to >> > if (dev->parent) >> > in device_add_class_symlinks()?) >> >> Yeah, that would be good to find out. >> >> Kay, did I mess up the merge here? > > It looks fine to me. "device" links must never point to anything else > than a bus device. -- Simon Arlott - 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/