Re: udevinfo patch

[Ananth N Mavinakayanahalli <ananth@in.ibm.com>]

On Fri, Mar 26, 2004 at 04:47:35AM +0100, Kay Sievers wrote:
> On Fri, Mar 26, 2004 at 01:26:46AM +0100, Carl-Daniel Hailfinger wrote:
> > Greg KH wrote:
> > > On Tue, Mar 23, 2004 at 01:51:01PM -0800, Daniel Stekloff wrote:
> > > 
> > >>
> > >>Hi Greg,
> > >>
> > >>I think this is what you want for udevinfo. Patched against the latest BK
> > >>tree. I tested it and it seemed to work.
> > 
> > It seems to have a bug:
> > 
> > carldani@4100xcdt:~/work/udev-023-orig> ./udevinfo -a -p /sys/block/
> > 
> > udevinfo starts with the device the node belongs to and then walks up the
> > [...]
> > to match the device for which the node will be created.
> > 
> > device '/sys/block' has major:minor 1:9
> >   looking at class device '/sys/block':
> > couldn't open class device directory
> > 
> > 
> > How can /sys/block have a major/minor number which changes everytime I
> > recompile udevinfo?
> 
> It's the attribute value read from the last sysfs device.
> 
> > > Thanks, I've applied this.
> > 
> > Could you also apply the attached patch to clean up a bit?
> 
> No, it breaks the net device handling. I think we should change
> libsysfs instead, not to return a class device for '/block', if
> we want to fix it.

/sys/block is considered a sysfs "class" and not a class_device. So, 
going by udevinfo's help, -p expects path to a class_device and _not_ 
a class itself and hence option /sys/block with -p is not a valid query. 

Kay?

Thanks,
Ananth


-------------------------------------------------------
This SF.Net email is sponsored by: IBM Linux Tutorials
Free Linux tutorial presented by Daniel Robbins, President and CEO of
GenToo technologies. Learn everything from fundamentals to system
administration.http://ads.osdn.com/?ad_id\x1470&alloc_id638&op=click
_______________________________________________
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