panic excl->shared for an AF_LOCAL socket

Rick Macklem rmacklem at uoguelph.ca
Wed Aug 22 00:01:59 UTC 2018


Matthew Macy wrote:
[stuff snipped]
>I don't know what's special in this case, but I did revamp the locking there several >months back so I'll take a look next weekend.

Thanks but don't worry about it for now. I think I figured out how the panic()
occurred. If the nfsd was accessing /var/run/nfsuserd.sock for a client and then
tried to soconnect() to it to do the upcall, the nfsd thread would already have
/var/run/nfsuserd.sock vnode locked.

The old way (and what FreeBSD-11 still does) was to use a UDP socket, which
isn't in the file system namespace. (I switched the default to AF_LOCAL so that
nfsuserd could be used in jails where 127.0.0.1 doesn't work, but I now think
it isn't safe to use an AF_LOCAL socket, since it is in the file system's namespace
and, therefore, can be accessed directly by the NFS code.

I think I'll revert the "switch to AF_LOCAL socket" patch.

Hopefully the reporter can help confirm this "theory", rick




More information about the freebsd-current mailing list