Changeset 89127 in vbox for trunk/src

Timestamp:

May 17, 2021 11:26:25 PM (4 years ago)

Author:

vboxsync

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

144468

Message:

ValKit/AudioTest: audioTestDriverStackStreamCreateOutput need to return the actual cMsSchedulingHint value when we supply it a default. bugref:10008

File:

• trunk/src/VBox/ValidationKit/utils/audio/vkat.cpp (modified) (6 diffs)

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

@@ -828 +828 @@
  * @param   cMsSchedulingHint   The scheduling hint in milliseconds.
  * @param   ppStream            Where to return the stream pointer on success.
+ * @param   pCfgAcq             Where to return the actual (well, not
+ *                              necessarily when using DrvAudio, but probably
+ *                              the same) stream config on success (not used as
+ *                              input).
  */
 static int audioTestDriverStackStreamCreateOutput(PAUDIOTESTDRVSTACK pDrvStack, PCPDMAUDIOPCMPROPS pProps,
                                                   uint32_t cMsBufferSize, uint32_t cMsPreBuffer, uint32_t cMsSchedulingHint,
-                                                  PPDMAUDIOSTREAM *ppStream)
+                                                  PPDMAUDIOSTREAM *ppStream, PPDMAUDIOSTREAMCFG pCfgAcq)
 {
     char szTmp[PDMAUDIOSTRMCFGTOSTRING_MAX + 16];
@@ -881 +885 @@
         if (RT_SUCCESS(rc))
         {
+            *pCfgAcq = CfgReq; /** @todo PDMIAUDIOCONNECTOR::pfnStreamCreate only does one utterly pointless change to the two configs (enmLayout) from what I can tell... */
+            pCfgAcq->Props = (*ppStream)->Props;
             RTMsgInfo("Created backend stream: %s\n", PDMAudioStrmCfgToString(&CfgReq, szTmp, sizeof(szTmp)));
             return rc;
@@ -932 +938 @@
                         {
                             *ppStream = &pStreamAt->Core;
+                            *pCfgAcq  = pStreamAt->Cfg;
                             return VINF_SUCCESS;
                         }
@@ -944 +951 @@
                             {
                                 *ppStream = &pStreamAt->Core;
+                                *pCfgAcq  = pStreamAt->Cfg;
                                 return VINF_SUCCESS;
                             }
@@ -2049 +2057 @@
          * Open a stream for the output.
          */
-        PPDMAUDIOSTREAM pStream = NULL;
+        PDMAUDIOSTREAMCFG CfgAcq;
+        PPDMAUDIOSTREAM   pStream = NULL;
         rc = audioTestDriverStackStreamCreateOutput(&DrvStack, &WaveFile.Props, cMsBufferSize,
-                                                    cMsPreBuffer, cMsSchedulingHint, &pStream);
+                                                    cMsPreBuffer, cMsSchedulingHint, &pStream, &CfgAcq);
         if (RT_SUCCESS(rc))
         {
@@ -2098 +2107 @@
                             }
                             else if (audioTestDriverStackStreamIsOkay(&DrvStack, pStream))
-                                RTThreadSleep(RT_MIN(RT_MAX(1, cMsSchedulingHint), 256));
+                                RTThreadSleep(RT_MIN(RT_MAX(1, CfgAcq.Device.cMsSchedulingHint), 256));
                             else
                             {