From: Alexander Lobakin <aleksander.lobakin@intel.com>
To: Lorenzo Bianconi <lorenzo@kernel.org>, Daniel Xu <dxu@dxuuu.xyz>
Cc: <bpf@vger.kernel.org>, <kuba@kernel.org>, <ast@kernel.org>,
<daniel@iogearbox.net>, <andrii@kernel.org>,
<john.fastabend@gmail.com>, <hawk@kernel.org>,
<martin.lau@linux.dev>, <davem@davemloft.net>,
<edumazet@google.com>, <pabeni@redhat.com>,
<netdev@vger.kernel.org>, <lorenzo.bianconi@redhat.com>
Subject: Re: [RFC/RFT v2 0/3] Introduce GRO support to cpumap codebase
Date: Wed, 9 Oct 2024 14:27:54 +0200 [thread overview]
Message-ID: <55d2ac1c-0619-4b24-b8ab-6eb5f553c1dd@intel.com> (raw)
In-Reply-To: <ZwZe6Bg5ZrXLkDGW@lore-desk>
From: Lorenzo Bianconi <lorenzo@kernel.org>
Date: Wed, 9 Oct 2024 12:46:00 +0200
>> Hi Lorenzo,
>>
>> On Mon, Sep 16, 2024 at 12:13:42PM GMT, Lorenzo Bianconi wrote:
>>> Add GRO support to cpumap codebase moving the cpu_map_entry kthread to a
>>> NAPI-kthread pinned on the selected cpu.
>>>
>>> Changes in rfc v2:
>>> - get rid of dummy netdev dependency
>>>
>>> Lorenzo Bianconi (3):
>>> net: Add napi_init_for_gro routine
>>> net: add napi_threaded_poll to netdevice.h
>>> bpf: cpumap: Add gro support
>>>
>>> include/linux/netdevice.h | 3 +
>>> kernel/bpf/cpumap.c | 123 ++++++++++++++++----------------------
>>> net/core/dev.c | 27 ++++++---
>>> 3 files changed, 73 insertions(+), 80 deletions(-)
>>>
>>> --
>>> 2.46.0
>>>
>>
>> Sorry about the long delay - finally caught up to everything after
>> conferences.
>>
>> I re-ran my synthetic tests (including baseline). v2 is somehow showing
>> 2x bigger gains than v1 (~30% vs ~14%) for tcp_stream. Again, the only
>> variable I changed is kernel version - steering prog is active for both.
>>
>>
>> Baseline (again)
>>
>> ./tcp_rr -c -H $TASK_IP -p 50,90,99 -T4 -F8 -l30 ./tcp_stream -c -H $TASK_IP -T8 -F16 -l30
>>
>> Transactions Latency P50 (s) Latency P90 (s) Latency P99 (s) Throughput (Mbit/s)
>> Run 1 2560252 0.00009087 0.00010495 0.00011647 Run 1 15479.31
>> Run 2 2665517 0.00008575 0.00010239 0.00013311 Run 2 15162.48
>> Run 3 2755939 0.00008191 0.00010367 0.00012287 Run 3 14709.04
>> Run 4 2595680 0.00008575 0.00011263 0.00012671 Run 4 15373.06
>> Run 5 2841865 0.00007999 0.00009471 0.00012799 Run 5 15234.91
>> Average 2683850.6 0.000084854 0.00010367 0.00012543 Average 15191.76
>>
>> cpumap NAPI patches v2
>>
>> Transactions Latency P50 (s) Latency P90 (s) Latency P99 (s) Throughput (Mbit/s)
>> Run 1 2577838 0.00008575 0.00012031 0.00013695 Run 1 19914.56
>> Run 2 2729237 0.00007551 0.00013311 0.00017663 Run 2 20140.92
>> Run 3 2689442 0.00008319 0.00010495 0.00013311 Run 3 19887.48
>> Run 4 2862366 0.00008127 0.00009471 0.00010623 Run 4 19374.49
>> Run 5 2700538 0.00008319 0.00010367 0.00012799 Run 5 19784.49
>> Average 2711884.2 0.000081782 0.00011135 0.000136182 Average 19820.388
>> Delta 1.04% -3.62% 7.41% 8.57% 30.47%
>>
>> Thanks,
>> Daniel
>
> Hi Daniel,
>
> cool, thx for testing it.
>
> @Olek: how do we want to proceed on it? Are you still working on it or do you want me
> to send a regular patch for it?
Hi,
I had a small vacation, sorry. I'm starting working on it again today.
>
> Regards,
> Lorenzo
Thanks,
Olek
next prev parent reply other threads:[~2024-10-09 12:28 UTC|newest]
Thread overview: 36+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-09-16 10:13 [RFC/RFT v2 0/3] Introduce GRO support to cpumap codebase Lorenzo Bianconi
2024-09-16 10:13 ` [RFC/RFT v2 1/3] net: Add napi_init_for_gro routine Lorenzo Bianconi
2024-09-16 10:13 ` [RFC/RFT v2 2/3] net: add napi_threaded_poll to netdevice.h Lorenzo Bianconi
2024-09-16 10:13 ` [RFC/RFT v2 3/3] bpf: cpumap: Add gro support Lorenzo Bianconi
2024-09-16 15:10 ` [RFC/RFT v2 0/3] Introduce GRO support to cpumap codebase Alexander Lobakin
2024-10-08 22:39 ` Daniel Xu
2024-10-09 10:46 ` Lorenzo Bianconi
2024-10-09 12:27 ` Alexander Lobakin [this message]
2024-10-09 12:47 ` Lorenzo Bianconi
2024-10-09 12:50 ` Alexander Lobakin
2024-10-22 15:51 ` Alexander Lobakin
2024-11-12 17:43 ` Alexander Lobakin
2024-11-13 23:39 ` Daniel Xu
2024-11-23 0:10 ` Daniel Xu
2024-11-25 15:12 ` Alexander Lobakin
2024-11-25 17:03 ` Daniel Xu
2024-11-25 18:50 ` Jesper Dangaard Brouer
2024-11-25 21:53 ` Daniel Xu
2024-11-25 22:19 ` Lorenzo Bianconi
2024-11-25 22:56 ` Daniel Xu
2024-11-26 10:36 ` Alexander Lobakin
2024-11-26 17:02 ` Lorenzo Bianconi
2024-11-26 17:12 ` Jesper Dangaard Brouer
2024-11-28 10:41 ` Alexander Lobakin
2024-11-28 10:56 ` Lorenzo Bianconi
2024-11-28 10:57 ` Alexander Lobakin
2024-12-02 22:47 ` Jakub Kicinski
2024-12-03 11:01 ` Alexander Lobakin
2024-12-04 0:51 ` Jakub Kicinski
2024-12-04 16:42 ` Alexander Lobakin
2024-12-04 21:51 ` Daniel Xu
2024-12-05 10:38 ` Alexander Lobakin
2024-12-05 11:06 ` Alexander Lobakin
2024-12-06 0:41 ` Daniel Xu
2024-12-06 15:06 ` Alexander Lobakin
2024-12-06 23:36 ` Daniel Xu
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=55d2ac1c-0619-4b24-b8ab-6eb5f553c1dd@intel.com \
--to=aleksander.lobakin@intel.com \
--cc=andrii@kernel.org \
--cc=ast@kernel.org \
--cc=bpf@vger.kernel.org \
--cc=daniel@iogearbox.net \
--cc=davem@davemloft.net \
--cc=dxu@dxuuu.xyz \
--cc=edumazet@google.com \
--cc=hawk@kernel.org \
--cc=john.fastabend@gmail.com \
--cc=kuba@kernel.org \
--cc=lorenzo.bianconi@redhat.com \
--cc=lorenzo@kernel.org \
--cc=martin.lau@linux.dev \
--cc=netdev@vger.kernel.org \
--cc=pabeni@redhat.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