diff options
author | Sebastian Huber <sebastian.huber@embedded-brains.de> | 2014-02-16 12:12:38 +0100 |
---|---|---|
committer | Michael Tokarev <mjt@tls.msk.ru> | 2014-03-15 13:54:18 +0400 |
commit | 9d5614d582d23ec96b167583557bf3f25f64f050 (patch) | |
tree | 9d9a45d251c701ee443ae4ff10c31505b8ea06e8 /hw | |
parent | 39d16d29c81295be72dbae6e6bc7adc58deacb41 (diff) | |
download | qemu-9d5614d582d23ec96b167583557bf3f25f64f050.zip qemu-9d5614d582d23ec96b167583557bf3f25f64f050.tar.gz qemu-9d5614d582d23ec96b167583557bf3f25f64f050.tar.bz2 |
hw/timer/grlib_gptimer: Avoid integer overflows
The GPTIMER uses 32-bit registers. Use a 64-bit operation to get the
ptimer count, otherwise we end up with a count of 0 for GPTIMER counter
values of 0xffffffff.
Use the GPTIMER counter value for tracing to avoid an overflow of the
32-bit value passed to trace_grlib_gptimer_enable().
Reviewed-by: Fabien Chouteau <chouteau@adacore.com>
Signed-off-by: Sebastian Huber <sebastian.huber@embedded-brains.de>
Signed-off-by: Michael Tokarev <mjt@tls.msk.ru>
Diffstat (limited to 'hw')
-rw-r--r-- | hw/timer/grlib_gptimer.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/hw/timer/grlib_gptimer.c b/hw/timer/grlib_gptimer.c index 74c16d6..7672d3a 100644 --- a/hw/timer/grlib_gptimer.c +++ b/hw/timer/grlib_gptimer.c @@ -106,9 +106,9 @@ static void grlib_gptimer_enable(GPTimer *timer) /* ptimer is triggered when the counter reach 0 but GPTimer is triggered at underflow. Set count + 1 to simulate the GPTimer behavior. */ - trace_grlib_gptimer_enable(timer->id, timer->counter + 1); + trace_grlib_gptimer_enable(timer->id, timer->counter); - ptimer_set_count(timer->ptimer, timer->counter + 1); + ptimer_set_count(timer->ptimer, (uint64_t)timer->counter + 1); ptimer_run(timer->ptimer, 1); } |