Changeset 89992 in vbox for trunk/src/VBox/ValidationKit

Timestamp:

Jul 2, 2021 8:56:55 AM (4 years ago)

Author:

vboxsync

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

145469

Message:

Audio/ValKit: More testing code fixes and additions. bugref:10008

Location:

trunk/src/VBox/ValidationKit/utils/audio

Files:

• vkat.cpp (modified) (11 diffs)
• vkatCommon.cpp (modified) (3 diffs)

trunk/src/VBox/ValidationKit/utils/audio/vkat.cpp

@@ -253 +253 @@
     pTstParmsAcq->enmDir      = PDMAUDIODIR_OUT;
 #ifdef DEBUG
-    pTstParmsAcq->cIterations = 1;
+    pTstParmsAcq->cIterations = 4;
 #else
     pTstParmsAcq->cIterations = RTRandU32Ex(1, 10);
@@ -265 +265 @@
     pToneParms->msPrequel      = 0; /** @todo Implement analyzing this first! */
 #ifdef DEBUG_andy
-    pToneParms->msDuration     = 2000;
+    pToneParms->msDuration     = RTRandU32Ex(50, 2500);
 #else
     pToneParms->msDuration     = RTRandU32Ex(0, RT_MS_10SEC); /** @todo Probably a bit too long, but let's see. */
@@ -286 +286 @@
     for (uint32_t i = 0; i < pTstParms->cIterations; i++)
     {
+        PAUDIOTESTTONEPARMS const pToneParms = &pTstParms->TestTone;
+
+        RTTestPrintf(g_hTest, RTTESTLVL_ALWAYS, "Test #%RU32/%RU16: Playing test tone (%RU16Hz, %RU32ms)\n",
+                     pTstParms->idxCurrent, i, (uint16_t)pToneParms->dbFreqHz, pToneParms->msDuration);
+
         /*
          * 1. Arm the (host) ValKit ATS with the recording parameters.
          */
-        PAUDIOTESTTONEPARMS const pToneParms = &pTstParms->TestTone;
         rc = AudioTestSvcClientToneRecord(&pTstEnv->u.Host.AtsClValKit, pToneParms);
         if (RT_SUCCESS(rc))
@@ -299 +303 @@
         }
 
-        if (RT_FAILURE(rc))
-            RTTestFailed(g_hTest, "Playing tone failed\n");
+        if (RT_SUCCESS(rc))
+        {
+            if (pTstParms->cIterations)
+                RTThreadSleep(RTRandU32Ex(2000, 5000));
+        }
+        else
+            RTTestFailed(g_hTest, "Test #%RU32/%RU16: Playing tone failed\n", pTstParms->idxCurrent, i);
     }
 
@@ -336 +345 @@
     pTstParmsAcq->enmDir      = PDMAUDIODIR_IN;
 #ifdef DEBUG
-    pTstParmsAcq->cIterations = 1;
+    pTstParmsAcq->cIterations = 4;
 #else
     pTstParmsAcq->cIterations = RTRandU32Ex(1, 10);
@@ -342 +351 @@
     pTstParmsAcq->idxCurrent  = 0;
 
+    PAUDIOTESTTONEPARMS pToneParms = &pTstParmsAcq->TestTone;
+
+    pToneParms->Props          = pTstParmsAcq->Props;
+    pToneParms->dbFreqHz       = AudioTestToneGetRandomFreq();
+    pToneParms->msPrequel      = 0; /** @todo Implement analyzing this first! */
+    pToneParms->Props          = pTstParmsAcq->Props;
+#ifdef DEBUG_andy
+    pToneParms->msDuration     = RTRandU32Ex(50 /* ms */, 2500);
+#else
+    pToneParms->msDuration     = RTRandU32Ex(50 /* ms */, RT_MS_30SEC); /** @todo Record even longer? */
+#endif
+    pToneParms->msSequel       = 0;   /** @todo Implement analyzing this first! */
+    pToneParms->uVolumePercent = 100; /** @todo Implement analyzing this first! */
+
     return rc;
 }
