Re: 01/02: services: Allow modprobe to use "/etc/modprobe.d".

[Brice Waegeneire]

Hello Ludo',

On 2020-04-05 21:15, Ludovic Courtès wrote:
> address@hidden skribis:
> >                    #~(begin
> >                        (setenv "LINUX_MODULE_DIRECTORY"
> >                                
> > "/run/booted-system/kernel/lib/modules")
> > +                      ;; FIXME: Remove this crutch when the patch 
> > #40422,
> > +                      ;; updating to kmod 27 is merged.
> > +                      (setenv "MODPROBE_OPTIONS"
> > +                              "-C /etc/modprobe.d")
>
> [...]
>
> > +  (services (cons* (service kernel-module-loader-service-type
> > +                            '("ddcci" "ddcci_backlight"))
> > +                   (simple-service 'ddcci-config etc-service-type
> > +                                   (list `("modprobe.d/ddcci.conf"
> > +                                           ,ddcci-config)))
> > +                   %base-services))
>
> Looking at this, I was wondering if it would be possible to not use
> /etc/modprobe.d and instead have a way to tell the modprobe wrapper to
> pass “-C /gnu/store/…-modprobe.d”, which would contain the right thing.
>
> Thoughts?

What's the issue with using /etc/modrpobe.d?

As noted in the comments I thought setting MODPROBE_OPTIONS was kinda of 
a
hack; #40422[0] was there to fix it. But if you think it's appropriate 
to
use this environment variable it can be done in a future
“kernel-module-configuration-service-type” we discussed with Danny[1].
Instead of extending “etc-service-type” we would use
“activation-service-type”, as “%modprobe-wrapper” is currently put
in place by a simple activation service.

[0]: https://issues.guix.info/issue/40422
[1]: https://issues.guix.info/issue/40274#29

- Brice