Changeset 70644 in vbox for trunk/src/VBox/Main/src-client/AudioDriver.cpp

Timestamp:

Jan 19, 2018 12:20:33 PM (7 years ago)

Author:

vboxsync

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

120374

Message:

Audio/Main: More code needed for attaching / detaching host backends at runtime.

File:

• trunk/src/VBox/Main/src-client/AudioDriver.cpp (modified) (11 diffs)

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

@@ -35 +35 @@
     : mpConsole(pConsole)
     , mfAttached(false)
-    , muLUN(UINT8_MAX)
 {
 }
@@ -43 +42 @@
 }
 
-/**
- * Returns the next free LUN of the audio device driver
- * chain.
- *
- * @return unsigned             Next free LUN in audio device driver chain.
- */
-unsigned AudioDriver::getFreeLUN(void)
-{
-    Console::SafeVMPtrQuiet ptrVM(mpConsole);
-    Assert(ptrVM.isOk());
-
-    PUVM pUVM = ptrVM.rawUVM();
-    AssertPtr(pUVM);
-
-    unsigned uLUN = 0;
-
-    PCFGMNODE pDevLUN;
-    for (;;)
-    {
-        pDevLUN = CFGMR3GetChildF(CFGMR3GetRootU(pUVM), "Devices/%s/%u/LUN#%u/", mCfg.strDev.c_str(), mCfg.uInst, uLUN);
-        if (!pDevLUN)
-            break;
-        uLUN++;
-    }
-
-    return uLUN;
-}
-
 
 /**
  * Initializes the audio driver with a certain (device) configuration.
- *
- * @note The driver's LUN will be determined on runtime when attaching the
- *       driver to the audio driver chain.
  *
  * @returns VBox status code.
@@ -84 +52 @@
 {
     AssertPtrReturn(pCfg, VERR_INVALID_POINTER);
+
+    /* Sanity. */
+    AssertReturn(pCfg->strDev.isNotEmpty(),  VERR_INVALID_PARAMETER);
+    AssertReturn(pCfg->uLUN != UINT8_MAX,    VERR_INVALID_PARAMETER);
+    AssertReturn(pCfg->strName.isNotEmpty(), VERR_INVALID_PARAMETER);
 
     /* Apply configuration. */
@@ -142 +115 @@
     Assert(ptrVM.isOk());
 
-
     if (pThis->mfAttached) /* Already attached? Bail out. */
     {
@@ -151 +123 @@
     AudioDriverCfg *pCfg = &pThis->mCfg;
 
-    unsigned uLUN = pThis->muLUN;
-    if (uLUN == UINT8_MAX) /* No LUN assigned / configured yet? Retrieve it. */
-        uLUN = pThis->getFreeLUN();
-
     LogFunc(("strName=%s, strDevice=%s, uInst=%u, uLUN=%u\n",
-             pCfg->strName.c_str(), pCfg->strDev.c_str(), pCfg->uInst, uLUN));
-
-    int vrc = pThis->configure(uLUN, true /* Attach */);
-    if (RT_SUCCESS(vrc))
-        vrc = PDMR3DeviceAttach(ptrVM.rawUVM(), pCfg->strDev.c_str(), pCfg->uInst, uLUN, 0 /* fFlags */,
-                                NULL /* ppBase */);
-    if (RT_SUCCESS(vrc))
-    {
-        pThis->muLUN      = uLUN;
+             pCfg->strName.c_str(), pCfg->strDev.c_str(), pCfg->uInst, pCfg->uLUN));
+
+    int rc = pThis->configure(pCfg->uLUN, true /* Attach */);
+    if (RT_SUCCESS(rc))
+        rc = PDMR3DriverAttach(ptrVM.rawUVM(), pCfg->strDev.c_str(), pCfg->uInst, pCfg->uLUN, 0 /* fFlags */,
+                               NULL /* ppBase */);
+    if (RT_SUCCESS(rc))
+    {
         pThis->mfAttached = true;
-
-        LogRel2(("%s: Driver attached (LUN #%u)\n", pCfg->strName.c_str(), pThis->muLUN));
+        LogRel2(("%s: Driver attached (LUN #%u)\n", pCfg->strName.c_str(), pCfg->uLUN));
     }
     else
