For correctness and whitespace. Signed-off-by: Andy Grover <agrover@xxxxxxxxxx> --- drivers/target/iscsi/iscsi_target_seq_pdu_list.c | 10 +++++----- 1 files changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/target/iscsi/iscsi_target_seq_pdu_list.c b/drivers/target/iscsi/iscsi_target_seq_pdu_list.c index 9f1c3e5..7d0effa 100644 --- a/drivers/target/iscsi/iscsi_target_seq_pdu_list.c +++ b/drivers/target/iscsi/iscsi_target_seq_pdu_list.c @@ -288,8 +288,8 @@ static void iscsit_determine_counts_for_list( /* - * Builds PDU and/or Sequence list, called while DataSequenceInOrder=No - * and DataPDUInOrder=No. + * Builds PDU and/or Sequence list, called while DataSequenceInOrder=No + * or DataPDUInOrder=No. */ static int iscsit_do_build_pdu_and_seq_lists( struct iscsi_cmd *cmd, @@ -494,9 +494,6 @@ static int iscsit_do_build_pdu_and_seq_lists( return 0; } -/* - * Only called while DataSequenceInOrder=No or DataPDUInOrder=No. - */ int iscsit_build_pdu_and_seq_lists( struct iscsi_cmd *cmd, u32 immediate_data_length) @@ -510,6 +507,9 @@ int iscsit_build_pdu_and_seq_lists( struct iscsi_session *sess = conn->sess; struct iscsi_node_attrib *na; + /* + * Do nothing if no OOO shenanigans + */ if (sess->sess_ops->DataSequenceInOrder && sess->sess_ops->DataPDUInOrder) return 0; -- 1.7.1 -- 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
