Message ID | 20190130145900.2614-1-gvaradar@cisco.com |
---|---|
State | Accepted |
Delegated to: | David Miller |
Headers | show |
Series | [net] enic: fix checksum validation for IPv6 | expand |
From: Govindarajulu Varadarajan <gvaradar@cisco.com> Date: Wed, 30 Jan 2019 06:59:00 -0800 > In case of IPv6 pkts, ipv4_csum_ok is 0. Because of this, driver does > not set skb->ip_summed. So IPv6 rx checksum is not offloaded. > > Signed-off-by: Govindarajulu Varadarajan <gvaradar@cisco.com> Applied and queued up for -stable, thanks.
diff --git a/drivers/net/ethernet/cisco/enic/enic_main.c b/drivers/net/ethernet/cisco/enic/enic_main.c index 60641e202534..9a7f70db20c7 100644 --- a/drivers/net/ethernet/cisco/enic/enic_main.c +++ b/drivers/net/ethernet/cisco/enic/enic_main.c @@ -1434,7 +1434,8 @@ static void enic_rq_indicate_buf(struct vnic_rq *rq, * csum is correct or is zero. */ if ((netdev->features & NETIF_F_RXCSUM) && !csum_not_calc && - tcp_udp_csum_ok && ipv4_csum_ok && outer_csum_ok) { + tcp_udp_csum_ok && outer_csum_ok && + (ipv4_csum_ok || ipv6)) { skb->ip_summed = CHECKSUM_UNNECESSARY; skb->csum_level = encap; }
In case of IPv6 pkts, ipv4_csum_ok is 0. Because of this, driver does not set skb->ip_summed. So IPv6 rx checksum is not offloaded. Signed-off-by: Govindarajulu Varadarajan <gvaradar@cisco.com> --- drivers/net/ethernet/cisco/enic/enic_main.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)