diff options
| author | Trond Myklebust <Trond.Myklebust@netapp.com> | 2013-03-14 16:57:48 -0400 |
|---|---|---|
| committer | Trond Myklebust <Trond.Myklebust@netapp.com> | 2013-03-25 12:04:10 -0400 |
| commit | 5d422301f97b821301efcdb6fc9d1a83a5c102d6 (patch) | |
| tree | ed7117feb89a0a804669f9f017d2ce3ef7fee10c /fs/nfs/pnfs.c | |
| parent | 3ed5e2a2c394df4e03a680842c2d07a8680f133b (diff) | |
| download | op-kernel-dev-5d422301f97b821301efcdb6fc9d1a83a5c102d6.zip op-kernel-dev-5d422301f97b821301efcdb6fc9d1a83a5c102d6.tar.gz | |
NFSv4: Fail I/O if the state recovery fails irrevocably
If state recovery fails with an ESTALE or a ENOENT, then we shouldn't
keep retrying. Instead, mark the stateid as being invalid and
fail the I/O with an EIO error.
For other operations such as POSIX and BSD file locking, truncate
etc, fail with an EBADF to indicate that this file descriptor is no
longer valid.
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
Diffstat (limited to 'fs/nfs/pnfs.c')
| -rw-r--r-- | fs/nfs/pnfs.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/fs/nfs/pnfs.c b/fs/nfs/pnfs.c index 4bdffe0..c5bd758e 100644 --- a/fs/nfs/pnfs.c +++ b/fs/nfs/pnfs.c @@ -718,6 +718,8 @@ pnfs_choose_layoutget_stateid(nfs4_stateid *dst, struct pnfs_layout_hdr *lo, spin_lock(&lo->plh_inode->i_lock); if (pnfs_layoutgets_blocked(lo, 1)) { status = -EAGAIN; + } else if (!nfs4_valid_open_stateid(open_state)) { + status = -EBADF; } else if (list_empty(&lo->plh_segs)) { int seq; |
