From owner-freebsd-scsi@FreeBSD.ORG Mon Aug 22 20:31:15 2011 Return-Path: Delivered-To: freebsd-scsi@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id C79581065676; Mon, 22 Aug 2011 20:31:15 +0000 (UTC) (envelope-from mavbsd@gmail.com) Received: from mail-fx0-f54.google.com (mail-fx0-f54.google.com [209.85.161.54]) by mx1.freebsd.org (Postfix) with ESMTP id 294A88FC12; Mon, 22 Aug 2011 20:31:14 +0000 (UTC) Received: by fxe4 with SMTP id 4so4697050fxe.13 for ; Mon, 22 Aug 2011 13:31:13 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=sender:message-id:date:from:user-agent:mime-version:to:cc:subject :references:in-reply-to:content-type:content-transfer-encoding; bh=4Ft18SarZFIwQoOceS8664wZzDNre51H8IUWwFuxFSc=; b=nf2CpwkR2mbxrH15ScgwiXq2t3zRCVWVc+eUc/wdMJ1TFyrfyjqeRvo4k7BIbeVPD0 /pbFDoHSG6otqCbzhyBqCeptYRi/6NaWfIGyEzSd0G9Tn9B9QbeczkahvAmbldKvDm29 SqKmMsWG0COjobenta6ONjBffGgRPKz+kTF0s= Received: by 10.223.54.134 with SMTP id q6mr4177604fag.130.1314045073632; Mon, 22 Aug 2011 13:31:13 -0700 (PDT) Received: from mavbook.mavhome.dp.ua (pc.mavhome.dp.ua [212.86.226.226]) by mx.google.com with ESMTPS id g16sm5377027faa.27.2011.08.22.13.31.09 (version=SSLv3 cipher=OTHER); Mon, 22 Aug 2011 13:31:10 -0700 (PDT) Sender: Alexander Motin Message-ID: <4E52BC7F.4040408@FreeBSD.org> Date: Mon, 22 Aug 2011 23:30:55 +0300 From: Alexander Motin User-Agent: Mozilla/5.0 (X11; FreeBSD amd64; rv:5.0) Gecko/20110709 Thunderbird/5.0 MIME-Version: 1.0 To: Dmitry Morozovsky References: <20110822194031.GR17489@deviant.kiev.zoral.com.ua> In-Reply-To: Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: freebsd-scsi@FreeBSD.org, re@FreeBSD.org Subject: Re: Documentating patch for defaults/loader.conf X-BeenThere: freebsd-scsi@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: SCSI subsystem List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 22 Aug 2011 20:31:15 -0000 On 22.08.2011 23:10, Dmitry Morozovsky wrote: > On Mon, 22 Aug 2011, Kostik Belousov wrote: >>> any objections for this simple patch? >>> >>> (we stumped into this yesterday, and no easy founded answers even on Google >>> except the single FreeBSD Forums thread was found) >> >> I do not think that the wording in right, but I am not CAM expert by any >> measure. From my reading of the code, it is a hook to delay the root mount, >> not to delay CAM bus registration. It would be better if you get the scsi@ >> or mav@ (author of the tunable) review. > > I based the comment on existing comment to the tunable in cam_xpt.c: > > cam_xpt.c:TUNABLE_INT("kern.cam.boot_delay",&xsoftc.boot_delay); > cam_xpt.c:SYSCTL_INT(_kern_cam, OID_AUTO, boot_delay, CTLFLAG_RDTUN, > cam_xpt.c:&xsoftc.boot_delay, 0, "Bus registration wait time"); > > > Alexander, colleagues, your thought on the following patch? > > -- 8< -- > Index: boot/forth/loader.conf > =================================================================== > --- boot/forth/loader.conf (revision 224542) > +++ boot/forth/loader.conf (working copy) > @@ -109,6 +109,8 @@ > #kern.ncallout="" # Set the maximum # of timer events > #kern.ngroups="1023" # Set the maximum # of supplemental groups > #kern.sgrowsiz="" # Set the amount to grow stack > +#kern.cam.boot_delay="10000" # CAM bus registration delay (in ms), > + # useful for USB sticks as root > #kern.cam.scsi_delay="2000" # Delay (in ms) before probing SCSI > #kern.ipc.maxsockets="" # Set the maximum number of sockets > avaliable > #kern.ipc.nmbclusters="" # Set the number of mbuf clusters > -- 8< -- Yes, I agree that wording is not perfect. It is not a delay of the bus registration, but instead boot delay for the bus registration. Original sysctl description is closer to the right meaning. -- Alexander Motin