diff options
author | Eric Dumazet <dada1@cosmosbay.com> | 2007-12-21 04:49:07 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2008-01-28 17:59:57 -0500 |
commit | b790cedd24a7f7d1639072b3faf35f1f56cb38ea (patch) | |
tree | b39d755577be2cc822b5340ba5e4d77a45a6b4a7 /net/ipv4/route.c | |
parent | 9cb5734e5b9b26097c7fa28a9c6426a204cc15e3 (diff) |
[INET]: Avoid an integer divide in rt_garbage_collect()
Since 'goal' is a signed int, compiler may emit an integer divide
to compute goal/2.
Using a right shift is OK here and less expensive.
Signed-off-by: Eric Dumazet <dada1@cosmosbay.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv4/route.c')
-rw-r--r-- | net/ipv4/route.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/net/ipv4/route.c b/net/ipv4/route.c index 1cc6c23cf758..933b093721ea 100644 --- a/net/ipv4/route.c +++ b/net/ipv4/route.c | |||
@@ -851,14 +851,14 @@ static int rt_garbage_collect(void) | |||
851 | equilibrium = ipv4_dst_ops.gc_thresh; | 851 | equilibrium = ipv4_dst_ops.gc_thresh; |
852 | goal = atomic_read(&ipv4_dst_ops.entries) - equilibrium; | 852 | goal = atomic_read(&ipv4_dst_ops.entries) - equilibrium; |
853 | if (goal > 0) { | 853 | if (goal > 0) { |
854 | equilibrium += min_t(unsigned int, goal / 2, rt_hash_mask + 1); | 854 | equilibrium += min_t(unsigned int, goal >> 1, rt_hash_mask + 1); |
855 | goal = atomic_read(&ipv4_dst_ops.entries) - equilibrium; | 855 | goal = atomic_read(&ipv4_dst_ops.entries) - equilibrium; |
856 | } | 856 | } |
857 | } else { | 857 | } else { |
858 | /* We are in dangerous area. Try to reduce cache really | 858 | /* We are in dangerous area. Try to reduce cache really |
859 | * aggressively. | 859 | * aggressively. |
860 | */ | 860 | */ |
861 | goal = max_t(unsigned int, goal / 2, rt_hash_mask + 1); | 861 | goal = max_t(unsigned int, goal >> 1, rt_hash_mask + 1); |
862 | equilibrium = atomic_read(&ipv4_dst_ops.entries) - goal; | 862 | equilibrium = atomic_read(&ipv4_dst_ops.entries) - goal; |
863 | } | 863 | } |
864 | 864 | ||