]> git.proxmox.com Git - mirror_ubuntu-kernels.git/commit
NFSv4.1/pnfs: Fix a close/delegreturn hang when return-on-close is set
authorTrond Myklebust <trond.myklebust@primarydata.com>
Wed, 19 Aug 2015 04:23:21 +0000 (23:23 -0500)
committerTrond Myklebust <trond.myklebust@primarydata.com>
Wed, 19 Aug 2015 04:23:21 +0000 (23:23 -0500)
commit4ff376feaf57af94e08c8df769e7c48b805ac897
tree00c24b5f282e6658fb0639f3b3069991ccd99013
parent7e94d6c4ab69562423fdc18801050a84462c2787
NFSv4.1/pnfs: Fix a close/delegreturn hang when return-on-close is set

The helper pnfs_roc() has already verified that we have no delegations,
and no further open files, hence no outstanding I/O and it has marked
all the return-on-close lsegs as being invalid.
Furthermore, it sets the NFS_LAYOUT_RETURN bit, thus serialising the
close/delegreturn with all future layoutget calls on this inode.

The checks in pnfs_roc_drain() for valid layout segments are therefore
redundant: those cannot exist until another layoutget completes.
The other check for whether or not NFS_LAYOUT_RETURN is set, actually
causes a hang, since we already know that we hold that flag.

To fix, we therefore strip out all the functionality in pnfs_roc_drain()
except the retrieval of the barrier state, and then rename the function
accordingly.

Reported-by: Christoph Hellwig <hch@infradead.org>
Fixes: 5c4a79fb2b1c ("Don't prevent layoutgets when doing return-on-close")
Signed-off-by: Trond Myklebust <trond.myklebust@primarydata.com>
fs/nfs/nfs4proc.c
fs/nfs/pnfs.c
fs/nfs/pnfs.h