Messages in this thread | | | Date | Wed, 3 Aug 2022 13:27:32 +0200 | From | Guillaume Nault <> | Subject | Re: [PATCH v2 net 0/4] Do not use RT_TOS for IPv6 flowlabel |
| |
On Tue, Aug 02, 2022 at 02:09:31PM +0200, Matthias May wrote: > According to Guillaume Nault RT_TOS should never be used for IPv6. > > Quote: > RT_TOS() is an old macro used to interprete IPv4 TOS as described in > the obsolete RFC 1349. It's conceptually wrong to use it even in IPv4 > code, although, given the current state of the code, most of the > existing calls have no consequence. > > But using RT_TOS() in IPv6 code is always a bug: IPv6 never had a "TOS" > field to be interpreted the RFC 1349 way. There's no historical > compatibility to worry about.
Apart from the not so informative commit messages, I'm fine with this series. Please keep my acked-by on all patches if you send a v3.
Thanks again for fixing this.
Acked-by: Guillaume Nault <gnault@redhat.com>
> --- > v1 -> v2: > - Fix spacing of "Fixes" tag. > - Add missing CCs > > Matthias May (4): > geneve: do not use RT_TOS for IPv6 flowlabel > vxlan: do not use RT_TOS for IPv6 flowlabel > mlx5: do not use RT_TOS for IPv6 flowlabel > ipv6: do not use RT_TOS for IPv6 flowlabel > > drivers/net/ethernet/mellanox/mlx5/core/en/tc_tun.c | 4 ++-- > drivers/net/geneve.c | 3 +-- > drivers/net/vxlan/vxlan_core.c | 2 +- > net/ipv6/ip6_output.c | 3 +-- > 4 files changed, 5 insertions(+), 7 deletions(-) > > -- > 2.35.1 >
| |