From owner-freebsd-hackers@FreeBSD.ORG Tue Oct 19 14:35:18 2010 Return-Path: Delivered-To: freebsd-hackers@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id C19C51065670 for ; Tue, 19 Oct 2010 14:35:18 +0000 (UTC) (envelope-from mj@feral.com) Received: from ns1.feral.com (ns1.feral.com [192.67.166.1]) by mx1.freebsd.org (Postfix) with ESMTP id 8A82C8FC19 for ; Tue, 19 Oct 2010 14:35:18 +0000 (UTC) Received: from [192.168.1.2] (m206-63.dsl.tsoft.com [198.144.206.63]) by ns1.feral.com (8.14.3/8.14.3) with ESMTP id o9JEZHcr017200 for ; Tue, 19 Oct 2010 07:35:17 -0700 (PDT) (envelope-from mj@feral.com) Message-ID: <4CBDACA9.3040701@feral.com> Date: Tue, 19 Oct 2010 07:35:21 -0700 From: Matthew Jacob Organization: Feral Software User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.2.9) Gecko/20100915 Thunderbird/3.1.4 MIME-Version: 1.0 To: freebsd-hackers@freebsd.org References: <20101018235318.GA87158@freebsd.org> <4CBCE67C.1070106@feral.com> <20101019103432.GA69208@freebsd.org> <4CBDA371.4080801@feral.com> <20101019143110.GA5802@freebsd.org> In-Reply-To: <20101019143110.GA5802@freebsd.org> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit X-Greylist: Default is to whitelist mail, not delayed by milter-greylist-4.2.6 (ns1.feral.com [192.67.166.1]); Tue, 19 Oct 2010 07:35:18 -0700 (PDT) Subject: Re: SCSI_DELAY cleanup X-BeenThere: freebsd-hackers@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Technical Discussions relating to FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 19 Oct 2010 14:35:18 -0000 I'd go for the gusto in -current, but it's ok to be conservative too. > On Tue Oct 19 10, Matthew Jacob wrote: >> It would be an effective behavioral change for those of us who remove >> that line. >> Personally, I think 5 seconds is too long- even 2 seconds is more than >> adequate even for moderately old 'other' hardware like scanners. >> >> For -current, why don't you simply remove all of the config lines and >> leave the default at 2000ms? > hmmm...i can only test the delay value on amd64. i was under the impression > that archs like arm and mips need the longer delay. > > also at some locations in the code SCSI_DELAY is being set to 15000. i believe > this is the case when certain drivers (cam, ahb, aha) get loaded as a kernel > module, but i'm not sure. it looks like this: > > .if !defined(KERNBUILDDIR) > opt_scsi.h: > echo "#define SCSI_DELAY 15000"> ${.TARGET} > .endif > > cheers. > alex > >