On Wed, Dec 09, 2020 at 06:00:52PM +0100, Greg Kurz wrote: > kvm_handle_nmi() directly calls spapr_mce_req_event() which is machine > level code. Apart from being ugly, this forces spapr_mce_req_event() > to rely on qdev_get_machine() to get a pointer to the machine state. > This is a bit unfortunate since POWER CPUs have a backlink to the > virtual hypervisor, which happens to be the machine itself with > sPAPR. > > Turn spapr_mce_req_event() into a PPC virtual hypervisor operation, > and adapt kvm_handle_nmi() to call it as such. > > Signed-off-by: Greg Kurz <gr...@kaod.org>
I have somewhat mixed thoughts on this. Putting it in vhyp makes a certain sense. But on the other hand, the MCE event from KVM is an explicitly PAPR specific interface, so it can't really go to any other implementation. -- David Gibson | I'll have my music baroque, and my code david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_ | _way_ _around_! http://www.ozlabs.org/~dgibson
signature.asc
Description: PGP signature