Skip to content

4.0 server tests: remove RPLY14 - #9

Merged
chucklever merged 1 commit into
testingfrom
retire-rply14
Nov 3, 2026
Merged

chucklever merged 1 commit into
testingfrom
retire-rply14

Conversation

@chucklever

Copy link
Copy Markdown
Collaborator

RPLY14 depends on an XID-based duplicate request cache that RFC 7530
does not specify. The test now fails against NFSD. Removing it leaves
the reply-cache tests that RFC 7530 Section 9.1.7 requires.

RPLY14 sends a CREATE, then resends it twice with the same XID on the
same TCP connection and expects NFS4_OK each time. The other RPLY tests
exercise the per-state-owner reply cache that RFC 7530 Section 9.1.7
requires, or replay an idempotent LOOKUP. RPLY14 alone depends on an
XID-based duplicate request cache, which RFC 7530 does not specify.
Section 3.1.1 also forbids an NFSv4.0 client from retrying a request
unless the transport connection has been broken.

A recent NFSD change drops a retransmission that arrives on the
connection that carried the original request, so RPLY14 now fails
against NFSD. Reconnecting before the replay does not rescue the test.
NFSD may evict a DRC entry once the client has acknowledged the cached
reply, so the replayed CREATE executes a second time and fails with
NFS4ERR_EXIST.

Remove RPLY14 and record its code as retired.

Signed-off-by: Chuck Lever <cel@kernel.org>

@cdmackay cdmackay left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fair enough.

@chucklever
chucklever merged commit f0b7f9b into testing Nov 3, 2026
8 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants