mbox series

[v2,0/3] Support outer layer FDIR for GTPoGRE packet

Message ID 20210706074258.1345934-1-lingyu.liu@intel.com (mailing list archive)
Headers
Series Support outer layer FDIR for GTPoGRE packet |

Message

Liu, Lingyu July 6, 2021, 7:42 a.m. UTC
  Add support for outer most L3 layer FDIR for GTPoGRE packet. GTPU TEID
FDIR is supported for GTPoGRE flow pattern without inner L3,L4
fields. GTPU enhance header is supported.

Lingyu Liu (3):
  net/iavf: add flow pattern for GTPoGRE
  net/iavf: add AVF FDIR for GTPoGRE tunnel packet
  doc: release note for GTPoGRE FDIR
---
  V2 change:
  - refine release note

 doc/guides/rel_notes/release_21_08.rst |  3 +-
 drivers/net/iavf/iavf_fdir.c           |  8 +++
 drivers/net/iavf/iavf_generic_flow.c   | 84 ++++++++++++++++++++++++++
 drivers/net/iavf/iavf_generic_flow.h   |  8 +++
 4 files changed, 102 insertions(+), 1 deletion(-)
  

Comments

Qi Zhang July 6, 2021, 2:40 a.m. UTC | #1
> -----Original Message-----
> From: Liu, Lingyu <lingyu.liu@intel.com>
> Sent: Tuesday, July 6, 2021 3:43 PM
> To: dev@dpdk.org; Zhang, Qi Z <qi.z.zhang@intel.com>; Xing, Beilei
> <beilei.xing@intel.com>; Wu, Jingjing <jingjing.wu@intel.com>
> Cc: Guo, Junfeng <junfeng.guo@intel.com>; Liu, Lingyu <lingyu.liu@intel.com>
> Subject: [PATCH v2 0/3] Support outer layer FDIR for GTPoGRE packet
> 
> Add support for outer most L3 layer FDIR for GTPoGRE packet. GTPU TEID FDIR
> is supported for GTPoGRE flow pattern without inner L3,L4 fields. GTPU
> enhance header is supported.
> 
> Lingyu Liu (3):
>   net/iavf: add flow pattern for GTPoGRE
>   net/iavf: add AVF FDIR for GTPoGRE tunnel packet
>   doc: release note for GTPoGRE FDIR
> ---
>   V2 change:
>   - refine release note
> 
>  doc/guides/rel_notes/release_21_08.rst |  3 +-
>  drivers/net/iavf/iavf_fdir.c           |  8 +++
>  drivers/net/iavf/iavf_generic_flow.c   | 84 ++++++++++++++++++++++++++
>  drivers/net/iavf/iavf_generic_flow.h   |  8 +++
>  4 files changed, 102 insertions(+), 1 deletion(-)
> 
> --
> 2.25.1

Acked-by: Qi Zhang <qi.z.zhang@intel.com>

Applied to dpdk-next-net-intel.

Thanks
Qi
  
Thomas Monjalon July 6, 2021, 10:10 p.m. UTC | #2
06/07/2021 04:40, Zhang, Qi Z:
> From: Liu, Lingyu <lingyu.liu@intel.com>
> > Add support for outer most L3 layer FDIR for GTPoGRE packet. GTPU TEID FDIR
> > is supported for GTPoGRE flow pattern without inner L3,L4 fields. GTPU
> > enhance header is supported.
> > 
> > Lingyu Liu (3):
> >   net/iavf: add flow pattern for GTPoGRE
> >   net/iavf: add AVF FDIR for GTPoGRE tunnel packet
> >   doc: release note for GTPoGRE FDIR
> 
> Acked-by: Qi Zhang <qi.z.zhang@intel.com>
> 
> Applied to dpdk-next-net-intel.

The difference with the series "support AVF RSS and FDIR for GTPoGRE packet"
is not reflected in titles.
I prefer not pulling these 2 series so you can rework the titles
and release notes in dpdk-next-net-intel.
Note: the release notes should be updated in each patch, not separately.
  
