Commit 752d8680 authored by Nicholas Bellinger's avatar Nicholas Bellinger

iscsi-target: Fix SNACK Type 1 + BegRun=0 handling

This patch fixes Status SNACK handling of BegRun=0 to allow
for all unacknowledged respones to be resent, instead of
always assuming that BegRun would be an explicit value less
than the current ExpStatSN.
Reported-by: default avatarsantosh kulkarni <santosh.kulkarni@calsoftinc.com>
Signed-off-by: default avatarNicholas Bellinger <nab@linux-iscsi.org>
parent 6a16d7be
...@@ -507,7 +507,9 @@ int iscsit_handle_status_snack( ...@@ -507,7 +507,9 @@ int iscsit_handle_status_snack(
u32 last_statsn; u32 last_statsn;
int found_cmd; int found_cmd;
if (conn->exp_statsn > begrun) { if (!begrun) {
begrun = conn->exp_statsn;
} else if (conn->exp_statsn > begrun) {
pr_err("Got Status SNACK Begrun: 0x%08x, RunLength:" pr_err("Got Status SNACK Begrun: 0x%08x, RunLength:"
" 0x%08x but already got ExpStatSN: 0x%08x on CID:" " 0x%08x but already got ExpStatSN: 0x%08x on CID:"
" %hu.\n", begrun, runlength, conn->exp_statsn, " %hu.\n", begrun, runlength, conn->exp_statsn,
......
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