From nobody@FreeBSD.org  Sun Jul 17 09:27:59 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 25E34106566C
	for <freebsd-gnats-submit@FreeBSD.org>; Sun, 17 Jul 2011 09:27:59 +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 15B5B8FC08
	for <freebsd-gnats-submit@FreeBSD.org>; Sun, 17 Jul 2011 09:27:59 +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 p6H9RwJD047239
	for <freebsd-gnats-submit@FreeBSD.org>; Sun, 17 Jul 2011 09:27:58 GMT
	(envelope-from nobody@red.freebsd.org)
Received: (from nobody@localhost)
	by red.freebsd.org (8.14.4/8.14.4/Submit) id p6H9RwHW047238;
	Sun, 17 Jul 2011 09:27:58 GMT
	(envelope-from nobody)
Message-Id: <201107170927.p6H9RwHW047238@red.freebsd.org>
Date: Sun, 17 Jul 2011 09:27:58 GMT
From: wayne mitchell <wayne.mitchell.iz@gmail.com>
To: freebsd-gnats-submit@FreeBSD.org
Subject: loader.conf man page
X-Send-Pr-Version: www-3.1
X-GNATS-Notify:

>Number:         158992
>Category:       docs
>Synopsis:       loader.conf(5) man page: examples are incorrect
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    jhb
>State:          closed
>Quarter:        
>Keywords:       
>Date-Required:  
>Class:          doc-bug
>Submitter-Id:   current-users
>Arrival-Date:   Sun Jul 17 09:30:12 UTC 2011
>Closed-Date:    Fri Sep 16 18:19:12 UTC 2011
>Last-Modified:  Fri Sep 16 18:19:12 UTC 2011
>Originator:     wayne mitchell
>Release:        8.1-RELEASE
>Organization:
>Environment:
FreeBSD 8.1-RELEASE FreeBSd 8.1-RELEASE #(date) root@mason.cse.buffalo.edu:/usr/obj/sys/GENERIC amd64
>Description:
man page for loader.conf shows values to set with examples
value for 'kernel' is incorrect
man page shows:

kernel  (''/boot/kernel/kernel'')

this does not work, the system prepends this value with the /boot/kernel/ path value, which would give /boot/kernel//boot/kernel/kernel
not sure what purpose this value has as the system boots with the standard kernel even when this value is incorrect - though it does knock out the module loading 

>How-To-Repeat:
set value in loader.conf as:
kernel="/boot/kernel/kernel"
this should knock out module loading (is all i noticed)
>Fix:
set value in loader.conf as:
kernel="kernel"

and correct the man page

>Release-Note:
>Audit-Trail:

From: John Baldwin <jhb@freebsd.org>
To: freebsd-doc@freebsd.org
Cc: wayne mitchell <wayne.mitchell.iz@gmail.com>,
 freebsd-gnats-submit@freebsd.org
Subject: Re: docs/158992: loader.conf man page
Date: Mon, 18 Jul 2011 13:15:45 -0400

 > this does not work, the system prepends this value with the /boot/kernel/ 
 path value, which would give /boot/kernel//boot/kernel/kernel
 > not sure what purpose this value has as the system boots with the standard 
 kernel even when this value is incorrect - though it does knock out the module 
 loading 
 
 The default value is actually "kernel".  You can set it to, e.g. "test" to 
 boot from /boot/test/kernel (and modules from /boot/test).
 
 How about this:
 
 Index: loader.conf.5
 ===================================================================
 --- loader.conf.5	(revision 224058)
 +++ loader.conf.5	(working copy)
 @@ -107,6 +107,9 @@ present file.
  Name of the kernel to be loaded.
  If no kernel name is set, no additional
  modules will be loaded.
 +The name must be a subdirectory of
 +.Pa /boot
 +that contains a kernel.
  .It Ar kernel_options
  Flags to be passed to the kernel.
  .It Ar password
 @@ -190,7 +193,7 @@ replacing it with
  .Dq spinning
  character (useful for embedded products and such).
  .It Va kernel
 -.Pq Dq Pa /boot/kernel/kernel
 +.Pq Dq kernel
  .It Va loader_conf_files
  .Pq Dq Pa /boot/loader.conf /boot/loader.conf.local
  .It Va splash_bmp_load
 
 -- 
 John Baldwin

From: dfilter@FreeBSD.ORG (dfilter service)
To: bug-followup@FreeBSD.org
Cc:  
Subject: Re: docs/158992: commit references a PR
Date: Wed, 20 Jul 2011 15:18:34 +0000 (UTC)

 Author: jhb
 Date: Wed Jul 20 15:18:24 2011
 New Revision: 224228
 URL: http://svn.freebsd.org/changeset/base/224228
 
 Log:
   Note that the "kernel" variable in loader.conf is the name of a directory
   containing a kernel under /boot and that it's default value is "kernel"
   not "/boot/kernel/kernel".
   
   PR:		docs/158992
   Reported by:	Wayne Mitchell  wayne.mitchell.iz at gmail
   Approved by:	re (kib)
   MFC after:	1 week
 
 Modified:
   head/sys/boot/forth/loader.conf.5
 
 Modified: head/sys/boot/forth/loader.conf.5
 ==============================================================================
 --- head/sys/boot/forth/loader.conf.5	Wed Jul 20 13:29:39 2011	(r224227)
 +++ head/sys/boot/forth/loader.conf.5	Wed Jul 20 15:18:24 2011	(r224228)
 @@ -23,7 +23,7 @@
  .\" SUCH DAMAGE.
  .\"
  .\" $FreeBSD$
 -.Dd January 16, 2008
 +.Dd July 20, 2011
  .Dt LOADER.CONF 5
  .Os
  .Sh NAME
 @@ -107,6 +107,9 @@ present file.
  Name of the kernel to be loaded.
  If no kernel name is set, no additional
  modules will be loaded.
 +The name must be a subdirectory of
 +.Pa /boot
 +that contains a kernel.
  .It Ar kernel_options
  Flags to be passed to the kernel.
  .It Ar password
 @@ -190,7 +193,7 @@ replacing it with
  .Dq spinning
  character (useful for embedded products and such).
  .It Va kernel
 -.Pq Dq Pa /boot/kernel/kernel
 +.Pq Dq kernel
  .It Va loader_conf_files
  .Pq Dq Pa /boot/loader.conf /boot/loader.conf.local
  .It Va splash_bmp_load
 _______________________________________________
 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->patched 
State-Changed-By: jhb 
State-Changed-When: Wed Jul 20 17:32:30 UTC 2011 
State-Changed-Why:  
Fix committed to HEAD. 


Responsible-Changed-From-To: freebsd-doc->jhb 
Responsible-Changed-By: jhb 
Responsible-Changed-When: Wed Jul 20 17:32:30 UTC 2011 
Responsible-Changed-Why:  
Fix committed to HEAD. 

http://www.freebsd.org/cgi/query-pr.cgi?pr=158992 
State-Changed-From-To: patched->closed 
State-Changed-By: jhb 
State-Changed-When: Fri Sep 16 18:18:46 UTC 2011 
State-Changed-Why:  
Merged to 8. 

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