From mboxrd@z Thu Jan 1 00:00:00 1970 From: Greg KH Date: Wed, 22 Oct 2003 01:46:39 +0000 Subject: Re: [ANNOUNCE] udev 004 release Message-Id: List-Id: References: In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: linux-hotplug@vger.kernel.org On Tue, Oct 21, 2003 at 03:13:38PM -0700, Greg KH wrote: > On Tue, Oct 21, 2003 at 02:45:55PM -0700, Jesse Barnes wrote: > > Thanks for the new release, Greg. I just tried it out on a system with > > some disks, but a bunch of udev processes ended up hanging. Is there > > something I'm missing or do you need a patch like this? > > Yeah, sorry, this kind of fix is required :( > > It's fixed in my bk tree now. > > Oh, and it looks like the LABEL rule is also broken due to the libsysfs > changes... I'm working on adding regression tests right now to prevent > things like this from slipping through. Here's a patch for this. It fixes the problem of LABEL rules on the device, not the class device. LABEL rules on the class device seem to work just fine. Thanks to Dan Stekloff for help in finding this bug. thanks, greg k-h # fix LABEL bug for device files (not class files.) diff -Nru a/namedev.c b/namedev.c --- a/namedev.c Tue Oct 21 18:44:59 2003 +++ b/namedev.c Tue Oct 21 18:44:59 2003 @@ -566,7 +566,7 @@ /* look in the class device directory if present */ if (class_dev->sysdevice) { - tmpattr = sysfs_get_classdev_attr(class_dev, dev->sysfs_file); + tmpattr = sysfs_get_device_attr(class_dev->sysdevice, dev->sysfs_file); if (tmpattr) goto label_found; } @@ -599,7 +599,7 @@ /* look in the class device directory if present */ if (class_dev_parent->sysdevice) { - tmpattr = sysfs_get_classdev_attr(class_dev_parent, dev->sysfs_file); + tmpattr = sysfs_get_device_attr(class_dev_parent->sysdevice, dev->sysfs_file); if (tmpattr) goto label_found; } ------------------------------------------------------- This SF.net email is sponsored by OSDN developer relations Here's your chance to show off your extensive product knowledge We want to know what you know. Tell us and you have a chance to win $100 http://www.zoomerang.com/survey.zgi?HRPT1X3RYQNC5V4MLNSV3E54 _______________________________________________ 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