OSDN Git Service

hw/timer/armv7m_systick: Rewrite to use ptimers
authorPeter Maydell <peter.maydell@linaro.org>
Thu, 15 Oct 2020 15:18:29 +0000 (16:18 +0100)
committerPeter Maydell <peter.maydell@linaro.org>
Tue, 27 Oct 2020 11:15:31 +0000 (11:15 +0000)
commit32bd322a0134ed89db00f2b9b3894982db3dedcb
tree883740b78d2bab137e2e75af39807ac34a9fd54e
parent68d59c6d8d85ae176d3cb2cd20a48d6a090ba288
hw/timer/armv7m_systick: Rewrite to use ptimers

The armv7m systick timer is a 24-bit decrementing, wrap-on-zero,
clear-on-write counter. Our current implementation has various
bugs and dubious workarounds in it (for instance see
https://bugs.launchpad.net/qemu/+bug/1872237).

We have an implementation of a simple decrementing counter
and we put a lot of effort into making sure it handles the
interesting corner cases (like "spend a cycle at 0 before
reloading") -- ptimer.

Rewrite the systick timer to use a ptimer rather than
a raw QEMU timer.

Unfortunately this is a migration compatibility break,
which will affect all M-profile boards.

Among other bugs, this fixes
https://bugs.launchpad.net/qemu/+bug/1872237 :
now writes to SYST_CVR when the timer is enabled correctly
do nothing; when the timer is enabled via SYST_CSR.ENABLE,
the ptimer code will (because of POLICY_NO_IMMEDIATE_RELOAD)
arrange that after one timer tick the counter is reloaded
from SYST_RVR and then counts down from there, as the
architecture requires.

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-id: 20201015151829.14656-3-peter.maydell@linaro.org
hw/timer/armv7m_systick.c
include/hw/timer/armv7m_systick.h