On 2/9/2015 6:07 PM, Sagi Grimberg wrote:
In case sendtargets response is larger than initiator MRDSL, we send a partial sendtargets response (setting F=0, C=1, TTT!=0xffffffff), accept a consecutive empty text message and send the rest of the payload. In case we are done, we set F=1, C=0, TTT=0xffffffff. We do that by storing the sendtargets response bytes done under the session. This patch makes iscsit_find_cmd_from_itt public for isert, and gets rid of maxcmdsn_inc variable. Signed-off-by: Sagi Grimberg <sagig@xxxxxxxxxxxx> ---
Forgot to list changes from v1: - Remove discovery session type restriction for multi-sequence text. - Remove session st_rsp_bytes - using cmd->read_data_done. - Don't free partial text responses (continue bit on). - In case a text request has ttt != 0xffffffff, locate the command by itt instead of allocating it (and avoid re-adding it to conn_cmd_list. - Removed maxcmdsn_inc bit to allow a sequence over 2 transactions. This flag seems redundant to me, but I'm not sure what is it's usage, Nic can you sched some light on this? do we really need it? -- 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