diff options
| author | Daniel Axtens <[email protected]> | 2018-03-01 06:13:38 +0000 |
|---|---|---|
| committer | David S. Miller <[email protected]> | 2018-03-04 22:49:17 +0000 |
| commit | ee78bbef8d63202ca0f2485aecf30b8c2b0088cc (patch) | |
| tree | fb4656122536f2df9964865d441751c31a0005da | |
| parent | net: rename skb_gso_validate_mtu -> skb_gso_validate_network_len (diff) | |
| download | kernel-ee78bbef8d63202ca0f2485aecf30b8c2b0088cc.tar.gz kernel-ee78bbef8d63202ca0f2485aecf30b8c2b0088cc.zip | |
net: sched: tbf: handle GSO_BY_FRAGS case in enqueue
tbf_enqueue() checks the size of a packet before enqueuing it.
However, the GSO size check does not consider the GSO_BY_FRAGS
case, and so will drop GSO SCTP packets, causing a massive drop
in throughput.
Use skb_gso_validate_mac_len() instead, as it does consider that
case.
Signed-off-by: Daniel Axtens <[email protected]>
Reviewed-by: Marcelo Ricardo Leitner <[email protected]>
Signed-off-by: David S. Miller <[email protected]>
| -rw-r--r-- | net/sched/sch_tbf.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/net/sched/sch_tbf.c b/net/sched/sch_tbf.c index 229172d509cc..03225a8df973 100644 --- a/net/sched/sch_tbf.c +++ b/net/sched/sch_tbf.c @@ -188,7 +188,8 @@ static int tbf_enqueue(struct sk_buff *skb, struct Qdisc *sch, int ret; if (qdisc_pkt_len(skb) > q->max_size) { - if (skb_is_gso(skb) && skb_gso_mac_seglen(skb) <= q->max_size) + if (skb_is_gso(skb) && + skb_gso_validate_mac_len(skb, q->max_size)) return tbf_segment(skb, sch, to_free); return qdisc_drop(skb, sch, to_free); } |