-
-
Notifications
You must be signed in to change notification settings - Fork 268
always load modules in /etc/modules (issue905 and others) #909
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
always load modules in /etc/modules (issue905 and others) #909
Conversation
…ted, so that we always load the modules listed in /etc/modules (set by MODULES_LOAD) Issue #905
|
Now it works well for me. In particulat I verified with the additional "debug" kernel I just merge it because I think it is now at least |
|
There is a small thing left where I don't know At the end of 40-start-udev-or-load-modules.sh RESCUE f121:~ # lsmod | grep dm dm_mod 110780 0 I only assume this modprobe call was also meant |
|
git forensic via git log -p usr/share/rear/skel/default/etc/scripts/system-setup.d/40-start-udev-or-load-modules.sh shows that the unconditioned "modprobe -q dm-mod" call is there FYI: git blame -w usr/share/rear/skel/default/etc/scripts/system-setup.d/40-start-udev-or-load-modules.sh shows it was this git commit: 844d50b |
|
@schlomo I think now I understand the reason for my confusion in In gereral I noticed at that time that in the rear recovery system I had more than 100 kernel modules loaded while in my original system I had something about 40 kernel modules loaded. From my point of view this indicates that there is something wrong in general in the rear recovery system that it has much more kernel modules loaded than in the original system... I don't know a mandatory reason why the explicitly specified modules But I could imagine a reason why the explicitly specified modules If the explicitly specified modules were loaded before the In contrast when the udev automatism runs first I think in theory the module dependencies should Currently usr/share/rear/conf/default.conf # Enforce to load these modules in the given order in the rescue/recovery system: MODULES_LOAD=() Currently it only means that MODULES_LOAD=( foo bar ) I think to make MODULES_LOAD really powerful Furthermore I would like to have a switch so that the MODULES_LOAD=( 'foo' 'bar' 'skip_automated_udev_module_loading' ) but this makes implementation a bit awkward because I Because I think I know now the root cause why the loaded |
|
@jsmeix please wait with your effort till I am done with my changes in https://github.com/rear/rear/tree/schlomo-2017-06 (and feel free to look there and comment). Maybe you don't need to do anything at all. I already took out one sort which should fix the ordering. In my experience modules auto-load their dependencies without problems. In this example I had I would really prefer to keep things simple here. That means that we use I am actually much more worried about us supporting systemd-modules-load.service properly. Do we start that? Does systemd start it when we copy it into the rescue system? Unfortunately a quick look did not enlighten me on these questions, our boot process is already too complex. FYI, on my Ubuntu 17.04 |
|
I am afraid I know basically nothing at all about I fully agree to load the MODULES_LOAD modules I watch your changes in |
|
@schlomo if ! grep -q $module_to_be_loaded $recovery_system_etc_modules ; then should be more specific to avoid that substrings match, |
|
excellent catch - thanks a lot! (please ignore the last commit there, it was wrong and I pulled it back). |
Tries to fix 0ec8716
by changing 40-start-udev-or-load-modules.sh
so taht the code that always loads modules
in /etc/modules is always run.