From josh@tcbug.org  Thu Sep 29 22:15:11 2005
Return-Path: <josh@tcbug.org>
Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125])
	by hub.freebsd.org (Postfix) with ESMTP id E677016A41F
	for <FreeBSD-gnats-submit@freebsd.org>; Thu, 29 Sep 2005 22:15:11 +0000 (GMT)
	(envelope-from josh@tcbug.org)
Received: from rwcrmhc11.comcast.net (rwcrmhc11.comcast.net [204.127.198.35])
	by mx1.FreeBSD.org (Postfix) with ESMTP id 948C143D49
	for <FreeBSD-gnats-submit@freebsd.org>; Thu, 29 Sep 2005 22:15:11 +0000 (GMT)
	(envelope-from josh@tcbug.org)
Received: from gimpy.tcbug.org (c-24-118-173-219.hsd1.mn.comcast.net[24.118.173.219])
          by comcast.net (rwcrmhc11) with SMTP
          id <20050929221510013003opk3e>; Thu, 29 Sep 2005 22:15:10 +0000
Message-Id: <1128032161.0@gimpy.tcbug.org>
Date: Thu, 29 Sep 2005 17:16:01 -0500
From: "Josh Paetzel" <josh@tcbug.org>
To: "FreeBSD gnats submit" <FreeBSD-gnats-submit@freebsd.org>
Subject: [patch] Add using kldload as an alternative to compiling in gbde in your kernel
X-Send-Pr-Version: gtk-send-pr 0.4.6 
X-GNATS-Notify:

>Number:         86733
>Category:       docs
>Synopsis:       [patch] handbook: add using kldload as an alternative to compiling in gbde in your kernel
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    trhodes
>State:          closed
>Quarter:        
>Keywords:       
>Date-Required:  
>Class:          doc-bug
>Submitter-Id:   current-users
>Arrival-Date:   Thu Sep 29 22:20:14 GMT 2005
>Closed-Date:    Sat Nov 04 11:05:21 GMT 2006
>Last-Modified:  Sat Nov  4 11:10:18 GMT 2006
>Originator:     Josh Paetzel
>Release:        FreeBSD 5.4-RELEASE-p7 i386
>Organization:
>Environment:


System: FreeBSD 5.4-RELEASE-p7 #8: Fri Sep  9 16:38:25 CDT 2005
    jpaetzel@gimpy.tcbug.org:/usr/obj/usr/src/sys/GIMPY



>Description:


http://www.freebsd.org/doc/en_US.ISO8859-1/books/handbook/disks-encrypting.html explains how to recompile your kernel with options GEOM_BDE but doesn't mention that there is a kernel module (geom_bde) available.


>How-To-Repeat:


Browse http://www.freebsd.org/doc/en_US.ISO8859-1/books/handbook/disks-encrypting.html



>Fix:


--- chapter.sgml.old    Thu Sep 29 16:44:10 2005
+++ chapter.sgml        Thu Sep 29 17:02:02 2005
@@ -3543,6 +3543,25 @@

          <para>Reboot into the new kernel.</para>
        </step>
+
+    <step>
+        <title>Load gbde(4) support using
+        <application>kldload</application>
+        </title>
+
+        <para>An alternative to recompiling your kernel is to use
+            <application>kldload</application> to load the gbde(4)
+            kernel module dynamically.
+            <screen>&prompt.root;
+                <userinput>kldload geom_bde
+                </userinput>
+            </screen>
+        </para>
+
+        <para>At this time there is no option in
+            <filename>/boot/loader.conf</filename>
+            to load the kernel module at boot time
+        </para>
       </procedure>
     </sect2>




>Release-Note:
>Audit-Trail:

From: David Adam <zanchey@ucc.gu.uwa.edu.au>
To: Josh Paetzel <josh@tcbug.org>
Cc: FreeBSD gnats submit <FreeBSD-gnats-submit@FreeBSD.org>,
	freebsd-doc@FreeBSD.org
Subject: Re: docs/86733: [patch] Add using kldload as an alternative to
 compiling in gbde in your kernel
Date: Fri, 30 Sep 2005 09:22:47 +0800 (WST)

 Josh,
 
 > +        <para>At this time there is no option in
 > +            <filename>/boot/loader.conf</filename>
 > +            to load the kernel module at boot time
 > +        </para>
 
 Don't forget the full stop (period) on the end of the sentence!
 
 Otherwise, looks good. Is there really no loader.conf option? That kinda
 sucks, although I can't volunteer to fix that!
 
 David Adam
 zanchey@ucc.gu.uwa.edu.au
 

From: Marc Fonvieille <blackend@FreeBSD.org>
To: Josh Paetzel <josh@tcbug.org>
Cc: FreeBSD gnats submit <FreeBSD-gnats-submit@FreeBSD.org>
Subject: Re: docs/86733: [patch] Add using kldload as an alternative to compiling in gbde in your kernel
Date: Fri, 30 Sep 2005 08:58:50 +0200

 On Thu, Sep 29, 2005 at 05:16:01PM -0500, Josh Paetzel wrote:
 > 
 > 
 > --- chapter.sgml.old    Thu Sep 29 16:44:10 2005
 > +++ chapter.sgml        Thu Sep 29 17:02:02 2005
 > @@ -3543,6 +3543,25 @@
 > 
 >           <para>Reboot into the new kernel.</para>
 >         </step>
 > +
 > +    <step>
 > +        <title>Load gbde(4) support using
 > +        <application>kldload</application>
 > +        </title>
 > +
 > +        <para>An alternative to recompiling your kernel is to use
 > +            <application>kldload</application> to load the gbde(4)
 > +            kernel module dynamically.
 > +            <screen>&prompt.root;
 > +                <userinput>kldload geom_bde
 > +                </userinput>
 > +            </screen>
 > +        </para>
 > +
 > +        <para>At this time there is no option in
 > +            <filename>/boot/loader.conf</filename>
 > +            to load the kernel module at boot time
 > +        </para>
 >        </procedure>
 >      </sect2>
 >
 
 Hello,
 
 Since you submit PRs from time to time, you should have a look to the
 FDP primer; it's impossible for us to use your patch like this (missing
 tags, missing punctuation, mix of inline and block elements, wrong
 "wrap-lines", indentation issues, misuse of SGML tags).
 
 Marc

