Changeset 35792 in vbox for trunk/src/VBox/Devices

Timestamp:

Jan 31, 2011 3:40:27 PM (14 years ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

69764

Message:

ATAController: Brought over device reset fix.

File:

• trunk/src/VBox/Devices/Storage/ATAController.cpp (modified) (1 diff)

trunk/src/VBox/Devices/Storage/ATAController.cpp

@@ -4260 +4260 @@
 
             case AHCIATA_AIO_ABORT:
-                /* Abort the current command only if it operates on the same interface. */
-                if (pCtl->iAIOIf == pReq->u.a.iIf)
+                /* Abort the current command no matter what. There cannot be
+                 * any command activity on the other drive otherwise using
+                 * one thread per controller wouldn't work at all. */
+                s = &pCtl->aIfs[pReq->u.a.iIf];
+
+                pCtl->uAsyncIOState = AHCIATA_AIO_NEW;
+                /* Do not change the DMA registers, they are not affected by the
+                 * ATA controller reset logic. It should be sufficient to issue a
+                 * new command, which is now possible as the state is cleared. */
+                if (pReq->u.a.fResetDrive)
                 {
-                    s = &pCtl->aIfs[pCtl->iAIOIf];
-
-                    pCtl->uAsyncIOState = AHCIATA_AIO_NEW;
-                    /* Do not change the DMA registers, they are not affected by the
-                     * ATA controller reset logic. It should be sufficient to issue a
-                     * new command, which is now possible as the state is cleared. */
-                    if (pReq->u.a.fResetDrive)
-                    {
-                        ataResetDevice(s);
-                        ataExecuteDeviceDiagnosticSS(s);
-                    }
-                    else
-                    {
-                        ataPIOTransferStop(s);
-                        ataUnsetStatus(s, ATA_STAT_BUSY | ATA_STAT_DRQ | ATA_STAT_SEEK | ATA_STAT_ERR);
-                        ataSetStatus(s, ATA_STAT_READY);
-                        ataSetIRQ(s);
-                    }
+                    ataResetDevice(s);
+                    ataExecuteDeviceDiagnosticSS(s);
+                }
+                else
+                {
+                    ataPIOTransferStop(s);
+                    ataUnsetStatus(s, ATA_STAT_BUSY | ATA_STAT_DRQ | ATA_STAT_SEEK | ATA_STAT_ERR);
+                    ataSetStatus(s, ATA_STAT_READY);
+                    ataSetIRQ(s);
                 }
                 break;