e1000: fix loss of multicast packets
authorJesse Brandeburg <jesse.brandeburg@intel.com>
Sat, 4 Apr 2009 23:36:53 +0000 (16:36 -0700)
committerDavid S. Miller <davem@davemloft.net>
Sat, 4 Apr 2009 23:36:53 +0000 (16:36 -0700)
commit81c522851436dbc058c9c0c11b32e60d76b180ce
treed665ad0664f8a362baf496da9d4578dab2d0c397
parent2b05e0021b21e1dc484a1237b7fa674e8a3704bb
e1000: fix loss of multicast packets

e1000 (and e1000e, igb, ixgbe, ixgb) all do a series of
operations each time a multicast address is added.  The flow goes
something like

1) stack adds one multicast address
2) stack passes whole current list of unicast and multicast
   addresses to driver
3) driver clears entire list in hardware
4) driver programs each multicast address using iomem in a loop

This was causing multicast packets to be lost during the
reprogramming process.

reference with test program:
http://kerneltrap.org/mailarchive/linux-netdev/2009/3/14/5160514/thread

Thanks to Dave Boutcher for his report and test program.

This driver fix prepares an array all at once in memory and
programs it in one shot to the hardware, not requiring an "erase"
cycle.

Signed-off-by: Jesse Brandeburg <jesse.brandeburg@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/e1000/e1000_main.c