Changeset 64514 in vbox for trunk/src/VBox/Devices

Timestamp:

Nov 1, 2016 7:10:01 PM (8 years ago)

Author:

vboxsync

Message:

Dev/E1000: (bugref:8624) Discard RX packets until the link is actually up

File:

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

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

@@ -2658 +2658 @@
     else
     {
+        /*
+         * When the guest changes 'Set Link Up' bit from 0 to 1 we check if
+         * the link is down and the cable is connected, and if they are we
+         * bring the link up, see @bugref{8624}.
+         */
         if (   (value & CTRL_SLU)
+            && !(CTRL & CTRL_SLU)
             && pThis->fCableConnected
             && !(STATUS & STATUS_LU))
@@ -2665 +2671 @@
             /* Do so in 5 seconds (by default). */
             e1kBringLinkUpDelayed(pThis);
-            /*
-             * Change the status (but not PHY status) anyway as Windows expects
-             * it for 82543GC.
-             */
-            STATUS |= STATUS_LU;
         }
         if (value & CTRL_VME)