dev.dpdk.org archive mirror
 help / color / mirror / Atom feed
From: Thomas Monjalon <thomas.monjalon-pdR9zngts4EAvxtiuMwx3w@public.gmane.org>
To: "Wu, Jingjing" <jingjing.wu-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
Cc: dev-VfR2kkLFssw@public.gmane.org
Subject: Re: [PATCH v2 0/4] NIC filters support for generic filter
Date: Wed, 11 Jun 2014 17:45:09 +0200	[thread overview]
Message-ID: <2116871.SYCYqgDKRI@xps13> (raw)
In-Reply-To: <9BB6961774997848B5B42BEC655768F8A78A07-0J0gbvR4kTg/UvCtAeCM4rfspsVTdybXVpNB7YpNyf8@public.gmane.org>

Hi Jingjing,

Please reply below the question.

2014-05-28 01:12, Wu, Jingjing:
> You are discussing whether the APIs provide for NIC filters is generic
> or not. About that we can use same API for a type of filter. For example,
> if we want to configure ethertype filter, we can use the same API, no
> matter the NIC is 82580, i350, 82576 or 82599. We think these NICs may be
> most common used.

I was wondering if this API can apply to non-Intel devices. But nobody talked 
about it. So let's forget it.

My main concern is that Vladimir Medvedkin suggested another API and I'd like 
you give your opinion about it:
	http://dpdk.org/ml/archives/dev/2014-June/003053.html
It offers pool number in configuration of the filters.

Last comment: patches would be more pleasant to read with right alignment and 
spaces in comments. This is an extract to illustrate what I'm talking about:
+       uint16_t priority; /**< used when more than one filter matches */
+       uint8_t dst_port_mask : 1,  /**<if mask is 1b, means not compare*/
+               protocol_mask   : 1;

Thanks
-- 
Thomas

  parent reply	other threads:[~2014-06-11 15:45 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-05-24  1:37 [PATCH v2 0/4] NIC filters support for generic filter Jingjing Wu
     [not found] ` <1400895442-32433-1-git-send-email-jingjing.wu-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2014-05-24  1:37   ` [PATCH v2 1/4]ethdev: add ethdev APIs for NIC filters of " Jingjing Wu
     [not found]     ` <1400895442-32433-2-git-send-email-jingjing.wu-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2014-05-27  7:57       ` Ming, LiX
     [not found]         ` <0976FC66838DDE4585886952506C9436FBA620-0J0gbvR4kTg/UvCtAeCM4rfspsVTdybXVpNB7YpNyf8@public.gmane.org>
2014-05-27  9:18           ` Thomas Monjalon
2014-05-24  1:37   ` [PATCH v2 2/4]e1000: add igb NIC filters of generic filter feature Jingjing Wu
     [not found]     ` <1400895442-32433-3-git-send-email-jingjing.wu-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2014-05-27  7:59       ` Ming, LiX
2014-05-24  1:37   ` [PATCH v2 3/4]ixgbe: add ixgbe " Jingjing Wu
     [not found]     ` <1400895442-32433-4-git-send-email-jingjing.wu-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2014-05-27  8:00       ` Ming, LiX
2014-05-24  1:37   ` [PATCH v2 4/4]app/test-pmd: add commands in testpmd for NIC filters Jingjing Wu
     [not found]     ` <1400895442-32433-5-git-send-email-jingjing.wu-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2014-05-27  8:00       ` Ming, LiX
2014-05-27 23:21   ` [PATCH v2 0/4] NIC filters support for generic filter Thomas Monjalon
2014-05-28  0:53     ` Wu, Jingjing
     [not found]       ` <9BB6961774997848B5B42BEC655768F8A789C6-0J0gbvR4kTg/UvCtAeCM4rfspsVTdybXVpNB7YpNyf8@public.gmane.org>
2014-05-28  1:12         ` Wu, Jingjing
     [not found]           ` <9BB6961774997848B5B42BEC655768F8A78A07-0J0gbvR4kTg/UvCtAeCM4rfspsVTdybXVpNB7YpNyf8@public.gmane.org>
2014-06-11 15:45             ` Thomas Monjalon [this message]
2014-06-12  8:08               ` Wu, Jingjing
     [not found]                 ` <9BB6961774997848B5B42BEC655768F8A8BBB5-0J0gbvR4kTg/UvCtAeCM4rfspsVTdybXVpNB7YpNyf8@public.gmane.org>
2014-06-12 15:36                   ` Thomas Monjalon
2014-06-13 11:51                     ` Vladimir Medvedkin
     [not found]                       ` <CANDrEH=8SeES7N5eybWaCvJdE0JkZpq-DuxSVYdXAXbm=Wf-Ww-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2014-06-13 14:12                         ` Wu, Jingjing
     [not found]                           ` <9BB6961774997848B5B42BEC655768F8A8D402-0J0gbvR4kTg/UvCtAeCM4rfspsVTdybXVpNB7YpNyf8@public.gmane.org>
2014-06-13 16:19                             ` Vladimir Medvedkin
     [not found]                               ` <CANDrEHkPvqmkA6vUukApzTfwJAWz6_R8Lf+RBcJq1J6UNq0uMw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2014-06-14  1:00                                 ` Wu, Jingjing
     [not found]                                   ` <9BB6961774997848B5B42BEC655768F8A8D780-0J0gbvR4kTg/UvCtAeCM4rfspsVTdybXVpNB7YpNyf8@public.gmane.org>
2014-06-14  9:00                                     ` Vladimir Medvedkin
2014-06-04  4:16   ` Cao, Waterman

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=2116871.SYCYqgDKRI@xps13 \
    --to=thomas.monjalon-pdr9zngts4eavxtiumwx3w@public.gmane.org \
    --cc=dev-VfR2kkLFssw@public.gmane.org \
    --cc=jingjing.wu-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).