NLM: Always start both UDP and TCP listeners
authorChuck Lever <chuck.lever@oracle.com>
Fri, 3 Oct 2008 21:15:23 +0000 (17:15 -0400)
committerJ. Bruce Fields <bfields@citi.umich.edu>
Sat, 4 Oct 2008 21:08:16 +0000 (17:08 -0400)
commit8c3916f4bdf9c8388bd70d0b399b3a43daf2087a
tree738c44f56265c6fbca20ce8c7ec8aa655ce56803
parent9a38a83880c224c6a3fd973ac9ae30a043487f0f
NLM: Always start both UDP and TCP listeners

Commit 24e36663, which first appeared in 2.6.19, changed lockd so that
the client side starts a UDP listener only if there is a UDP NFSv2/v3
mount.  Its description notes:

    This... means that lockd will *not* listen on UDP if the only
    mounts are TCP mount (and nfsd hasn't started).

    The latter is the only one that concerns me at all - I don't know
    if this might be a problem with some servers.

Unfortunately it is a problem for Linux itself.  The rpc.statd daemon
on Linux uses UDP for contacting the local lockd, no matter which
protocol is used for NFS mounts.  Without a local lockd UDP listener,
NFSv2/v3 lock recovery from Linux NFS clients always fails.

Revert parts of commit 24e36663 so lockd_up() always starts both
listeners.

Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Cc: Neil Brown <neilb@suse.de>
Signed-off-by: J. Bruce Fields <bfields@citi.umich.edu>
fs/lockd/svc.c