Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 21 Aug 2002 02:46:14 +0200
From:      Thomas Quinot <thomas@cuivre.fr.eu.org>
To:        Nate Lawson <nate@root.org>
Cc:        scsi@freebsd.org, current@freebsd.org
Subject:   Re: Proliferating quirk table entries
Message-ID:  <20020821024614.A722@melusine.cuivre.fr.eu.org>
In-Reply-To: <Pine.BSF.4.21.0208161526550.43909-100000@root.org>; from nate@root.org on Fri, Aug 16, 2002 at 03:39:01PM -0700
References:  <Pine.BSF.4.21.0208161526550.43909-100000@root.org>

next in thread | previous in thread | raw e-mail | index | archive | help
Le 2002-08-17, Nate Lawson écrivait :

> I'm working on cleaning up quirk entries in scsi_da.c, especially ones
> related to READ/WRITE 6->10 escalation.  For those just joining in, there
> is a function (cmd6workaround) that handles a R/W6 error by translating
> the cdb to 10 bytes and restarting it.

It might be worthwhile moving this to some generic part in the CAM
framework, instead of having it in the da driver. Similar promotion
is performed for some commands (MODE_{SELECT,SENSE}_6 as well as
{READ,WRITE}_6) in a rather ad hoc fashion in atapi-cam. At least
the cmd6workaround function should be factored in some way; as for the
try 6 -> fail -> retry 10 process, however, I am not sure this can
be readily generalised to ATAPI devices (which are explicitly specified
to only support the _10 variants) as these tend to have very strange
reactions to CDBs they cannot handle properly.

Thomas.

-- 
    Thomas.Quinot@Cuivre.FR.EU.ORG

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




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20020821024614.A722>