mbox series

[v2,0/3] net/i40e: add FDIR ID to vector rx

Message ID 20191009152006.5768-1-harry.van.haaren@intel.com (mailing list archive)
Headers
Series net/i40e: add FDIR ID to vector rx |

Message

Van Haaren, Harry Oct. 9, 2019, 3:20 p.m. UTC 
  v2:
- Improve RSS clearning in SSE 16B descriptor
- Fix AVX 16B descriptor data handling

---

Hey All,

This series adds FDIR ID support to the vector Rx routines.
Support for both the SSE and AVX drivers is added, and both
the 16 byte and 32 byte descriptors are implemented.

A v1/RFC for this functionality was sent in 19.08 timeframe,
refer to the patch as archived on patchwork here:
http://patches.dpdk.org/patch/53969/

Regards, -Harry

Harry van Haaren (3):
  net/i40e: cache fdir enable value in rx queue
  net/i40e: add flow mark capability to SSE vector routine
  net/i40e: add flow director support to avx rx path

 drivers/net/i40e/i40e_ethdev.h        |   1 +
 drivers/net/i40e/i40e_fdir.c          |  20 +++
 drivers/net/i40e/i40e_flow.c          |   4 +
 drivers/net/i40e/i40e_rxtx.c          |   6 -
 drivers/net/i40e/i40e_rxtx.h          |   8 ++
 drivers/net/i40e/i40e_rxtx_vec_avx2.c | 193 +++++++++++++++++++++++++-
 drivers/net/i40e/i40e_rxtx_vec_sse.c  | 154 ++++++++++++++++++--
 7 files changed, 367 insertions(+), 19 deletions(-)

Comments

Qi Zhang Oct. 10, 2019, 12:02 a.m. UTC | #1 
> -----Original Message-----
> From: Van Haaren, Harry
> Sent: Wednesday, October 9, 2019 11:20 PM
> To: dev@dpdk.org
> Cc: Zhang, Qi Z <qi.z.zhang@intel.com>; Ergin, Mesut A
> <mesut.a.ergin@intel.com>; Van Haaren, Harry
> <harry.van.haaren@intel.com>
> Subject: [PATCH v2 0/3] net/i40e: add FDIR ID to vector rx
> 
> v2:
> - Improve RSS clearning in SSE 16B descriptor
> - Fix AVX 16B descriptor data handling
> 
> ---
> 
> Hey All,
> 
> This series adds FDIR ID support to the vector Rx routines.
> Support for both the SSE and AVX drivers is added, and both the 16 byte and
> 32 byte descriptors are implemented.
> 
> A v1/RFC for this functionality was sent in 19.08 timeframe, refer to the
> patch as archived on patchwork here:
> http://patches.dpdk.org/patch/53969/
> 
> Regards, -Harry
> 
> Harry van Haaren (3):
>   net/i40e: cache fdir enable value in rx queue
>   net/i40e: add flow mark capability to SSE vector routine
>   net/i40e: add flow director support to avx rx path
> 
>  drivers/net/i40e/i40e_ethdev.h        |   1 +
>  drivers/net/i40e/i40e_fdir.c          |  20 +++
>  drivers/net/i40e/i40e_flow.c          |   4 +
>  drivers/net/i40e/i40e_rxtx.c          |   6 -
>  drivers/net/i40e/i40e_rxtx.h          |   8 ++
>  drivers/net/i40e/i40e_rxtx_vec_avx2.c | 193
> +++++++++++++++++++++++++-  drivers/net/i40e/i40e_rxtx_vec_sse.c  |
> 154 ++++++++++++++++++--
>  7 files changed, 367 insertions(+), 19 deletions(-)
> 
> --
> 2.17.1

