Changeset 91647 in vbox

Timestamp:

Oct 8, 2021 6:48:43 PM (3 years ago)

Author:

vboxsync

Message:

Audio/Validation Kit: Added a VBOX_WITH_AUDIO_VALKIT_DUMP_STREAMS define for dumping output audio streams to disks to the Validation Kit audio driver. ​bugref:10008

File:

• trunk/src/VBox/Devices/Audio/DrvHostAudioValidationKit.cpp (modified) (7 diffs)

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

@@ -39 +39 @@
 
 
+#ifdef DEBUG_andy
+/** Enables dumping audio streams to the temporary directory for debugging. */
+# define VBOX_WITH_AUDIO_VALKIT_DUMP_STREAMS
+#endif
+
+
 /*********************************************************************************************************************************
 *   Structures and Typedefs                                                                                                      *
@@ -53 +59 @@
     /** How much bytes are available to read (only for capturing streams). */
     uint32_t                cbAvail;
+#ifdef VBOX_WITH_AUDIO_VALKIT_DUMP_STREAMS
+    /** Audio file to dump output to. */
+    PAUDIOHLPFILE           pFile;
+#endif
 } VALKITAUDIOSTREAM;
 /** Pointer to a Validation Kit stream. */
@@ -670 +680 @@
     int rc = VINF_SUCCESS;
     PDMAudioStrmCfgCopy(&pStreamDbg->Cfg, pCfgAcq);
+
+#ifdef VBOX_WITH_AUDIO_VALKIT_DUMP_STREAMS
+    int rc2 = AudioHlpFileCreateAndOpenEx(&pStreamDbg->pFile, AUDIOHLPFILETYPE_WAV, NULL /*use temp dir*/,
+                                          pThis->pDrvIns->iInstance, AUDIOHLPFILENAME_FLAGS_NONE, AUDIOHLPFILE_FLAGS_NONE,
+                                          &pCfgReq->Props, RTFILE_O_WRITE | RTFILE_O_DENY_WRITE | RTFILE_O_CREATE_REPLACE,
+                                          pCfgReq->enmDir == PDMAUDIODIR_IN ? "ValKitAudioIn" : "ValKitAudioOut");
+    if (RT_FAILURE(rc2))
+        LogRel(("ValKit: Failed to creating debug file for %s stream '%s' in the temp directory: %Rrc\n",
+                pCfgReq->enmDir == PDMAUDIODIR_IN ? "input" : "output", pCfgReq->szName, rc2));
+#endif
+
     return rc;
 }
@@ -682 +703 @@
     PVALKITAUDIOSTREAM  pStreamDbg = (PVALKITAUDIOSTREAM)pStream;
     AssertPtrReturn(pStreamDbg, VERR_INVALID_POINTER);
+
+#ifdef VBOX_WITH_AUDIO_VALKIT_DUMP_STREAMS
+    if (pStreamDbg->pFile)
+    {
+        AudioHlpFileDestroy(pStreamDbg->pFile);
+        pStreamDbg->pFile = NULL;
+    }
+#endif
 
     return VINF_SUCCESS;
@@ -858 +887 @@
     PVALKITTESTDATA     pTst  = NULL;
 
+    int rc2;
+#ifdef VBOX_WITH_AUDIO_VALKIT_DUMP_STREAMS
+    PVALKITAUDIOSTREAM  pStrmValKit = (PVALKITAUDIOSTREAM)pStream;
+    rc2 = AudioHlpFileWrite(pStrmValKit->pFile, pvBuf, cbBuf);
+    AssertRC(rc2);
+#endif
+
     bool const fIsSilence = PDMAudioPropsIsBufferSilence(&pStream->pStream->Cfg.Props, pvBuf, cbBuf);
 
@@ -879 +915 @@
         pTst = pThis->pTestCurPlay;
 
-        int rc2 = RTCritSectLeave(&pThis->CritSect);
+        rc2 = RTCritSectLeave(&pThis->CritSect);
         AssertRC(rc2);
     }
@@ -970 +1006 @@
                     pTst                = NULL;
 
-                    int rc2 = RTCritSectLeave(&pThis->CritSect);
+                    rc2 = RTCritSectLeave(&pThis->CritSect);
                     if (RT_SUCCESS(rc))
                         rc = rc2;