sky2: prevent dual port receiver problems
authorStephen Hemminger <shemminger@osdl.org>
Thu, 11 May 2006 22:07:28 +0000 (15:07 -0700)
committerStephen Hemminger <shemminger@osdl.org>
Mon, 15 May 2006 19:35:01 +0000 (12:35 -0700)
When both ports are receiving simultaneously, the receive logic gets confused
and may pass up a packet before it is full. This causes hangs, and IP will see
lots of garbage packets. There is even the potential for data corruption if
a later arriving packet DMA's into freed memory.

It looks like a hardware bug because status arrives for a packet but no
data is there. Until this bug is worked out, block the user from bringing
up both ports at once.

Signed-off-by: Stephen Hemminger <shemminger@osdl.org>
drivers/net/sky2.c

index ffd267fab21da998ed0a349716ba4594706b9aec..62be6d99d05cd3623e1d0df74cff1495adc9b47c 100644 (file)
@@ -1020,8 +1020,19 @@ static int sky2_up(struct net_device *dev)
        struct sky2_hw *hw = sky2->hw;
        unsigned port = sky2->port;
        u32 ramsize, rxspace, imask;
-       int err = -ENOMEM;
+       int err;
+       struct net_device *otherdev = hw->dev[sky2->port^1];
 
+       /* Block bringing up both ports at the same time on a dual port card.
+        * There is an unfixed bug where receiver gets confused and picks up
+        * packets out of order. Until this is fixed, prevent data corruption.
+        */
+       if (otherdev && netif_running(otherdev)) {
+               printk(KERN_INFO PFX "dual port support is disabled.\n");
+               return -EBUSY;
+       }
+
+       err = -ENOMEM;
        if (netif_msg_ifup(sky2))
                printk(KERN_INFO PFX "%s: enabling interface\n", dev->name);