libata: fix double-completion on error
authorDarrick J. Wong <djwong@us.ibm.com>
Tue, 14 Nov 2006 14:55:41 +0000 (09:55 -0500)
committerJeff Garzik <jeff@garzik.org>
Tue, 14 Nov 2006 14:55:41 +0000 (09:55 -0500)
A curious thing happens, however, when ata_qc_new_init fails to get
an ata_queued_cmd:

First, ata_qc_new_init handles the failure like this:
    cmd->result = (DID_OK << 16) | (QUEUE_FULL << 1);
    done(cmd);

Then, we return to ata_scsi_translate and do this:
    err_mem:
        cmd->result = (DID_ERROR << 16);
        done(cmd);

It appears to me that first we set a status code indicating that we're
ok but the device queue is full and finish the command,  but then
we blow away that status code and replace it with an error flag and
finish the command a second time!  That does not seem to be desirable
behavior since we merely want the I/O to wait until a command slot
frees up, not send errors up the block layer.

In the err_mem case, we should simply exit out of ata_scsi_translate
instead.

Signed-off-by: Darrick J. Wong <djwong@us.ibm.com>
Signed-off-by: Jeff Garzik <jeff@garzik.org>
drivers/ata/libata-scsi.c

index 7af2a4ba49905e068d7eb38c4ad0f778ccc3702e..5c1fc467fc7fd0ed25172def2417f6c420993cf3 100644 (file)
@@ -1612,9 +1612,9 @@ early_finish:
 
 err_did:
        ata_qc_free(qc);
-err_mem:
        cmd->result = (DID_ERROR << 16);
        done(cmd);
+err_mem:
        DPRINTK("EXIT - internal\n");
        return 0;