Thanks for enabling this!
Acked-by: Qi Zhang <qi.z.zhang@intel.com>
Ergin, Mesut A Oct. 10, 2019, 4:23 p.m. UTC | #2 
> -----Original Message-----
> From: Zhang, Qi Z
> Sent: Wednesday, October 9, 2019 5:02 PM
> To: Van Haaren, Harry <harry.van.haaren@intel.com>; dev@dpdk.org
> Cc: Ergin, Mesut A <mesut.a.ergin@intel.com>
> Subject: RE: [PATCH v2 0/3] net/i40e: add FDIR ID to vector rx
> 
> 
> 
> > -----Original Message-----
> > From: Van Haaren, Harry
> > Sent: Wednesday, October 9, 2019 11:20 PM
> > To: dev@dpdk.org
> > Cc: Zhang, Qi Z <qi.z.zhang@intel.com>; Ergin, Mesut A
> > <mesut.a.ergin@intel.com>; Van Haaren, Harry
> > <harry.van.haaren@intel.com>
> > Subject: [PATCH v2 0/3] net/i40e: add FDIR ID to vector rx
> >
> > v2:
> > - Improve RSS clearning in SSE 16B descriptor
> > - Fix AVX 16B descriptor data handling
> >
> > ---
> >
> > Hey All,
> >
> > This series adds FDIR ID support to the vector Rx routines.
> > Support for both the SSE and AVX drivers is added, and both the 16 byte and
> > 32 byte descriptors are implemented.
> >
> > A v1/RFC for this functionality was sent in 19.08 timeframe, refer to the
> > patch as archived on patchwork here:
> > http://patches.dpdk.org/patch/53969/
> >
> > Regards, -Harry
> >
> > Harry van Haaren (3):
> >   net/i40e: cache fdir enable value in rx queue
> >   net/i40e: add flow mark capability to SSE vector routine
> >   net/i40e: add flow director support to avx rx path
> >
> >  drivers/net/i40e/i40e_ethdev.h        |   1 +
> >  drivers/net/i40e/i40e_fdir.c          |  20 +++
> >  drivers/net/i40e/i40e_flow.c          |   4 +
> >  drivers/net/i40e/i40e_rxtx.c          |   6 -
> >  drivers/net/i40e/i40e_rxtx.h          |   8 ++
> >  drivers/net/i40e/i40e_rxtx_vec_avx2.c | 193
> > +++++++++++++++++++++++++-  drivers/net/i40e/i40e_rxtx_vec_sse.c  |
> > 154 ++++++++++++++++++--
> >  7 files changed, 367 insertions(+), 19 deletions(-)
> >
> > --
> > 2.17.1
> 
> Thanks for enabling this!
> Acked-by: Qi Zhang <qi.z.zhang@intel.com>

This patchset tested out as expected without any regressions on my setup.

Tested-by: Mesut A Ergin <mesut.a.ergin@intel.com>
Xiaolong Ye Oct. 21, 2019, 6:04 a.m. UTC | #3 
On 10/09, Harry van Haaren wrote:
>v2:
>- Improve RSS clearning in SSE 16B descriptor
>- Fix AVX 16B descriptor data handling
>
>---
>
>Hey All,
>
>This series adds FDIR ID support to the vector Rx routines.
>Support for both the SSE and AVX drivers is added, and both
>the 16 byte and 32 byte descriptors are implemented.
>
>A v1/RFC for this functionality was sent in 19.08 timeframe,
>refer to the patch as archived on patchwork here:
>http://patches.dpdk.org/patch/53969/
>
>Regards, -Harry
>
>Harry van Haaren (3):
>  net/i40e: cache fdir enable value in rx queue
>  net/i40e: add flow mark capability to SSE vector routine
>  net/i40e: add flow director support to avx rx path
>
> drivers/net/i40e/i40e_ethdev.h        |   1 +
> drivers/net/i40e/i40e_fdir.c          |  20 +++
> drivers/net/i40e/i40e_flow.c          |   4 +
> drivers/net/i40e/i40e_rxtx.c          |   6 -
> drivers/net/i40e/i40e_rxtx.h          |   8 ++
> drivers/net/i40e/i40e_rxtx_vec_avx2.c | 193 +++++++++++++++++++++++++-
> drivers/net/i40e/i40e_rxtx_vec_sse.c  | 154 ++++++++++++++++++--
> 7 files changed, 367 insertions(+), 19 deletions(-)
>
>-- 
>2.17.1
>

Series applied to dpdk-next-net-intel.

Thanks,
Xiaolong