cvs commit: src/sys/cam/scsi scsi_da.c

[Nate Lawson]

On Thu, 17 Jul 2003, Thomas Quinot wrote:
>   Modified files:
>     sys/cam/scsi         scsi_da.c
>   Log:
>   Add quirk entry for IntelligentStick disc-on-key USB devices.
>   Reported by Samuel Tardieu <sam at rfc1149.net>.
>
>   Reviewed by:    roberto
>   MFC after:      1 week
>
>   Revision  Changes    Path
>   1.145     +8 -0      src/sys/cam/scsi/scsi_da.c

Since I haven't updated people on the current status of quirks, this is a
good chance to let everyone know the direction this is going.  Currently,
nearly every UMASS device on the planet needs a NO_6_BYTE quirk because
they hang (not respond with an error) to 6 byte commands.  Ones that
respond with an error cause an auto-renegotiate to 10 byte commands and
don't need a quirk.  Obviously, the right solution is to not send 10 byte
commands in the first place.  To do this right, CAM needs to be aware of
the transport protocol when a SIM is registered and needs to have
different settings for different transports.  This is a lot of work,
although there have been ongoing discussions about it.

In the interim, I marked myself as da(4) maintainer and have been
gathering information on quirks as they come in.  Each da(4) quirk needs
to have a PR to track the type of device and any important info (usbdevs
-v, camcontrol inquiry da0, ...).  See this url for more details:
  http://root.org/~nate/freebsd/quirks.html

Since the ultimate goal of full transport support is farther off than we'd
like, I am working on a patch that adds a "no 6 byte" flag to the SIM.
USB (and probably firewire) SIM drivers will set that flag in their attach
routines.  This will solve the problem (in a non-elegant way) and remove
the need for almost all of our USB quirks in da(4).  But for me to be able
to remove those quirks when they're no longer needed, I need a PR and
audit trail for it (see the web page) so there can be a minimum of grief
when I remove them.  I have not been committing quirks for a few weeks
since I'm getting ready to test this patch and remove them (hopefully
before 5.2 if all goes well).

So please coordinate quirks with me as we work towards eliminating them.
For this particular commit, please create a PR and replace the user email
address in the comment with the PR number (see surrounding quirk style).

Thanks,
Nate