diff options
author | Bruno Randolf <br1@einfach.org> | 2010-12-02 19:50:37 +0900 |
---|---|---|
committer | John W. Linville <linville@tuxdriver.com> | 2010-12-06 15:58:43 -0500 |
commit | af5568843594fb71055debe36e521fa8072fcecc (patch) | |
tree | 4f35966cb4e1018b05c87d7a10c337d888949e22 /include/linux/average.h | |
parent | 5dcc03fe29537edd7819f5b121bf3d779693f37b (diff) | |
download | blackbird-op-linux-af5568843594fb71055debe36e521fa8072fcecc.tar.gz blackbird-op-linux-af5568843594fb71055debe36e521fa8072fcecc.zip |
lib: Improve EWMA efficiency by using bitshifts
Using bitshifts instead of division and multiplication should improve
performance. That requires weight and factor to be powers of two, but i think
this is something we can live with.
Thanks to Peter Zijlstra for the improved formula!
Signed-off-by: Bruno Randolf <br1@einfach.org>
--
v2: use log2.h functions
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Diffstat (limited to 'include/linux/average.h')
-rw-r--r-- | include/linux/average.h | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/include/linux/average.h b/include/linux/average.h index 7706e40f95fa..c6028fd742c1 100644 --- a/include/linux/average.h +++ b/include/linux/average.h @@ -1,8 +1,6 @@ #ifndef _LINUX_AVERAGE_H #define _LINUX_AVERAGE_H -#include <linux/kernel.h> - /* Exponentially weighted moving average (EWMA) */ /* For more documentation see lib/average.c */ @@ -26,7 +24,7 @@ extern struct ewma *ewma_add(struct ewma *avg, unsigned long val); */ static inline unsigned long ewma_read(const struct ewma *avg) { - return DIV_ROUND_CLOSEST(avg->internal, avg->factor); + return avg->internal >> avg->factor; } #endif /* _LINUX_AVERAGE_H */ |