From 1fb44832079319cd98647bcdc25f2d148431733a Mon Sep 17 00:00:00 2001 From: Jesper Dangaard Brouer Date: Sat, 14 Nov 2020 18:44:40 +0100 Subject: [PATCH] traffic-pacing-edt: tc_fq_pacer.sh adjust packet per flow_limit This was causing strange issues, where a TCP single flow could not achieve the correct bandwidth. Signed-off-by: Jesper Dangaard Brouer --- traffic-pacing-edt/edt_pacer02.c | 1 + traffic-pacing-edt/tc_fq_pacer.sh | 8 +++++++- 2 files changed, 8 insertions(+), 1 deletion(-) diff --git a/traffic-pacing-edt/edt_pacer02.c b/traffic-pacing-edt/edt_pacer02.c index ed66ba1..83e2823 100644 --- a/traffic-pacing-edt/edt_pacer02.c +++ b/traffic-pacing-edt/edt_pacer02.c @@ -10,6 +10,7 @@ char _license[] SEC("license") = "GPL"; /* skb->len in bytes, thus easier to keep rate in bytes */ #define RATE_IN_BITS (1000 * 1000 * 1000ULL) +//#define RATE_IN_BITS (200 * 1000 * 1000ULL) //#define RATE_IN_BITS (500 * 1000 * 1000ULL) #define RATE_IN_BYTES (RATE_IN_BITS / 8) diff --git a/traffic-pacing-edt/tc_fq_pacer.sh b/traffic-pacing-edt/tc_fq_pacer.sh index 92aa21d..1ad9805 100755 --- a/traffic-pacing-edt/tc_fq_pacer.sh +++ b/traffic-pacing-edt/tc_fq_pacer.sh @@ -40,6 +40,12 @@ i=0 for dir in /sys/class/net/$DEV/queues/tx-*; do # Details: cause-off-by-one, as tx-0 becomes handle 1: ((i++)) || true - tc qdisc add dev $DEV parent 7FFF:$i handle $i: fq + #tc qdisc add dev $DEV parent 7FFF:$i handle $i: fq + # + # The higher 'flow_limit' is needed for high-BW pacing + tc qdisc add dev $DEV parent 7FFF:$i handle $i: fq \ + flow_limit 1000 + # + # quantum $((1514*4)) initial_quantum $((1514*20)) # tc qdisc add dev $DEV parent 7FFF:$i handle $i: fq maxrate 930mbit done