nfsd4: do not throw away 4.1 lock state on last unlock
authorJ. Bruce Fields <bfields@redhat.com>
Mon, 17 Jun 2013 21:29:40 +0000 (17:29 -0400)
committerJ. Bruce Fields <bfields@redhat.com>
Mon, 1 Jul 2013 21:32:06 +0000 (17:32 -0400)
commit0a262ffb75275e48caa17e80b96504354e94f6c2
tree5f75b398e0ea586e6d1664cf817072d566fe100b
parent89f6c3362cb5a6bce96dbe6aa15b4749c2262b21
nfsd4: do not throw away 4.1 lock state on last unlock

This reverts commit eb2099f31b0f090684a64ef8df44a30ff7c45fc2 "nfsd4:
release lockowners on last unlock in 4.1 case".  Trond identified
language in rfc 5661 section 8.2.4 which forbids this behavior:

Stateids associated with byte-range locks are an exception.
They remain valid even if a LOCKU frees all remaining locks, so
long as the open file with which they are associated remains
open, unless the client frees the stateids via the FREE_STATEID
operation.

And bakeathon 2013 testing found a 4.1 freebsd client was getting an
incorrect BAD_STATEID return from a FREE_STATEID in the above situation
and then failing.

The spec language honestly was probably a mistake but at this point with
implementations already following it we're probably stuck with that.

Signed-off-by: J. Bruce Fields <bfields@redhat.com>
fs/nfsd/nfs4state.c