diff options
author | Eric Dumazet <dada1@cosmosbay.com> | 2008-01-03 23:40:01 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2008-01-28 18:00:35 -0500 |
commit | 789675e216617b1331875c42a81f58227a06df91 (patch) | |
tree | bec5c846982ef2ebe5e6970242be146531add9b1 /net/core/gen_estimator.c | |
parent | e870a8efcddaaa3da7e180b6ae21239fb96aa2bb (diff) |
[NET]: Avoid divides in net/core/gen_estimator.c
We can void divides (as seen with CONFIG_CC_OPTIMIZE_FOR_SIZE=y on x86)
changing ((HZ<<idx)/4) to ((HZ/4) << idx)
Signed-off-by: Eric Dumazet <dada1@cosmosbay.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/core/gen_estimator.c')
-rw-r--r-- | net/core/gen_estimator.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/net/core/gen_estimator.c b/net/core/gen_estimator.c index daadbcc4e8d..86037d16f19 100644 --- a/net/core/gen_estimator.c +++ b/net/core/gen_estimator.c | |||
@@ -135,7 +135,7 @@ skip: | |||
135 | } | 135 | } |
136 | 136 | ||
137 | if (!list_empty(&elist[idx].list)) | 137 | if (!list_empty(&elist[idx].list)) |
138 | mod_timer(&elist[idx].timer, jiffies + ((HZ<<idx)/4)); | 138 | mod_timer(&elist[idx].timer, jiffies + ((HZ/4) << idx)); |
139 | rcu_read_unlock(); | 139 | rcu_read_unlock(); |
140 | } | 140 | } |
141 | 141 | ||
@@ -191,7 +191,7 @@ int gen_new_estimator(struct gnet_stats_basic *bstats, | |||
191 | } | 191 | } |
192 | 192 | ||
193 | if (list_empty(&elist[idx].list)) | 193 | if (list_empty(&elist[idx].list)) |
194 | mod_timer(&elist[idx].timer, jiffies + ((HZ<<idx)/4)); | 194 | mod_timer(&elist[idx].timer, jiffies + ((HZ/4) << idx)); |
195 | 195 | ||
196 | list_add_rcu(&est->list, &elist[idx].list); | 196 | list_add_rcu(&est->list, &elist[idx].list); |
197 | return 0; | 197 | return 0; |