| Summary: | Path for modprobe.d and modconf.d should be set through variables. | ||
|---|---|---|---|
| Product: | [Build System, Metadata & Runtime] OE-Core | Reporter: | Ola Nilsson <ola.x.nilsson> |
| Component: | configuration | Assignee: | Ola Nilsson <ola.x.nilsson> |
| Status: | RESOLVED FIXED | QA Contact: | |
| Severity: | enhancement | ||
| Priority: | Medium | CC: | quaresma.jose, randy.macleod |
| Version: | unspecified | ||
| Target Milestone: | Future | ||
| Hardware: | All | ||
| OS: | Multiple | ||
| Whiteboard: | |||
| OS type for building Yocto: | --- | Type of Regression: | --- |
| Verified: | Documentation change: | Don't know | |
|
Description
Ola Nilsson
2017-10-10 09:41:50 UTC
I have no objection to this in principle however I do wonder what assumptions the underlying software has about these paths. Its one thing to make it configurable, its another whether all of the software using the paths responds to the configuration options. I won't want to do this if it meant carrying a lot of patches to software with no hope of upstreaming them as I don't see this as an important enough issue based on the information here. systemd loads modules according to files in /etc/modules-load.d/*.conf /run/modules-load.d/*.conf /usr/lib/modules-load.d/*.conf according to 'man modules-load.d' modprobe uses conf files in /lib/modprobe.d/*.conf /etc/modprobe.d/*.conf /run/modprobe.d/*.conf according to 'man modprobe.d'. I'm not sure what other software you are thinking about. We put our files in /usr and /lib (read-only on our products) and it is possible to handle that with the current kernel.bbclass and kernel-modules.bbclass. But when we want to use the usrmerge feature it requires a rather ugly hack to split_module_packages. Ola, Are you able to send a patch? I guess it's too late for the zeus release, so there is no hurry? Then yes, I can prepare a patch. Eventually :) Thanks Ola. It's possible that the change could be backported to zeus if it's not too disruptive or risky. Any update Ola? Sorry, there is always something more pressing to do... I still want to add this functionality. I have submit a patch for this [2] but because it looks like busybox don't support other path than /etc [1] so the implementation only relocate the default /etc location when systemd is enabled [3]. [1] https://git.busybox.net/busybox/tree/modutils/modprobe.c?id=669c40ed8ebf480c95ce36135104e474e361a7e6#n658 [2] https://lists.openembedded.org/g/openembedded-core/message/177186 [3] https://lists.openembedded.org/g/openembedded-core/message/183123 This bug should be fixed with [1] and [2] also adds systemd configurations. [1] https://git.yoctoproject.org/poky/commit/?id=303421dce02fec67daab0dc86d85e11963d5741f [2] https://lists.openembedded.org/g/openembedded-core/message/183823 This bug should be fixed with [1] and [2] also adds systemd configurations. [1] https://git.yoctoproject.org/poky/commit/?id=303421dce02fec67daab0dc86d85e11963d5741f [2] https://git.yoctoproject.org/poky/commit/?id=6704606b351c4d570cce96ee9afb38c565829790 |