@@ -356 +379 @@
     for (uint32_t i = 0; i < pTstParms->cIterations; i++)
     {
-        pTstParms->TestTone.Props      = pTstParms->Props;
-#ifdef DEBUG_andy
-        pTstParms->TestTone.msDuration = RTRandU32Ex(50 /* ms */, 2000);
-#else
-        pTstParms->TestTone.msDuration = RTRandU32Ex(50 /* ms */, RT_MS_30SEC); /** @todo Record even longer? */
-#endif
+        PAUDIOTESTTONEPARMS const pToneParms = &pTstParms->TestTone;
+
+        RTTestPrintf(g_hTest, RTTESTLVL_ALWAYS, "Test #%RU32/%RU16: Recording test tone (%RU16Hz, %RU32ms)\n",
+                     pTstParms->idxCurrent, i, (uint16_t)pToneParms->dbFreqHz, pToneParms->msDuration);
+
         /*
          * 1. Arm the (host) ValKit ATS with the playback parameters.
@@ -375 +397 @@
 
         if (RT_FAILURE(rc))
-            RTTestFailed(g_hTest, "Recording tone failed\n");
+            RTTestFailed(g_hTest, "Test #%RU32/%RU16: Recording tone failed\n", pTstParms->idxCurrent, i);
+
+        /* Wait a bit to let the left over audio bits being processed. */
+        if (pTstParms->cIterations)
+            RTThreadSleep(RTRandU32Ex(2000, 5000));
     }
 
@@ -427 +453 @@
     if (pTstDesc->fExcluded)
     {
-        RTTestSkipped(g_hTest, "Excluded from list");
+        RTTestSkipped(g_hTest, "Test #%u is excluded from list, skipping", uSeq);
         return VINF_SUCCESS;
     }
@@ -438 +464 @@
         if (RT_FAILURE(rc))
         {
-            RTTestFailed(g_hTest, "Test setup failed with %Rrc\n", rc);
+            RTTestFailed(g_hTest, "Test #%u setup failed with %Rrc\n", uSeq, rc);
             return rc;
         }
     }
 
-    RTTestPrintf(g_hTest, RTTESTLVL_ALWAYS, "Test #%u: %RU32 iterations\n", uSeq, TstParms.cIterations);
+    RTTestPrintf(g_hTest, RTTESTLVL_ALWAYS, "Test #%u (%RU32 iterations total)\n", uSeq, TstParms.cIterations);
 
     AssertPtr(pTstDesc->pfnExec);
     rc = pTstDesc->pfnExec(pTstEnv, pvCtx, &TstParms);
     if (RT_FAILURE(rc))
-        RTTestFailed(g_hTest, "Test failed with %Rrc\n", rc);
+        RTTestFailed(g_hTest, "Test #%u failed with %Rrc\n", uSeq, rc);
 
     RTTestSubDone(g_hTest);
@@ -459 +485 @@
 
         if (RT_FAILURE(rc2))
-            RTTestFailed(g_hTest, "Test destruction failed with %Rrc\n", rc2);
+            RTTestFailed(g_hTest, "Test #%u destruction failed with %Rrc\n", uSeq, rc2);
     }
 

trunk/src/VBox/ValidationKit/utils/audio/vkatCommon.cpp

@@ -368 +368 @@
     const char *pcszPathOut = pTstEnv->Set.szPathAbs;
 
-    RTTestPrintf(g_hTest, RTTESTLVL_ALWAYS, "Recording test tone (for %RU32ms)\n", pParms->msDuration);
+    RTTestPrintf(g_hTest, RTTESTLVL_ALWAYS, "Recording test tone (tone frequency is %RU16Hz, %RU32ms)\n", (uint16_t)pParms->dbFreqHz, pParms->msDuration);
     RTTestPrintf(g_hTest, RTTESTLVL_DEBUG,  "Writing to '%s'\n", pcszPathOut);
 
@@ -376 +376 @@
     AssertRCReturn(rc, rc);
 
