From: "Dumitrescu, Cristian" <cristian.dumitrescu@intel.com>
To: "Richardson, Bruce" <bruce.richardson@intel.com>,
"Singh, Jasvinder" <jasvinder.singh@intel.com>
Cc: "dev@dpdk.org" <dev@dpdk.org>
Subject: Re: [dpdk-dev] [PATCH v2 1/3] lib/librte_port: enable file descriptor port support
Date: Fri, 9 Sep 2016 21:19:57 +0000 [thread overview]
Message-ID: <3EB4FA525960D640B5BDFFD6A3D8912647A67934@IRSMSX108.ger.corp.intel.com> (raw)
In-Reply-To: <20160905101218.GA18384@bricha3-MOBL3>
> -----Original Message-----
> From: Richardson, Bruce
> Sent: Monday, September 5, 2016 11:12 AM
> To: Singh, Jasvinder <jasvinder.singh@intel.com>
> Cc: dev@dpdk.org; Dumitrescu, Cristian <cristian.dumitrescu@intel.com>
> Subject: Re: [dpdk-dev] [PATCH v2 1/3] lib/librte_port: enable file descriptor
> port support
>
> On Sun, Sep 04, 2016 at 03:38:39PM +0100, Jasvinder Singh wrote:
> > This patch adds File Descriptor(FD) port type (e.g. TAP port) to the
> > packet framework library that allows interface with the kernel network
> > stack. The FD port APIs are defined that allow port creation, writing
> > and reading packet from the kernel interface.
> >
> > Signed-off-by: Jasvinder Singh <jasvinder.singh@intel.com>
> > Acked-by: Cristian Dumitrescu <cristian.dumitrescu@intel.com>
> > ---
> > v2:
> > - fix checkpatch warnings
>
> Rather than adding a special TAP port for use by packet framework alone,
> why not
> create a TAP PMD/ethdev and then it can be used both by regular DPDK
> apps, as well
> as by packet framework too - since packet framework already has an ethdev
> port
> type that presumably works with all ethdevs?
>
> /Bruce
Great idea, but we don't have the bandwidth to create a TAP PMD right now. Any volunteers?
Please also note that the non-blocking file descriptor is actually significantly more generic/has broader applicability than just the TAP device, as it can be used to interface with any file descriptor, not just a TAP file descriptor, e.g. any Linux kernel character device or sockets (probably a small number of sockets), etc. I am sure it will prove itself useful to more people and we'll probably find even more places to use it going forward. Unfortunately, AFAIK this cannot be fitted as a PMD right now due to EAL limitations, as this would be virtual device and the file descriptor ID would have to be known before the DPDK application is started and passed to the app through the EAL vdev parameter, right?
This being said, I propose we go ahead with this new type of port. Whenever a TAP PMD becomes available, I don't mind changing the IP pipeline app to use it if people would prefer it. Our primary motivation for adding TAP support to IP pipeline app was to serve as a base of performance comparison between TAP and KNI when the same setup is used, but I am sure there are other potential usages for it.
next prev parent reply other threads:[~2016-09-09 21:20 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-08-05 21:21 [dpdk-dev] [PATCH " Jasvinder Singh
2016-09-04 14:38 ` [dpdk-dev] [PATCH v2 " Jasvinder Singh
2016-09-04 14:38 ` [dpdk-dev] [PATCH v2 2/3] examples/ip_pipeline: integrate TAP port Jasvinder Singh
2016-09-04 14:38 ` [dpdk-dev] [PATCH v2 3/3] examples/ip_pipeline: add sample config file with TAP port usage Jasvinder Singh
[not found] ` <20160905101218.GA18384@bricha3-MOBL3>
2016-09-09 21:19 ` Dumitrescu, Cristian [this message]
2016-10-12 20:33 ` [dpdk-dev] [PATCH v2 1/3] lib/librte_port: enable file descriptor port support Thomas Monjalon
2016-10-12 20:44 ` Dumitrescu, Cristian
2016-10-14 15:08 ` Thomas Monjalon
2016-10-13 9:17 ` [dpdk-dev] [PATCH v3 " Jasvinder Singh
2016-10-13 9:17 ` [dpdk-dev] [PATCH v3 2/3] examples/ip_pipeline: integrate TAP port Jasvinder Singh
2016-10-13 9:17 ` [dpdk-dev] [PATCH v3 3/3] examples/ip_pipeline: add sample config file with TAP port usage Jasvinder Singh
2016-10-13 12:38 ` [dpdk-dev] [PATCH v3 1/3] lib/librte_port: enable file descriptor port support Thomas Monjalon
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=3EB4FA525960D640B5BDFFD6A3D8912647A67934@IRSMSX108.ger.corp.intel.com \
--to=cristian.dumitrescu@intel.com \
--cc=bruce.richardson@intel.com \
--cc=dev@dpdk.org \
--cc=jasvinder.singh@intel.com \
/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).