Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 24 Dec 2018 13:07:54 +0000
From:      Chris Rees <chris@rees.space>
To:        Chris Rees <crees@bayofrum.net>, Konstantin Belousov <kostikbel@gmail.com>, freebsd-rc@freebsd.org
Cc:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   Re: svn commit: r342389 - head/share/man/man5
Message-ID:  <dd115035-34c1-b73a-1ea5-f108407bda8d@rees.space>
In-Reply-To: <9f786428-7fea-4fa4-a29e-ed91997a87fd@email.android.com>
References:  <9f786428-7fea-4fa4-a29e-ed91997a87fd@email.android.com>

next in thread | previous in thread | raw e-mail | index | archive | help
Hi again,

On 24/12/2018 11:23, Chris Rees wrote:
> Hi Konstantin,
>
> On 24 Dec 2018 11:17, Konstantin Belousov <kostikbel@gmail.com> wrote:
>
>     On Mon, Dec 24, 2018 at 10:47:48AM +0000, Chris Rees wrote:
>     > Author: crees (doc,ports committer)
>     > Date: Mon Dec 24 10:47:48 2018
>     > New Revision: 342389
>     > URL: https://svnweb.freebsd.org/changeset/base/342389
>     >
>     > Log:
>     >   Clarify kld_list format
>     >  
>     >   PR: docs/234248
>     >   Submitted by: David Fiander
>     >   Submitted by: Miroslav Lachman
>     >
>     > Modified:
>     >   head/share/man/man5/rc.conf.5
>     >
>     > Modified: head/share/man/man5/rc.conf.5
>     >
>     ==============================================================================
>     > --- head/share/man/man5/rc.conf.5 Mon Dec 24 06:14:32 2018
>     (r342388)
>     > +++ head/share/man/man5/rc.conf.5 Mon Dec 24 10:47:48 2018
>     (r342389)
>     > @@ -248,12 +248,14 @@ Default
>     >  .Pa /etc/ddb.conf .
>     >  .It Va kld_list
>     >  .Pq Vt str
>     > -A list of kernel modules to load right after the local
>     > -disks are mounted.
>     > +A whitespace-separated list of kernel modules to load right after
>     > +the local disks are mounted, without any
>     > +.Pa .ko
>     > +extension or path.
>     I think both extension and path are accepted if supplied.
>     It is the behaviour described in kldload(8).
>
>
> That's true, but the kld rc script adds .ko, so providing the
> extension will probably break, and it checks for existing modules
> using the provided name as a regex, so that will also fail.
>
> I don't think that'd be hard to fix though, so I'll fix that and put a
> patch up for review later.

Having looked again, rc.subr uses kldstat -v, so the path is indeed not
a problem, but the extension is-- removing any extension from _kld will
ensure that it will always match correctly.  At the moment it is
fragile, because it will load correctly the first time but hit an error
if the user has put the extension in and the module is already loaded.

@RC people, does this look acceptable (I'll need approval please)?

https://www.bayofrum.net/~crees/patches/rc-kld_list-extension.diff

Chris


-- 
This message has been scanned for viruses and
dangerous content by MailScanner, and is
believed to be clean.




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?dd115035-34c1-b73a-1ea5-f108407bda8d>