svn commit: r339218 - in head: . stand/defaults stand/lua

[Mateusz Piotrowski]

On Sun, 7 Oct 2018 01:53:43 +0000 (UTC) Kyle Evans <kevans at FreeBSD.org> wrote:

>Author: kevans
>Date: Sun Oct  7 01:53:43 2018
>New Revision: 339218
>URL: https://svnweb.freebsd.org/changeset/base/339218
>
>Log:
>  lualoader: Create a module blacklist, add DRM modules to it
>  
>  This is a step in the process of easing migration into the new world order
>  of DRM drivers. Strongly encourage users towards loading DRM modules via
>  rc.conf(5) instead of loader.conf(5) by failing the load from loader(8).
>  Users so inclined may wipe out the blacklist via module_blacklist="" in
>  loader.conf(5), and it is expected that these modules will eventually be
>  removed from the blacklist. They may still be loaded as dependencies of
>  other modules or explicitly via the loader prompt, but this should not be a
>  major problem.
>  
>  Approved by:	re (rgrimes)
>  Relnotes:	yes
>  Differential Revision:	https://reviews.freebsd.org/D16914
>
>Modified:
>  head/UPDATING
>  head/stand/defaults/loader.conf
>  head/stand/defaults/loader.conf.5
>  head/stand/lua/config.lua
>
> [...]
>
>Modified: head/stand/defaults/loader.conf.5
>==============================================================================
>--- head/stand/defaults/loader.conf.5	Sun Oct  7 00:40:56 2018	(r339217)
>+++ head/stand/defaults/loader.conf.5	Sun Oct  7 01:53:43 2018	(r339218)
>@@ -23,7 +23,7 @@
> .\" SUCH DAMAGE.
> .\"
> .\" $FreeBSD$
>-.Dd August 28, 2018
>+.Dd October 6, 2018
> .Dt LOADER.CONF 5
> .Os
> .Sh NAME
>@@ -147,6 +147,15 @@ If a password is set, the user must provide specified 
> If set to
> .Dq YES ,
> module names will be displayed as they are loaded.
>+.It Ar module_blacklist
>+Blacklist of modules.
>+Modules specified in the blacklist may not be loaded automatically with a
>+.Ar *_load
>+directive, but they may be loaded directly at the
>+.Xr loader 8
>+prompt.
>+Blacklisted modules may still be loaded indirectly as dependencies of other
>+moduled.

Do you think that the users might benefit from adding a note
about the default set of modules included in the list?  Something like:

Default: 
.Dq drm drm2 radeonkms i915kms amdgpu .