diff options
author | Eric Dumazet <edumazet@google.com> | 2013-10-08 15:16:00 -0700 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2013-10-08 21:54:01 -0400 |
commit | 7eec4174ff29cd42f2acfae8112f51c228545d40 (patch) | |
tree | 435e9d12eec1944c4302d95cedc7fa46deb73a04 /net/core/sock.c | |
parent | 2b1f18a4d6ae8057a93e736a34cdcca925279403 (diff) | |
download | op-kernel-dev-7eec4174ff29cd42f2acfae8112f51c228545d40.zip op-kernel-dev-7eec4174ff29cd42f2acfae8112f51c228545d40.tar.gz |
pkt_sched: fq: fix non TCP flows pacing
Steinar reported FQ pacing was not working for UDP flows.
It looks like the initial sk->sk_pacing_rate value of 0 was
a wrong choice. We should init it to ~0U (unlimited)
Then, TCA_FQ_FLOW_DEFAULT_RATE should be removed because it makes
no real sense. The default rate is really unlimited, and we
need to avoid a zero divide.
Reported-by: Steinar H. Gunderson <sesse@google.com>
Signed-off-by: Eric Dumazet <edumazet@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/core/sock.c')
-rw-r--r-- | net/core/sock.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/net/core/sock.c b/net/core/sock.c index 5b6beba..0b39e7a 100644 --- a/net/core/sock.c +++ b/net/core/sock.c @@ -2319,6 +2319,7 @@ void sock_init_data(struct socket *sock, struct sock *sk) sk->sk_ll_usec = sysctl_net_busy_read; #endif + sk->sk_pacing_rate = ~0U; /* * Before updating sk_refcnt, we must commit prior changes to memory * (Documentation/RCU/rculist_nulls.txt for details) |