Changeset 90776 in vbox for trunk/src/VBox/ValidationKit/utils/audio

Timestamp:

Aug 23, 2021 8:16:58 AM (3 years ago)

Author:

vboxsync

Message:

Audio/VKAT: Implemented "--probe-backends" commands to auto-probe all (available) backends. Needed for Linux hosts/guests which might or might not have certain backends installed/enabled; also nice for regression testing of backends which aren't available on certain platforms. bugref:10008

Location:

trunk/src/VBox/ValidationKit/utils/audio

Files:

• vkat.cpp (modified) (6 diffs)
• vkatDriverStack.cpp (modified) (1 diff)

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

@@ -127 +127 @@
     VKAT_TEST_OPT_PCM_CHAN,
     VKAT_TEST_OPT_PCM_SIGNED,
+    VKAT_TEST_OPT_PROBE_BACKENDS,
     VKAT_TEST_OPT_TAG,
     VKAT_TEST_OPT_TEMPDIR,
@@ -175 +176 @@
     { "--pcm-hz",            VKAT_TEST_OPT_PCM_HZ,         RTGETOPT_REQ_UINT16  },
     { "--pcm-signed",        VKAT_TEST_OPT_PCM_SIGNED,     RTGETOPT_REQ_BOOL    },
+    { "--probe-backends",    VKAT_TEST_OPT_PROBE_BACKENDS, RTGETOPT_REQ_NOTHING },
     { "--mode",              VKAT_TEST_OPT_MODE,           RTGETOPT_REQ_STRING  },
     { "--no-verify",         VKAT_TEST_OPT_NO_VERIFY,      RTGETOPT_REQ_NOTHING },
@@ -693 +695 @@
         case VKAT_TEST_OPT_PCM_SIGNED:     return "Specifies whether to use signed (true) or unsigned (false) samples\n"
                                                   "    Default: true";
+        case VKAT_TEST_OPT_PROBE_BACKENDS:  return "Specifies whether to probe all (available) backends until a working one is found\n"
+                                                  "    Default: false";
         case VKAT_TEST_OPT_TAG:            return "Specifies the test set tag to use";
         case VKAT_TEST_OPT_TEMPDIR:        return "Specifies the temporary directory to use";
@@ -722 +726 @@
     uint32_t    uPcmHz        = 0;
     bool        fPcmSigned    = true;
+    bool        fProbeBackends = false;
 
     const char *pszGuestTcpAddr  = NULL;
@@ -821 +826 @@
                 break;
 
+            case VKAT_TEST_OPT_PROBE_BACKENDS:
+                fProbeBackends = ValueUnion.f;
+                break;
+
             case VKAT_TEST_OPT_TAG:
                 pszTag = ValueUnion.psz;
@@ -858 +867 @@
 
     AUDIOTESTDRVSTACK DrvStack;
-    rc = audioTestDriverStackInitEx(&DrvStack, pDrvReg,
-                                    true /* fEnabledIn */, true /* fEnabledOut */, fWithDrvAudio); /** @todo Make in/out configurable, too. */
+    for (size_t i = 0; i < RT_ELEMENTS(g_aBackends); i++)
+    {
+        if (fProbeBackends)
+        {
+            pDrvReg = g_aBackends[i].pDrvReg;
+            RTTestPrintf(g_hTest, RTTESTLVL_ALWAYS, "Probing for backend '%s' ...\n", g_aBackends[i].pszName);
+        }
+        rc = audioTestDriverStackInitEx(&DrvStack, pDrvReg,
+                                        true /* fEnabledIn */, true /* fEnabledOut */, fWithDrvAudio); /** @todo Make in/out configurable, too. */
+        if (RT_SUCCESS(rc))
+        {
+            if (fProbeBackends)
+                RTTestPrintf(g_hTest, RTTESTLVL_ALWAYS, "Probing backend '%s' successful\n", g_aBackends[i].pszName);
+            break;
+        }
+        if (fProbeBackends)
+        {
+            RTTestPrintf(g_hTest, RTTESTLVL_ALWAYS, "Probing backend '%s' failed with %Rrc, trying next one\n",
+                         g_aBackends[i].pszName, rc);
+            continue;
+        }
+        return RTMsgErrorExit(RTEXITCODE_SYNTAX, "Unable to init driver stack: %Rrc\n", rc);
+    }
+
     if (RT_FAILURE(rc))
-        return RTMsgErrorExit(RTEXITCODE_SYNTAX, "Unable to init driver stack: %Rrc\n", rc);
+        return RTMsgErrorExit(RTEXITCODE_SYNTAX, "Probing all backends failed, unable to continue\n");
 
     PPDMAUDIOHOSTDEV pDev;

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

@@ -418 +418 @@
     }
 
-    RTTestFailed(g_hTest, "Failed to construct audio driver '%s': %Rrc", pDrvReg->szName, rc);
     if (pDrvReg->pfnDestruct)
         pDrvReg->pfnDestruct(pDrvIns);