Changeset 87759 in vbox for trunk/src/VBox/Main/src-client

Timestamp:

Feb 15, 2021 12:15:18 PM (4 years ago)

Author:

vboxsync

Message:

Audio/HDA: Lots more code in the hope to resolve issue ticketoem2ref:36, namely:

• Decoupled async I/O timing from guest driver-specific DMA timing to further reduce EMT workload.
• Added data transfer heuristics (based on set-up DMA buffers) to detect Windows 10 guests (enabled by default).
• Also expose and support 16kHz + 22,5kHz streams (16-bit signed).

Only tested on Win10 20H2 and various Ubuntu guests so far.

File:

• trunk/src/VBox/Main/src-client/ConsoleImpl2.cpp (modified) (2 diffs)

trunk/src/VBox/Main/src-client/ConsoleImpl2.cpp

@@ -3052 +3052 @@
                      * HDA-specific parameters.
                      */
-
                     uint64_t uTmp;
                     GetExtraDataBoth(virtualBox, pMachine, "VBoxInternal2/Audio/Device/PosAdjustEnabled", &strTmp);
                     if (strTmp.isNotEmpty())
-                    {
-                        uTmp = strTmp.toUInt64(); /* Returns 0 if not set / invalid -> means disabled. */
-                        InsertConfigInteger(pCfg,          "PosAdjustEnabled",     uTmp);
-                    }
+                        InsertConfigInteger(pCfg,          "PosAdjustEnabled",             strTmp.equalsIgnoreCase("true")
+                                                                                        || strTmp.equalsIgnoreCase("1"));
 
                     GetExtraDataBoth(virtualBox, pMachine, "VBoxInternal2/Audio/Device/PosAdjustFrames", &strTmp);
@@ -3065 +3062 @@
                     {
                         uTmp = strTmp.toUInt64(); /* Ditto. */
-                        InsertConfigInteger(pCfg,          "PosAdjustFrames",      uTmp);
+                        InsertConfigInteger(pCfg,          "PosAdjustFrames",           uTmp);
                     }
 
+                    GetExtraDataBoth(virtualBox, pMachine, "VBoxInternal2/Audio/Device/TransferHeuristicsEnabled", &strTmp);
+                    if (strTmp.isNotEmpty())
+                        InsertConfigInteger(pCfg,          "TransferHeuristicsEnabled",    strTmp.equalsIgnoreCase("true")
+                                                                                        || strTmp.equalsIgnoreCase("1"));
                     break;
                 }