[SCSI] target: Fix multi task->task_sg[] chaining logic bug
authorNicholas Bellinger <nab@linux-iscsi.org>
Fri, 20 May 2011 03:19:09 +0000 (20:19 -0700)
committerNicholas Bellinger <nab@linux-iscsi.org>
Fri, 27 May 2011 11:49:16 +0000 (11:49 +0000)
commit4a8fcc2cc34b7546584bf9ebd8f0caf433e4736e
treebbf6fc116baba7191f9f31a986e5389eab677dcd
parent129257c6af28fadf7c81b796a6ed06db1fcf1b1d
[SCSI] target: Fix multi task->task_sg[] chaining logic bug

This patch fixes a bug in transport_do_task_sg_chain() used by HW target
mode modules with sg_chain() to provide a single sg_next() walkable memory
layout for use with pci_map_sg() and friends.  This patch addresses an
issue with mapping multiple small block max_sector tasks across multiple
struct se_task->task_sg[] mappings for HW target mode operation.

This was causing OOPs with (cmd->t_task->t_tasks_no > 1) I/O traffic for
HW target drivers using transport_do_task_sg_chain(), and has been tested
so far with tcm_fc(openfcoe), tcm_qla2xxx, and ib_srpt fabrics with
t_tasks_no > 1 IBLOCK backends using a smaller max_sectors to trigger the
original issue.

Signed-off-by: Nicholas Bellinger <nab@linux-iscsi.org>
Acked-by: Kiran Patil <kiran.patil@intel.com>
Cc: stable@kernel.org
Signed-off-by: James Bottomley <jbottomley@parallels.com>
drivers/target/target_core_transport.c