From owner-cvs-all  Mon May 20  7:39:11 2002
Delivered-To: cvs-all@freebsd.org
Received: from mail.sat.t.u-tokyo.ac.jp (nat.keisu.t.u-tokyo.ac.jp [133.11.68.2])
	by hub.freebsd.org (Postfix) with ESMTP
	id 1BB3337B403; Mon, 20 May 2002 07:39:03 -0700 (PDT)
Received: from ett.sat.t.u-tokyo.ac.jp (ett.sat.t.u-tokyo.ac.jp [10.6.1.30])
	by mail.sat.t.u-tokyo.ac.jp (Postfix) with ESMTP
	id BCE132DAC8; Mon, 20 May 2002 23:39:01 +0900 (JST)
Date: Mon, 20 May 2002 23:39:01 +0900
Message-ID: <ybsznyudgu2.wl@ett.sat.t.u-tokyo.ac.jp>
From: Hidetoshi Shimokawa <simokawa@sat.t.u-tokyo.ac.jp>
To: "Kenneth D. Merry" <ken@kdm.org>
Cc: Matthew Dillon <dillon@apollo.backplane.com>,
	cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org
Subject: Re: cvs commit: src/sys/cam/scsi scsi_da.c
In-Reply-To: <20020519172841.A51662@panzer.kdm.org>
References: <200205192159.g4JLxSx22676@freefall.freebsd.org>
	<20020519170221.A51589@panzer.kdm.org>
	<200205192312.g4JNCsKs098352@apollo.backplane.com>
User-Agent: Wanderlust/2.9.9 (Unchained Melody) REMI/1.14.3 (Matsudai)
 FLIM/1.14.3 (=?ISO-8859-1?Q?Unebigory=F2mae?=) APEL/10.3 MULE XEmacs/21.2
 (beta19) (Shinjuku) (i386-unknown-freebsd3.2)
X-Face: OE([KxWyJI0r[R~S/>7ia}SJ)i%a,$-9%7{*yihQk|]gl}2p#"oXmX/fT}Bn7:#j7i14gu$
 jgR\S*&C3R/pJX<mI-4Z8'mqt1R'CjWdw-xqVoilv`\JgZcquM5oVuO^<vt/~051rCDM(`{rM~?{Ok
 r,XC+}Ycm9IJO)odT*-0nyJjBJYzL{GsZ
MIME-Version: 1.0 (generated by REMI 1.14.3 - "Matsudai")
Content-Type: text/plain; charset=US-ASCII
Sender: owner-cvs-all@FreeBSD.ORG
Precedence: bulk
List-ID: <cvs-all.FreeBSD.ORG>
List-Archive: <http://docs.freebsd.org/mail/> (Web Archive)
List-Help: <mailto:majordomo@FreeBSD.ORG?subject=help> (List Instructions)
List-Subscribe: <mailto:majordomo@FreeBSD.ORG?subject=subscribe%20cvs-all>
List-Unsubscribe: <mailto:majordomo@FreeBSD.ORG?subject=unsubscribe%20cvs-all>
X-Loop: FreeBSD.ORG

At Sun, 19 May 2002 17:28:41 -0600,
Kenneth D. Merry <ken@kdm.org> wrote:
> -stable and -current both have the code to handle devices that don't like 6
> byte commands.
> 
> The reason all those quirks are still in there is that we would need to
> contact everyone to see if the code we've got to detect devices with this
> issue actually works for their device.  If the device returns a
> non-standard error code, it won't get flagged as a device that doesn't
> support 6 byte commands.

Unfortunately, there are many USB devices which don't return any SCSI
error for 6 byte commands. For such devices, you have to do one of the
following:

1. sysctl kern.cam.da.no_6_byte=1
2. Replace two '#if 0' with '#if 1' in dadone() of scsi_da.c.
3. Add a quirk.

> Well, they're more ATAPI than anything else.  The better fix in -current
> at least would be to use the new transport settings code to determine that
> devices on transport X don't support 6 byte commands.  What we've got now

It sounds good.

/\ Hidetoshi Shimokawa
\/  simokawa@sat.t.u-tokyo.ac.jp
PGP public key: http://www.sat.t.u-tokyo.ac.jp/~simokawa/pgp.html

To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message