> -----Original Message-----
> From: Xing, Beilei <[email protected]>
> Sent: Wednesday, February 9, 2022 3:31 PM
> To: Wang, Jie1X <[email protected]>; [email protected]
> Cc: Yang, SteveX <[email protected]>; [email protected]; Singh, Aman
> Deep <[email protected]>; Yigit, Ferruh <[email protected]>;
> [email protected]; [email protected]; Wu, Jingjing
> <[email protected]>; Zhang, Qi Z <[email protected]>;
> [email protected]
> Subject: RE: [PATCH v4 6/6] net/iavf: support L2TPv2 for AVF FDIR
>
>
>
> > -----Original Message-----
> > From: Wang, Jie1X <[email protected]>
> > Sent: Tuesday, February 8, 2022 4:39 PM
> > To: [email protected]
> > Cc: Yang, SteveX <[email protected]>; [email protected]; Singh,
> > Aman Deep <[email protected]>; Yigit, Ferruh
> > <[email protected]>; [email protected];
> > [email protected]; Wu, Jingjing <[email protected]>;
> > Xing, Beilei <[email protected]>; Zhang, Qi Z
> > <[email protected]>; [email protected]; Wang, Jie1X
> > <[email protected]>
> > Subject: [PATCH v4 6/6] net/iavf: support L2TPv2 for AVF FDIR
> >
> > Add support for L2TPv2(include PPP over L2TPv2) protocols FDIR based
> > on outer MAC src address and L2TPv2 session ID.
> >
> > Add support for PPPoL2TPv2oUDP protocols FDIR based on inner IP
> > src/dst address and UDP/TCP src/dst port.
> >
> > Patterns are listed below:
> > eth/ipv4(6)/udp/l2tpv2
> > eth/ipv4(6)/udp/l2tpv2/ppp
> >
> > eth/ipv4(6)/udp/l2tpv2/ppp/ipv4(6)
> > eth/ipv4(6)/udp/l2tpv2/ppp/ipv4(6)/udp
> > eth/ipv4(6)/udp/l2tpv2/ppp/ipv4(6)/tcp
> >
> > Signed-off-by: Jie Wang <[email protected]>
> > ---
> > doc/guides/rel_notes/release_22_03.rst | 8 +-
> > drivers/net/iavf/iavf_fdir.c | 174 +++++++++++++++++++++----
> > drivers/net/iavf/iavf_generic_flow.h | 4 +
> > 3 files changed, 156 insertions(+), 30 deletions(-)
> >
> > diff --git a/doc/guides/rel_notes/release_22_03.rst
> > b/doc/guides/rel_notes/release_22_03.rst
> > index 0d1e4a0b61..5a73ccc14e 100644
> > --- a/doc/guides/rel_notes/release_22_03.rst
> > +++ b/doc/guides/rel_notes/release_22_03.rst
> > @@ -66,8 +66,12 @@ New Features
> >
> > * **Updated Intel iavf driver.**
> >
> > - Added L2TPv2(include PPP over L2TPv2) RSS hash distribute packets
> > - based on outer MAC src address and L2TPv2 session ID.
> > + * Added L2TPv2(include PPP over L2TPv2) RSS hash distribute packets
> > + based on outer MAC src address and L2TPv2 session ID.
>
> Should fix in the patch 5/6.
>
> > + * Added L2TPv2(include PPP over L2TPv2) FDIR distribute packets
> > + based on outer MAC src address and L2TPv2 session ID.
> > + * Added PPPoL2TPv2oUDP FDIR distribute packets based on inner IP
> > + src/dst address and UDP/TCP src/dst port.
> >
> >
> > Removed Items
> > diff --git a/drivers/net/iavf/iavf_fdir.c
> > b/drivers/net/iavf/iavf_fdir.c index b63aaca91d..2583b899aa 100644
> > --- a/drivers/net/iavf/iavf_fdir.c
> > +++ b/drivers/net/iavf/iavf_fdir.c
> > @@ -168,6 +168,31 @@
> > IAVF_FDIR_INSET_GRE_IPV6 | IAVF_INSET_TUN_UDP_SRC_PORT | \
> > IAVF_INSET_TUN_UDP_DST_PORT)
> >
> > +#define IAVF_FDIR_INSET_L2TPV2 (\
> > + IAVF_INSET_SMAC | IAVF_INSET_L2TPV2)
>
> The same comment for FDIR: should we limit with source MAC?
>
It is same as the RSS hash input set.
> > +
> > +#define IAVF_FDIR_INSET_L2TPV2_PPP_IPV4 (\
> > + IAVF_INSET_TUN_IPV4_SRC | IAVF_INSET_TUN_IPV4_DST)
> > +
>
> <...>
>
> BR,
> Beilei
