Hi Nicholas,
Thanks for the clarification. I will also go ahead and verify this
patch. (Not this week, by next week).
Thanks,
Arshad
On 11/14/2013 2:43 AM, Nicholas A. Bellinger wrote:
On Wed, 2013-11-13 at 11:27 +0530, Arshad Hussain wrote:
Hi,
I am sending "SendTargets=All" as a text request. I am setting 'F' = 0.
(f bit to zero).
I was expecting target to respond with 'F' bit as 0. Instead it is
returning 'F' bit as 1.
RFC says that "A Text Response with a F bit set to 1 in response to a
Text Request with
the F bit set to 0 is a protocol error." (Section 10.11.1)
Are we looking at an error?
So given that multi PDU sequences of text commands + text responses are
currently not supported, rejecting text commands with F_BIT=0 || C_BIT=1
probably makes the most sense here until proper support is implemented.
I'll go ahead and apply the following patch.
--nab
diff --git a/drivers/target/iscsi/iscsi_target.c b/drivers/target/iscsi/iscsi_target.c
index ab64cbb..ba9787d 100644
--- a/drivers/target/iscsi/iscsi_target.c
+++ b/drivers/target/iscsi/iscsi_target.c
@@ -1948,6 +1948,13 @@ iscsit_setup_text_cmd(struct iscsi_conn *conn, struct iscsi_cmd *cmd,
(unsigned char *)hdr);
}
+ if (!(hdr->flags & ISCSI_FLAG_CMD_FINAL) ||
+ (hdr->flags & ISCSI_FLAG_TEXT_CONTINUE)) {
+ pr_err("Multi sequence text commands currently not supported\n");
+ return iscsit_reject_cmd(cmd, ISCSI_REASON_CMD_NOT_SUPPORTED,
+ (unsigned char *)hdr);
+ }
+
pr_debug("Got Text Request: ITT: 0x%08x, CmdSN: 0x%08x,"
" ExpStatSN: 0x%08x, Length: %u\n", hdr->itt, hdr->cmdsn,
hdr->exp_statsn, payload_length);
--
To unsubscribe from this list: send the line "unsubscribe target-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html