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

Timestamp:

Apr 22, 2021 6:24:15 PM (4 years ago)

Author:

vboxsync

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

143976

Message:

DevHda: Added release profiling of stream starting, resetting and stopping (in nanoseconds for better relrelatability). bugref:9890

Location:

trunk/src/VBox/Devices/Audio

Files:

• DevHda.cpp (modified) (5 diffs)
• DevHdaStream.h (modified) (1 diff)

trunk/src/VBox/Devices/Audio/DevHda.cpp

@@ -1339 +1339 @@
         LogFunc(("[SD%RU8] Reset enter\n", uSD));
 
+        STAM_REL_PROFILE_START_NS(&pStreamR3->State.StatReset, a);
         hdaStreamLock(pStreamShared);
 
@@ -1358 +1359 @@
 # endif
         hdaStreamUnlock(pStreamShared);
+        STAM_REL_PROFILE_STOP_NS(&pStreamR3->State.StatReset, a);
     }
     else
@@ -1366 +1368 @@
         if (fInRun != fRun)
         {
+            STAM_REL_PROFILE_START_NS((fRun ? &pStreamR3->State.StatStart : &pStreamR3->State.StatStop), r);
             Assert(!fReset && !fInReset); /* (code change paranoia, currently impossible ) */
             LogFunc(("[SD%RU8] State changed (fRun=%RTbool)\n", uSD, fRun));
@@ -1468 +1471 @@
             /* Make sure to leave the lock before (eventually) starting the timer. */
             hdaStreamUnlock(pStreamShared);
+            STAM_REL_PROFILE_STOP_NS((fRun ? &pStreamR3->State.StatStart : &pStreamR3->State.StatStop), r);
         }
     }
@@ -5327 +5331 @@
         PDMDevHlpSTAMRegisterF(pDevIns, &pThisCC->aStreams[idxStream].State.StatDmaBufUsed, STAMTYPE_U32, STAMVISIBILITY_USED, STAMUNIT_BYTES,
                                "Number of bytes used in the internal DMA buffer.",  "Stream%u/DMABufUsed", idxStream);
+
+        PDMDevHlpSTAMRegisterF(pDevIns, &pThisCC->aStreams[idxStream].State.StatStart, STAMTYPE_PROFILE, STAMVISIBILITY_USED, STAMUNIT_NS_PER_CALL,
+                               "Starting the stream.",  "Stream%u/Start", idxStream);
+        PDMDevHlpSTAMRegisterF(pDevIns, &pThisCC->aStreams[idxStream].State.StatStop, STAMTYPE_PROFILE, STAMVISIBILITY_USED, STAMUNIT_NS_PER_CALL,
+                               "Stopping the stream.",  "Stream%u/Stop", idxStream);
+        PDMDevHlpSTAMRegisterF(pDevIns, &pThisCC->aStreams[idxStream].State.StatReset, STAMTYPE_PROFILE, STAMVISIBILITY_USED, STAMUNIT_NS_PER_CALL,
+                               "Resetting the stream.",  "Stream%u/Reset", idxStream);
     }
 

trunk/src/VBox/Devices/Audio/DevHdaStream.h

@@ -326 +326 @@
         /** Number of bytes involved in unresolved flow errors. */
         STAMCOUNTER             StatDmaFlowErrorBytes;
+
+        STAMPROFILE             StatStart;
+        STAMPROFILE             StatReset;
+        STAMPROFILE             StatStop;
+        STAMPROFILE             StatUnusedPadding;
     } State;
     /** Debug bits. */