Changeset 43397 in vbox for trunk/src

Timestamp:

Sep 21, 2012 12:17:22 PM (12 years ago)

Author:

vboxsync

Message:

AHCI: A few more statistics and release logging if a request is active for too long

File:

• trunk/src/VBox/Devices/Storage/DevAHCI.cpp (modified) (4 diffs)

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

@@ -279 +279 @@
     /** Task state. */
     volatile AHCITXSTATE       enmTxState;
+    /** Start timestamp of the request. */
+    uint64_t                   tsStart;
     /** Tag of the task. */
     uint32_t                   uTag;
@@ -5615 +5617 @@
     bool fXchg = false;
     bool fRedo = false;
+    uint64_t tsNow = RTTimeMilliTS();
+
+    /*
+     * Leave a release log entry if the request was active for more than 25 seconds
+     * (30 seconds is the timeout of the guest).
+     */
+    if (tsNow - pAhciReq->tsStart >= 25 * 1000)
+    {
+        const char *pcszReq = NULL;
+
+        switch (pAhciReq->enmTxDir)
+        {
+            case AHCITXDIR_READ:
+                pcszReq = "Read";
+                break;
+            case AHCITXDIR_WRITE:
+                pcszReq = "Write";
+                break;
+            case AHCITXDIR_FLUSH:
+                pcszReq = "Flush";
+                break;
+            case AHCITXDIR_TRIM:
+                pcszReq = "Trim";
+                break;
+            default:
+                pcszReq = "<Invalid>";
+        }
+
+        LogRel(("AHCI#%u: %s request was active for %llu seconds\n",
+                pAhciPort->iLUN, pcszReq, (tsNow - pAhciReq->tsStart) / 1000));
+    }
 
     ASMAtomicCmpXchgSize(&pAhciReq->enmTxState, AHCITXSTATE_FREE, AHCITXSTATE_ACTIVE, fXchg);
@@ -6220 +6253 @@
             AssertMsg(fXchg, ("Task is already active\n"));
 
+            pAhciReq->tsStart = RTTimeMilliTS();
             pAhciReq->uATARegStatus = 0;
             pAhciReq->uATARegError  = 0;
@@ -6403 +6437 @@
             STAM_PROFILE_START(&pAhciPort->StatProfileProcessTime, a);
 
+            pAhciReq->tsStart       = RTTimeMilliTS();
             pAhciReq->uATARegStatus = 0;
             pAhciReq->uATARegError  = 0;