net: generic dev_disable_lro() stacked device handling
authorMichal Kubeček <mkubecek@suse.cz>
Thu, 13 Nov 2014 06:54:50 +0000 (07:54 +0100)
committerDavid S. Miller <davem@davemloft.net>
Thu, 13 Nov 2014 19:48:56 +0000 (14:48 -0500)
commitfbe168ba91f7c327856f205699404284c2f09e36
treecc69d067c84f90e105f098127fdd25330c5baf99
parentb6267d3e803b9adf73893f932f7c7f3a2167c908
net: generic dev_disable_lro() stacked device handling

Large receive offloading is known to cause problems if received packets
are passed to other host. Therefore the kernel disables it by calling
dev_disable_lro() whenever a network device is enslaved in a bridge or
forwarding is enabled for it (or globally). For virtual devices we need
to disable LRO on the underlying physical device (which is actually
receiving the packets).

Current dev_disable_lro() code handles this  propagation for a vlan
(including 802.1ad nested vlan), macvlan or a vlan on top of a macvlan.
It doesn't handle other stacked devices and their combinations, in
particular propagation from a bond to its slaves which often causes
problems in virtualization setups.

As we now have generic data structures describing the upper-lower device
relationship, dev_disable_lro() can be generalized to disable LRO also
for all lower devices (if any) once it is disabled for the device
itself.

For bonding and teaming devices, it is necessary to disable LRO not only
on current slaves at the moment when dev_disable_lro() is called but
also on any slave (port) added later.

v2: use lower device links for all devices (including vlan and macvlan)

Signed-off-by: Michal Kubecek <mkubecek@suse.cz>
Acked-by: Veaceslav Falico <vfalico@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/bonding/bond_main.c
drivers/net/team/team.c
net/core/dev.c