Changeset 11538

Timestamp:

08/21/08 17:16:07 (3 months ago)

Author:

vboxsync

Message:

pcnet: Rearranged the giant frame detection, adding the VLAN EtherType? to iprt/net.h. Btw. to make things easier to read, don't do if (error) do-lots; else do-what-we're-here-for; because then the reader will have to skip lots of code when following the normal execution path. Also, there is hardly any point in using the 'register' keyword these days.

Files:

• trunk/include/iprt/net.h (modified) (1 diff)
• trunk/src/VBox/Devices/Network/DevPCNet.cpp (modified) (4 diffs)
• trunk/src/VBox/Runtime/testcase/tstUuid.cpp (modified) (1 diff)

trunk/include/iprt/net.h

@@ -134 +134 @@
-#define RTNET_ETHERTYPE_ARP     UINT16_C(0x0806)
-#define RTNET_ETHERTYPE_IPV6    UINT16_C(0x86dd)
+#define RTNET_ETHERTYPE_VLAN    UINT16_C(0x8100)
-#define RTNET_ETHERTYPE_IPX_1   UINT16_C(0x8037)
-#define RTNET_ETHERTYPE_IPX_2   UINT16_C(0x8137)

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

@@ -64 +64 @@
-#include <iprt/string.h>
-#include <iprt/time.h>
+#include <iprt/net.h>
-#ifdef IN_RING3
-# include <iprt/mem.h>
@@ -4386 +4387 @@
-    PCNetState *pThis = INETWORKPORT_2_DATA(pInterface);
-    int         rc;
-    static bool firstBigFrameLoss = true;
-
-    STAM_PROFILE_ADV_START(&pThis->StatReceive, a);
@@ -4393 +4393 @@
-
-    /*
-Determine max ethernet frame size with vlan tag (TPID == 0x8100) or without it
-(excluding CRC which is not transferred)
-It consists of 14-byte header [+ 4-byte vlan tag] +
+Check for the max ethernet frame size, taking the IEEE 802.1Q (VLAN) tag into
+account. Note that we are *not* expecting the CRC Checksum
+Ethernet frames consists of a 14-byte header [+ 4-byte vlan tag] + a
-     */
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
+
+
+
-    {
-        if (cb > 70) /* unqualified guess */
@@ -4418 +4406 @@
-        pThis->Led.Actual.s.fReading = 0;
-    }
+#ifdef LOG_ENABLED
+    else
+    {
+        static bool s_fFirstBigFrameLoss = true;
+        unsigned cbMaxFrame = ((PCRTNETETHERHDR)pvBuf)->EtherType == RT_H2BE_U16_C(RTNET_ETHERTYPE_VLAN)
+                            ? 1518 : 1514;
+        if (s_fFirstBigFrameLoss)
+        {
+            s_fFirstBigFrameLoss = false;
+            Log(("PCNet#%d: Received giant frame %zu, max %u. (Further giants will be reported at level5.)\n",
+                 PCNET_INST_NR, cb, cbMaxFrame));
+        }
+        else
+            Log5(("PCNet#%d: Received giant frame %zu bytes, max %u.\n",
+                  PCNET_INST_NR, cb, cbMaxFrame));
+    }
+#endif /* LOG_ENABLED */
-
-    PDMCritSectLeave(&pThis->CritSect);

trunk/src/VBox/Runtime/testcase/tstUuid.cpp

@@ -106 +106 @@
-    CHECK_EXPR(memcmp(&Uuid3, &Uuid, sizeof(Uuid)) == 0);
-
-0
+1
-    /*
-     * checking the clock seq and time hi and version bits...