From: Steve Dickson <steved@redhat.com>
To: Salvatore Bonaccorso <carnil@debian.org>,
NeilBrown <neilb@suse.de>,
linux-nfs@vger.kernel.org
Cc: Andras Korn <korn-debbugs@elan.rulez.org>,
Marco d'Itri <md@linux.it>, Michael Prokop <mika@debian.org>
Subject: Re: [PATCH v2 0/4] Replace sysctl setting invocations triggered from udev rule instead of modprobe configuration
Date: Tue, 6 Dec 2022 08:32:24 -0500 [thread overview]
Message-ID: <b4474890-4d36-730c-af29-2d4541a9c362@redhat.com> (raw)
In-Reply-To: <20221125140656.1985137-1-carnil@debian.org>
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
>
prev parent reply other threads:[~2022-12-06 13:33 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-11-25 14:06 [PATCH v2 0/4] Replace sysctl setting invocations triggered from udev rule instead of modprobe configuration Salvatore Bonaccorso
2022-11-25 14:06 ` [PATCH v2 1/4] Revert "configure: make modprobe.d directory configurable." Salvatore Bonaccorso
2022-11-25 14:06 ` [PATCH v2 2/4] Revert "modprobe: protect against sysctl errors" Salvatore Bonaccorso
2022-11-25 14:06 ` [PATCH v2 3/4] Revert "systemd: Apply all sysctl settings when NFS-related modules are loaded" Salvatore Bonaccorso
2022-11-25 14:06 ` [PATCH v2 4/4] systemd: Apply all sysctl settings through udev rule when NFS-related modules are loaded Salvatore Bonaccorso
2022-12-06 13:32 ` Steve Dickson [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=b4474890-4d36-730c-af29-2d4541a9c362@redhat.com \
--to=steved@redhat.com \
--cc=carnil@debian.org \
--cc=korn-debbugs@elan.rulez.org \
--cc=linux-nfs@vger.kernel.org \
--cc=md@linux.it \
--cc=mika@debian.org \
--cc=neilb@suse.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.