On Tue, Nov 15, 2022 at 10:00:43PM +0100, Salvatore Bonaccorso wrote:

Hi,

> > Try something like this in /lib/udev/rules.d/60-nfs.rules:
> > 
> > ACTION=="add", SUBSYSTEM=="module", KERNEL=="sunrpc", \
> >   RUN+="/sbin/sysctl -q --pattern ^sunrpc --system"
> > ACTION=="add", SUBSYSTEM=="module", KERNEL=="rpcrdma", \
> >   RUN+="/sbin/sysctl -q --pattern ^sunrpc.svc_rdma --system"
> > ACTION=="add", SUBSYSTEM=="module", KERNEL=="lockd", \
> >   RUN+="/sbin/sysctl -q --pattern ^fs.nfs.n[sl]m --system"
> > ACTION=="add", SUBSYSTEM=="module", KERNEL=="nfsv4", \
> >   RUN+="/sbin/sysctl -q --pattern 
> > ^fs.nfs.(nfs_callback_tcpport|idmap_cache_timeout) --system"
> > ACTION=="add", SUBSYSTEM=="module", KERNEL=="nfs", \
> >   RUN+="/sbin/sysctl -q --pattern ^fs.nfs --system"
> > 
> > Differently from the original file I tired anchoring the patterns, which 
> > looks more correct to me.
> 
> Thanks, I will try to test this and bring to nfs-utils upstream.
> Michael or Andras, if you can help testing it in your use cases that
> would be helpful.

I don't have a real world test case for this; I don't set any of those sysctls.

Incidentally, busybox sysctl doesn't support --pattern, so if this 
functionality is important, nfs-kernel-server needs to force inclusion of the 
"big" /sbin/sysctl in the initramfs.

AndrĂ¡s

-- 
         To understand recursion, we must first understand recursion.

Reply via email to