Changeset 88906 in vbox for trunk/src/VBox/Devices/Audio/DevSB16.cpp

Timestamp:

May 6, 2021 2:24:47 PM (4 years ago)

Author:

vboxsync

Message:

Audio/Dev*: Overhauled the attach/detach code a little, making attach fail if the configured driver doesn't have a PDMIADUIOCONNECTOR interface among other things. [more] bugref:9890

File:

• trunk/src/VBox/Devices/Audio/DevSB16.cpp (modified) (5 diffs)

trunk/src/VBox/Devices/Audio/DevSB16.cpp

@@ -2850 +2850 @@
 
 /**
- * Attach command, internal version.
- *
- * This is called to let the device attach to a driver for a specified LUN
- * during runtime. This is not called during VM construction, the device
- * constructor has to attach to all the available drivers.
+ * Worker for sb16Construct() and sb16Attach().
  *
  * @returns VBox status code.
  * @param   pThis       SB16 state.
  * @param   uLUN        The logical unit which is being detached.
- * @param   fFlags      Flags, combination of the PDMDEVATT_FLAGS_* \#defines.
  * @param   ppDrv       Attached driver instance on success. Optional.
  */
-static int sb16AttachInternal(PSB16STATE pThis, unsigned uLUN, uint32_t fFlags, PSB16DRIVER *ppDrv)
-{
-    RT_NOREF(fFlags);
-
+static int sb16AttachInternal(PSB16STATE pThis, unsigned uLUN, PSB16DRIVER *ppDrv)
+{
     /*
      * Allocate a new driver structure and try attach the driver.
@@ -2903 +2896 @@
             if (ppDrv)
                 *ppDrv = pDrv;
-            LogFunc(("iLUN=%u, fFlags=0x%x: VINF_SUCCESS\n", uLUN, fFlags));
+            LogFunc(("LUN#%u: VINF_SUCCESS\n", uLUN));
             return VINF_SUCCESS;
         }
@@ -2914 +2907 @@
     RTMemFree(pDrv);
 
-    LogFunc(("iLUN=%u, fFlags=0x%x: rc=%Rrc\n", uLUN, fFlags, rc));
+    LogFunc(("LUN#%u: rc=%Rrc\n", uLUN, rc));
     return rc;
 }
@@ -2924 +2917 @@
 {
     PSB16STATE pThis = PDMDEVINS_2_DATA(pDevIns, PSB16STATE);
-
-    LogFunc(("iLUN=%u, fFlags=0x%x\n", iLUN, fFlags));
+    RT_NOREF(fFlags);
+    LogFunc(("iLUN=%u, fFlags=%#x\n", iLUN, fFlags));
 
     /** @todo r=andy Any locking required here? */
 
     PSB16DRIVER pDrv;
-    int rc = sb16AttachInternal(pThis, iLUN, fFlags, &pDrv);
+    int rc = sb16AttachInternal(pThis, iLUN, &pDrv);
     if (RT_SUCCESS(rc))
     {
@@ -3263 +3256 @@
         AssertBreak(iLun < UINT8_MAX);
         LogFunc(("Trying to attach driver for LUN#%u ...\n", iLun));
-        rc = sb16AttachInternal(pThis, iLun, 0 /* fFlags */, NULL /* ppDrv */);
+        rc = sb16AttachInternal(pThis, iLun, NULL /* ppDrv */);
         if (rc == VERR_PDM_NO_ATTACHED_DRIVER)
         {