Re: [PATCH 2/2] x86/reboot: Disable virtualization in an emergency if SVM is supported

From: Thomas Gleixner
Date: Thu May 12 2022 - 11:47:55 EST


On Thu, May 12 2022 at 14:39, Sean Christopherson wrote:
> What about leaving cpu_crash_disable_virtualization() inside the if-statement?
> It feels wierd to "disable" virtualization on the current CPU but ignore others,
> e.g. if there's some newfangled type of virtualization in the future, I would be
> quite surprised if only the CPU doing the transfer needed to disable virtualization.

No real preference, though you have the unconditional invocation already
in the crash code IIRC.

Thanks,

tglx