Qi Zhang July 6, 2021, 11:27 p.m. UTC | #3
> -----Original Message-----
> From: Thomas Monjalon <thomas@monjalon.net>
> Sent: Wednesday, July 7, 2021 6:11 AM
> To: Zhang, Qi Z <qi.z.zhang@intel.com>
> Cc: Liu, Lingyu <lingyu.liu@intel.com>; dev@dpdk.org; Xing, Beilei
> <beilei.xing@intel.com>; Wu, Jingjing <jingjing.wu@intel.com>; Guo,
> Junfeng <junfeng.guo@intel.com>; Yigit, Ferruh <ferruh.yigit@intel.com>
> Subject: Re: [dpdk-dev] [PATCH v2 0/3] Support outer layer FDIR for
> GTPoGRE packet
> 
> 06/07/2021 04:40, Zhang, Qi Z:
> > From: Liu, Lingyu <lingyu.liu@intel.com>
> > > Add support for outer most L3 layer FDIR for GTPoGRE packet. GTPU
> > > TEID FDIR is supported for GTPoGRE flow pattern without inner L3,L4
> > > fields. GTPU enhance header is supported.
> > >
> > > Lingyu Liu (3):
> > >   net/iavf: add flow pattern for GTPoGRE
> > >   net/iavf: add AVF FDIR for GTPoGRE tunnel packet
> > >   doc: release note for GTPoGRE FDIR
> >
> > Acked-by: Qi Zhang <qi.z.zhang@intel.com>
> >
> > Applied to dpdk-next-net-intel.
> 
> The difference with the series "support AVF RSS and FDIR for GTPoGRE
> packet"
> is not reflected in titles.
> I prefer not pulling these 2 series so you can rework the titles and release
> notes in dpdk-next-net-intel.
> Note: the release notes should be updated in each patch, not separately.
> 

OK, I think we can merge 2 series into one as well as the following fix.
  
Qi Zhang July 7, 2021, 7:25 a.m. UTC | #4
> -----Original Message-----
> From: Liu, Lingyu <lingyu.liu@intel.com>
> Sent: Wednesday, July 7, 2021 8:58 PM
> To: dev@dpdk.org; Zhang, Qi Z <qi.z.zhang@intel.com>; Xing, Beilei
> <beilei.xing@intel.com>; Wu, Jingjing <jingjing.wu@intel.com>
> Cc: Guo, Junfeng <junfeng.guo@intel.com>; Liu, Lingyu <lingyu.liu@intel.com>
> Subject: [PATCH v1 0/4] support AVF RSS and FDIR for GTPoGRE packet
> 
> Support AVF RSS and FDIR for GTPoGRE packet.
> 
> Lingyu Liu (4):
>   net/iavf: support flow pattern for GTPoGRE
>   net/iavf: support AVF FDIR for GTPoGRE tunnel packet
>   net/iavf: support AVF RSS for GTPoGRE packet
>   doc: update iavf driver FDIR/RSS for GTPoGRE
> 
>  doc/guides/rel_notes/release_21_08.rst |   2 +
>  drivers/net/iavf/iavf_fdir.c           |  56 ++
>  drivers/net/iavf/iavf_generic_flow.c   | 684 +++++++++++++++++++++++++
>  drivers/net/iavf/iavf_generic_flow.h   |  88 ++++
>  drivers/net/iavf/iavf_hash.c           | 142 ++++-
>  5 files changed, 948 insertions(+), 24 deletions(-)
> 
> --
> 2.25.1

Applied to dpdk-next-net-intel.
Squash the release notes update into patch [3/4].

Thanks
Qi
  
Liu, Lingyu July 7, 2021, 12:57 p.m. UTC | #5
Support AVF RSS and FDIR for GTPoGRE packet.

Lingyu Liu (4):
  net/iavf: support flow pattern for GTPoGRE
  net/iavf: support AVF FDIR for GTPoGRE tunnel packet
  net/iavf: support AVF RSS for GTPoGRE packet
  doc: update iavf driver FDIR/RSS for GTPoGRE

 doc/guides/rel_notes/release_21_08.rst |   2 +
 drivers/net/iavf/iavf_fdir.c           |  56 ++
 drivers/net/iavf/iavf_generic_flow.c   | 684 +++++++++++++++++++++++++
 drivers/net/iavf/iavf_generic_flow.h   |  88 ++++
 drivers/net/iavf/iavf_hash.c           | 142 ++++-
 5 files changed, 948 insertions(+), 24 deletions(-)