e1000e: fix loss of multicast packets
authorJesse Brandeburg <jesse.brandeburg@intel.com>
Wed, 25 Mar 2009 22:05:21 +0000 (22:05 +0000)
committerDavid S. Miller <davem@davemloft.net>
Thu, 26 Mar 2009 08:09:59 +0000 (01:09 -0700)
commita72d2b2cc63994cb8d592a004bf5331be6905814
treec55a5b6d1f471e0aae21881d3bce8b047c46baa4
parent12d04a3c12b420f23398b4d650127642469a60a6
e1000e: fix loss of multicast packets

e1000e (and e1000, 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.  It would still
be possible for packets to be dropped while the receiver is off during
reprogramming.

Signed-off-by: Jesse Brandeburg <jesse.brandeburg@intel.com>
CC: Dave Boutcher <daveboutcher@gmail.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/e1000e/lib.c