Re: [Xen-devel] [PATCH] xen-netfront: reset skb transport header before checksum

2015-05-12 Thread Venkat Venkatsubra
> > This looks bogus for IPv6. > > I would rather fix skb_checksum_setup_ip() and > skb_checksum_setup_ipv6(), since you already use correctly > skb_maybe_pull_tail() there. > > In general, setting network and transport header at this point seems > extra work. > Yes, I missed the IPv6 case. ;-

Re: [Xen-devel] [PATCH] xen-netfront: reset skb transport header before checksum

2015-05-11 Thread Eric Dumazet
On Mon, 2015-05-11 at 18:34 -0700, Venkat Venkatsubra wrote: > In ed1f50c3a ("net: add skb_checksum_setup") some checksum functions > were introduced in core. Subsequent change b5cf66cd1 ("xen-netfront: > use new skb_checksum_setup function") made use of those functions to > replace its own impleme

[Xen-devel] [PATCH] xen-netfront: reset skb transport header before checksum

2015-05-11 Thread Venkat Venkatsubra
In ed1f50c3a ("net: add skb_checksum_setup") some checksum functions were introduced in core. Subsequent change b5cf66cd1 ("xen-netfront: use new skb_checksum_setup function") made use of those functions to replace its own implementation. With that change ip_hdr() and tcp_hdr() were not pointing at