Alex Bennée <[email protected]> writes:
> Junjie Mao <[email protected]> writes: > >> The peripheral and PrimeCell identification registers of pl011 are located at >> offset 0xFE0 - 0xFFC. To check if a read falls to such registers, the C >> implementation checks if the offset-shifted-by-2 (not the offset itself) is >> in >> the range 0x3F8 - 0x3FF. >> >> Use the same check in the Rust implementation. >> >> This fixes the timeout of the following avocado tests: >> >> * tests/avocado/boot_linux_console.py:BootLinuxConsole.test_arm_virt >> * tests/avocado/replay_kernel.py:ReplayKernelNormal.test_arm_virt >> * tests/avocado/replay_kernel.py:ReplayKernelNormal.test_arm_vexpressa9 >> >> Reported-by: Peter Maydell <[email protected]> >> Signed-off-by: Junjie Mao <[email protected]> > > This certainly fixes the avocado failures. > > Tested-by: Alex Bennée <[email protected]> > Reviewed-by: Alex Bennée <[email protected]> > Thanks for reviewing and testing, Alex! Meanwhile, Manos has submitted another fix [1] which also replaces arrays of constant register values with more explicit register getters. His change may supercedes mine. [1] https://lore.kernel.org/qemu-devel/[email protected] -- Best Regards Junjie Mao
