ibmveth: Fix checksum offload failure handling
authorAnton Blanchard <anton@samba.org>
Wed, 7 Sep 2011 14:41:06 +0000 (14:41 +0000)
committerDavid S. Miller <davem@davemloft.net>
Fri, 16 Sep 2011 19:26:32 +0000 (15:26 -0400)
commitfb82fd204b6e6c67661bbd37df032edafb2da56e
treea97071b481dd80b553b98e2158c951e618cecb80
parent91aae1e5c407d4fc79f6983e6c6ba04756c004cb
ibmveth: Fix checksum offload failure handling

Fix a number of issues in ibmveth_set_csum_offload:

- set_attr6 and clr_attr6 may be used uninitialised

- We store the result of the IPV4 checksum change in ret but overwrite
  it in a couple of places before checking it again later. Add ret4
  to make it obvious what we are doing.

- We weren't clearing the NETIF_F_IP_CSUM and NETIF_F_IPV6_CSUM flags
  if the enable of that hypervisor feature failed.

Signed-off-by: Anton Blanchard <anton@samba.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ibmveth.c