From garys@opusnet.com  Fri Aug 19 17:16:20 2005
Return-Path: <garys@opusnet.com>
Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125])
	by hub.freebsd.org (Postfix) with ESMTP id 4A54F16A41F
	for <FreeBSD-gnats-submit@freebsd.org>; Fri, 19 Aug 2005 17:16:20 +0000 (GMT)
	(envelope-from garys@opusnet.com)
Received: from opusnet.com (mail.opusnet.com [209.210.200.6])
	by mx1.FreeBSD.org (Postfix) with ESMTP id 739C743D46
	for <FreeBSD-gnats-submit@freebsd.org>; Fri, 19 Aug 2005 17:16:19 +0000 (GMT)
	(envelope-from garys@opusnet.com)
Received: from localhost.localhost [70.98.246.232] by opusnet.com with ESMTP
  (SMTPD32-8.05) id A3E1AA600C6; Fri, 19 Aug 2005 10:16:17 -0700
Received: from localhost.localhost (localhost.localhost [127.0.0.1])
	by localhost.localhost (8.13.3/8.13.3) with ESMTP id j7JHHRhi016164
	for <FreeBSD-gnats-submit@freebsd.org>; Fri, 19 Aug 2005 10:17:27 -0700 (PDT)
	(envelope-from garys@opusnet.com)
Received: (from jojo@localhost)
	by localhost.localhost (8.13.3/8.13.3/Submit) id j7JHHMDm016163;
	Fri, 19 Aug 2005 10:17:22 -0700 (PDT)
	(envelope-from garys@opusnet.com)
Message-Id: <hlu0hlrgm5.0hl@mail.opusnet.com>
Date: Fri, 19 Aug 2005 10:17:22 -0700
From: "Gary W. Swearingen" <garys@opusnet.com>
Reply-To: garys@opusnet.com
To: FreeBSD-gnats-submit@freebsd.org
Subject: loader.conf autoboot_delay incompletly described
X-GNATS-Notify:

>Number:         85128
>Category:       docs
>Synopsis:       [patch] loader.conf(5) autoboot_delay incompletly described
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    freebsd-doc
>State:          open
>Quarter:        
>Keywords:       
>Date-Required:  
>Class:          doc-bug
>Submitter-Id:   current-users
>Arrival-Date:   Fri Aug 19 17:20:09 GMT 2005
>Closed-Date:    
>Last-Modified:  Sat Jun 21 09:06:28 UTC 2008
>Originator:     Gary W. Swearingen
>Release:        FreeBSD 5.4-RELEASE i386
>Organization:
none
>Environment:
n/a
>Description:

The description of "autoboot_delay" variable in
/boot/defaults/loader.conf does not mention it's "NO" special
value.

A variable of the same name is described in loader(8), but it is by no
means obvious whether it's the same variable or just another set at the
loader prompt.

The variable is only the one in loader.conf that has more than a brief
comment, leading me to wonder if it shouldn't be described in
loader.conf(5) manpage instead, but the first paragraph of its
"Default Settings" section  makes me wonder.

>How-To-Repeat:
>Fix:

Decide whether full description belongs in loader(8), loader.conf(5),
or /boot/defaults/loader.conf.  I don't know about loader(8), but I'm
thinking a full description should be in loader.conf(5) and referenced
from comment in "/boot/defaults/loader.conf".

I'll write patch after change is agreed on.
>Release-Note:
>Audit-Trail:

From: Giorgos Keramidas <keramida@ceid.upatras.gr>
To: "Gary W. Swearingen" <garys@opusnet.com>
Cc: bug-followup@freebsd.org
Subject: Re: docs/85128: loader.conf autoboot_delay incompletly described
Date: Sat, 20 Aug 2005 04:28:50 +0300

 On 2005-08-19 10:17, "Gary W. Swearingen" <garys@opusnet.com> wrote:
 > The description of "autoboot_delay" variable in
 > /boot/defaults/loader.conf does not mention it's "NO" special
 > value.
 >
 > A variable of the same name is described in loader(8), but it is by no
 > means obvious whether it's the same variable or just another set at the
 > loader prompt.
 >
 > The variable is only the one in loader.conf that has more than a brief
 > comment, leading me to wonder if it shouldn't be described in
 > loader.conf(5) manpage instead, but the first paragraph of its
 > "Default Settings" section  makes me wonder.
 >
 > >Fix:
 >
 > Decide whether full description belongs in loader(8), loader.conf(5),
 > or /boot/defaults/loader.conf.  I don't know about loader(8), but I'm
 > thinking a full description should be in loader.conf(5) and referenced
 > from comment in "/boot/defaults/loader.conf".
 
 That seems reasonable.  Duplicating the description is not really useful
 and having the loader options described in loader.conf(5) seems a good
 thing.
 

