mbox series

[0/7] support RXDID22 and FDID22

Message ID 20200826075501.50052-1-guinanx.sun@intel.com (mailing list archive)
Headers
Series support RXDID22 and FDID22 |

Message

Guinan Sun Aug. 26, 2020, 7:54 a.m. UTC 
  Support flexible descriptor RXDID22 and FDID offload in vPMD.
Remove devargs flow-mark-support.

Guinan Sun (4):
  net/ice: remove devargs flow-mark-support
  net/ice: add flow director enabled switch value
  net/ice: support Flex Rx desc and flow mark in AVX path
  net/ice: support Flex Rx desc and flow mark in SSE path

Junyu Jiang (3):
  net/ice: change RSS hash parsing in AVX path
  net/ice: change RSS hash parsing in SSE path
  net/ice: support flexible descriptor RxDID #22

 doc/guides/nics/ice.rst               |  12 --
 drivers/net/ice/ice_ethdev.c          |  10 +-
 drivers/net/ice/ice_ethdev.h          |   3 +-
 drivers/net/ice/ice_fdir_filter.c     |   9 +-
 drivers/net/ice/ice_rxtx.c            |  16 +--
 drivers/net/ice/ice_rxtx.h            |  72 ++++++++++++
 drivers/net/ice/ice_rxtx_vec_avx2.c   | 162 +++++++++++++++++++++++++-
 drivers/net/ice/ice_rxtx_vec_common.h |   6 -
 drivers/net/ice/ice_rxtx_vec_sse.c    | 138 +++++++++++++++++++---
 9 files changed, 371 insertions(+), 57 deletions(-)

Comments

Qi Zhang Sept. 7, 2020, 5:43 a.m. UTC | #1 
> -----Original Message-----
> From: Sun, GuinanX <guinanx.sun@intel.com>
> Sent: Wednesday, August 26, 2020 3:55 PM
> To: dev@dpdk.org
> Cc: Zhang, Qi Z <qi.z.zhang@intel.com>; Yang, Qiming
> <qiming.yang@intel.com>; Sun, GuinanX <guinanx.sun@intel.com>
> Subject: [PATCH 0/7] support RXDID22 and FDID22
> 
> Support flexible descriptor RXDID22 and FDID offload in vPMD.
> Remove devargs flow-mark-support.
> 
> Guinan Sun (4):
>   net/ice: remove devargs flow-mark-support
>   net/ice: add flow director enabled switch value
>   net/ice: support Flex Rx desc and flow mark in AVX path
>   net/ice: support Flex Rx desc and flow mark in SSE path
> 
> Junyu Jiang (3):
>   net/ice: change RSS hash parsing in AVX path
>   net/ice: change RSS hash parsing in SSE path
>   net/ice: support flexible descriptor RxDID #22

Can you combine above 3 patches into 1 and reorder other 4 patches as below？
 
net/ice: add flow director enabled switch value
net/ice: support Flex Rx desc and flow mark in AVX path
net/ice: support Flex Rx desc and flow mark in SSE path
net/ice: remove devargs flow-mark-support

so it will not break regression test.
> 
>  doc/guides/nics/ice.rst               |  12 --
>  drivers/net/ice/ice_ethdev.c          |  10 +-
>  drivers/net/ice/ice_ethdev.h          |   3 +-
>  drivers/net/ice/ice_fdir_filter.c     |   9 +-
>  drivers/net/ice/ice_rxtx.c            |  16 +--
>  drivers/net/ice/ice_rxtx.h            |  72 ++++++++++++
>  drivers/net/ice/ice_rxtx_vec_avx2.c   | 162 +++++++++++++++++++++++++-
>  drivers/net/ice/ice_rxtx_vec_common.h |   6 -
>  drivers/net/ice/ice_rxtx_vec_sse.c    | 138 +++++++++++++++++++---
>  9 files changed, 371 insertions(+), 57 deletions(-)
> 
> --
> 2.17.1
Junyu Jiang Sept. 7, 2020, 5:55 a.m. UTC | #2 
> -----Original Message-----
> From: Zhang, Qi Z <qi.z.zhang@intel.com>
> Sent: Monday, September 7, 2020 1:44 PM
> To: Sun, GuinanX <guinanx.sun@intel.com>; Jiang, JunyuX
> <junyux.jiang@intel.com>; dev@dpdk.org
> Cc: Yang, Qiming <qiming.yang@intel.com>
> Subject: RE: [PATCH 0/7] support RXDID22 and FDID22
> 
> 
> 
> 
> 
> > -----Original Message-----
> > From: Sun, GuinanX <guinanx.sun@intel.com>
> > Sent: Wednesday, August 26, 2020 3:55 PM
> > To: dev@dpdk.org
> > Cc: Zhang, Qi Z <qi.z.zhang@intel.com>; Yang, Qiming
> > <qiming.yang@intel.com>; Sun, GuinanX <guinanx.sun@intel.com>
> > Subject: [PATCH 0/7] support RXDID22 and FDID22
> >
> > Support flexible descriptor RXDID22 and FDID offload in vPMD.
> > Remove devargs flow-mark-support.
> >
> > Guinan Sun (4):
> >   net/ice: remove devargs flow-mark-support
> >   net/ice: add flow director enabled switch value
> >   net/ice: support Flex Rx desc and flow mark in AVX path
> >   net/ice: support Flex Rx desc and flow mark in SSE path
> >
> > Junyu Jiang (3):
> >   net/ice: change RSS hash parsing in AVX path
> >   net/ice: change RSS hash parsing in SSE path
> >   net/ice: support flexible descriptor RxDID #22
> 
> Can you combine above 3 patches into 1 and reorder other 4 patches as
> below？
>
OK.
 
> net/ice: add flow director enabled switch value
> net/ice: support Flex Rx desc and flow mark in AVX path
> net/ice: support Flex Rx desc and flow mark in SSE path
> net/ice: remove devargs flow-mark-support
> 
> so it will not break regression test.
> >
> >  doc/guides/nics/ice.rst               |  12 --
> >  drivers/net/ice/ice_ethdev.c          |  10 +-
> >  drivers/net/ice/ice_ethdev.h          |   3 +-
> >  drivers/net/ice/ice_fdir_filter.c     |   9 +-
> >  drivers/net/ice/ice_rxtx.c            |  16 +--
> >  drivers/net/ice/ice_rxtx.h            |  72 ++++++++++++
> >  drivers/net/ice/ice_rxtx_vec_avx2.c   | 162
> +++++++++++++++++++++++++-
> >  drivers/net/ice/ice_rxtx_vec_common.h |   6 -
> >  drivers/net/ice/ice_rxtx_vec_sse.c    | 138 +++++++++++++++++++---
> >  9 files changed, 371 insertions(+), 57 deletions(-)
> >
> > --
> > 2.17.1