diff options
| author | Ian McDonald <ian.mcdonald@jandi.co.nz> | 2006-07-12 20:58:53 -0400 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2006-07-12 20:58:53 -0400 |
| commit | a6f157a88d1398d7ccb743c5a56138edf6f6ef0b (patch) | |
| tree | 8244a8f1e359830829b0845cc677edd2d3959a47 | |
| parent | fd1278d720b48ad7576f64014b916cc77365cb3f (diff) | |
[NET]: fix __sk_stream_mem_reclaim
__sk_stream_mem_reclaim is only called by sk_stream_mem_reclaim.
As such the check on sk->sk_forward_alloc is not needed and can be
removed.
Signed-off-by: Ian McDonald <ian.mcdonald@jandi.co.nz>
Acked-by: Herbert Xu <herbert@gondor.apana.org.au>
Signed-off-by: David S. Miller <davem@davemloft.net>
| -rw-r--r-- | net/core/stream.c | 16 |
1 files changed, 7 insertions, 9 deletions
diff --git a/net/core/stream.c b/net/core/stream.c index e9489696f694..d1d7decf70b0 100644 --- a/net/core/stream.c +++ b/net/core/stream.c | |||
| @@ -196,15 +196,13 @@ EXPORT_SYMBOL(sk_stream_error); | |||
| 196 | 196 | ||
| 197 | void __sk_stream_mem_reclaim(struct sock *sk) | 197 | void __sk_stream_mem_reclaim(struct sock *sk) |
| 198 | { | 198 | { |
| 199 | if (sk->sk_forward_alloc >= SK_STREAM_MEM_QUANTUM) { | 199 | atomic_sub(sk->sk_forward_alloc / SK_STREAM_MEM_QUANTUM, |
| 200 | atomic_sub(sk->sk_forward_alloc / SK_STREAM_MEM_QUANTUM, | 200 | sk->sk_prot->memory_allocated); |
| 201 | sk->sk_prot->memory_allocated); | 201 | sk->sk_forward_alloc &= SK_STREAM_MEM_QUANTUM - 1; |
| 202 | sk->sk_forward_alloc &= SK_STREAM_MEM_QUANTUM - 1; | 202 | if (*sk->sk_prot->memory_pressure && |
| 203 | if (*sk->sk_prot->memory_pressure && | 203 | (atomic_read(sk->sk_prot->memory_allocated) < |
| 204 | (atomic_read(sk->sk_prot->memory_allocated) < | 204 | sk->sk_prot->sysctl_mem[0])) |
| 205 | sk->sk_prot->sysctl_mem[0])) | 205 | *sk->sk_prot->memory_pressure = 0; |
| 206 | *sk->sk_prot->memory_pressure = 0; | ||
| 207 | } | ||
| 208 | } | 206 | } |
| 209 | 207 | ||
| 210 | EXPORT_SYMBOL(__sk_stream_mem_reclaim); | 208 | EXPORT_SYMBOL(__sk_stream_mem_reclaim); |
