From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ananth N Mavinakayanahalli Date: Fri, 26 Mar 2004 12:50:57 +0000 Subject: Re: udevinfo patch Message-Id: <20040326133632.GA9525@in.ibm.com> List-Id: References: <20040323215101.GA30721@us.ibm.com> In-Reply-To: <20040323215101.GA30721@us.ibm.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: linux-hotplug@vger.kernel.org On Fri, Mar 26, 2004 at 11:24:39AM +0100, Kay Sievers wrote: > On Fri, Mar 26, 2004 at 11:21:29AM +0500, Ananth N Mavinakayanahalli wrote: > > 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: > > > > > > 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? > > Yes, it's invalid, but we shouldn't print major minor for a invalid > path. sysfs_open_class_device_path("/block") returns a device. If this is > the right behavior for libsysfs, I will change the get_device_type("/block") > not to return a 'b'-type. Libsysfs validates the path given to it for opening a class_device to be a valid directory; it does not however validate if the path is a valid class_device path. So, in the case of udevinfo, a 'b' type should not be returned if the path is just /sys/block or /sys/block/ 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_id70&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