Changeset 28095 in vbox

Timestamp:

Apr 8, 2010 2:44:41 PM (15 years ago)

Author:

vboxsync

Message:

DrvIntNet.cpp,VBox/param.h: Adjusted the Send buffer so it can fit a 64KB frame. Added VBOX_MAX_GSO_SIZE.

Location:

trunk

Files:

• include/VBox/param.h (modified) (1 diff)
• src/VBox/Devices/Network/DrvIntNet.cpp (modified) (3 diffs)

trunk/include/VBox/param.h

@@ -156 +156 @@
 /** @} */
 
+
+/** @defgroup grp_vbox_param_misc  Misc
+ * @{ */
+
+#define VBOX_MAX_GSO_SIZE           0xfff0
+
+/** @} */
+
+
 /** @} */
 

trunk/src/VBox/Devices/Network/DrvIntNet.cpp

@@ -33 +33 @@
 #include <VBox/err.h>
 
+#include <VBox/param.h>
 #include <VBox/log.h>
 #include <iprt/asm.h>
@@ -1120 +1121 @@
         OpenReq.fFlags |= INTNET_OPEN_FLAGS_SHARED_MAC_ON_WIRE;
 
-    /** @cfgm{ReceiveBufferSize, uint32_t, 218 KB}
+    /** @cfgm{ReceiveBufferSize, uint32_t, 318 KB}
      * The size of the receive buffer.
      */
     rc = CFGMR3QueryU32(pCfg, "ReceiveBufferSize", &OpenReq.cbRecv);
     if (rc == VERR_CFGM_VALUE_NOT_FOUND)
-        OpenReq.cbRecv = 218 * _1K ;
+        OpenReq.cbRecv = 318 * _1K ;
     else if (RT_FAILURE(rc))
         return PDMDRV_SET_ERROR(pDrvIns, rc,
                                 N_("Configuration error: Failed to get the \"ReceiveBufferSize\" value"));
 
-    /** @cfgm{SendBufferSize, uint32_t, 36 KB}
+    /** @cfgm{SendBufferSize, uint32_t, 196 KB}
      * The size of the send (transmit) buffer.
      * This should be more than twice the size of the larges frame size because
@@ -1140 +1141 @@
     rc = CFGMR3QueryU32(pCfg, "SendBufferSize", &OpenReq.cbSend);
     if (rc == VERR_CFGM_VALUE_NOT_FOUND)
-        OpenReq.cbSend = 36*_1K;
+        OpenReq.cbSend = RT_ALIGN_Z(VBOX_MAX_GSO_SIZE * 3, _1K);
     else if (RT_FAILURE(rc))
         return PDMDRV_SET_ERROR(pDrvIns, rc,
                                 N_("Configuration error: Failed to get the \"SendBufferSize\" value"));
-    if (OpenReq.cbSend < 32)
+    if (OpenReq.cbSend < 128)
         return PDMDRV_SET_ERROR(pDrvIns, rc,
                                 N_("Configuration error: The \"SendBufferSize\" value is too small"));
-    if (OpenReq.cbSend < 16384*2 + 64)
-        LogRel(("DrvIntNet: Warning! SendBufferSize=%u, Recommended minimum size %u butes.\n", OpenReq.cbSend, 16384*2 + 64));
+    if (OpenReq.cbSend < VBOX_MAX_GSO_SIZE * 4)
+        LogRel(("DrvIntNet: Warning! SendBufferSize=%u, Recommended minimum size %u butes.\n", OpenReq.cbSend, VBOX_MAX_GSO_SIZE * 4));
 
     /** @cfgm{IsService, boolean, true}