Date: Tue, 13 Jan 2009 22:00:31 +0000 From: Frank Shute <frank@shute.org.uk> To: Ben Kaduk <kaduk@mit.edu> Cc: freebsd-doc@freebsd.org Subject: Re: docs/130482: misleading loader documentation in handbook Message-ID: <20090113220031.GA20634@melon.esperance-linux.co.uk> In-Reply-To: <200901130225.n0D2PRJX015696@www.freebsd.org> References: <200901130225.n0D2PRJX015696@www.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Tue, Jan 13, 2009 at 02:25:27AM +0000, Ben Kaduk wrote: > > > >Number: 130482 > >Category: docs > >Synopsis: misleading loader documentation in handbook > >Confidential: no > >Severity: non-critical > >Priority: low > >Responsible: freebsd-doc > >State: open > >Quarter: > >Keywords: > >Date-Required: > >Class: doc-bug > >Submitter-Id: current-users > >Arrival-Date: Tue Jan 13 02:30:03 UTC 2009 > >Closed-Date: > >Last-Modified: > >Originator: Ben Kaduk > >Release: 7.1-PRERELEASE > >Organization: > MIT SIPB > >Environment: > FreeBSD periphrasis.mit.edu 7.1-PRERELEASE FreeBSD 7.1-PRERELEASE #5: Sat Jan 3 17:44:27 EST 2009 kadku@periphrasis.mit.edu:/usr/obj/usr/src/sys/PERIPHRASIS amd64 > >Description: > The handbook chapter on the Boot Manager and boot stages includes section 12.3.3.2 describing the loader built-in commands. The description for the 'boot' command describes the optional kernelname parameter, but does not mention that this parameter is overrided if there is already a kernel image loaded. > (The correct procedure to boot a different kernel is mentioned later in the page, though.) > >How-To-Repeat: > http://www.freebsd.org/doc/en_US.ISO8859-1/books/handbook/boot-blocks.html > >Fix: > The patch available here is also inlined below: > http://stuff.mit.edu/afs/sipb.mit.edu/user/kaduk/freebsd/patches/chapter.boot.diff.20090112 > > > --------------- begin patch here -------------- > --- chapter.sgml.orig Mon Jan 12 21:13:56 2009 > +++ chapter.sgml Mon Jan 12 21:18:37 2009 > @@ -343,7 +343,10 @@ > <listitem> > <para>Immediately proceeds to boot the kernel, with the > given options, if any, and with the kernel name given, > - if it is.</para> > + if it is. Providing a kernel name on the command-line > + is only applicable after an <term>unlad</term> command > + has been issued, otherwise the previously-loaded kernel > + will be used.</para> > </listitem> > </varlistentry> > > > >Release-Note: > >Audit-Trail: > >Unformatted: > _______________________________________________ Thought I'd just point out the inlined patch has a typo in it (s/unlad/unload). The one at the URL you gave seems OK though. Regards, -- Frank Contact info: http://www.shute.org.uk/misc/contact.html
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20090113220031.GA20634>