Re: [usb-storage] Re: RFC drivers/usb/storage/libusual
From: Alan Stern
Date: Sat Oct 08 2005 - 10:20:22 EST
On Sat, 8 Oct 2005, Pete Zaitcev wrote:
> On Fri, 7 Oct 2005 10:41:14 -0400 (EDT), Alan Stern <stern@xxxxxxxxxxxxxxxxxxx> wrote:
> > > --- linux-2.6.14-rc2/drivers/usb/storage/usb.h 2005-09-24 20:32:56.000000000 -0700
> > > +++ linux-2.6.14-rc2-wip/drivers/usb/storage/usb.h 2005-10-06 21:37:10.000000000 -0700
> > ...
> > > -/* Dynamic flag definitions: used in set_bit() etc. */
> > > -#define US_FLIDX_URB_ACTIVE 18 /* 0x00040000 current_urb is in use */
> > > -#define US_FLIDX_SG_ACTIVE 19 /* 0x00080000 current_sg is in use */
> > I would prefer to keep these definitions in the usb-storage driver. They
> > refer to dynamic aspects of an individual device, not static blacklist or
> > ID-matching for all devices of a particular type. As such, they are of no
> > interest to ub or libusual.
> > > + * Observe that usb-storage blatantly mixes set_bit() and normal
> > > + * shift and mask operations on flags, which is strictly illegal.
> > > + * And it probably even works for all flags except GO_SLOW and NO_WP_DETECT.
> > Once the device is running, the shift/mask operations are used _only_ for
> > reading, never for writing.
> How about now?
This is better, thank you.
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/