Commit 32f16b38 authored by J. Bruce Fields's avatar J. Bruce Fields

nfsd4: remove some dprintk's and a comment

The comment is redundant, and if we really want dprintk's here they'd
probably be better in the common (check-slot_seqid) code.
Signed-off-by: default avatarJ. Bruce Fields <bfields@redhat.com>
parent 778df3f0
...@@ -1750,16 +1750,10 @@ nfsd4_create_session(struct svc_rqst *rqstp, ...@@ -1750,16 +1750,10 @@ nfsd4_create_session(struct svc_rqst *rqstp,
cs_slot = &conf->cl_cs_slot; cs_slot = &conf->cl_cs_slot;
status = check_slot_seqid(cr_ses->seqid, cs_slot->sl_seqid, 0); status = check_slot_seqid(cr_ses->seqid, cs_slot->sl_seqid, 0);
if (status == nfserr_replay_cache) { if (status == nfserr_replay_cache) {
dprintk("Got a create_session replay! seqid= %d\n",
cs_slot->sl_seqid);
/* Return the cached reply status */
status = nfsd4_replay_create_session(cr_ses, cs_slot); status = nfsd4_replay_create_session(cr_ses, cs_slot);
goto out; goto out;
} else if (cr_ses->seqid != cs_slot->sl_seqid + 1) { } else if (cr_ses->seqid != cs_slot->sl_seqid + 1) {
status = nfserr_seq_misordered; status = nfserr_seq_misordered;
dprintk("Sequence misordered!\n");
dprintk("Expected seqid= %d but got seqid= %d\n",
cs_slot->sl_seqid, cr_ses->seqid);
goto out; goto out;
} }
} else if (unconf) { } else if (unconf) {
...@@ -1768,7 +1762,6 @@ nfsd4_create_session(struct svc_rqst *rqstp, ...@@ -1768,7 +1762,6 @@ nfsd4_create_session(struct svc_rqst *rqstp,
status = nfserr_clid_inuse; status = nfserr_clid_inuse;
goto out; goto out;
} }
cs_slot = &unconf->cl_cs_slot; cs_slot = &unconf->cl_cs_slot;
status = check_slot_seqid(cr_ses->seqid, cs_slot->sl_seqid, 0); status = check_slot_seqid(cr_ses->seqid, cs_slot->sl_seqid, 0);
if (status) { if (status) {
...@@ -1776,7 +1769,6 @@ nfsd4_create_session(struct svc_rqst *rqstp, ...@@ -1776,7 +1769,6 @@ nfsd4_create_session(struct svc_rqst *rqstp,
status = nfserr_seq_misordered; status = nfserr_seq_misordered;
goto out; goto out;
} }
confirm_me = true; confirm_me = true;
conf = unconf; conf = unconf;
} else { } else {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment