Re: RFC: [2.6 patch] let BLK_DEV_UB depend on USB_STORAGE=n

[Greg KH]

On Thu, Dec 23, 2004 at 03:40:31AM +0100, Adrian Bunk wrote:
> On Sun, Dec 19, 2004 at 04:31:46PM -0800, Greg KH wrote:
> > On Mon, Dec 20, 2004 at 01:16:44AM +0100, Adrian Bunk wrote:
> > > I've already seen people crippling their usb-storage driver with 
> > > enabling BLK_DEV_UB - and I doubt the warning in the help text added 
> > > after 2.6.9 will fix all such problems.
> > > 
> > > Is there except for kernel size any good reason for using BLK_DEV_UB 
> > > instead of USB_STORAGE?
> > 
> > You don't want to use the scsi layer?  You like the stability of it at
> > times?  :)
> > 
> > > If not, I'd suggest the patch below to let BLK_DEV_UB depend
> > > on EMBEDDED.
> > 
> > No, it's good for non-embedded boxes too.
> 
> 
> My current understanding is:
> - BLK_DEV_UB supports a subset of what USB_STORAGE can support
> - for an average user, there's no reason to enable BLK_DEV_UB
> - if you really know what you are doing, there might be several reasons
>   why you might want to use BLK_DEV_UB

I have been running with just the code portion of this patch for a while
now, with good results (no Kconfig changes.)

Pete and Matt, do you mind me applying the following portion of the
patch to the kernel tree?

thanks,

greg k-h
> --- linux-2.6.10-rc3-mm1-full/drivers/usb/storage/unusual_devs.h.old	2004-12-23 03:09:42.000000000 +0100
> +++ linux-2.6.10-rc3-mm1-full/drivers/usb/storage/unusual_devs.h	2004-12-23 03:09:53.000000000 +0100
> @@ -549,13 +549,11 @@
>  		US_SC_SCSI, US_PR_CB, NULL,
>  		US_FL_SINGLE_LUN ),
>  
> -#if !defined(CONFIG_BLK_DEV_UB) && !defined(CONFIG_BLK_DEV_UB_MODULE)
>  UNUSUAL_DEV(  0x0781, 0x0002, 0x0009, 0x0009, 
>  		"Sandisk",
>  		"ImageMate SDDR-31",
>  		US_SC_DEVICE, US_PR_DEVICE, NULL,
>  		US_FL_IGNORE_SER ),
> -#endif
>  
>  UNUSUAL_DEV(  0x0781, 0x0100, 0x0100, 0x0100,
>  		"Sandisk",
> --- linux-2.6.10-rc3-mm1-full/drivers/usb/storage/usb.c.old	2004-12-23 03:10:00.000000000 +0100
> +++ linux-2.6.10-rc3-mm1-full/drivers/usb/storage/usb.c	2004-12-23 03:10:13.000000000 +0100
> @@ -144,9 +144,7 @@
>  	{ USB_INTERFACE_INFO(USB_CLASS_MASS_STORAGE, US_SC_QIC, US_PR_BULK) },
>  	{ USB_INTERFACE_INFO(USB_CLASS_MASS_STORAGE, US_SC_UFI, US_PR_BULK) },
>  	{ USB_INTERFACE_INFO(USB_CLASS_MASS_STORAGE, US_SC_8070, US_PR_BULK) },
> -#if !defined(CONFIG_BLK_DEV_UB) && !defined(CONFIG_BLK_DEV_UB_MODULE)
>  	{ USB_INTERFACE_INFO(USB_CLASS_MASS_STORAGE, US_SC_SCSI, US_PR_BULK) },
> -#endif
>  
>  	/* Terminating entry */
>  	{ }
> @@ -220,10 +218,8 @@
>  	  .useTransport = US_PR_BULK},
>  	{ .useProtocol = US_SC_8070,
>  	  .useTransport = US_PR_BULK},
> -#if !defined(CONFIG_BLK_DEV_UB) && !defined(CONFIG_BLK_DEV_UB_MODULE)
>  	{ .useProtocol = US_SC_SCSI,
>  	  .useTransport = US_PR_BULK},
> -#endif
>  
>  	/* Terminating entry */
>  	{ NULL }

-- 
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/