On 2/4/21 11:37 PM, Hao Wu wrote: > I don't see this error. It could be some error in the clock that the > timer module does not get a correct clock input. > How do you reproduce this?
I got it only once, can not reproduce, but it happened on our CI too: https://gitlab.com/qemu-project/qemu/-/jobs/1006073367#L4430 > > On Thu, Feb 4, 2021 at 1:39 AM Philippe Mathieu-Daudé <[email protected] > <mailto:[email protected]>> wrote: > > Hi, > > On Tue, Jan 12, 2021 at 6:20 PM Peter Maydell > <[email protected] <mailto:[email protected]>> > wrote: > > > > From: Hao Wu <[email protected] <mailto:[email protected]>> > > > > This patch makes NPCM7XX Timer to use a the timer clock generated > by the > > CLK module instead of the magic number TIMER_REF_HZ. > > > > Reviewed-by: Havard Skinnemoen <[email protected] > <mailto:[email protected]>> > > Reviewed-by: Tyrone Ting <[email protected] > <mailto:[email protected]>> > > Signed-off-by: Hao Wu <[email protected] > <mailto:[email protected]>> > > Message-id: [email protected] > <mailto:[email protected]> > > Reviewed-by: Peter Maydell <[email protected] > <mailto:[email protected]>> > > Signed-off-by: Peter Maydell <[email protected] > <mailto:[email protected]>> > > --- > > include/hw/misc/npcm7xx_clk.h | 6 ----- > > include/hw/timer/npcm7xx_timer.h | 1 + > > hw/arm/npcm7xx.c | 5 ++++ > > hw/timer/npcm7xx_timer.c | 39 > +++++++++++++++----------------- > > 4 files changed, 24 insertions(+), 27 deletions(-) > > Is that a spurious error (building with Clang)? > > Running test qtest-arm/npcm7xx_timer-test > ERROR:../tests/qtest/npcm7xx_timer-test.c:475:test_periodic_interrupt: > assertion failed (tim_read(td, TISR) == tim_timer_bit(td)): (0x00000000 > == 0x00000004) > ERROR:../tests/qtest/npcm7xx_timer-test.c:476:test_periodic_interrupt: > 'qtest_get_irq(global_qtest, tim_timer_irq(td))' should be TRUE > FAIL 155 qtest-arm/npcm7xx_timer-test > /arm/npcm7xx_timer/tim[2]/timer[2]/periodic_interrupt > make: *** [Makefile.mtest:1033: run-test-127] Error 1 >
