[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-Id: <20180410.110435.2099683036018893839.davem@davemloft.net>
Date: Tue, 10 Apr 2018 11:04:35 -0400 (EDT)
From: David Miller <davem@...emloft.net>
To: sd@...asysnail.net
Cc: netdev@...r.kernel.org, lorenzo.bianconi@...hat.com,
lucien.xin@...il.com, u9012063@...il.com
Subject: Re: [PATCH net] ip_gre: clear feature flags when incompatible
o_flags are set
From: Sabrina Dubroca <sd@...asysnail.net>
Date: Tue, 10 Apr 2018 12:57:18 +0200
> Commit dd9d598c6657 ("ip_gre: add the support for i/o_flags update via
> netlink") added the ability to change o_flags, but missed that the
> GSO/LLTX features are disabled by default, and only enabled some gre
> features are unused. Thus we also need to disable the GSO/LLTX features
> on the device when the TUNNEL_SEQ or TUNNEL_CSUM flags are set.
>
> These two examples should result in the same features being set:
>
> ip link add gre_none type gre local 192.168.0.10 remote 192.168.0.20 ttl 255 key 0
>
> ip link set gre_none type gre seq
> ip link add gre_seq type gre local 192.168.0.10 remote 192.168.0.20 ttl 255 key 1 seq
>
> Fixes: dd9d598c6657 ("ip_gre: add the support for i/o_flags update via netlink")
> Signed-off-by: Sabrina Dubroca <sd@...asysnail.net>
Applied and queued up for -stable, thank you.
Powered by blists - more mailing lists