Changeset 108630 in vbox

Timestamp:

Mar 20, 2025 11:15:51 AM (7 weeks ago)

Author:

vboxsync

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

168042

Message:

Devices/Audio/testcase/tstAudioMixBuffer.cpp: Make it work for hosts where we don't know the host page size when building, bugref:10391

File:

• trunk/src/VBox/Devices/Audio/testcase/tstAudioMixBuffer.cpp (modified) (2 diffs)

trunk/src/VBox/Devices/Audio/testcase/tstAudioMixBuffer.cpp

@@ -37 +37 @@
 #include <iprt/string.h>
 #include <iprt/test.h>
+#include <iprt/system.h>
 
 #include <VBox/vmm/pdm.h>
@@ -168 +169 @@
 
     /* DrvAudioHlpClearBuf: */
+    size_t const cbPageSize = RTSystemGetPageSize();
     uint8_t *pbPage;
-    int rc = RTTestGuardedAlloc(hTest, HOST_PAGE_SIZE, 0, false /*fHead*/, (void **)&pbPage);
+    int rc = RTTestGuardedAlloc(hTest, cbPageSize, 0, false /*fHead*/, (void **)&pbPage);
     RTTESTI_CHECK_RC_OK_RETV(rc);
 
-    memset(pbPage, 0x42, HOST_PAGE_SIZE);
-    PDMAudioPropsClearBuffer(&Cfg441StereoS16, pbPage, HOST_PAGE_SIZE, HOST_PAGE_SIZE / 4);
-    RTTESTI_CHECK(ASMMemIsZero(pbPage, HOST_PAGE_SIZE));
-
-    memset(pbPage, 0x42, HOST_PAGE_SIZE);
-    PDMAudioPropsClearBuffer(&Cfg441StereoU16, pbPage, HOST_PAGE_SIZE, HOST_PAGE_SIZE / 4);
-    for (uint32_t off = 0; off < HOST_PAGE_SIZE; off += 2)
+    memset(pbPage, 0x42, cbPageSize);
+    PDMAudioPropsClearBuffer(&Cfg441StereoS16, pbPage, cbPageSize, cbPageSize / 4);
+    RTTESTI_CHECK(ASMMemIsZero(pbPage, cbPageSize));
+
+    memset(pbPage, 0x42, cbPageSize);
+    PDMAudioPropsClearBuffer(&Cfg441StereoU16, pbPage, cbPageSize, cbPageSize / 4);
+    for (uint32_t off = 0; off < cbPageSize; off += 2)
         RTTESTI_CHECK_MSG(pbPage[off] == 0 && pbPage[off + 1] == 0x80, ("off=%#x: %#x %x\n", off, pbPage[off], pbPage[off + 1]));
 
-    memset(pbPage, 0x42, HOST_PAGE_SIZE);
-    PDMAudioPropsClearBuffer(&Cfg441StereoU32, pbPage, HOST_PAGE_SIZE, HOST_PAGE_SIZE / 8);
-    for (uint32_t off = 0; off < HOST_PAGE_SIZE; off += 4)
+    memset(pbPage, 0x42, cbPageSize);
+    PDMAudioPropsClearBuffer(&Cfg441StereoU32, pbPage, cbPageSize, cbPageSize / 8);
+    for (uint32_t off = 0; off < cbPageSize; off += 4)
         RTTESTI_CHECK(pbPage[off] == 0 && pbPage[off + 1] == 0 && pbPage[off + 2] == 0 && pbPage[off + 3] == 0x80);
 
 
     RTTestDisableAssertions(hTest);
-    memset(pbPage, 0x42, HOST_PAGE_SIZE);
-    PDMAudioPropsClearBuffer(&Cfg441StereoS16, pbPage, HOST_PAGE_SIZE, HOST_PAGE_SIZE); /* should adjust down the frame count. */
-    RTTESTI_CHECK(ASMMemIsZero(pbPage, HOST_PAGE_SIZE));
-
-    memset(pbPage, 0x42, HOST_PAGE_SIZE);
-    PDMAudioPropsClearBuffer(&Cfg441StereoU16, pbPage, HOST_PAGE_SIZE, HOST_PAGE_SIZE); /* should adjust down the frame count. */
-    for (uint32_t off = 0; off < HOST_PAGE_SIZE; off += 2)
+    memset(pbPage, 0x42, cbPageSize);
+    PDMAudioPropsClearBuffer(&Cfg441StereoS16, pbPage, cbPageSize, cbPageSize); /* should adjust down the frame count. */
+    RTTESTI_CHECK(ASMMemIsZero(pbPage, cbPageSize));
+
+    memset(pbPage, 0x42, cbPageSize);
+    PDMAudioPropsClearBuffer(&Cfg441StereoU16, pbPage, cbPageSize, cbPageSize); /* should adjust down the frame count. */
+    for (uint32_t off = 0; off < cbPageSize; off += 2)
         RTTESTI_CHECK_MSG(pbPage[off] == 0 && pbPage[off + 1] == 0x80, ("off=%#x: %#x %x\n", off, pbPage[off], pbPage[off + 1]));
 
-    memset(pbPage, 0x42, HOST_PAGE_SIZE);
-    PDMAudioPropsClearBuffer(&Cfg441StereoU32, pbPage, HOST_PAGE_SIZE, HOST_PAGE_SIZE); /* should adjust down the frame count. */
-    for (uint32_t off = 0; off < HOST_PAGE_SIZE; off += 4)
+    memset(pbPage, 0x42, cbPageSize);
+    PDMAudioPropsClearBuffer(&Cfg441StereoU32, pbPage, cbPageSize, cbPageSize); /* should adjust down the frame count. */
+    for (uint32_t off = 0; off < cbPageSize; off += 4)
         RTTESTI_CHECK(pbPage[off] == 0 && pbPage[off + 1] == 0 && pbPage[off + 2] == 0 && pbPage[off + 3] == 0x80);
     RTTestRestoreAssertions(hTest);