mbox series

[v6,0/3] Enable Protocol Agnostic Flow Offloading in AVF

Message ID 20220523023138.3777313-1-junfeng.guo@intel.com (mailing list archive)
Headers
Series Enable Protocol Agnostic Flow Offloading in AVF |

Message

Junfeng Guo May 23, 2022, 2:31 a.m. UTC
  This patch set enabled Protocol Agnostic Flow (raw flow) Offloading
for FDIR and RSS in AVF, based on the Parser Library feature and the
existing rte_flow `raw` API.

[PATCH v6 1/3] common/iavf: support raw packet in protocol header
[PATCH v6 2/3] net/iavf: enable Protocol Agnostic Flow Offloading FDIR
[PATCH v6 3/3] net/iavf: support Protocol Agnostic Flow Offloading VF RSS

v6:
merge two commit into one for single commit compilation.

v5:
code rebase and update commit messages.

v4:
add support raw flow for RSS in AVF.

v3:
fix CI build issue.

v2:
add release notes and document update.


Junfeng Guo (2):
  common/iavf: support raw packet in protocol header
  net/iavf: enable Protocol Agnostic Flow Offloading FDIR

Ting Xu (1):
  net/iavf: support Protocol Agnostic Flow Offloading VF RSS

 doc/guides/rel_notes/release_22_07.rst |   1 +
 drivers/common/iavf/virtchnl.h         |  20 +-
 drivers/net/iavf/iavf_fdir.c           |  67 ++++++
 drivers/net/iavf/iavf_generic_flow.c   |   6 +
 drivers/net/iavf/iavf_generic_flow.h   |   3 +
 drivers/net/iavf/iavf_hash.c           | 276 +++++++++++++++++--------
 6 files changed, 281 insertions(+), 92 deletions(-)
  

Comments

Qi Zhang May 23, 2022, 5:09 a.m. UTC | #1
> -----Original Message-----
> From: Guo, Junfeng <junfeng.guo@intel.com>
> Sent: Monday, May 23, 2022 10:32 AM
> To: Zhang, Qi Z <qi.z.zhang@intel.com>; Wu, Jingjing <jingjing.wu@intel.com>;
> Xing, Beilei <beilei.xing@intel.com>
> Cc: dev@dpdk.org; Xu, Ting <ting.xu@intel.com>; Guo, Junfeng
> <junfeng.guo@intel.com>
> Subject: [PATCH v6 0/3] Enable Protocol Agnostic Flow Offloading in AVF
> 
> This patch set enabled Protocol Agnostic Flow (raw flow) Offloading for FDIR
> and RSS in AVF, based on the Parser Library feature and the existing rte_flow
> `raw` API.
> 
> [PATCH v6 1/3] common/iavf: support raw packet in protocol header [PATCH v6
> 2/3] net/iavf: enable Protocol Agnostic Flow Offloading FDIR [PATCH v6 3/3]
> net/iavf: support Protocol Agnostic Flow Offloading VF RSS
> 
> v6:
> merge two commit into one for single commit compilation.
> 
> v5:
> code rebase and update commit messages.
> 
> v4:
> add support raw flow for RSS in AVF.
> 
> v3:
> fix CI build issue.
> 
> v2:
> add release notes and document update.
> 
> 
> Junfeng Guo (2):
>   common/iavf: support raw packet in protocol header
>   net/iavf: enable Protocol Agnostic Flow Offloading FDIR
> 
> Ting Xu (1):
>   net/iavf: support Protocol Agnostic Flow Offloading VF RSS
> 
>  doc/guides/rel_notes/release_22_07.rst |   1 +
>  drivers/common/iavf/virtchnl.h         |  20 +-
>  drivers/net/iavf/iavf_fdir.c           |  67 ++++++
>  drivers/net/iavf/iavf_generic_flow.c   |   6 +
>  drivers/net/iavf/iavf_generic_flow.h   |   3 +
>  drivers/net/iavf/iavf_hash.c           | 276 +++++++++++++++++--------
>  6 files changed, 281 insertions(+), 92 deletions(-)
> 
> --
> 2.25.1

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

