From mboxrd@z Thu Jan  1 00:00:00 1970
From: Thomas Monjalon <thomas.monjalon-pdR9zngts4EAvxtiuMwx3w@public.gmane.org>
Subject: Re: [PATCH][PMD][GENERIC_FILTER] add NIC filters support
	for generic filter feature
Date: Mon, 19 May 2014 12:17:12 +0200
Message-ID: <2436549.UhLCUXjEFZ@xps13>
References: <1400319309-25231-1-git-send-email-jingjing.wu@intel.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7Bit
Cc: dev-VfR2kkLFssw@public.gmane.org
To: Jingjing Wu <jingjing.wu-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
Return-path: <dev-bounces-VfR2kkLFssw@public.gmane.org>
In-Reply-To: <1400319309-25231-1-git-send-email-jingjing.wu-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
List-Id: patches and discussions about DPDK <dev.dpdk.org>
List-Unsubscribe: <http://dpdk.org/ml/options/dev>,
 <mailto:dev-request-VfR2kkLFssw@public.gmane.org?subject=unsubscribe>
List-Archive: <http://dpdk.org/ml/archives/dev/>
List-Post: <mailto:dev-VfR2kkLFssw@public.gmane.org>
List-Help: <mailto:dev-request-VfR2kkLFssw@public.gmane.org?subject=help>
List-Subscribe: <http://dpdk.org/ml/listinfo/dev>,
 <mailto:dev-request-VfR2kkLFssw@public.gmane.org?subject=subscribe>
Errors-To: dev-bounces-VfR2kkLFssw@public.gmane.org
Sender: "dev" <dev-bounces-VfR2kkLFssw@public.gmane.org>

Hi Jingjing,

2014-05-17 17:35, Jingjing Wu:
> A generic filter mechanism for handling special packet is required. It will
> allows filters  to be set in HW when available for so that specific packets
> may be filtered by NICs to specific desriptor queues for processing.
> Currently only the Flow Director for Intel's 10GbE 82599 devices is
> available. Other types of filter  are not supported.
> 
> This pacth adds following NIC filters used to assign different packets to
> certain receive queues. ethertype filter/syn filter/2tuple filter/flex
> filter for E1000(82580, i350) ethertype filter/syn filter/5tuple filter for
> 10G(82599)
> 
> Signed-off-by: jingjing.wu <jingjing.wu-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
> ---
>  app/test-pmd/cmdline.c              | 905
> +++++++++++++++++++++++++++++++++++- app/test-pmd/config.c               |
> 143 ++++++
>  app/test-pmd/testpmd.h              |   6 +
>  lib/librte_ether/rte_ethdev.c       | 300 ++++++++++++
>  lib/librte_ether/rte_ethdev.h       | 429 ++++++++++++++++-
>  lib/librte_pmd_e1000/e1000_ethdev.h |  38 ++
>  lib/librte_pmd_e1000/igb_ethdev.c   | 512 ++++++++++++++++++++
>  lib/librte_pmd_ixgbe/ixgbe_ethdev.c | 365 +++++++++++++++
>  lib/librte_pmd_ixgbe/ixgbe_ethdev.h |  33 ++
>  9 files changed, 2729 insertions(+), 2 deletions(-)

It would be really easier to review if you split it in 4 parts:
- ethdev API
- igb implementation
- ixgbe implementation
- testpmd

Thanks
-- 
Thomas