[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <1f48019a-fb72-324c-7626-ba5ccb9307b0@iogearbox.net>
Date: Wed, 24 Jan 2024 11:29:27 +0100
From: Daniel Borkmann <daniel@...earbox.net>
To: Amery Hung <ameryhung@...il.com>, netdev@...r.kernel.org
Cc: bpf@...r.kernel.org, yangpeihao@...u.edu.cn, toke@...hat.com,
jhs@...atatu.com, jiri@...nulli.us, sdf@...gle.com,
xiyou.wangcong@...il.com, yepeilin.cs@...il.com
Subject: Re: [RFC PATCH v7 7/8] samples/bpf: Add an example of bpf fq qdisc
On 1/17/24 10:56 PM, Amery Hung wrote:
> tc_sch_fq.bpf.c
> A simple bpf fair queueing (fq) qdisc that gives each flow a euqal chance
> to transmit data. The qdisc respects the timestamp in a skb set by an
> clsact rate limiter. It can also inform the rate limiter about packet drop
> when enabled to adjust timestamps. The implementation does not prevent hash
> collision of flows nor does it recycle flows.
>
> tc_sch_fq.c
> A user space program to load and attach the eBPF-based fq qdisc, which
> by default add the bpf fq to the loopback device, but can also add to other
> dev and class with '-d' and '-p' options.
>
> To test the bpf fq qdisc with the EDT rate limiter:
> $ tc qdisc add dev lo clsact
> $ tc filter add dev lo egress bpf obj tc_clsact_edt.bpf.o sec classifier
> $ ./tc_sch_fq -s
Would be nice if you also include a performance comparison (did you do
production tests with it?) with side-by-side to native fq and if you see
a delta elaborate on what would be needed to address it.
> Signed-off-by: Amery Hung <amery.hung@...edance.com>
> ---
> samples/bpf/Makefile | 8 +-
> samples/bpf/bpf_experimental.h | 134 +++++++
> samples/bpf/tc_clsact_edt.bpf.c | 103 +++++
> samples/bpf/tc_sch_fq.bpf.c | 666 ++++++++++++++++++++++++++++++++
> samples/bpf/tc_sch_fq.c | 321 +++++++++++++++
> 5 files changed, 1231 insertions(+), 1 deletion(-)
> create mode 100644 samples/bpf/bpf_experimental.h
> create mode 100644 samples/bpf/tc_clsact_edt.bpf.c
> create mode 100644 samples/bpf/tc_sch_fq.bpf.c
> create mode 100644 samples/bpf/tc_sch_fq.c
Powered by blists - more mailing lists