Changeset 104523 in vbox for trunk/src/VBox/Devices/Audio

Timestamp:

May 6, 2024 4:28:42 PM (12 months ago)

Author:

vboxsync

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

163048

Message:

Audio/AudioMixer: Better diagnostics via (verbose + limited) release logging when a mixer's master volume (+ mute status) and its sinks are being changed.

File:

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

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

@@ -312 +312 @@
     LogFlowFunc(("[%s] fMuted=%RTbool auChannels=%.*Rhxs => fMuted=%RTbool auChannels=%.*Rhxs\n", pMixer->pszName,
                  pMixer->VolMaster.fMuted, sizeof(pMixer->VolMaster.auChannels), pMixer->VolMaster.auChannels,
-                 pVol->fMuted, sizeof(pVol->auChannels), pVol->auChannels ));
+                 pVol->fMuted, sizeof(pVol->auChannels), pVol->auChannels));
     memcpy(&pMixer->VolMaster, pVol, sizeof(PDMAUDIOVOLUME));
+
+    LogRelMax(256, ("Audio Mixer: %s master volume of '%s' -- channel volumes: %.*Rhxs\n",
+                    pMixer->VolMaster.fMuted ? "MUTING" : "Setting",
+                    pMixer->pszName, sizeof(pMixer->VolMaster.auChannels), pMixer->VolMaster.auChannels));
 
     /*
@@ -1109 +1113 @@
                  pSink->VolumeCombined.fMuted, sizeof(pSink->VolumeCombined.auChannels), pSink->VolumeCombined.auChannels ));
 
+    LogRelMax(256, ("Audio Mixer: %s sink '%s/%s' -- channel volumes: %.*Rhxs\n",
+                    pSink->VolumeCombined.fMuted ? "MUTING" : "Setting",
+                    pSink->pParent->pszName, pSink->pszName,
+                    sizeof(pSink->VolumeCombined.auChannels), pSink->VolumeCombined.auChannels));
+
     AudioMixBufSetVolume(&pSink->MixBuf, &pSink->VolumeCombined);
     return VINF_SUCCESS;
@@ -1131 +1140 @@
 
     memcpy(&pSink->Volume, pVol, sizeof(PDMAUDIOVOLUME));
-
-    LogRel2(("Audio Mixer: Setting volume of sink '%s' to fMuted=%RTbool auChannels=%.*Rhxs\n",
-              pSink->pszName, pVol->fMuted, sizeof(pVol->auChannels), pVol->auChannels));
 
     Assert(pSink->pParent);