On Sat, Apr 09, 2005 at 09:55:21AM +0400, Roman Kagan wrote:
> On Fri, Apr 08, 2005 at 02:49:13PM -0700, Greg KH wrote:
> > On Thu, Mar 31, 2005 at 07:40:46PM +0400, Roman Kagan wrote:
> > > On Wed, Mar 30, 2005 at 01:47:55PM -0800, Greg KH wrote:
> > > > ChangeSet 1.2181.4.57, 2005/03/24 14:34:40-08:00, [EMAIL PROTECTED]
> > > > 
> > > > [PATCH] drivers/usb/core/usb.c: add MODALIAS env var to hotplug
> > > > 
> > > > The patch below adds MODALIAS environment variable to usb hotplug
> > > > callout, allowing for its straightforward use with modprobe.
> > > > 
> > > > Signed-off-by: Roman Kagan <[EMAIL PROTECTED]>
> > > > Signed-off-by: Greg Kroah-Hartman <[EMAIL PROTECTED]>
> > > 
> > > Any chance to get my other patch re. modifying the module alias patterns
> > > in usb modules to correctly handle bcdDevice ranges included?  I'm a bit
> > > worried about this one included without that one, because it's sort of
> > > encouraging the interface which used to be broken.
> > 
> > In thinking about it a lot, I don't want to accept that change.  I don't
> > think it is wise to break userspace tools in such a bad way.  Especially
> > as I don't see a need for this (no one ever relied on matching based on
> > the bcdDevice range before, and if they did, the kernel handled it
> > properly internally anyway.)
> 
> I disagree with both your arguments :)
> 
> 1) I'm unaware of any existing userspace tool which would be broken by
>    that, exactly because it was practically impossible to create one;

Um, all of the existing hotplug scripts would break, if you modified the
file by removing a field there, right?  That wouldn't be good at all.

> 2) now that MODALIAS is generated by the kernel, it will be the kernel
>    which will be broken: for a number of drivers _no_ MODALIAS will
>    match the MODULE_ALIAS_TABLE, and the userspace will have little to
>    do about it.

No, see how I modified the patch.  It's in the -mm tree.

thanks,

greg k-h


-------------------------------------------------------
SF email is sponsored by - The IT Product Guide
Read honest & candid reviews on hundreds of IT Products from real users.
Discover which products truly live up to the hype. Start reading now.
http://ads.osdn.com/?ad_id=6595&alloc_id=14396&op=click
_______________________________________________
linux-usb-devel@lists.sourceforge.net
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel

Reply via email to