Re: [PATCH net-next] geneve: always fill CSUM6_RX configuration

2017-05-22 Thread Eric Garver
On Sat, May 20, 2017 at 09:56:44PM -0700, Pravin Shelar wrote: > On Sat, May 20, 2017 at 6:35 AM, Eric Garver wrote: > > On Fri, May 19, 2017 at 06:57:46PM -0700, Pravin Shelar wrote: > >> On Thu, May 18, 2017 at 12:59 PM, Eric Garver wrote: > >> > CSMU6_RX is

Re: [PATCH net-next] geneve: always fill CSUM6_RX configuration

2017-05-20 Thread Pravin Shelar
On Sat, May 20, 2017 at 6:35 AM, Eric Garver wrote: > On Fri, May 19, 2017 at 06:57:46PM -0700, Pravin Shelar wrote: >> On Thu, May 18, 2017 at 12:59 PM, Eric Garver wrote: >> > CSMU6_RX is relevant for collect_metadata as well. As such leave it >> > outside of the

Re: [PATCH net-next] geneve: always fill CSUM6_RX configuration

2017-05-20 Thread Eric Garver
On Fri, May 19, 2017 at 06:57:46PM -0700, Pravin Shelar wrote: > On Thu, May 18, 2017 at 12:59 PM, Eric Garver wrote: > > CSMU6_RX is relevant for collect_metadata as well. As such leave it > > outside of the dev's IPv4/IPv6 checks. > > > Can you explain it bit? is this flag used

Re: [PATCH net-next] geneve: always fill CSUM6_RX configuration

2017-05-19 Thread Pravin Shelar
On Thu, May 18, 2017 at 12:59 PM, Eric Garver wrote: > CSMU6_RX is relevant for collect_metadata as well. As such leave it > outside of the dev's IPv4/IPv6 checks. > Can you explain it bit? is this flag used with ipv4 tunnels? > Fixes: 9b4437a5b870 ("geneve: Unify LWT and netdev

[PATCH net-next] geneve: always fill CSUM6_RX configuration

2017-05-18 Thread Eric Garver
CSMU6_RX is relevant for collect_metadata as well. As such leave it outside of the dev's IPv4/IPv6 checks. Fixes: 9b4437a5b870 ("geneve: Unify LWT and netdev handling.") Signed-off-by: Eric Garver --- drivers/net/geneve.c | 8 1 file changed, 4 insertions(+), 4