From bb7cf681e90d5aa2d867aeff4948ac605447de7d Mon Sep 17 00:00:00 2001 From: Ondrej Bilka Date: Wed, 8 May 2013 08:21:05 +0200 Subject: Preheat CPU in benchtests. A benchmark could be skewed by CPU initialy working on minimal frequency and speeding up later. We first run code in loop to partialy fix this issue. --- ChangeLog | 4 ++++ benchtests/bench-skeleton.c | 17 +++++++++++++++++ 2 files changed, 21 insertions(+) diff --git a/ChangeLog b/ChangeLog index fa126348c2..c6812711c0 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,7 @@ +2013-05-08 Ondřej Bílka + + * benchtests/bench-skeleton.c (main): Preheat CPU. + 2013-05-07 Aurelien Jarno * misc/sys/param.h (DEV_BSIZE): Define only if not already defined. diff --git a/benchtests/bench-skeleton.c b/benchtests/bench-skeleton.c index 7359184ba8..a13ad02177 100644 --- a/benchtests/bench-skeleton.c +++ b/benchtests/bench-skeleton.c @@ -22,6 +22,21 @@ #include #include +volatile unsigned int dontoptimize = 0; +void startup () +{ + /* This loop should cause CPU to switch to maximal freqency. + This makes subsequent measurement more accurate. We need a side effect + to prevent the loop being deleted by compiler. + This should be enough to cause CPU to speed up and it is simpler than + running loop for constant time. This is used when user does not have root + access to set a constant freqency. */ + + int k; + for (k = 0; k < 10000000; k++) + dontoptimize += 23 * dontoptimize + 2; +} + #define TIMESPEC_AFTER(a, b) \ (((a).tv_sec == (b).tv_sec) ? \ ((a).tv_nsec > (b).tv_nsec) : \ @@ -32,6 +47,8 @@ main (int argc, char **argv) unsigned long i, k; struct timespec start, end, runtime; + startup(); + memset (&runtime, 0, sizeof (runtime)); memset (&start, 0, sizeof (start)); memset (&end, 0, sizeof (end)); -- cgit v1.2.3