From jgabel@thilelli.net  Sat Aug 27 17:43:56 2005
Return-Path: <jgabel@thilelli.net>
Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125])
	by hub.freebsd.org (Postfix) with ESMTP id 70C8416A41F
	for <FreeBSD-gnats-submit@freebsd.org>; Sat, 27 Aug 2005 17:43:56 +0000 (GMT)
	(envelope-from jgabel@thilelli.net)
Received: from smtp.thilelli.net (smtp.thilelli.net [213.41.129.161])
	by mx1.FreeBSD.org (Postfix) with ESMTP id D9E5643D46
	for <FreeBSD-gnats-submit@freebsd.org>; Sat, 27 Aug 2005 17:43:55 +0000 (GMT)
	(envelope-from jgabel@thilelli.net)
Received: from localhost (localhost [127.0.0.1])
	by bento.thilelli.net (Postfix) with ESMTP id 5D7A75C7D
	for <FreeBSD-gnats-submit@freebsd.org>; Sat, 27 Aug 2005 19:43:54 +0200 (CEST)
Received: from bento.thilelli.net ([127.0.0.1])
 by localhost (bento.thilelli.net [127.0.0.1]) (amavisd-new, port 10024)
 with LMTP id 87781-06 for <FreeBSD-gnats-submit@freebsd.org>;
 Sat, 27 Aug 2005 19:43:49 +0200 (CEST)
Received: from titeuf.thilelli.net (titeuf.thilelli.net [192.168.1.20])
	by bento.thilelli.net (Postfix) with ESMTP id 39B755C7C
	for <FreeBSD-gnats-submit@freebsd.org>; Sat, 27 Aug 2005 19:43:49 +0200 (CEST)
Received: by titeuf.thilelli.net (Postfix, from userid 1001)
	id 2AECE73028; Sat, 27 Aug 2005 19:43:49 +0200 (CEST)
Message-Id: <20050827174349.2AECE73028@titeuf.thilelli.net>
Date: Sat, 27 Aug 2005 19:43:49 +0200 (CEST)
From: Julien Gabel <jpeg@thilelli.net>
Reply-To: Julien Gabel <jpeg@thilelli.net>
To: FreeBSD-gnats-submit@freebsd.org
Cc:
Subject: [patch] Make it clear that the extention name of the module isn't needed with kldload(8).
X-Send-Pr-Version: 3.113
X-GNATS-Notify:

>Number:         85356
>Category:       docs
>Synopsis:       [patch] Make it clear that the extention name of the module isn't needed with kldload(8).
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    keramida
>State:          closed
>Quarter:        
>Keywords:       
>Date-Required:  
>Class:          doc-bug
>Submitter-Id:   current-users
>Arrival-Date:   Sat Aug 27 17:50:13 GMT 2005
>Closed-Date:    Sat Aug 27 22:27:29 GMT 2005
>Last-Modified:  Sat Aug 27 22:27:29 GMT 2005
>Originator:     Julien Gabel
>Release:        FreeBSD 5.4-STABLE i386
>Organization:
>Environment:
System: FreeBSD titeuf.thilelli.net 5.4-STABLE FreeBSD 5.4-STABLE #0: Sun Aug 21 14:49:33 CEST 2005 root@titeuf.thilelli.net:/usr/obj/usr/src/sys/TITEUF i386

>Description:
Make it clear that put the extention name of a module is not yet necessary,
as in 'kldload geom_mirror' vs. 'kldload geom_mirror.ko'.

Add more example of automatically loaded modules at boot time.

>How-To-Repeat:
n.a.

>Fix:
Here is a patch:

--- kldload.8.diff begins here ---
--- /usr/src/sbin/kldload/kldload.8	Mon Mar  3 23:48:06 2003
+++ /tmp/kldload.8	Sat Aug 27 19:31:22 2005
@@ -43,6 +43,12 @@
 into the kernel using the kernel linker.
 Note that if multiple modules are specified then an attempt will
 be made to load them all, even if some fail.
