From owner-cvs-src@FreeBSD.ORG Sun Oct 26 22:15:56 2003 Return-Path: Delivered-To: cvs-src@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 00E0816A4BF; Sun, 26 Oct 2003 22:15:56 -0800 (PST) Received: from repoman.freebsd.org (repoman.freebsd.org [216.136.204.115]) by mx1.FreeBSD.org (Postfix) with ESMTP id 85DC643FE0; Sun, 26 Oct 2003 22:15:55 -0800 (PST) (envelope-from ken@FreeBSD.org) Received: from repoman.freebsd.org (localhost [127.0.0.1]) by repoman.freebsd.org (8.12.9/8.12.9) with ESMTP id h9R6FtXJ038958; Sun, 26 Oct 2003 22:15:55 -0800 (PST) (envelope-from ken@repoman.freebsd.org) Received: (from ken@localhost) by repoman.freebsd.org (8.12.9/8.12.9/Submit) id h9R6FtQY038957; Sun, 26 Oct 2003 22:15:55 -0800 (PST) (envelope-from ken) Message-Id: <200310270615.h9R6FtQY038957@repoman.freebsd.org> From: "Kenneth D. Merry" Date: Sun, 26 Oct 2003 22:15:55 -0800 (PST) To: src-committers@FreeBSD.org, cvs-src@FreeBSD.org, cvs-all@FreeBSD.org X-FreeBSD-CVS-Branch: HEAD Subject: cvs commit: src/sys/cam cam_periph.c src/sys/cam/scsi scsi_cd.c X-BeenThere: cvs-src@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 27 Oct 2003 06:15:56 -0000 ken 2003/10/26 22:15:55 PST FreeBSD src repository Modified files: sys/cam cam_periph.c sys/cam/scsi scsi_cd.c Log: In camperiphdone(), make sure we check for fatal errors and bail out instead of retrying them blindly. This should fix some of the problems people have been having with cdrom drives taking a long time to probe. This should also eliminate the need for the initial TUR in cdsize(). cam_periph.c: Don't keep retrying if the error we get back is a fatal error. This should help us detect the transition from "Logical unit not ready, cause not reportable" to "Medium not present" in the "TUR many" handler. (The TUR many handler gets triggered for Logical unit not ready, cause not reportable errors.) scsi_cd.c: Remove the initial test unit ready in cdsize(). Hopefully it isn't necessary after the above change. Submitted by: gibbs (mostly) Tested by: peter MFC After: 2 weeks Revision Changes Path 1.55 +17 -2 src/sys/cam/cam_periph.c 1.88 +0 -14 src/sys/cam/scsi/scsi_cd.c