On Fri, Sep 21, 2018 at 11:16:55PM +0100, James Morse wrote: > ghes_copy_tofrom_phys() takes different locks depending on in_nmi(). > This doesn't work when we have multiple NMI-like notifications, that > can interrupt each other. > > Now that NOTIFY_SEA is always called as an NMI, move the lock-taking > to the notification helper. The helper will always know which lock > to take. This avoids ghes_copy_tofrom_phys() taking a guess based > on in_nmi(). > > This splits NOTIFY_NMI and NOTIFY_SEA to use different locks. All > the other notifications use ghes_proc(), and are called in process > or IRQ context. Move the spin_lock_irqsave() around their ghes_proc() > calls.
Right, should ghes_proc() be renamed to ghes_proc_irq() now, to be absolutely clear on the processing context it is operating in? Other than that: Reviewed-by: Borislav Petkov <[email protected]> -- Regards/Gruss, Boris. Good mailing practices for 400: avoid top-posting and trim the reply. _______________________________________________ kvmarm mailing list [email protected] https://lists.cs.columbia.edu/mailman/listinfo/kvmarm