+The
+.Pa .ko
+extension name is not yet mandatory when loading a given module
+using
+.Nm .
+It doesn't hurt to keep it though.
 .Pp
 The following option is available:
 .Bl -tag -width indent
@@ -59,8 +65,8 @@
 .Sh DIAGNOSTICS
 .Ex -std
 .Sh AUTOMATICALLY LOADING MODULES
-Some modules (ipfw, ipf, etc.) may be automatically loaded at boot
-time by having their operation enabled in
+Some modules (lomac, pf, ipfw, ipf, etc.) may be automatically loaded
+at boot time by having their operation enabled in
 .Xr rc.conf 5 .
 Modules may also be auto-loaded through their addition to
 .Xr loader.conf 5 .
--- kldload.8.diff ends here ---
>Release-Note:
>Audit-Trail:

From: Giorgos Keramidas <keramida@freebsd.org>
To: Julien Gabel <jpeg@thilelli.net>
Cc: bug-followup@freebsd.org
Subject: Re: docs/85356: [patch] Make it clear that the extention name of the module isn't needed with kldload(8).
Date: Sat, 27 Aug 2005 23:55:26 +0300

 On 2005-08-27 19:43, Julien Gabel <jpeg@thilelli.net> wrote:
 > --- /usr/src/sbin/kldload/kldload.8	Mon Mar  3 23:48:06 2003
 > +++ /tmp/kldload.8	Sat Aug 27 19:31:22 2005
 > @@ -43,6 +43,12 @@
 >  into the kernel using the kernel linker.
 >  Note that if multiple modules are specified then an attempt will
 >  be made to load them all, even if some fail.
 > +The
 > +.Pa .ko
 > +extension name is not yet mandatory when loading a given module
 > +using
 > +.Nm .
 > +It doesn't hurt to keep it though.
 
 Not ``yet''?  You mean there are plans to make it mandatory?
 
 > -Some modules (ipfw, ipf, etc.) may be automatically loaded at boot
 > -time by having their operation enabled in
 > +Some modules (lomac, pf, ipfw, ipf, etc.) may be automatically loaded
 > +at boot time by having their operation enabled in
 >  .Xr rc.conf 5 .
 
 The word ``operation'' is not very nice here.  Modules may be required
 for some ``feature'' but they don't ``operate'' on anything.
 

From: "Julien Gabel" <jpeg@thilelli.net>
To: "Giorgos Keramidas" <keramida@freebsd.org>
Cc: bug-followup@freebsd.org
Subject: Re: docs/85356: [patch] Make it clear that the extention name of 
     the module isn't needed with kldload(8).
