From owner-freebsd-scsi@FreeBSD.ORG Sat Apr 13 10:05:04 2013 Return-Path: Delivered-To: freebsd-scsi@freebsd.org Received: from mx1.freebsd.org (mx1.FreeBSD.org [8.8.178.115]) by hub.freebsd.org (Postfix) with ESMTP id 945522EE; Sat, 13 Apr 2013 10:05:04 +0000 (UTC) (envelope-from brde@optusnet.com.au) Received: from mail04.syd.optusnet.com.au (mail04.syd.optusnet.com.au [211.29.132.185]) by mx1.freebsd.org (Postfix) with ESMTP id 1C49DACA; Sat, 13 Apr 2013 10:05:03 +0000 (UTC) Received: from c211-30-173-106.carlnfd1.nsw.optusnet.com.au (c211-30-173-106.carlnfd1.nsw.optusnet.com.au [211.30.173.106]) by mail04.syd.optusnet.com.au (8.13.1/8.13.1) with ESMTP id r3DA4s6F029512 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO); Sat, 13 Apr 2013 20:04:55 +1000 Date: Sat, 13 Apr 2013 20:04:54 +1000 (EST) From: Bruce Evans X-X-Sender: bde@besplex.bde.org To: sbruno@freebsd.org Subject: Re: kern/161809: [cam] [patch] set kern.cam.boot_delay via build options (usb boot) In-Reply-To: <201304122227.r3CMRg46091388@freefall.freebsd.org> Message-ID: <20130413200213.U1165@besplex.bde.org> References: <201304122227.r3CMRg46091388@freefall.freebsd.org> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII; format=flowed X-Optus-CM-Score: 0 X-Optus-CM-Analysis: v=2.0 cv=S7iBW/QP c=1 sm=1 a=RjrtvuHzA0YA:10 a=kj9zAlcOel0A:10 a=PO7r1zJSAAAA:8 a=JzwRw_2MAAAA:8 a=h_RcaxWYXjwA:10 a=6I5d2MoRAAAA:8 a=G6wGxQ0sGGlQ4ZYG1NQA:9 a=CjuIK1q_8ugA:10 a=SV7veod9ZcQA:10 a=TEtd8y5WR3g2ypngnwZWYw==:117 Cc: freebsd-scsi@freebsd.org, rozhuk.im@gmail.com X-BeenThere: freebsd-scsi@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: SCSI subsystem List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 13 Apr 2013 10:05:04 -0000 On Fri, 12 Apr 2013 sbruno@freebsd.org wrote: > Synopsis: [cam] [patch] set kern.cam.boot_delay via build options (usb boot) > > State-Changed-From-To: open->feedback > ... > I'm leaning towards rejecting this patch. I see why you would want a compile > time option for the boot delay variable, but you can accomplish the same thing > with a loader.conf tuneable now. > > kern.cam.boot_delay: 0 Or better using a static env variable. I don't remember if the patches to merge static env variables with loader env variables got committed. Bruce