ieee1394: sbp2: slightly reorder sbp2scsi_abort
authorStefan Richter <stefanr@s5r6.in-berlin.de>
Mon, 11 Sep 2006 18:17:14 +0000 (20:17 +0200)
committerStefan Richter <stefanr@s5r6.in-berlin.de>
Thu, 7 Dec 2006 20:27:30 +0000 (21:27 +0100)
Put the target's fetch agent into reset state before the underlying ORB
DMA is unmapped and the ->done handler is called.  It is highly unlikely
but the target could access that ORB right before sbp2 sends the reset
request.

Signed-off-by: Stefan Richter <stefanr@s5r6.in-berlin.de>
drivers/ieee1394/sbp2.c

index cd156d4e779e973511e67562ab0f1d9f9919bed5..e5ba55bbd93566375afe024df2c205eb48fabca6 100644 (file)
@@ -2564,11 +2564,9 @@ static int sbp2scsi_abort(struct scsi_cmnd *SCpnt)
        scsi_print_command(SCpnt);
 
        if (sbp2util_node_is_available(scsi_id)) {
+               sbp2_agent_reset(scsi_id, 1);
 
-               /*
-                * Right now, just return any matching command structures
-                * to the free pool.
-                */
+               /* Return a matching command structure to the free pool. */
                spin_lock_irqsave(&scsi_id->sbp2_command_orb_lock, flags);
                command = sbp2util_find_command_for_SCpnt(scsi_id, SCpnt);
                if (command) {
@@ -2589,10 +2587,6 @@ static int sbp2scsi_abort(struct scsi_cmnd *SCpnt)
                }
                spin_unlock_irqrestore(&scsi_id->sbp2_command_orb_lock, flags);
 
-               /*
-                * Initiate a fetch agent reset.
-                */
-               sbp2_agent_reset(scsi_id, 1);
                sbp2scsi_complete_all_commands(scsi_id, DID_BUS_BUSY);
        }