tracing: Convert stddev into u64 in tracepoint benchmark
authorSteven Rostedt (Red Hat) <rostedt@goodmis.org>
Fri, 6 Jun 2014 00:35:30 +0000 (20:35 -0400)
committerSteven Rostedt <rostedt@goodmis.org>
Fri, 6 Jun 2014 00:35:30 +0000 (20:35 -0400)
I've been told that do_div() expects an unsigned 64 bit number, and
is undefined if a signed is used. This gave a warning on the MIPS
build. I'm not sure if a signed 64 bit dividend is really an issue
or not, but the calculation this is used for is standard deviation,
and that isn't going to be negative. We can just convert it to
unsigned and be safe.

Reported-by: David Daney <ddaney.cavm@gmail.com>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
kernel/trace/trace_benchmark.c

index a10adc7095cdf6a362900c958d4c23f4f95254a5..8bd3365a65b27fd6421f05e57e3418eff210b84f 100644 (file)
@@ -33,7 +33,7 @@ static void trace_do_benchmark(void)
        u64 start;
        u64 stop;
        u64 delta;
-       s64 stddev;
+       u64 stddev;
        u64 seed;
        u64 last_seed;
        unsigned int avg;