[dpdk-dev] [PATCH v2 0/6] new ntuple filter replaces 2tuple and 5tuple filters

2015-01-30 Thread Wu, Jingjing


> -Original Message-
> From: De Lara Guarch, Pablo
> Sent: Wednesday, January 28, 2015 10:29 PM
> To: Wu, Jingjing; dev at dpdk.org
> Cc: Cao, Min; Xu, HuilongX
> Subject: RE: [PATCH v2 0/6] new ntuple filter replaces 2tuple and 5tuple
> filters
> 
> 
> 
> > -Original Message-
> > From: Wu, Jingjing
> > Sent: Thursday, January 22, 2015 7:38 AM
> > To: dev at dpdk.org
> > Cc: Wu, Jingjing; De Lara Guarch, Pablo; Cao, Min; Xu, HuilongX
> > Subject: [PATCH v2 0/6] new ntuple filter replaces 2tuple and 5tuple
> > filters
> >
> > v2 changes:
> >   - remove the code which is already applied in patch "Integrate ethertype
> > filter in igb/ixgbe driver to new API".
> >   - modify commands' description in doc testpmd_funcs.rst.
> >
> > The patch set uses filter_ctrl API to replace old 2tuple and 5tuple filter 
> > APIs.
> > It defines ntuple filter to combine 2tuple and 5tuple types.
> > It uses new functions and structure to replace old ones in igb/ixgbe
> > driver, new commands to replace old ones in testpmd, and removes the
> old APIs.
> > It removes the filter's index parameters from user interface, only the
> > filter's key and assigned queue are visible to user.
> >
> > Jingjing Wu (6):
> >   ethdev: define ntuple filter type and its structure
> >   ixgbe: ntuple filter functions replace old ones for 5tuple filter
> >   e1000: ntuple filter functions replace old ones for 2tuple and 5tuple
> > filter
> >   testpmd: new commands for ntuple filter
> >   ethdev: remove old APIs and structures of 5tuple and 2tuple filters
> >   doc: commands changed in testpmd_funcs for 2tuple amd 5tuple filter
> >
> >  app/test-pmd/cmdline.c  | 406 ++---
> >  app/test-pmd/config.c   |  65 ---
> >  doc/guides/testpmd_app_ug/testpmd_funcs.rst |  99 +---
> >  lib/librte_ether/rte_eth_ctrl.h |  57 ++
> >  lib/librte_ether/rte_ethdev.c   | 116 
> >  lib/librte_ether/rte_ethdev.h   | 192 --
> >  lib/librte_pmd_e1000/e1000_ethdev.h |  69 ++-
> >  lib/librte_pmd_e1000/igb_ethdev.c   | 869 +++---
> ---
> > ---
> >  lib/librte_pmd_ixgbe/ixgbe_ethdev.c | 468 +++
> >  lib/librte_pmd_ixgbe/ixgbe_ethdev.h |  52 +-
> >  10 files changed, 1300 insertions(+), 1093 deletions(-)
> >
> > --
> > 1.9.3
> 
> Acked-by: Pablo de Lara 
> 
> Just mind that the last patch (changing the documentation) does not apply
> properly,  as there was another patch (from you I think), that modifies that
> document.
> Could you send another version of the last patch?
> Not sure if that's OK or if it is better to send the full patchset again.
> 

Thank you, Pablo.
Yes. It's due to another patch of doc change is applied before this one.
But I think many patches will update this the document. Let's wait to
see whether there are more comments about this patch set at first. 




[dpdk-dev] [PATCH v2 0/6] new ntuple filter replaces 2tuple and 5tuple filters

2015-01-28 Thread De Lara Guarch, Pablo


