diff options
author | Radim Krčmář <rkrcmar@redhat.com> | 2016-08-08 20:16:23 +0200 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2016-10-07 15:23:46 +0200 |
commit | ae9ba37c044fdad5b760d34a27ef65b54ccaa5fe (patch) | |
tree | 9e5ecb11748111ea9fe55117f25c80b67db82fb6 /drivers/ps3 | |
parent | 564d03966e4446eae39bc9c1a5b6f607eb0eb961 (diff) | |
download | lwn-ae9ba37c044fdad5b760d34a27ef65b54ccaa5fe.tar.gz lwn-ae9ba37c044fdad5b760d34a27ef65b54ccaa5fe.zip |
KVM: nVMX: postpone VMCS changes on MSR_IA32_APICBASE write
commit dccbfcf52cebb8963246eba5b177b77f26b34da0 upstream.
If vmcs12 does not intercept APIC_BASE writes, then KVM will handle the
write with vmcs02 as the current VMCS.
This will incorrectly apply modifications intended for vmcs01 to vmcs02
and L2 can use it to gain access to L0's x2APIC registers by disabling
virtualized x2APIC while using msr bitmap that assumes enabled.
Postpone execution of vmx_set_virtual_x2apic_mode until vmcs01 is the
current VMCS. An alternative solution would temporarily make vmcs01 the
current VMCS, but it requires more care.
Fixes: 8d14695f9542 ("x86, apicv: add virtual x2apic support")
Reported-by: Jim Mattson <jmattson@google.com>
Reviewed-by: Wanpeng Li <wanpeng.li@hotmail.com>
Signed-off-by: Radim Krčmář <rkrcmar@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/ps3')
0 files changed, 0 insertions, 0 deletions