NFSv4: When returning a delegation, don't reclaim an incompatible open mode.
authorNeilBrown <neilb@suse.com>
Mon, 29 Jun 2015 04:28:54 +0000 (14:28 +1000)
committerTrond Myklebust <trond.myklebust@primarydata.com>
Mon, 29 Jun 2015 13:17:13 +0000 (09:17 -0400)
commit39f897fdbd46873defc8501102124f862e8f4d80
tree0c55285eb2b01e4c764a11e7220f21494954ef01
parent6c5a0d891543873aefc3aaf846c1e7afe0982ff9
NFSv4: When returning a delegation, don't reclaim an incompatible open mode.

It is possible to have an active open with one mode, and a delegation
for the same file with a different mode.
In particular, a WR_ONLY open and an RD_ONLY delegation.
This happens if a WR_ONLY open is followed by a RD_ONLY open which
provides a delegation, but is then close.

When returning the delegation, we currently try to claim opens for
every open type (n_rdwr, n_rdonly, n_wronly).  As there is no harm
in claiming an open for a mode that we already have, this is often
simplest.

However if the delegation only provides a subset of the modes that we
currently have open, this will produce an error from the server.

So when claiming open modes prior to returning a delegation, skip the
open request if the mode is not covered by the delegation - the open_stateid
must already cover that mode, so there is nothing to do.

Signed-off-by: NeilBrown <neilb@suse.de>
Signed-off-by: Trond Myklebust <trond.myklebust@primarydata.com>
fs/nfs/nfs4proc.c