Commit bfdd89f2 authored by J. Bruce Fields's avatar J. Bruce Fields Committed by Chuck Lever
Browse files

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: 624322f1 "NFSD add COPY_NOTIFY operation"
Signed-off-by: default avatarJ. Bruce Fields <bfields@redhat.com>
Signed-off-by: default avatarChuck Lever <chuck.lever@oracle.com>
parent 70052273

fs/nfsd/nfs4state.c

+1 −1
Original line number Diff line number Diff line
@@ -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);