Re: getting rid of sys/nfs/nfs_lock.c

From: Dennis Clarke <>
Date: Sun, 29 Dec 2019 01:42:35 -0500
On 12/28/19 7:30 PM, Rick Macklem wrote:
> Hi,
> sys/nfs/nfs_lock.c uses Giant. Since it has not been used by default since
> March 2008, I suspect it can be removed from head without any impact.
> Post March 2008, the only way this code could be executed is by both
> building a kernel without "options NFSLOCKD" and deleting nfslockd.ko
> from the kernel boot directory and then running rpc.lockd on the system.
> I doubt anyone has been doing both of the above, but if you think it is
> still useful, please speak up. (I have an untested patch that replaces Giant
> with a regular mutex. I realized this code is not used when I trying to test it.;-)
> Also, if it seems appropriate, I could commit a patch that makes it print out
> "deprecated and going away before FreeBSD 13" message, but I doubt anyone
> will ever see it.
> Should I do such a message and wait a few months for the deletion?

Such a message is a good idea.

I am curious if there is any way in which we would see that message when
creating an NFS share via ZFS set sharenfs='foo' ?

Dennis Clarke
UNIX and Linux spoken
GreyBeard and suspenders optional
Received on Sun Dec 29 2019 - 05:42:40 UTC

This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:41:22 UTC