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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.