Date: Sun, 28 Aug 2005 00:06:10 +0200 (CEST)

 ------=_20050828000610_39119
 Content-Type: text/plain; charset="iso-8859-1"
 Content-Transfer-Encoding: 8bit
 
 >> --- /usr/src/sbin/kldload/kldload.8	Mon Mar  3 23:48:06 2003
 >> +++ /tmp/kldload.8	Sat Aug 27 19:31:22 2005
 >> @@ -43,6 +43,12 @@
 >>  into the kernel using the kernel linker.
 >>  Note that if multiple modules are specified then an attempt will
 >>  be made to load them all, even if some fail.
 >> +The
 >> +.Pa .ko
 >> +extension name is not yet mandatory when loading a given module
 >> +using
 >> +.Nm .
 >> +It doesn't hurt to keep it though.
 
 > Not ``yet''?  You mean there are plans to make it mandatory?
 
 Ouch!  Because i think it was necessary to keep the extension in the
 past, i wanted this to happear in the manual page: so i wanted to say
 "anymore", not "yet"...  sorry for the mistake.  But maybe it is not
 worth noting it.
 
 >> -Some modules (ipfw, ipf, etc.) may be automatically loaded at boot
 >> -time by having their operation enabled in
 >> +Some modules (lomac, pf, ipfw, ipf, etc.) may be automatically loaded
 >> +at boot time by having their operation enabled in
 >>  .Xr rc.conf 5 .
 
 > The word ``operation'' is not very nice here.  Modules may be required
 > for some ``feature'' but they don't ``operate'' on anything.
 
 Ok.  I changed the sentence here, see the new attached diff please.
 
 -- 
 -jpeg.
 ------=_20050828000610_39119
 Content-Type: application/octet-stream; name="kldload.8.diff"
 Content-Transfer-Encoding: base64
 Content-Disposition: attachment; filename="kldload.8.diff"
 
 LS0tIC91c3Ivc3JjL3NiaW4va2xkbG9hZC9rbGRsb2FkLjgJTW9uIE1hciAgMyAyMzo0ODowNiAy
 MDAzCisrKyAvdG1wL2tsZGxvYWQuOAlTdW4gQXVnIDI4IDAwOjAyOjMxIDIwMDUKQEAgLTQzLDYg
 KzQzLDEyIEBACiBpbnRvIHRoZSBrZXJuZWwgdXNpbmcgdGhlIGtlcm5lbCBsaW5rZXIuCiBOb3Rl
 IHRoYXQgaWYgbXVsdGlwbGUgbW9kdWxlcyBhcmUgc3BlY2lmaWVkIHRoZW4gYW4gYXR0ZW1wdCB3
 aWxsCiBiZSBtYWRlIHRvIGxvYWQgdGhlbSBhbGwsIGV2ZW4gaWYgc29tZSBmYWlsLgorVGhlCisu
 UGEgLmtvCitleHRlbnNpb24gbmFtZSBpcyBub3QgbWFuZGF0b3J5IGFueW1vcmUgd2hlbiBsb2Fk
 aW5nIGEgZ2l2ZW4gbW9kdWxlCit1c2luZworLk5tIC4KK0l0IGRvZXNuJ3QgaHVydCB0byBrZWVw
 IGl0IHRob3VnaC4KIC5QcAogVGhlIGZvbGxvd2luZyBvcHRpb24gaXMgYXZhaWxhYmxlOgogLkJs
 IC10YWcgLXdpZHRoIGluZGVudApAQCAtNTksOSArNjUsMTAgQEAKIC5TaCBESUFHTk9TVElDUwog
 LkV4IC1zdGQKIC5TaCBBVVRPTUFUSUNBTExZIExPQURJTkcgTU9EVUxFUwotU29tZSBtb2R1bGVz
 IChpcGZ3LCBpcGYsIGV0Yy4pIG1heSBiZSBhdXRvbWF0aWNhbGx5IGxvYWRlZCBhdCBib290Ci10
 aW1lIGJ5IGhhdmluZyB0aGVpciBvcGVyYXRpb24gZW5hYmxlZCBpbgotLlhyIHJjLmNvbmYgNSAu
 CitTb21lIG1vZHVsZXMgKGxvbWFjLCBwZiwgaXBmdywgaXBmLCBldGMuKSBtYXkgYmUgYXV0b21h
 dGljYWxseSBsb2FkZWQKK2F0IGJvb3QgdGltZSB3aGVuIHRoZSBjb3JyZXNwb25kaW5nCisuWHIg
 cmMuY29uZiA1CitzdGF0ZW1lbnQgaXMgdXNlZC4KIE1vZHVsZXMgbWF5IGFsc28gYmUgYXV0by1s
 b2FkZWQgdGhyb3VnaCB0aGVpciBhZGRpdGlvbiB0bwogLlhyIGxvYWRlci5jb25mIDUgLgogLlNo
 IFNFRSBBTFNPCg==
 ------=_20050828000610_39119--
 
 
State-Changed-From-To: open->closed 
State-Changed-By: keramida 
State-Changed-When: Sat Aug 27 22:26:17 GMT 2005 
State-Changed-Why:  
Committed a _slightly_ modified version of the second patch. 


Responsible-Changed-From-To: freebsd-doc->keramida 
Responsible-Changed-By: keramida 
Responsible-Changed-When: Sat Aug 27 22:26:17 GMT 2005 
Responsible-Changed-Why:  
I'll handle the MFC. 

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