diff options
| author | David S. Miller <davem@davemloft.net> | 2008-01-10 21:56:38 -0800 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2008-01-28 15:01:42 -0800 |
| commit | 9993e7d313e80bdc005d09c7def91903e0068f07 (patch) | |
| tree | ed7c65b35fd3392948ab2f6b225f69225fdcbba8 /include/net/sock.h | |
| parent | e186932b3d26bd975022a1e254407e95dddceae7 (diff) | |
| download | lwn-9993e7d313e80bdc005d09c7def91903e0068f07.tar.gz lwn-9993e7d313e80bdc005d09c7def91903e0068f07.zip | |
[TCP]: Do not purge sk_forward_alloc entirely in tcp_delack_timer().
Otherwise we beat heavily on the global tcp_memory atomics
when all of the sockets in the system are slowly sending
perioding packet clumps.
Noticed and suggested by Eric Dumazet.
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'include/net/sock.h')
| -rw-r--r-- | include/net/sock.h | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/include/net/sock.h b/include/net/sock.h index 786fae858e77..902324488d0f 100644 --- a/include/net/sock.h +++ b/include/net/sock.h @@ -759,6 +759,14 @@ static inline void sk_mem_reclaim(struct sock *sk) __sk_mem_reclaim(sk); } +static inline void sk_mem_reclaim_partial(struct sock *sk) +{ + if (!sk_has_account(sk)) + return; + if (sk->sk_forward_alloc > SK_MEM_QUANTUM) + __sk_mem_reclaim(sk); +} + static inline void sk_mem_charge(struct sock *sk, int size) { if (!sk_has_account(sk)) |