From nobody@FreeBSD.org  Mon Aug 15 01:02:05 2011
Return-Path: <nobody@FreeBSD.org>
Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34])
	by hub.freebsd.org (Postfix) with ESMTP id 1EB59106567E
	for <freebsd-gnats-submit@FreeBSD.org>; Mon, 15 Aug 2011 01:02:05 +0000 (UTC)
	(envelope-from nobody@FreeBSD.org)
Received: from red.freebsd.org (red.freebsd.org [IPv6:2001:4f8:fff6::22])
	by mx1.freebsd.org (Postfix) with ESMTP id 0E5B68FC08
	for <freebsd-gnats-submit@FreeBSD.org>; Mon, 15 Aug 2011 01:02:05 +0000 (UTC)
Received: from red.freebsd.org (localhost [127.0.0.1])
	by red.freebsd.org (8.14.4/8.14.4) with ESMTP id p7F124xa031653
	for <freebsd-gnats-submit@FreeBSD.org>; Mon, 15 Aug 2011 01:02:04 GMT
	(envelope-from nobody@red.freebsd.org)
Received: (from nobody@localhost)
	by red.freebsd.org (8.14.4/8.14.4/Submit) id p7F124Zr031652;
	Mon, 15 Aug 2011 01:02:04 GMT
	(envelope-from nobody)
Message-Id: <201108150102.p7F124Zr031652@red.freebsd.org>
Date: Mon, 15 Aug 2011 01:02:04 GMT
From: Colin Percival <cperciva@FreeBSD.org>
To: freebsd-gnats-submit@FreeBSD.org
Subject: handling of autoboot_delay="-1" has changed in 9.0
X-Send-Pr-Version: www-3.1
X-GNATS-Notify:

>Number:         159775
>Category:       conf
>Synopsis:       [boot] handling of autoboot_delay="-1" has changed in 9.0
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    freebsd-bugs
>State:          closed
>Quarter:        
>Keywords:       
>Date-Required:  
>Class:          sw-bug
>Submitter-Id:   current-users
>Arrival-Date:   Mon Aug 15 01:10:08 UTC 2011
>Closed-Date:    Fri Sep 02 19:38:54 UTC 2011
>Last-Modified:  Fri Sep 02 19:38:54 UTC 2011
>Originator:     Colin Percival
>Release:        9.0-BETA1
>Organization:
>Environment:
FreeBSD 9.0-BETA1
>Description:
The handling of autoboot_delay="-1" has changed with the new boot loader scripts.  

In /boot/defaults/loader.conf, the old behaviour is documented:
#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, set to
                                # "NO" to disable autobooting

In the menu.4th man page, the new behaviour is documented:
           autoboot_delay
                     Number of seconds menu-display will wait before executing
                     menu_timeout_command (boot by default) unless a key is
                     pressed.  If set to ``NO'' (case-insensitive) or ``-1'',
                     menu-display will wait for user input and never execute
                     menu_timeout_command.  Default is ``10''.  See loader(8)
                     for additional information.

Note that "user can't interrupt boot process" and "menu-display will never execute menu_timeout_command" are nearly exact opposite behaviours.
>How-To-Repeat:
Set autoboot_delay="-1" on 9.0-BETA1.  Be astonished by what happens.
>Fix:
Either change menu.4th to restore the old behaviour, or update the comments in /boot/defaults/loader.conf (and possibly add an entry to the release notes to warn users).

>Release-Note:
>Audit-Trail:

From: dfilter@FreeBSD.ORG (dfilter service)
To: bug-followup@FreeBSD.org
Cc:  
Subject: Re: conf/159775: commit references a PR
Date: Fri,  2 Sep 2011 19:29:48 +0000 (UTC)

 Author: jh
 Date: Fri Sep  2 19:29:37 2011
 New Revision: 225353
 URL: http://svn.freebsd.org/changeset/base/225353
 
 Log:
   Restore behavior of the autoboot_delay="-1" boot menu setting to the
   pre-r222417 state. The behavior was essentially reversed in r222417
   which can cause confusion.
   
   PR:		159775
   Submitted by:	Devin Teske
   Approved by:	re (kib)
 
 Modified:
   head/sys/boot/forth/menu.4th
   head/sys/boot/forth/menu.4th.8
 
 Modified: head/sys/boot/forth/menu.4th
 ==============================================================================
 --- head/sys/boot/forth/menu.4th	Fri Sep  2 18:52:28 2011	(r225352)
 +++ head/sys/boot/forth/menu.4th	Fri Sep  2 19:29:37 2011	(r225353)
 @@ -742,11 +742,12 @@ create init_text8 255 allot
  			else
  				-rot 2drop
  
 -				\ disable timeout if less than zero
 +				\ boot immediately if less than zero
  				dup 0< if
  					drop
 -					0 menu_timeout_enabled !
 -					0 ( assigned to menu_timeout below )
 +					menu-create
 +					0 25 at-xy
 +					0 boot
  				then
  			then
  		then
 
 Modified: head/sys/boot/forth/menu.4th.8
 ==============================================================================
 --- head/sys/boot/forth/menu.4th.8	Fri Sep  2 18:52:28 2011	(r225352)
 +++ head/sys/boot/forth/menu.4th.8	Fri Sep  2 19:29:37 2011	(r225353)
 @@ -24,7 +24,7 @@
  .\"
  .\" $FreeBSD$
  .\"
 -.Dd May 18, 2011
 +.Dd Aug 29, 2011
  .Dt MENU.4TH 8
  .Os
  .Sh NAME
 @@ -96,11 +96,15 @@ will wait before executing
  by default) unless a key is pressed.
  If set to
  .Dq Li NO
 -(case-insensitive) or
 -.Dq Li -1 ,
 +(case-insensitive)
  .Ic menu-display
  will wait for user input and never execute
  .Ic menu_timeout_command .
 +If set to
 +.Dq Li -1 ,
 +.Ic menu-display
 +will boot immediately, preventing both interruption of the autoboot process and
 +escaping to the loader prompt.
  Default is
  .Dq Li 10 .
  See
 _______________________________________________
 svn-src-all@freebsd.org mailing list
 http://lists.freebsd.org/mailman/listinfo/svn-src-all
 To unsubscribe, send any mail to "svn-src-all-unsubscribe@freebsd.org"
 
State-Changed-From-To: open->closed 
State-Changed-By: jh 
State-Changed-When: Fri Sep 2 19:38:53 UTC 2011 
State-Changed-Why:  
Fixed in r225353. 

http://www.freebsd.org/cgi/query-pr.cgi?pr=159775 
>Unformatted:
