mbox series

[v2,0/4] Replace sysctl setting invocations triggered from udev rule instead of modprobe configuration

Message ID 20221125140656.1985137-1-carnil@debian.org (mailing list archive)
Headers show
Series Replace sysctl setting invocations triggered from udev rule instead of modprobe configuration | expand

Message

Salvatore Bonaccorso Nov. 25, 2022, 2:06 p.m. UTC
Hi Neil, hi Steve

In Debian for the update including the systemd/50-nfs.conf there was a
report that sunrpc is not included anymore in the initrd through the
initramfs-tools hooks.

The report is at https://bugs.debian.org/1022172

Marco d'Itri suggested there three possible solutions, of which one
could be done in nfs-utils (whereas the other two are either in kmod
upstream or initramfs-tools upstream). The nfs-utils one would be to
replace the modprobe configuration with a set of udev rules instead.

This series reverts the commits which introduce the use of the modprobe
configuration and instead replaces it with an udev rule which triggers
setting the sysctl settings when the respective modules are loaded.

Regards,
Salvatore

Changes:
--------
v2:
 - Fix a series of spelling and typos in the commit messages.

Salvatore Bonaccorso (4):
  Revert "configure: make modprobe.d directory configurable."
  Revert "modprobe: protect against sysctl errors"
  Revert "systemd: Apply all sysctl settings when NFS-related modules
    are loaded"
  systemd: Apply all sysctl settings through udev rule when NFS-related
    modules are loaded

 configure.ac         | 12 ------------
 systemd/50-nfs.conf  | 16 ----------------
 systemd/60-nfs.rules | 21 +++++++++++++++++++++
 systemd/Makefile.am  | 15 ++++++---------
 4 files changed, 27 insertions(+), 37 deletions(-)
 delete mode 100644 systemd/50-nfs.conf
 create mode 100644 systemd/60-nfs.rules

Comments

Steve Dickson Dec. 6, 2022, 1:32 p.m. UTC | #1
On 11/25/22 9:06 AM, Salvatore Bonaccorso wrote:
> Hi Neil, hi Steve
> 
> In Debian for the update including the systemd/50-nfs.conf there was a
> report that sunrpc is not included anymore in the initrd through the
> initramfs-tools hooks.
> 
> The report is at https://bugs.debian.org/1022172
> 
> Marco d'Itri suggested there three possible solutions, of which one
> could be done in nfs-utils (whereas the other two are either in kmod
> upstream or initramfs-tools upstream). The nfs-utils one would be to
> replace the modprobe configuration with a set of udev rules instead.
> 
> This series reverts the commits which introduce the use of the modprobe
> configuration and instead replaces it with an udev rule which triggers
> setting the sysctl settings when the respective modules are loaded.
> 
> Regards,
> Salvatore
> 
> Changes:
> --------
> v2:
>   - Fix a series of spelling and typos in the commit messages.
Committed... (tag: nfs-utils-2-6-3-rc5)

steved.
> 
> Salvatore Bonaccorso (4):
>    Revert "configure: make modprobe.d directory configurable."
>    Revert "modprobe: protect against sysctl errors"
>    Revert "systemd: Apply all sysctl settings when NFS-related modules
>      are loaded"
>    systemd: Apply all sysctl settings through udev rule when NFS-related
>      modules are loaded
> 
>   configure.ac         | 12 ------------
>   systemd/50-nfs.conf  | 16 ----------------
>   systemd/60-nfs.rules | 21 +++++++++++++++++++++
>   systemd/Makefile.am  | 15 ++++++---------
>   4 files changed, 27 insertions(+), 37 deletions(-)
>   delete mode 100644 systemd/50-nfs.conf
>   create mode 100644 systemd/60-nfs.rules
>