Message ID | 20211126163915.1048353-1-peter.maydell@linaro.org |
---|---|
Headers | show |
Series | arm_gicv3: Fix handling of LPIs in list registers | expand |
Hi Peter, On Fri, 26 Nov 2021 16:39:13 +0000, Peter Maydell <peter.maydell@linaro.org> wrote: > > (Marc: cc'd you on this one in case you're still using QEMU > to test KVM stuff with, in which case you might have run into > the bug this is fixing.) Amusingly enough, I have recently fixed [1] a very similar issue with the ICV_*_EL1 emulation that KVM uses when dealing with sub-par HW (ThunderX and M1). When writing this a very long while ago, I modelled it so that LPIs wouldn't have an Active state, similar to bare metal. As it turns out, the pseudocode actually treats LPIs almost like any other interrupt, and is quite happy to carry an active bit that eventually gets exposed to the hypervisor. I don't think this ever caused any issue, but I'd be pretty happy to see the QEMU implementation fixed. For the whole series: Reviewed-by: Marc Zyngier <maz@kernel.org> Thanks, M. [1] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/arch/arm64/kvm/hyp/vgic-v3-sr.c?id=9d449c71bd8f74282e84213c8f0b8328293ab0a7