Changeset 90912 in vbox for trunk/src/VBox/Devices/Audio

Timestamp:

Aug 26, 2021 1:38:34 PM (4 years ago)

Author:

vboxsync

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

146548

Message:

Audio/Validation Kit: More connection mode handling. bugref:10008

Location:

trunk/src/VBox/Devices/Audio

Files:

• AudioTestService.h (modified) (1 diff)
• AudioTestServiceTcp.cpp (modified) (10 diffs)
• DrvHostAudioValidationKit.cpp (modified) (1 diff)
• testcase/tstAudioTestService.cpp (modified) (2 diffs)

trunk/src/VBox/Devices/Audio/AudioTestService.h

@@ -178 +178 @@
 int AudioTestSvcShutdown(PATSSERVER pThis);
 
+/**
+ * Enumeration for the server connection mode.
+ * Only applies to certain transport implementation like TCP/IP.
+ */
+typedef enum ATSCONNMODE
+{
+    /** Both: Uses parallel client and server connection methods (via threads). */
+    ATSCONNMODE_BOTH = 0,
+    /** Client only: Connects to a server. */
+    ATSCONNMODE_CLIENT,
+    /** Server only: Listens for new incoming client connections. */
+    ATSCONNMODE_SERVER,
+    /** 32bit hack. */
+    ATSCONNMODE_32BIT_HACK = 0x7fffffff
+} ATSCONNMODE;
+
 /** TCP/IP options for the ATS server.
  *  @todo Make this more abstract later. */
 enum ATSTCPOPT
 {
-    ATSTCPOPT_MODE = 5000,
+    ATSTCPOPT_CONN_MODE = 5000,
     ATSTCPOPT_BIND_ADDRESS,
     ATSTCPOPT_BIND_PORT,

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

@@ -68 +68 @@
 
 /**
- * Enumeration for the TCP/IP connection mode.
- */
-typedef enum ATSTCPMODE
-{
-    /** Both: Uses parallel client and server connection methods (via threads). */
-    ATSTCPMODE_BOTH = 0,
-    /** Client only: Connects to a server. */
-    ATSTCPMODE_CLIENT,
-    /** Server only: Listens for new incoming client connections. */
-    ATSTCPMODE_SERVER
-} ATSTCPMODE;
-
-/**
  * Structure for keeping Audio Test Service (ATS) transport instance-specific data.
  */
@@ -88 +75 @@
     RTCRITSECT                         CritSect;
     /** Connection mode to use. */
-    ATSTCPMODE                         enmMode;
+    ATSCONNMODE                        enmConnMode;
     /** The addresses to bind to.  Empty string means any. */
     char                               szBindAddr[256];
@@ -314 +301 @@
     int rc;
 
-    LogFunc(("enmMode=%#x\n", pThis->enmMode));
-
-    if (pThis->enmMode == ATSTCPMODE_SERVER)
+    LogFunc(("enmConnMode=%#x\n", pThis->enmConnMode));
+
+    if (pThis->enmConnMode == ATSCONNMODE_SERVER)
     {
         pClient->fFromServer = true;
@@ -322 +309 @@
         LogFunc(("RTTcpServerListen2 -> %Rrc\n", rc));
     }
-    else if (pThis->enmMode == ATSTCPMODE_CLIENT)
+    else if (pThis->enmConnMode == ATSCONNMODE_CLIENT)
     {
         pClient->fFromServer = false;
@@ -339 +326 @@
     else
     {
-        Assert(pThis->enmMode == ATSTCPMODE_BOTH);
+        Assert(pThis->enmConnMode == ATSCONNMODE_BOTH);
 
         /*
@@ -731 +718 @@
     int rc = VINF_SUCCESS;
 
-    if (pThis->enmMode != ATSTCPMODE_CLIENT)
+    if (pThis->enmConnMode != ATSCONNMODE_CLIENT)
     {
         rc = RTTcpServerCreateEx(pThis->szBindAddr[0] ? pThis->szBindAddr : NULL, pThis->uBindPort, &pThis->pTcpServer);
@@ -771 +758 @@
     switch (ch)
     {
-        case ATSTCPOPT_MODE:
-            if (!strcmp(pVal->psz, "both"))
-                pThis->enmMode = ATSTCPMODE_BOTH;
-            else if (!strcmp(pVal->psz, "client"))
-                pThis->enmMode = ATSTCPMODE_CLIENT;
-            else if (!strcmp(pVal->psz, "server"))
-                pThis->enmMode = ATSTCPMODE_SERVER;
-            else
-                return RTMsgErrorRc(VERR_INVALID_PARAMETER, "Invalid TCP mode: '%s'\n", pVal->psz);
+        case ATSTCPOPT_CONN_MODE:
+            pThis->enmConnMode = (ATSCONNMODE)pVal->u32;
             return VINF_SUCCESS;
 
@@ -818 +798 @@
 {
     RTStrmPrintf(pStream,
-                 "  --tcp-mode <both|client|server>\n"
-                 "       Selects the mode of operation.\n"
-                 "       Default: both\n"
-                 "  --tcp-bind-address <address>\n"
+                 "  --tcp-conn-mode <0=both|1=client|2=server>\n"
+                 "       Selects the connection mode.\n"
+                 "       Default: 0 (both)\n"
+                 "  --tcp-bind-addr[ess] <address>\n"
                  "       The address(es) to listen to TCP connection on.  Empty string\n"
                  "       means any address, this is the default.\n"
@@ -827 +807 @@
                  "       The port to listen to TCP connections on.\n"
                  "       Default: %u\n"
-                 "  --tcp-connect-address <address>\n"
+                 "  --tcp-connect-addr[ess] <address>\n"
                  "       The address of the server to try connect to in client mode.\n"
                  "       Default: " ATS_TCP_DEF_CONNECT_GUEST_STR "\n"
@@ -839 +819 @@
 static const RTGETOPTDEF  g_TcpOpts[] =
 {
-    { "--tcp-mode",             ATSTCPOPT_MODE,             RTGETOPT_REQ_STRING },
+    { "--tcp-conn-mode",        ATSTCPOPT_CONN_MODE,        RTGETOPT_REQ_STRING },
+    { "--tcp-bind-addr",        ATSTCPOPT_BIND_ADDRESS,     RTGETOPT_REQ_STRING },
     { "--tcp-bind-address",     ATSTCPOPT_BIND_ADDRESS,     RTGETOPT_REQ_STRING },
     { "--tcp-bind-port",        ATSTCPOPT_BIND_PORT,        RTGETOPT_REQ_UINT16 },
+    { "--tcp-connect-addr",     ATSTCPOPT_CONNECT_ADDRESS,  RTGETOPT_REQ_STRING },
     { "--tcp-connect-address",  ATSTCPOPT_CONNECT_ADDRESS,  RTGETOPT_REQ_STRING },
     { "--tcp-connect-port",     ATSTCPOPT_CONNECT_PORT,     RTGETOPT_REQ_UINT16 }

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

@@ -1114 +1114 @@
         RT_ZERO(Val);
 
-        Val.psz = "server"; /** @ŧodo No client connection mode needed here (yet). Make this configurable via CFGM. */
-        rc2 = AudioTestSvcHandleOption(&pThis->Srv, ATSTCPOPT_MODE, &Val);
+        Val.u32 = ATSCONNMODE_SERVER; /** @todo No client connection mode needed here (yet). Make this configurable via CFGM. */
+        rc2 = AudioTestSvcHandleOption(&pThis->Srv, ATSTCPOPT_CONN_MODE, &Val);
         AssertRC(rc2);
 

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

@@ -84 +84 @@
 
         Val.psz = "server";
-        rc = AudioTestSvcHandleOption(&Srv, ATSTCPOPT_MODE, &Val);
+        rc = AudioTestSvcHandleOption(&Srv, ATSTCPOPT_CONN_MODE, &Val);
         RTTEST_CHECK_RC_OK(hTest, rc);
 
@@ -118 +118 @@
 
                 Val.psz = "client";
-                rc = AudioTestSvcClientHandleOption(&Client, ATSTCPOPT_MODE, &Val);
+                rc = AudioTestSvcClientHandleOption(&Client, ATSTCPOPT_CONN_MODE, &Val);
                 RTTEST_CHECK_RC_OK(hTest, rc);