diff options
| author |   J. Bruce Fields <bfields@redhat.com> | 2021-02-24 13:39:50 -0500 |
|---|---|---|
| committer |   Chuck Lever <chuck.lever@oracle.com> | 2021-03-06 16:41:48 -0500 |
| commit | bfdd89f232aa2de5a4b3fc985cba894148b830a8 (patch) | |
| tree | 47e50128eed0e3563cdcda3394568d6c8e33174a /fs/nfsd/nfs4state.c | |
| parent | fs: nfsd: fix kconfig dependency warning for NFSD_V4 (diff) | |
| download | linux-dev-bfdd89f232aa2de5a4b3fc985cba894148b830a8.tar.xz linux-dev-bfdd89f232aa2de5a4b3fc985cba894148b830a8.zip | |
nfsd: don't abort copies early
The typical result of the backwards comparison here is that the source
server in a server-to-server copy will return BAD_STATEID within a few
seconds of the copy starting, instead of giving the copy a full lease
period, so the copy_file_range() call will end up unnecessarily
returning a short read.
Fixes: 624322f1adc5 "NFSD add COPY_NOTIFY operation"
Signed-off-by: J. Bruce Fields <bfields@redhat.com>
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Diffstat (limited to '')
| -rw-r--r-- | fs/nfsd/nfs4state.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/nfsd/nfs4state.c b/fs/nfsd/nfs4state.c index 423fd6683f3a..61552e89bd89 100644 --- a/fs/nfsd/nfs4state.c +++ b/fs/nfsd/nfs4state.c @@ -5389,7 +5389,7 @@ nfs4_laundromat(struct nfsd_net *nn) idr_for_each_entry(&nn->s2s_cp_stateids, cps_t, i) { cps = container_of(cps_t, struct nfs4_cpntf_state, cp_stateid); if (cps->cp_stateid.sc_type == NFS4_COPYNOTIFY_STID && - cps->cpntf_time > cutoff) + cps->cpntf_time < cutoff) _free_cpntf_state_locked(nn, cps); } spin_unlock(&nn->s2s_cp_lock); |