-        LogRel(("%s: Failed to attach audio driver, rc=%Rrc\n", pCfg->strName.c_str(), vrc));
-
-    return vrc;
+        LogRel(("%s: Failed to attach audio driver, rc=%Rrc\n", pCfg->strName.c_str(), rc));
+
+    LogFunc(("Returning %Rrc\n", rc));
+    return rc;
 }
 
@@ -232 +199 @@
     Assert(ptrVM.isOk());
 
-    Assert(pThis->muLUN != UINT8_MAX);
-
     AudioDriverCfg *pCfg = &pThis->mCfg;
 
+    Assert(pCfg->uLUN != UINT8_MAX);
+
     LogFunc(("strName=%s, strDevice=%s, uInst=%u, uLUN=%u\n",
-             pCfg->strName.c_str(), pCfg->strDev.c_str(), pCfg->uInst, pThis->muLUN));
-
-    int vrc = PDMR3DriverDetach(ptrVM.rawUVM(), pCfg->strDev.c_str(), pCfg->uInst, pThis->muLUN, "AUDIO",
-                            0 /* iOccurrence */, 0 /* fFlags */);
-    if (RT_SUCCESS(vrc))
-        vrc = pThis->configure(pThis->muLUN, false /* Detach */);
-
-    if (RT_SUCCESS(vrc))
-    {
-        pThis->muLUN      = UINT8_MAX;
+             pCfg->strName.c_str(), pCfg->strDev.c_str(), pCfg->uInst, pCfg->uLUN));
+
+    int rc = PDMR3DeviceDetach(ptrVM.rawUVM(), pCfg->strDev.c_str(), pCfg->uInst, pCfg->uLUN, 0 /* fFlags */);
+    if (RT_SUCCESS(rc))
+        rc = pThis->configure(pCfg->uLUN, false /* Detach */);
+
+    if (RT_SUCCESS(rc))
+    {
         pThis->mfAttached = false;
-
         LogRel2(("%s: Driver detached\n", pCfg->strName.c_str()));
     }
     else
-        LogRel(("%s: Failed to detach audio driver, rc=%Rrc\n", pCfg->strName.c_str(), vrc));
-
-    return vrc;
+        LogRel(("%s: Failed to detach audio driver, rc=%Rrc\n", pCfg->strName.c_str(), rc));
+
+    LogFunc(("Returning %Rrc\n", rc));
+    return rc;
 }
 
@@ -268 +233 @@
 int AudioDriver::configure(unsigned uLUN, bool fAttach)
 {
-    int rc = VINF_SUCCESS;
-
     Console::SafeVMPtrQuiet ptrVM(mpConsole);
     Assert(ptrVM.isOk());
@@ -276 +239 @@
     AssertPtr(pUVM);
 
-    PCFGMNODE pRoot   = CFGMR3GetRootU(pUVM);
+    PCFGMNODE pRoot = CFGMR3GetRootU(pUVM);
     AssertPtr(pRoot);
-    PCFGMNODE pDev0   = CFGMR3GetChildF(pRoot, "Devices/%s/%u/", mCfg.strDev.c_str(), mCfg.uInst);
+    PCFGMNODE pDev0 = CFGMR3GetChildF(pRoot, "Devices/%s/%u/", mCfg.strDev.c_str(), mCfg.uInst);
 
     if (!pDev0) /* No audio device configured? Bail out. */
@@ -286 +249 @@
     }
 
