From: Jesper Dangaard Brouer <brouer@redhat.com>
To: David Miller <davem@davemloft.net>
Cc: john.fastabend@gmail.com, netdev@vger.kernel.org,
andy@greyhouse.net, daniel@iogearbox.net, ast@fb.com,
alexander.duyck@gmail.com, bjorn.topel@intel.com,
jakub.kicinski@netronome.com, ecree@solarflare.com,
sgoutham@cavium.com, Yuval.Mintz@cavium.com, saeedm@mellanox.com,
brouer@redhat.com
Subject: Re: [RFC PATCH 00/12] Implement XDP bpf_redirect vairants
Date: Tue, 11 Jul 2017 17:36:58 +0200 [thread overview]
Message-ID: <20170711173658.6188b0a2@redhat.com> (raw)
In-Reply-To: <20170708210617.249059b9@redhat.com>
On Sat, 8 Jul 2017 21:06:17 +0200
Jesper Dangaard Brouer <brouer@redhat.com> wrote:
> My plan is to test this latest patchset again, Monday and Tuesday.
> I'll try to assess stability and provide some performance numbers.
Performance numbers:
14378479 pkt/s = XDP_DROP without touching memory
9222401 pkt/s = xdp1: XDP_DROP with reading packet data
6344472 pkt/s = xdp2: XDP_TX with swap mac (writes into pkt)
4595574 pkt/s = xdp_redirect: XDP_REDIRECT with swap mac (simulate XDP_TX)
5066243 pkt/s = xdp_redirect_map: XDP_REDIRECT with swap mac + devmap
The performance drop between xdp2 and xdp_redirect, was expected due
to the HW-tailptr flush per packet, which is costly.
(1/6344472-1/4595574)*10^9 = -59.98 ns
The performance drop between xdp2 and xdp_redirect_map, is higher than
I expected, which is not good! The avoidance of the tailptr flush per
packet was expected to give a higher boost. The cost increased with
40 ns, which is too high compared to the code added (on a 4GHz machine
approx 160 cycles).
(1/6344472-1/5066243)*10^9 = -39.77 ns
This system doesn't have DDIO, thus we are stalling on cache-misses,
but I was actually expecting that the added code could "hide" behind
these cache-misses.
I'm somewhat surprised to see this large a performance drop.
--
Best regards,
Jesper Dangaard Brouer
MSc.CS, Principal Kernel Engineer at Red Hat
LinkedIn: http://www.linkedin.com/in/brouer
Results::
# XDP_DROP with reading packet data
[jbrouer@canyon bpf]$ sudo ./xdp1 3
proto 17: 6449727 pkt/s
proto 17: 9222639 pkt/s
proto 17: 9222401 pkt/s
proto 17: 9223083 pkt/s
proto 17: 9223515 pkt/s
proto 17: 9222477 pkt/s
^C
# XDP_TX with swap mac
[jbrouer@canyon bpf]$ sudo ./xdp2 3
proto 17: 934682 pkt/s
proto 17: 6344845 pkt/s
proto 17: 6344472 pkt/s
proto 17: 6345265 pkt/s
proto 17: 6345238 pkt/s
proto 17: 6345338 pkt/s
^C
# XDP_REDIRECT with swap mac (simulate XDP_TX via same ifindex)
[jbrouer@canyon bpf]$ sudo ./xdp_redirect 3 3
ifindex 3: 749567 pkt/s
ifindex 3: 4595025 pkt/s
ifindex 3: 4595574 pkt/s
ifindex 3: 4595429 pkt/s
ifindex 3: 4595340 pkt/s
ifindex 3: 4595352 pkt/s
ifindex 3: 4595364 pkt/s
^C
# XDP_REDIRECT with swap mac + devmap (still simulate XDP_TX)
[jbrouer@canyon bpf]$ sudo ./xdp_redirect_map 3 3
map[0] (vports) = 4, map[1] (map) = 5, map[2] (count) = 0
ifindex 3: 3076506 pkt/s
ifindex 3: 5066282 pkt/s
ifindex 3: 5066243 pkt/s
ifindex 3: 5067376 pkt/s
ifindex 3: 5067226 pkt/s
ifindex 3: 5067622 pkt/s
My own tools::
[jbrouer@canyon prototype-kernel]$
sudo ./xdp_bench01_mem_access_cost --dev ixgbe1 --sec 2 \
--action XDP_DROP
XDP_action pps pps-human-readable mem
XDP_DROP 0 0 no_touch
XDP_DROP 9894401 9,894,401 no_touch
XDP_DROP 14377459 14,377,459 no_touch
XDP_DROP 14378228 14,378,228 no_touch
XDP_DROP 14378400 14,378,400 no_touch
XDP_DROP 14378319 14,378,319 no_touch
XDP_DROP 14378479 14,378,479 no_touch
XDP_DROP 14377332 14,377,332 no_touch
XDP_DROP 14378411 14,378,411 no_touch
XDP_DROP 14378095 14,378,095 no_touch
^CInterrupted: Removing XDP program on ifindex:3 device:ixgbe1
[jbrouer@canyon prototype-kernel]$
sudo ./xdp_bench01_mem_access_cost --dev ixgbe1 --sec 2 \
--action XDP_DROP --read
XDP_action pps pps-human-readable mem
XDP_DROP 0 0 read
XDP_DROP 6994114 6,994,114 read
XDP_DROP 8979414 8,979,414 read
XDP_DROP 8979636 8,979,636 read
XDP_DROP 8980087 8,980,087 read
XDP_DROP 8979097 8,979,097 read
XDP_DROP 8978970 8,978,970 read
^CInterrupted: Removing XDP program on ifindex:3 device:ixgbe1
[jbrouer@canyon prototype-kernel]$
sudo ./xdp_bench01_mem_access_cost --dev ixgbe1 --sec 2 \
--action XDP_TX --swap --read
XDP_action pps pps-human-readable mem
XDP_TX 0 0 swap_mac
XDP_TX 2141556 2,141,556 swap_mac
XDP_TX 6171984 6,171,984 swap_mac
XDP_TX 6171955 6,171,955 swap_mac
XDP_TX 6171767 6,171,767 swap_mac
XDP_TX 6171680 6,171,680 swap_mac
XDP_TX 6172201 6,172,201 swap_mac
^CInterrupted: Removing XDP program on ifindex:3 device:ixgbe1
Setting tuned-adm network-latency ::
$ sudo tuned-adm list
[...]
Current active profile: network-latency
next prev parent reply other threads:[~2017-07-11 15:37 UTC|newest]
Thread overview: 43+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-07-07 17:34 [RFC PATCH 00/12] Implement XDP bpf_redirect vairants John Fastabend
2017-07-07 17:34 ` [RFC PATCH 01/12] ixgbe: NULL xdp_tx rings on resource cleanup John Fastabend
2017-07-07 17:35 ` [RFC PATCH 02/12] net: xdp: support xdp generic on virtual devices John Fastabend
2017-07-07 17:35 ` [RFC PATCH 03/12] xdp: add bpf_redirect helper function John Fastabend
2017-07-09 13:37 ` Saeed Mahameed
2017-07-10 17:23 ` John Fastabend
2017-07-11 14:09 ` Andy Gospodarek
2017-07-11 18:38 ` John Fastabend
2017-07-11 19:38 ` Jesper Dangaard Brouer
2017-07-12 11:00 ` Saeed Mahameed
2017-07-07 17:35 ` [RFC PATCH 04/12] xdp: sample program for new bpf_redirect helper John Fastabend
2017-07-07 17:36 ` [RFC PATCH 05/12] net: implement XDP_REDIRECT for xdp generic John Fastabend
2017-07-07 17:36 ` [RFC PATCH 06/12] ixgbe: add initial support for xdp redirect John Fastabend
2017-07-07 17:36 ` [RFC PATCH 07/12] xdp: add trace event " John Fastabend
2017-07-07 17:37 ` [RFC PATCH 08/12] bpf: add devmap, a map for storing net device references John Fastabend
2017-07-08 18:57 ` Jesper Dangaard Brouer
2017-07-07 17:37 ` [RFC PATCH 09/12] bpf: add bpf_redirect_map helper routine John Fastabend
2017-07-07 17:37 ` [RFC PATCH 10/12] xdp: Add batching support to redirect map John Fastabend
2017-07-10 17:53 ` Jesper Dangaard Brouer
2017-07-10 17:56 ` John Fastabend
2017-07-07 17:38 ` [RFC PATCH 11/12] net: add notifier hooks for devmap bpf map John Fastabend
2017-07-07 17:38 ` [RFC PATCH 12/12] xdp: bpf redirect with map sample program John Fastabend
2017-07-07 17:48 ` [RFC PATCH 00/12] Implement XDP bpf_redirect vairants John Fastabend
2017-07-08 9:46 ` David Miller
2017-07-08 19:06 ` Jesper Dangaard Brouer
2017-07-10 18:30 ` Jesper Dangaard Brouer
2017-07-11 0:59 ` John Fastabend
2017-07-11 14:23 ` Jesper Dangaard Brouer
2017-07-11 18:26 ` John Fastabend
2017-07-13 11:14 ` Jesper Dangaard Brouer
2017-07-13 16:16 ` Jesper Dangaard Brouer
2017-07-13 17:00 ` John Fastabend
2017-07-13 18:21 ` David Miller
2017-07-11 15:36 ` Jesper Dangaard Brouer [this message]
2017-07-11 17:48 ` John Fastabend
2017-07-11 18:01 ` Jesper Dangaard Brouer
2017-07-11 18:29 ` John Fastabend
2017-07-11 18:44 ` Jesper Dangaard Brouer
2017-07-11 18:56 ` John Fastabend
2017-07-11 19:19 ` Jesper Dangaard Brouer
2017-07-11 19:37 ` John Fastabend
2017-07-16 8:23 ` Jesper Dangaard Brouer
2017-07-17 17:04 ` Jesse Brandeburg
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=20170711173658.6188b0a2@redhat.com \
--to=brouer@redhat.com \
--cc=Yuval.Mintz@cavium.com \
--cc=alexander.duyck@gmail.com \
--cc=andy@greyhouse.net \
--cc=ast@fb.com \
--cc=bjorn.topel@intel.com \
--cc=daniel@iogearbox.net \
--cc=davem@davemloft.net \
--cc=ecree@solarflare.com \
--cc=jakub.kicinski@netronome.com \
--cc=john.fastabend@gmail.com \
--cc=netdev@vger.kernel.org \
--cc=saeedm@mellanox.com \
--cc=sgoutham@cavium.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).