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>
This commit is contained in:
parent
6133e78f41
commit
6b015a523f
|
@ -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)
|
static struct sk_buff *fq_dequeue(struct Qdisc *sch)
|
||||||
{
|
{
|
||||||
struct fq_sched_data *q = qdisc_priv(sch);
|
struct fq_sched_data *q = qdisc_priv(sch);
|
||||||
u64 now = ktime_get_ns();
|
|
||||||
struct fq_flow_head *head;
|
struct fq_flow_head *head;
|
||||||
struct sk_buff *skb;
|
struct sk_buff *skb;
|
||||||
struct fq_flow *f;
|
struct fq_flow *f;
|
||||||
unsigned long rate;
|
unsigned long rate;
|
||||||
u32 plen;
|
u32 plen;
|
||||||
|
u64 now;
|
||||||
|
|
||||||
|
if (!sch->q.qlen)
|
||||||
|
return NULL;
|
||||||
|
|
||||||
skb = fq_dequeue_head(sch, &q->internal);
|
skb = fq_dequeue_head(sch, &q->internal);
|
||||||
if (skb)
|
if (skb)
|
||||||
goto out;
|
goto out;
|
||||||
|
|
||||||
|
now = ktime_get_ns();
|
||||||
fq_check_throttled(q, now);
|
fq_check_throttled(q, now);
|
||||||
begin:
|
begin:
|
||||||
head = &q->new_flows;
|
head = &q->new_flows;
|
||||||
|
|
Loading…
Reference in New Issue