+    int rc = VINF_SUCCESS;
+
     PCFGMNODE pDevLun = CFGMR3GetChildF(pDev0, "LUN#%u/", uLUN);
 
     if (fAttach)
     {
-        AssertMsg(uLUN != UINT8_MAX, ("%s: LUN is undefined when it must not\n", mCfg.strName.c_str()));
-
-        if (!pDevLun)
+        do
         {
-            LogRel2(("%s: Configuring audio driver (to LUN #%RU8)\n", mCfg.strName.c_str(), uLUN));
-
-            PCFGMNODE pLunL0;
-            CFGMR3InsertNodeF(pDev0, &pLunL0, "LUN#%RU8", uLUN);
-            CFGMR3InsertString(pLunL0, "Driver", "AUDIO");
-
-            PCFGMNODE pLunCfg;
-            CFGMR3InsertNode(pLunL0,   "Config", &pLunCfg);
-                CFGMR3InsertStringF(pLunCfg, "DriverName",    "%s", mCfg.strName.c_str());
-                CFGMR3InsertInteger(pLunCfg, "InputEnabled",  0); /* Play safe by default. */
-                CFGMR3InsertInteger(pLunCfg, "OutputEnabled", 1);
-
-            PCFGMNODE pLunL1;
-            CFGMR3InsertNode(pLunL0, "AttachedDriver", &pLunL1);
-                CFGMR3InsertStringF(pLunL1, "Driver", "%s", mCfg.strName.c_str());
-
-                CFGMR3InsertNode(pLunL1, "Config", &pLunCfg);
+            AssertMsgBreakStmt(pDevLun, ("%s: Device LUN #%u not found\n", mCfg.strName.c_str(), uLUN), rc = VERR_NOT_FOUND);
+
+            LogRel2(("%s: Configuring audio driver (to LUN #%u)\n", mCfg.strName.c_str(), uLUN));
+
+            PCFGMNODE pLunCfg = CFGMR3GetChild(pDevLun, "Config");
+            AssertBreakStmt(pLunCfg, rc = VERR_NOT_FOUND);
+
+                rc = CFGMR3InsertStringF(pLunCfg, "DriverName",    "%s", mCfg.strName.c_str());    AssertRCBreak(rc);
+
+                rc = CFGMR3InsertInteger(pLunCfg, "InputEnabled",  0); /* Play safe by default. */ AssertRCBreak(rc);
+                rc = CFGMR3InsertInteger(pLunCfg, "OutputEnabled", 1);                             AssertRCBreak(rc);
+
+            PCFGMNODE pAttachedDriver, pAttachedDriverCfg;
+            rc = CFGMR3InsertNode(pDevLun, "AttachedDriver", &pAttachedDriver);                    AssertRCBreak(rc);
+                rc = CFGMR3InsertStringF(pAttachedDriver, "Driver", "%s", mCfg.strName.c_str());   AssertRCBreak(rc);
+                rc = CFGMR3InsertNode(pAttachedDriver, "Config", &pAttachedDriverCfg);             AssertRCBreak(rc);
 
                 /* Call the (virtual) method for driver-specific configuration. */
-                configureDriver(pLunCfg);
-        }
-        else
-            rc = VERR_ALREADY_EXISTS;
+                rc = configureDriver(pAttachedDriverCfg);                                          AssertRCBreak(rc);
+
+        } while (0);
     }
     else /* Detach */
@@ -323 +284 @@
         {
             LogRel2(("%s: Unconfiguring audio driver\n", mCfg.strName.c_str()));
-            CFGMR3RemoveNode(pDevLun);
+
+            PCFGMNODE pLunCfg = CFGMR3GetChild(pDevLun, "Config");
+            if (pLunCfg)
+                CFGMR3RemoveNode(pLunCfg);
+
+            rc = CFGMR3InsertNode(pDevLun, "Config", &pLunCfg);
+
+            PCFGMNODE pLunAttachedDriver = CFGMR3GetChild(pDevLun, "AttachedDriver");
+            if (pLunAttachedDriver)
+                CFGMR3RemoveNode(pLunAttachedDriver);
         }
-        else
-            rc = VERR_NOT_FOUND;
-    }
-
-#ifdef DEBUG_andy
-    CFGMR3Dump(pDev0);
-#endif
+    }
 
     if (RT_FAILURE(rc))
@@ -337 +301 @@
                 mCfg.strName.c_str(), fAttach ? "Configuring" : "Unconfiguring", rc));
 
+    LogFunc(("Returning %Rrc\n", rc));
     return rc;
 }