On 15/05/2018 12:26, Christoffer Dall wrote: > On Fri, May 11, 2018 at 03:20:11PM +0100, Andre Przywara wrote: >> Jan recently reported lockdep complaints regarding various locks in our >> VGIC emulation [1][2]. >> This boiled down to two separate issues: >> - When promoting the vgic_irq->irq_lock to require IRQs being disabled, >> we forgot to amend some instances of this lock on the way. Also this >> needs to be applied to dependent locks as well. The first two patches >> fix that. The patch split is designed to simplify backporting. >> Those patches have been posted before, I am resending them as part >> of this series. >> - Calling kvm_read_guest() requires us to be inside an SRCU critical >> section. On some architectures we are always in it when handling VCPU >> exits, but on ARM we need to lock it individually. Patches 3 and 4 >> fix that, the split is again made to ease backporting. >> Each of the hunks fix an indiviual commit, but I refrained from >> splitting this down into eight patches just to put proper Fixes: tags >> on it. Eventually those commits are part of one out of two series, I put >> the respective kernel release version as a tag to the Cc: stable line. >> >> I couldn't reproduce the full lockdep splat on my setup, but at least >> could show one instance and prove that these patches fixes that. >> >> > For the series: > > Acked-by: Christoffer Dall <christoffer.dall@xxxxxxx> Shall I put the patches on their route to Linus? Thanks, Paolo
