Changeset 28131 in vbox

Timestamp:

Apr 9, 2010 9:50:19 AM (15 years ago)

Author:

vboxsync

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

59854

Message:

#4286: e1000: Force RX checksum in guest software.

File:

• trunk/src/VBox/Devices/Network/DevE1000.cpp (modified) (2 diffs)

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

@@ -1952 +1952 @@
     E1kLog2(("%s e1kRxChecksumOffload: EtherType=%x\n", INSTANCE(pState), uEtherType));
 
-    //pStatus->fIPE   = false;
-    //pStatus->fTCPE  = false;
     switch (uEtherType)
     {
+        /* @todo
+         * It is not safe to bypass checksum verification for packets coming
+         * from real wire. We currently unable to tell where packets are
+         * coming from so we tell the driver to ignore our checksum flags
+         * and do verification in software.
+         */
+#if 0
         case 0x800: /* IPv4 */
             pStatus->fIXSM  = false;
@@ -1968 +1973 @@
             pStatus->fTCPCS = true;
             break;
+#endif
         default: /* ARP, VLAN, etc. */
             pStatus->fIXSM = true;