-    if (audioTestDriverStackStreamIsOkay(&pTstEnv->DrvStack, pStream->pStream))
-    {
-        const uint32_t cbPerSched = PDMAudioPropsMilliToBytes(&pParms->Props, pTstEnv->cMsSchedulingHint);
-        AssertStmt(cbPerSched, rc = VERR_INVALID_PARAMETER);
-              uint32_t cbToRead   = PDMAudioPropsMilliToBytes(&pParms->Props, pParms->msDuration);
-        AssertStmt(cbToRead,   rc = VERR_INVALID_PARAMETER);
-
+    PAUDIOTESTDRVMIXSTREAM pMix = &pStream->Mix;
+
+    rc = AudioTestMixStreamEnable(pMix);
+    if (RT_SUCCESS(rc))
+    {
+        uint64_t cbToRecTotal = PDMAudioPropsMilliToBytes(&pStream->Cfg.Props, pParms->msDuration);
+
+        RTTestPrintf(g_hTest, RTTESTLVL_DEBUG, "Recording %RU32 bytes total\n", cbToRecTotal);
+
+        AudioTestSetObjAddMetadataStr(pObj, "stream_to_record_bytes=%RU32\n", cbToRecTotal);
+        AudioTestSetObjAddMetadataStr(pObj, "stream_buffer_size_ms=%RU32\n", pTstEnv->cMsBufferSize);
+        AudioTestSetObjAddMetadataStr(pObj, "stream_prebuf_size_ms=%RU32\n", pTstEnv->cMsPreBuffer);
+        /* Note: This mostly is provided by backend (e.g. PulseAudio / ALSA / ++) and
+         *       has nothing to do with the device emulation scheduling hint. */
+        AudioTestSetObjAddMetadataStr(pObj, "device_scheduling_hint_ms=%RU32\n", pTstEnv->cMsSchedulingHint);
+
+        uint8_t         abSamples[16384];
+        uint32_t const  cbSamplesAligned = PDMAudioPropsFloorBytesToFrame(pMix->pProps, sizeof(abSamples));
+        uint64_t        cbRecTotal  = 0;
+        while (!g_fTerminate && cbRecTotal < cbToRecTotal)
+        {
+            /*
+             * Anything we can read?
+             */
+            uint32_t const cbCanRead = AudioTestMixStreamGetReadable(pMix);
+            if (cbCanRead)
+            {
+                uint32_t const cbToRead   = RT_MIN(cbCanRead, cbSamplesAligned);
+                uint32_t       cbRecorded = 0;
+                rc = AudioTestMixStreamCapture(pMix, abSamples, cbToRead, &cbRecorded);
+                if (RT_SUCCESS(rc))
+                {
+                    if (cbRecorded)
+                    {
+                        rc = AudioTestSetObjWrite(pObj, abSamples, cbRecorded);
+                        if (RT_SUCCESS(rc))
+                        {
+                            cbRecTotal += cbRecorded;
+
+                            /** @todo Clamp result? */
+                        }
+                    }
+                }
+            }
+            else if (AudioTestMixStreamIsOkay(pMix))
+                RTThreadSleep(RT_MIN(RT_MAX(1, pTstEnv->cMsSchedulingHint), 256));
+
+            if (RT_FAILURE(rc))
+                break;
+        }
+
+        int rc2 = AudioTestMixStreamDisable(pMix);
         if (RT_SUCCESS(rc))
-        {
-            AudioTestSetObjAddMetadataStr(pObj, "buffer_size_ms=%RU32\n", pTstEnv->cMsBufferSize);
-            AudioTestSetObjAddMetadataStr(pObj, "prebuf_size_ms=%RU32\n", pTstEnv->cMsPreBuffer);
-            AudioTestSetObjAddMetadataStr(pObj, "scheduling_hint_ms=%RU32\n", pTstEnv->cMsSchedulingHint);
-
-            uint8_t abBuf[_4K];
-
-            while (cbToRead)
-            {
-                const uint32_t cbChunk = RT_MIN(cbToRead, RT_MIN(cbPerSched, sizeof(abBuf)));
-
-                uint32_t cbRead = 0;
-                rc = audioTestDriverStackStreamCapture(&pTstEnv->DrvStack, pStream->pStream, (void *)abBuf, cbChunk, &cbRead);
-                if (RT_SUCCESS(rc))
-                    rc = AudioTestSetObjWrite(pObj, abBuf, cbRead);
-
-                if (RT_FAILURE(rc))
-                    break;
-
-                RTThreadSleep(pTstEnv->cMsSchedulingHint);
-
-                Assert(cbToRead >= cbRead);
-                cbToRead -= cbRead;
-            }
-        }
-    }
-    else
-        rc = VERR_AUDIO_STREAM_NOT_READY;
+            rc = rc2;
+    }
 
     int rc2 = AudioTestSetObjClose(pObj);
@@ -465 +484 @@
     PATSCALLBACKCTX pCtx    = (PATSCALLBACKCTX)pvUser;
     PAUDIOTESTENV   pTstEnv = pCtx->pTstEnv;
-
-    AUDIOTESTTONE TstTone;
-    AudioTestToneInitRandom(&TstTone, &pToneParms->Props);
 
     const PAUDIOTESTSTREAM pTstStream = &pTstEnv->aStreams[0]; /** @todo Make this dynamic. */