From owner-cvs-all Thu Aug 30 13:30:33 2001 Delivered-To: cvs-all@freebsd.org Received: from postfix2-1.free.fr (postfix2-1.free.fr [213.228.0.9]) by hub.freebsd.org (Postfix) with ESMTP id 2312337B417; Thu, 30 Aug 2001 13:30:21 -0700 (PDT) (envelope-from groudier@free.fr) Received: from nas-cbv-7-23-113.dial.proxad.net (nas-cbv-7-23-113.dial.proxad.net [213.228.23.113]) by postfix2-1.free.fr (Postfix) with ESMTP id CE54B3D0; Thu, 30 Aug 2001 21:47:12 +0200 (CEST) Date: Thu, 30 Aug 2001 21:44:23 +0200 (CEST) From: =?ISO-8859-1?Q?G=E9rard_Roudier?= X-X-Sender: To: Matt Jacob Cc: , Subject: Re: cvs commit: src/sys/cam/scsi scsi_sa.c In-Reply-To: <200108301625.f7UGPOB07591@freefall.freebsd.org> Message-ID: <20010830212344.A1692-100000@gerard> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=ISO-8859-1 Content-Transfer-Encoding: QUOTED-PRINTABLE Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG On Thu, 30 Aug 2001, Matt Jacob wrote: [...] > 2. If we get a Bus Reset, had a BDR sent, or get the cam status of > CAM_REQUEUE_REQ, check the retry count on the command. The default > error handler, cam_periph_error, doesn't honor retry count in these > cases. This may change in the future, but for now, make sure we > set EIO and return without calling cam_periph_error if the retry > count for the command with an error is zero. You certainly didn't miss that commands returned by SIMs with CAM_REQUEUE_REQ status haven't ever being tried. Checking against the retry count for these ones does not make sense, in my opinion. The retry could should only be counted for the command that failed and that made the SIM ask for some other commands to be requeued, in my humble :) opinion. It may well be very unlikely and probably just bogus for a batch of commands to be completed with CAM_REQUEUE_REQ and not even a single command being completed with a true failure condition (for a given device). [...] G=E9rard. To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message