> -Original Message-
> From: Wu, Jingjing
> Sent: Thursday, January 22, 2015 7:38 AM
> To: dev at dpdk.org
> Cc: Wu, Jingjing; De Lara Guarch, Pablo; Cao, Min; Xu, HuilongX
> Subject: [PATCH v2 0/6] new ntuple filter replaces 2tuple and 5tuple filters
> 
> v2 changes:
>   - remove the code which is already applied in patch "Integrate ethertype
> filter in igb/ixgbe driver to new API".
>   - modify commands' description in doc testpmd_funcs.rst.
> 
> The patch set uses filter_ctrl API to replace old 2tuple and 5tuple filter 
> APIs.
> It defines ntuple filter to combine 2tuple and 5tuple types.
> It uses new functions and structure to replace old ones in igb/ixgbe driver,
> new commands to replace old ones in testpmd, and removes the old APIs.
> It removes the filter's index parameters from user interface, only the
> filter's key and assigned queue are visible to user.
> 
> Jingjing Wu (6):
>   ethdev: define ntuple filter type and its structure
>   ixgbe: ntuple filter functions replace old ones for 5tuple filter
>   e1000: ntuple filter functions replace old ones for 2tuple and 5tuple
> filter
>   testpmd: new commands for ntuple filter
>   ethdev: remove old APIs and structures of 5tuple and 2tuple filters
>   doc: commands changed in testpmd_funcs for 2tuple amd 5tuple filter
> 
>  app/test-pmd/cmdline.c  | 406 ++---
>  app/test-pmd/config.c   |  65 ---
>  doc/guides/testpmd_app_ug/testpmd_funcs.rst |  99 +---
>  lib/librte_ether/rte_eth_ctrl.h |  57 ++
>  lib/librte_ether/rte_ethdev.c   | 116 
>  lib/librte_ether/rte_ethdev.h   | 192 --
>  lib/librte_pmd_e1000/e1000_ethdev.h |  69 ++-
>  lib/librte_pmd_e1000/igb_ethdev.c   | 869 +++--
> ---
>  lib/librte_pmd_ixgbe/ixgbe_ethdev.c | 468 +++
>  lib/librte_pmd_ixgbe/ixgbe_ethdev.h |  52 +-
>  10 files changed, 1300 insertions(+), 1093 deletions(-)
> 
> --
> 1.9.3

Acked-by: Pablo de Lara 

Just mind that the last patch (changing the documentation) does not apply 
properly,
 as there was another patch (from you I think), that modifies that document.
Could you send another version of the last patch? 
Not sure if that's OK or if it is better to send the full patchset again.



[dpdk-dev] [PATCH v2 0/6] new ntuple filter replaces 2tuple and 5tuple filters

2015-01-22 Thread Jingjing Wu
v2 changes:
  - remove the code which is already applied in patch "Integrate ethertype
filter in igb/ixgbe driver to new API".
  - modify commands' description in doc testpmd_funcs.rst.

The patch set uses filter_ctrl API to replace old 2tuple and 5tuple filter APIs.
It defines ntuple filter to combine 2tuple and 5tuple types. 
It uses new functions and structure to replace old ones in igb/ixgbe driver,
new commands to replace old ones in testpmd, and removes the old APIs.
It removes the filter's index parameters from user interface, only the
filter's key and assigned queue are visible to user.

Jingjing Wu (6):
  ethdev: define ntuple filter type and its structure
  ixgbe: ntuple filter functions replace old ones for 5tuple filter
  e1000: ntuple filter functions replace old ones for 2tuple and 5tuple
filter
  testpmd: new commands for ntuple filter
  ethdev: remove old APIs and structures of 5tuple and 2tuple filters
  doc: commands changed in testpmd_funcs for 2tuple amd 5tuple filter

 app/test-pmd/cmdline.c  | 406 ++---
 app/test-pmd/config.c   |  65 ---
 doc/guides/testpmd_app_ug/testpmd_funcs.rst |  99 +---
 lib/librte_ether/rte_eth_ctrl.h |  57 ++
 lib/librte_ether/rte_ethdev.c   | 116 
 lib/librte_ether/rte_ethdev.h   | 192 --
 lib/librte_pmd_e1000/e1000_ethdev.h |  69 ++-
 lib/librte_pmd_e1000/igb_ethdev.c   | 869 +++-
 lib/librte_pmd_ixgbe/ixgbe_ethdev.c | 468 +++
 lib/librte_pmd_ixgbe/ixgbe_ethdev.h |  52 +-
 10 files changed, 1300 insertions(+), 1093 deletions(-)

-- 
1.9.3