Bug 85356

Summary: [patch] Make it clear that the extention name of the module isn't needed with kldload(8).
Product: Documentation Reporter: Julien Gabel <jpeg>
Component: Books & ArticlesAssignee: Giorgos Keramidas <keramida>
Status: Closed FIXED    
Severity: Affects Only Me    
Priority: Normal    
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
kldload.8.diff
none
kldload.8.diff none

Description Julien Gabel 2005-08-27 18:50:13 UTC 
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.

Fix: Here is a patch:
How-To-Repeat: n.a.
Comment 1 Giorgos Keramidas freebsd_committer freebsd_triage 2005-08-27 21:55:26 UTC 
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.
Comment 2 Julien Gabel 2005-08-27 23:06:10 UTC 
>> --- /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.
Comment 3 Giorgos Keramidas freebsd_committer freebsd_triage 2005-08-27 23:26:17 UTC 
State Changed
From-To: open->closed

Committed a _slightly_ modified version of the second patch.
Comment 4 Giorgos Keramidas freebsd_committer freebsd_triage 2005-08-27 23:26:17 UTC 
Responsible Changed
From-To: freebsd-doc->keramida

I'll handle the MFC.