On Sat, Oct 12, 2019 at 01:55:05PM +0200, Davide Caratti wrote: > this series contains two fixes for TC 'act_mpls', that try to address > two problems that can be observed configuring simple 'push' / 'pop' > operations: > - patch 1/2 avoids dropping non-MPLS packets that pass through the MPLS > 'pop' action. > - patch 2/2 fixes corruption of the L2 header that occurs when 'push' > or 'pop' actions are configured in TC egress path. > > v2: - change commit message in patch 1/2 to better describe that the > patch impacts only TC, thanks to Simon Horman > - fix missing documentation of 'mac_len' in patch 2/2
Thanks for the follow-up, this looks good to me. > Davide Caratti (2): > net: avoid errors when trying to pop MLPS header on non-MPLS packets > net/sched: fix corrupted L2 header with MPLS 'push' and 'pop' actions > > include/linux/skbuff.h | 5 +++-- > net/core/skbuff.c | 21 ++++++++++++--------- > net/openvswitch/actions.c | 5 +++-- > net/sched/act_mpls.c | 12 ++++++++---- > 4 files changed, 26 insertions(+), 17 deletions(-) > > -- > 2.21.0 >
