Edward Goggin wrote:
Patch 1 of 1
This patch initializes the max_target_blocked field of a scsi target structure
so that a queuecommand return value of SCSI_MLQUEUE_TARGET_BUSY will actually
result in having the scsi_queue_insert blocking the device queue before requeuing
the command and running the queue. Otherwise, can and does cause livelock on
single CPU configurations if/when open-iSCSI software initiator's command PDU
window fills.
Signed-off-by: Ed Goggin <egoggin@xxxxxxxxxx>
--- linux-2.6.29/drivers/scsi/scsi_scan.c 2009-03-23 19:12:14.000000000 -0400
+++ linux-2.6.29-fix/drivers/scsi/scsi_scan.c 2009-04-09 11:11:01.000000000 -0400
@@ -427,6 +427,7 @@ static struct scsi_target *scsi_alloc_ta
INIT_LIST_HEAD(&starget->devices);
starget->state = STARGET_CREATED;
starget->scsi_level = SCSI_2;
+ starget->max_target_blocked = SCSI_DEFAULT_TARGET_BLOCKED;
retry:
spin_lock_irqsave(shost->host_lock, flags);
Hi James,
I see you picked up my patch to fix the q lock not being held:
http://marc.info/?l=linux-scsi&m=123931064705280&w=2
I think you need Ed's patch above too. Without it, target_blocked is
always zero, so we never hit the code I fixed up. I tested his patch
with my patch, and combined they fix all the issues here.
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html