Applied to dpdk-next-net-intel.

Thanks
Qi
  
Thomas Monjalon June 5, 2022, 5:43 p.m. UTC | #2
23/05/2022 07:09, Zhang, Qi Z:
> > Junfeng Guo (2):
> >   common/iavf: support raw packet in protocol header
> >   net/iavf: enable Protocol Agnostic Flow Offloading FDIR
> > 
> > Ting Xu (1):
> >   net/iavf: support Protocol Agnostic Flow Offloading VF RSS
> > 
> >  doc/guides/rel_notes/release_22_07.rst |   1 +
> >  drivers/common/iavf/virtchnl.h         |  20 +-
> >  drivers/net/iavf/iavf_fdir.c           |  67 ++++++
> >  drivers/net/iavf/iavf_generic_flow.c   |   6 +
> >  drivers/net/iavf/iavf_generic_flow.h   |   3 +
> >  drivers/net/iavf/iavf_hash.c           | 276 +++++++++++++++++--------
> >  6 files changed, 281 insertions(+), 92 deletions(-)
> > 
> > --
> > 2.25.1
> 
> Acked-by: Qi Zhang <qi.z.zhang@intel.com>
> 
> Applied to dpdk-next-net-intel.

You should not have merged this, it is triggering an error
with devtools/check-doc-vs-code.sh

rte_flow doc out of sync for iavf
        item raw

Once again, I will fix it while pulling the tree.
There are a lot of mistakes in this tree,
I hope you understand why I don't have time to pull it
as frequently as you would like.
  
Qi Zhang June 5, 2022, 11:10 p.m. UTC | #3
> -----Original Message-----
> From: Thomas Monjalon <thomas@monjalon.net>
> Sent: Monday, June 6, 2022 1:43 AM
> To: Zhang, Qi Z <qi.z.zhang@intel.com>
> Cc: Guo, Junfeng <junfeng.guo@intel.com>; Wu, Jingjing
> <jingjing.wu@intel.com>; Xing, Beilei <beilei.xing@intel.com>; dev@dpdk.org;
> Xu, Ting <ting.xu@intel.com>; ferruh.yigit@amd.com; Mcnamara, John
> <john.mcnamara@intel.com>; david.marchand@redhat.com
> Subject: Re: [PATCH v6 0/3] Enable Protocol Agnostic Flow Offloading in AVF
> 
> 23/05/2022 07:09, Zhang, Qi Z:
> > > Junfeng Guo (2):
> > >   common/iavf: support raw packet in protocol header
> > >   net/iavf: enable Protocol Agnostic Flow Offloading FDIR
> > >
> > > Ting Xu (1):
> > >   net/iavf: support Protocol Agnostic Flow Offloading VF RSS
> > >
> > >  doc/guides/rel_notes/release_22_07.rst |   1 +
> > >  drivers/common/iavf/virtchnl.h         |  20 +-
> > >  drivers/net/iavf/iavf_fdir.c           |  67 ++++++
> > >  drivers/net/iavf/iavf_generic_flow.c   |   6 +
> > >  drivers/net/iavf/iavf_generic_flow.h   |   3 +
> > >  drivers/net/iavf/iavf_hash.c           | 276 +++++++++++++++++--------
> > >  6 files changed, 281 insertions(+), 92 deletions(-)
> > >
> > > --
> > > 2.25.1
> >
> > Acked-by: Qi Zhang <qi.z.zhang@intel.com>
> >
> > Applied to dpdk-next-net-intel.
> 
> You should not have merged this, it is triggering an error with devtools/check-
> doc-vs-code.sh

my bad, I was reminded by Ferruh previously.
I should add check-doc-vs-code.sh into my routine operation.   

> 
> rte_flow doc out of sync for iavf
>         item raw
> 
> Once again, I will fix it while pulling the tree.

Thanks

> There are a lot of mistakes in this tree, I hope you understand why I don't have
> time to pull it as frequently as you would like.

Understand your concern, I can't guarantee there will be no mistake, but I will try to avoid the same mistake happen again.

>