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 | |
| 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')
| -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 daadbcc4e8dd..86037d16f199 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; |