From: "Simon L. Nielsen" <simon@FreeBSD.org>
To: Josh Paetzel <josh@tcbug.org>
Cc: FreeBSD gnats submit <FreeBSD-gnats-submit@FreeBSD.org>
Subject: Re: docs/86733: [patch] Add using kldload as an alternative to compiling in gbde in your kernel
Date: Fri, 30 Sep 2005 09:54:05 +0200

 --qDbXVdCdHGoSgWSk
 Content-Type: text/plain; charset=us-ascii
 Content-Disposition: inline
 Content-Transfer-Encoding: quoted-printable
 
 On 2005.09.29 17:16:01 -0500, Josh Paetzel wrote:
 >=20
 > +        <para>At this time there is no option in
 > +            <filename>/boot/loader.conf</filename>
 > +            to load the kernel module at boot time
 > +        </para>
 
 Sure about that?  It's been a while since I played with gbde (now it's
 just running) but I'm rather sure 'geom_bde_load=3D"YES"' in loader.conf
 works just fine.
 
 I only have a 7.0 handy that I can reboot, but there it works just
 fine loading gbde from loader.conf.
 
 --=20
 Simon L. Nielsen
 
 --qDbXVdCdHGoSgWSk
 Content-Type: application/pgp-signature
 Content-Disposition: inline
 
 -----BEGIN PGP SIGNATURE-----
 Version: GnuPG v1.4.2 (FreeBSD)
 
 iD8DBQFDPO8ch9pcDSc1mlERAsGYAKDIQI8M0tDxLfL27+hekbUfT1qXwgCgm1lD
 AW3N65IUhM+XkaU+nMF2MsM=
 =kIIu
 -----END PGP SIGNATURE-----
 
 --qDbXVdCdHGoSgWSk--

From: Josh Paetzel <josh@tcbug.org>
To: "Simon L. Nielsen" <simon@freebsd.org>
Cc: FreeBSD gnats submit <FreeBSD-gnats-submit@freebsd.org>
Subject: Re: docs/86733: [patch] Add using kldload as an alternative to compiling in gbde in your kernel
Date: Fri, 30 Sep 2005 09:35:40 -0500

 On Friday 30 September 2005 02:54, Simon L. Nielsen wrote:
 > On 2005.09.29 17:16:01 -0500, Josh Paetzel wrote:
 > > +        <para>At this time there is no option in
 > > +            <filename>/boot/loader.conf</filename>
 > > +            to load the kernel module at boot time
 > > +        </para>
 >
 > Sure about that?  It's been a while since I played with gbde (now
 > it's just running) but I'm rather sure 'geom_bde_load="YES"' in
 > loader.conf works just fine.
 >
 > I only have a 7.0 handy that I can reboot, but there it works just
 > fine loading gbde from loader.conf.
 
 I see geom_bde.ko in /boot/kernel but it's not 
 in /boot/default/loader.conf
 This is on a 5.4 system.
 
 -- 
 Thanks,
 
 Josh Paetzel
State-Changed-From-To: open->closed 
State-Changed-By: trhodes 
State-Changed-When: Sat Nov 4 11:03:03 UTC 2006 
State-Changed-Why:  
Your patch was modified to fit document style and make it 
buildable.  You're right, there is no defaults/loader.conf 
option, so I left that information.  Thanks for the submission. 
In the future, to help your patches get committed, I recommend 
reading over the FDP. 


Responsible-Changed-From-To: freebsd-doc->trhodes 
Responsible-Changed-By: trhodes 
Responsible-Changed-When: Sat Nov 4 11:03:03 UTC 2006 
Responsible-Changed-Why:  
Over to me. 

http://www.freebsd.org/cgi/query-pr.cgi?pr=86733 

From: dfilter@FreeBSD.ORG (dfilter service)
To: bug-followup@FreeBSD.org
Cc:  
Subject: Re: docs/86733: commit references a PR
Date: Sat,  4 Nov 2006 11:03:02 +0000 (UTC)

 trhodes     2006-11-04 11:02:41 UTC
 
   FreeBSD doc repository
 
   Modified files:
     en_US.ISO8859-1/books/handbook/disks chapter.sgml 
   Log:
   Add a note (step) showing that you can load gdbe using kldload(8);
   at this time there is no loader.conf variable in defaults/loader.conf in 6,
   note that.
   Remove some old cruft and reword near sentence to keep flow going smoothly.
   Two spaces between the end of a sentence and the beginning of a new sentence.
   
   PR:             86733
   Submitted by:   Josh Paetzel (patch rewritten by me)
   
   Revision  Changes    Path
   1.266     +16 -5     doc/en_US.ISO8859-1/books/handbook/disks/chapter.sgml
 _______________________________________________
 cvs-all@freebsd.org mailing list
 http://lists.freebsd.org/mailman/listinfo/cvs-all
 To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"
 
>Unformatted:
