diff options
author | Eric Dumazet <edumazet@google.com> | 2018-11-19 17:30:19 -0800 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2018-11-20 09:51:32 -0800 |
commit | 6b015a523fa3ba57ccd9c72697163b560fcdedb4 (patch) | |
tree | 531594a49b69a9843804d30d7e1f2cd4adb53bf7 /net/sched/sch_fq.c | |
parent | 6133e78f4102405b1505b749d3bd06564ddafea9 (diff) |
net_sched: sch_fq: avoid calling ktime_get_ns() if not needed
There are two cases were we can avoid calling ktime_get_ns() :
1) Queue is empty.
2) Internal queue is not empty.
Signed-off-by: Eric Dumazet <edumazet@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/sched/sch_fq.c')
-rw-r--r-- | net/sched/sch_fq.c | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/net/sched/sch_fq.c b/net/sched/sch_fq.c index 1da8864502d4..1a662f2bb7bb 100644 --- a/net/sched/sch_fq.c +++ b/net/sched/sch_fq.c @@ -414,16 +414,21 @@ static void fq_check_throttled(struct fq_sched_data *q, u64 now) static struct sk_buff *fq_dequeue(struct Qdisc *sch) { struct fq_sched_data *q = qdisc_priv(sch); - u64 now = ktime_get_ns(); struct fq_flow_head *head; struct sk_buff *skb; struct fq_flow *f; unsigned long rate; u32 plen; + u64 now; + + if (!sch->q.qlen) + return NULL; skb = fq_dequeue_head(sch, &q->internal); if (skb) goto out; + + now = ktime_get_ns(); fq_check_throttled(q, now); begin: head = &q->new_flows; |