DPDK patches and discussions
 help / color / mirror / Atom feed
From: Bruce Richardson <bruce.richardson@intel.com>
To: Yoshinobu Inoue <inoue.yoshinobu@jp.fujitsu.com>
Cc: dev@dpdk.org
Subject: Re: [dpdk-dev] 2.3 Roadmap
Date: Tue, 1 Dec 2015 11:58:16 +0000	[thread overview]
Message-ID: <20151201115816.GA29824@bricha3-MOBL3> (raw)
In-Reply-To: <20151201.202639.45956937.shin@lenovo>

On Tue, Dec 01, 2015 at 08:26:39PM +0900, Yoshinobu Inoue wrote:
> Hello DPDK list,
> 
> I've been so far just roughly reading messages, as I've been working on my
> company's product based on DPDK1.6 for some time and haven't yet much
> catched up to newer releases,
> but as I implemented packet capture for my product just in a way as commented
> here, (using KNI),
> 
> > Our current thinking is to use kni to mirror packets into the kernel itself,
> > so that all standard linux capture tools can then be used.
> > 
> > /Bruce
> 
> I felt like giving commets from my humble experiences,,,
> 
>  - In our case, KNI is always enabeld for each DPDK port,
>    as it seemed handy that packet rx/tx stat and up/down status can be checked
>    by ifconfig as well.
>    Also, iface MIB become available as well via /sys/devices/pci.../net.
> 
>    As far as we checked it, it seemed that just creating KNI itself didn't much
>    affect Dplane performance.
>    (Just when we update statistics, there is a bit of overhead.)
> 
>  - I inserted rte_kni_tx_burst() call to
>    packet RX path (after rte_eth_rx_burst) and TX path (after rte_eth_tx_burst,
>    based on an assumption that just sent out pkt is not yet freed, it will be
>    freed when the tx descriptor is overwritten by some time later tx packet.).
> 
>    The call to rte_kni_tx_burst() is enabled/disabled by some external capture
>    enable/disable command.
> 
>    I copy the packet beforehand and pass the copied one to rte_kni_tx_burst().
>    In TX path, we might not need to copy if we just increment the packet refcnt
>    by 1, but haven't yet tried such hack much.
> 
>    The packets sent to rte_kni_tx_burst() can then be captured by normal libpcap
>    tools like tcpdump on the correspondent KNI.
> 
>    The performance loss when the capture was enabled was roughly 20-30%.
>    (Perhaps it might change based on many factors.)
> 
>    By the way, in this way, we can enable tx-only or rx-only capture.
> 
> 
>  - Some considerations,
> 
>    -  Someone might not like capture on/off check everytime on normal fast path
>       tx/rx route.
>       I too, so created fastpath send routine and slowpath send routine,
>       and switched the function pointer when the capture is enabled/disabled.
>       But not sure if it was worth for the effort.
> 
>    - In this approach, everyone needs to create their own capture enable/disable
>      command in their implementation, and it could be a bit bothering.
> 
>      I myself am not sure if it's possible, but if as in normal tcpdump,
>      an invocation of tcpdump to a KNI interfce could be somehow notified to
>      the correspondent DPDK port user application, and then the call to
>      rte_kni_tx_burst() could be automatically enabled/disabled, that's cool.
> 
>    
> Thanks,
> Yoshinobu Inoue
> 
Hi,

that is indeed very similar to what we are thinking ourselves. Is there any of
what you have already done that you could contribute publically to save us
duplicating some of your effort? [The one big difference, is that we are not
thinking of enabling kni permanently for each port, as the ethtool support is
only present for a couple of NIC types, and solving that is a separate issue.:-)]

/Bruce

  reply	other threads:[~2015-12-01 11:58 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-11-30 20:50 O'Driscoll, Tim
2015-11-30 21:50 ` Thomas Monjalon
2015-11-30 22:19 ` Dave Neary
2015-12-01 11:57   ` O'Driscoll, Tim
2015-11-30 22:30 ` Hobywan Kenoby
2015-12-01 11:52   ` O'Driscoll, Tim
2015-11-30 22:53 ` Kyle Larose
2015-12-01  1:16   ` Stephen Hemminger
2015-12-01 10:03     ` Bruce Richardson
2015-12-01 11:26       ` Yoshinobu Inoue
2015-12-01 11:58         ` Bruce Richardson [this message]
2015-12-01 13:42           ` Matthew Hall
2015-12-01 14:45             ` Kyle Larose
2015-12-01 19:28               ` Matthew Hall
2015-12-02  0:53           ` Yoshinobu Inoue
2015-12-01 14:27       ` Panu Matilainen
2015-12-01 14:48         ` Vincent JARDIN
2015-12-01 14:58           ` Panu Matilainen
2015-12-01 15:16             ` Christian Ehrhardt
2015-12-01 15:19             ` Bruce Richardson
2015-12-01 15:31               ` Aaron Conole
2015-12-01 15:54                 ` Richardson, Bruce
2015-12-02  1:38                   ` Wiles, Keith
2015-12-02  2:42                     ` Matthew Hall
2015-12-01 19:32                 ` Matthew Hall
2015-12-02 11:24           ` Neil Horman
2015-12-01 12:59 ` Matthew Hall
2015-12-01 13:16   ` O'Driscoll, Tim
2015-12-01 13:44     ` Matthew Hall
2015-12-01 13:57       ` Bruce Richardson
2015-12-01 19:49         ` Matthew Hall
2015-12-02 12:35           ` Bruce Richardson
2015-12-02 15:47             ` Matthew Hall

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=20151201115816.GA29824@bricha3-MOBL3 \
    --to=bruce.richardson@intel.com \
    --cc=dev@dpdk.org \
    --cc=inoue.yoshinobu@jp.fujitsu.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).