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

Timestamp:

Aug 9, 2018 9:50:52 AM (6 years ago)

Author:

vboxsync

Message:

Audio/Mixer: Skip handling input streams which currently are not marked as the recording source -- should save a few CPU cycles.

File:

• trunk/src/VBox/Devices/Audio/AudioMixer.cpp (modified) (2 diffs)

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

@@ -1536 +1536 @@
         return rc;
 
-    /* Number of detected disabled streams of this sink. */
-    uint8_t cStreamsDisabled = 0;
+    /* Number of disabled streams of this sink. */
+    uint8_t cStreamsDisabled = pSink->cStreams;
 
     PAUDMIXSTREAM pMixStream, pMixStreamNext;
@@ -1587 +1587 @@
                 continue;
             }
-
-            PDMAUDIOSTREAMSTS strmSts = pConn->pfnStreamGetStatus(pConn, pStream);
-
-            /* Is the stream not enabled and also is not in a pending disable state anymore? */
-            if (   !(strmSts & PDMAUDIOSTREAMSTS_FLAG_ENABLED)
-                && !(strmSts & PDMAUDIOSTREAMSTS_FLAG_PENDING_DISABLE))
-            {
-                cStreamsDisabled++;
-            }
+        }
+
+        PDMAUDIOSTREAMSTS strmSts = pConn->pfnStreamGetStatus(pConn, pStream);
+
+        /* Is the stream enabled or in pending disable state?
+         * Don't consider this stream as being disabled then. */
+        if (   (strmSts & PDMAUDIOSTREAMSTS_FLAG_ENABLED)
+            || (strmSts & PDMAUDIOSTREAMSTS_FLAG_PENDING_DISABLE))
+        {
+            cStreamsDisabled--;
         }