Load modules from /etc/modules#131
Open
fhunleth wants to merge 1 commit into
Open
Conversation
Auto-load modules listed in the `/etc/modules` file. This makes it possible for systems to specify (and users to override) the load of kernel modules that may not be detectable via `uevent` messages. Previously these modules were compiled into the kernel, but this does not support cases where they could be optional. For example, it would be nice to have a few USB gadget driver options without hardcoding them into the kernel. This lets systems be made where the default case loads a Ethernet/serial gadget driver (for example), but that can be disabled by overriding `/etc/modules` with a different set of drivers.
fhunleth
force-pushed
the
load-etc-modules
branch
from
04c0866 to
f369212
Compare
Contributor
|
I presume this would enable this to work dynamically: nerves-project/nerves_system_rpi0#62 |
ConnorRigby
approved these changes
Mar 25, 2019
Member
Author
|
FYI - there's some out-of-band discussion on how this is handled. E.g., by file in |
Contributor
|
I'm curious if there's any update / recap on the discussion/trade-offs because I lost track of it and assumed it was already decided/merged. |
Member
Author
|
It hasn't been resolved. Justin got busy and I haven't prototyped an Application config version so that we can properly compare options. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Intended to be merged after #130.
Auto-load modules listed in the
/etc/modulesfile. This makes itpossible for systems to specify (and users to override) the load of
kernel modules that may not be detectable via
ueventmessages.Previously these modules were compiled into the kernel, but this does
not support cases where they could be optional. For example, it would
be nice to have a few USB gadget driver options without hardcoding them
into the kernel. This lets systems be made where the default case loads
a Ethernet/serial gadget driver (for example), but that can be disabled
by overriding
/etc/moduleswith a different set of drivers.