From: Daniel Gerzo <danger@rulez.sk>
To: bug-followup@FreeBSD.org
Cc:  
Subject: Re: docs/85128: loader.conf(5) autoboot_delay incompletly described
Date: Fri, 30 Dec 2005 23:18:13 +0100

 --C7zPtVaVf+AK4Oqc
 Content-Type: text/plain; charset=us-ascii
 Content-Disposition: inline
 
 Hello,
 
    Included diff should fix your report, it also adds some more
    information to the autoboot_delay when set to -1.
 
    discussed with, and help provided by: ceri@
 
 -- 
 Cheers,
    Daniel Gerzo
 
 --C7zPtVaVf+AK4Oqc
 Content-Type: text/plain; charset=us-ascii
 Content-Disposition: attachment; filename="loader.diff"
 
 diff -ruN loader.orig/loader.8 loader/loader.8
 --- loader.orig/loader.8	Fri Dec 30 21:03:24 2005
 +++ loader/loader.8	Fri Dec 30 21:23:08 2005
 @@ -333,35 +333,8 @@
  Number of seconds
  .Ic autoboot
  will wait before booting.
 -If this variable is not defined,
 -.Ic autoboot
 -will default to 10 seconds.
 -.Pp
 -If set to
 -.Dq Li NO ,
 -no
 -.Ic autoboot
 -will be automatically attempted after processing
 -.Pa /boot/loader.rc ,
 -though explicit
 -.Ic autoboot Ns 's
 -will be processed normally, defaulting to 10 seconds delay.
 -.Pp
 -If set to
 -.Dq Li 0 ,
 -no delay will be inserted, but user still will be able to interrupt
 -.Ic autoboot
 -process and escape into the interactive mode by pressing some key
 -on the console while kernel and
 -modules are being loaded.
 -.Pp
 -If set to
 -.Dq Li -1 ,
 -no delay will be inserted and
 -.Nm
 -will engage interactive mode only if
 -.Ic autoboot
 -has failed for some reason.
 +Configuration options are described in
 +.Xr loader.conf 5 .
  .It Va boot_askname
  Instructs the kernel to prompt the user for the name of the root device
  when the kernel is booted.
 diff -ruN loader.orig/loader.conf loader/loader.conf
 --- loader.orig/loader.conf	Fri Dec 30 21:03:24 2005
 +++ loader/loader.conf	Fri Dec 30 21:15:18 2005
 @@ -41,10 +41,7 @@
  ###  Loader settings  ########################################
  ##############################################################
  
 -#autoboot_delay="10"		# Delay in seconds before autobooting,
 -				# set to -1 if you don't want user to be
 -				# allowed to interrupt autoboot process and
 -				# escape to the loader prompt
 +#autoboot_delay="10"		# Delay in seconds before autobooting
  #beastie_disable="NO"		# Turn the beastie boot menu on and off
  #loader_logo="fbsdbw"		# Desired logo: fbsdbw, beastiebw, beastie, none
  #console="vidconsole"		# A comma separated list of console(s)
 diff -ruN loader.orig/loader.conf.5 loader/loader.conf.5
 --- loader.orig/loader.conf.5	Fri Dec 30 21:03:24 2005
 +++ loader/loader.conf.5	Fri Dec 30 23:10:09 2005
 @@ -90,6 +90,44 @@
  All such settings sharing a common
  prefix refer to the same module.
  .Bl -tag -width Ar
 +.It Ar autoboot_delay
 +Delay in seconds before autobooting.  If set to
 +.Dq Li NO ,
 +no
 +.Ic autoboot
 +will be automatically attempted after processing
 +.Pa /boot/loader.rc ,
 +though explicit
 +.Ic autoboot Ns 's
 +will be processed normally, defaulting to 10 seconds delay.
 +.Pp
 +If set to
 +.Dq Li 0 ,
 +no delay will be inserted, but a user with console access will be able
 +to interrupt the
 +.Ic autoboot
 +process and escape into the interactive mode by pressing some key on
 +the console while kernel and modules are being loaded.
 +.Pp
 +If set to
 +.Dq Li -1 ,
 +no delay will be inserted and
 +.Nm
 +will engage interactive mode only if
 +.Ic autoboot
 +has failed for some reason.
 +In combination with the
 +.Va beastie_disable
 +option, this option prevents users with console access from being able
 +to interrupt the
 +.Ic autoboot
 +process and escape to the loader prompt.
 +To use the
 +.Va autoboot_delay
 +option in this manner,
 +.Va beastie_disable
 +must be set to
 +.Dq Li YES .
  .It Ar exec
  Immediately executes a
  .Xr loader 8
 
 --C7zPtVaVf+AK4Oqc--
>Unformatted:
