* [PATCH] KVM: nVMX: unconditionally cancel preemption timer in free_nested (CVE-2019-7221)
@ 2019-02-07 18:09 Paolo Bonzini
2019-02-11 17:26 ` Sasha Levin
2019-02-11 17:26 ` Sasha Levin
0 siblings, 2 replies; 3+ messages in thread
From: Paolo Bonzini @ 2019-02-07 18:09 UTC (permalink / raw)
To: linux-kernel, kvm; +Cc: Peter Shier, stable
From: Peter Shier <pshier@google.com>
Bugzilla: 1671904
There are multiple code paths where an hrtimer may have been started to
emulate an L1 VMX preemption timer that can result in a call to free_nested
without an intervening L2 exit where the hrtimer is normally
cancelled. Unconditionally cancel in free_nested to cover all cases.
Embargoed until Feb 7th 2019.
Signed-off-by: Peter Shier <pshier@google.com>
Reported-by: Jim Mattson <jmattson@google.com>
Reviewed-by: Jim Mattson <jmattson@google.com>
Reported-by: Felix Wilhelm <fwilhelm@google.com>
Cc: stable@kernel.org
Message-Id: <20181011184646.154065-1-pshier@google.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
---
arch/x86/kvm/vmx/nested.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/x86/kvm/vmx/nested.c b/arch/x86/kvm/vmx/nested.c
index 8ff20523661b..d8ea4ebd79e7 100644
--- a/arch/x86/kvm/vmx/nested.c
+++ b/arch/x86/kvm/vmx/nested.c
@@ -211,6 +211,7 @@ static void free_nested(struct kvm_vcpu *vcpu)
if (!vmx->nested.vmxon && !vmx->nested.smm.vmxon)
return;
+ hrtimer_cancel(&vmx->nested.preemption_timer);
vmx->nested.vmxon = false;
vmx->nested.smm.vmxon = false;
free_vpid(vmx->nested.vpid02);
--
1.8.3.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] KVM: nVMX: unconditionally cancel preemption timer in free_nested (CVE-2019-7221)
2019-02-07 18:09 [PATCH] KVM: nVMX: unconditionally cancel preemption timer in free_nested (CVE-2019-7221) Paolo Bonzini
@ 2019-02-11 17:26 ` Sasha Levin
2019-02-11 17:26 ` Sasha Levin
1 sibling, 0 replies; 3+ messages in thread
From: Sasha Levin @ 2019-02-11 17:26 UTC (permalink / raw)
To: Sasha Levin, Paolo Bonzini, Peter Shier, linux-kernel, kvm
Cc: Peter Shier, stable, stable, stable
<20181011184646.154065-1-pshier@google.com>
References: <1549562945-5503-3-git-send-email-pbonzini@redhat.com>
<20181011184646.154065-1-pshier@google.com>
Hi,
[This is an automated email]
This commit has been processed because it contains a -stable tag.
The stable tag indicates that it's relevant for the following trees: .+
The bot has tested the following trees: v4.20.7, v4.19.20, v4.14.98, v4.9.155, v4.4.173, v3.18.134.
v4.20.7: Failed to apply! Possible dependencies:
199b118ab3d5 ("KVM: VMX: Alphabetize the includes in vmx.c")
55d2375e58a6 ("KVM: nVMX: Move nested code to dedicated files")
609363cf81fc ("KVM: nVMX: Move vmcs12 code to dedicated files")
75edce8a4548 ("KVM: VMX: Move eVMCS code to dedicated files")
a821bab2d1ee ("KVM: VMX: Move VMX specific files to a "vmx" subdirectory")
v4.19.20: Failed to apply! Possible dependencies:
09abb5e3e5e5 ("KVM: nVMX: call kvm_skip_emulated_instruction in nested_vmx_{fail,succeed}")
09abe3200266 ("KVM: nVMX: split pieces of prepare_vmcs02() to prepare_vmcs02_early()")
1438921c6dc1 ("KVM: nVMX: Flush TLB entries tagged by dest EPTP on L1<->L2 transitions")
199b118ab3d5 ("KVM: VMX: Alphabetize the includes in vmx.c")
1abf23fb42f5 ("KVM: nVMX: use vm_exit_controls_init() to write exit controls for vmcs02")
327c072187f7 ("KVM: nVMX: Flush linear and combined mappings on VPID02 related flushes")
3d5bdae8b164 ("KVM: nVMX: Use correct VPID02 when emulating L1 INVVPID")
3df5c37e55c8 ("KVM: nVMX: try to set EFER bits correctly when initializing controls")
55d2375e58a6 ("KVM: nVMX: Move nested code to dedicated files")
5b8ba41dafd7 ("KVM: nVMX: move vmcs12 EPTP consistency check to check_vmentry_prereqs()")
609363cf81fc ("KVM: nVMX: Move vmcs12 code to dedicated files")
75edce8a4548 ("KVM: VMX: Move eVMCS code to dedicated files")
7671ce21b13b ("KVM: nVMX: move check_vmentry_postreqs() call to nested_vmx_enter_non_root_mode()")
945679e301ea ("KVM: nVMX: add enlightened VMCS state")
a633e41e7362 ("KVM: nVMX: assimilate nested_vmx_entry_failure() into nested_vmx_enter_non_root_mode()")
a821bab2d1ee ("KVM: VMX: Move VMX specific files to a "vmx" subdirectory")
b8bbab928fb1 ("KVM: nVMX: implement enlightened VMPTRLD and VMCLEAR")
d63907dc7dd1 ("KVM: nVMX: rename enter_vmx_non_root_mode to nested_vmx_enter_non_root_mode")
efebf0aaec3d ("KVM: nVMX: Do not flush TLB on L1<->L2 transitions if L1 uses VPID and EPT")
v4.14.98: Failed to apply! Possible dependencies:
0234bf885236 ("KVM: x86: introduce ISA specific SMM entry/exit callbacks")
199b118ab3d5 ("KVM: VMX: Alphabetize the includes in vmx.c")
44900ba65e16 ("KVM: VMX: optimize shadow VMCS copying")
55d2375e58a6 ("KVM: nVMX: Move nested code to dedicated files")
5b15706dbf5b ("kvm: vmx: Introduce VMCS12_MAX_FIELD_INDEX")
609363cf81fc ("KVM: nVMX: Move vmcs12 code to dedicated files")
6677f3dad874 ("KVM: nVMX: introduce struct nested_vmx_msrs")
72d7b374b14d ("KVM: x86: introduce ISA specific smi_allowed callback")
72e9cbdb4338 ("KVM: nVMX: fix SMI injection in guest mode")
736fdf72518b ("KVM: VMX: rename RDSEED and RDRAND vmx ctrls to reflect exiting")
74a497fae754 ("KVM: nVMX: track dirty state of non-shadowed VMCS fields")
75edce8a4548 ("KVM: VMX: Move eVMCS code to dedicated files")
945679e301ea ("KVM: nVMX: add enlightened VMCS state")
a821bab2d1ee ("KVM: VMX: Move VMX specific files to a "vmx" subdirectory")
b8bbab928fb1 ("KVM: nVMX: implement enlightened VMPTRLD and VMCLEAR")
c5d167b27e00 ("KVM: vmx: shadow more fields that are read/written on every vmexits")
c9e9deae76b8 ("KVM: VMX: split list of shadowed VMCS field to a separate file")
f4160e459c57 ("kvm: nVMX: Add support for "VMWRITE to any supported field"")
v4.9.155: Failed to apply! Possible dependencies:
1279a6b124e4 ("KVM: nVMX: single function for switching between vmcs")
199b118ab3d5 ("KVM: VMX: Alphabetize the includes in vmx.c")
27c42a1bb867 ("KVM: nVMX: Enable VMFUNC for the L1 hypervisor")
2a499e49c2ec ("KVM: vmx: Enable VMFUNCs")
3899152ccbf4 ("KVM: nVMX: fix checks on CR{0,4} during virtual VMX operation")
44900ba65e16 ("KVM: VMX: optimize shadow VMCS copying")
4f350c6dbcb9 ("kvm: nVMX: Handle deferred early VMLAUNCH/VMRESUME failure properly")
55d2375e58a6 ("KVM: nVMX: Move nested code to dedicated files")
5a6a9748b4b4 ("KVM: nVMX: load GUEST_EFER after GUEST_CR0 during emulated VM-entry")
609363cf81fc ("KVM: nVMX: Move vmcs12 code to dedicated files")
62cc6b9dc61e ("KVM: nVMX: support restore of VMX capability MSRs")
72e9cbdb4338 ("KVM: nVMX: fix SMI injection in guest mode")
74a497fae754 ("KVM: nVMX: track dirty state of non-shadowed VMCS fields")
75edce8a4548 ("KVM: VMX: Move eVMCS code to dedicated files")
7ca29de21362 ("KVM: nVMX: fix CR3 load if L2 uses PAE paging and EPT")
858e25c06fb0 ("kvm: nVMX: Refactor nested_vmx_run()")
85fd514e2423 ("kvm: nVMX: Shadow "high" parts of shadowed 64-bit VMCS fields")
945679e301ea ("KVM: nVMX: add enlightened VMCS state")
9ed38ffad473 ("KVM: nVMX: introduce nested_vmx_load_cr3 and call it on vmentry")
a821bab2d1ee ("KVM: VMX: Move VMX specific files to a "vmx" subdirectory")
a8bc284eb70f ("kvm: nVMX: Refactor handle_vmptrld()")
b8bbab928fb1 ("KVM: nVMX: implement enlightened VMPTRLD and VMCLEAR")
c5d167b27e00 ("KVM: vmx: shadow more fields that are read/written on every vmexits")
c7c2c709b60e ("kvm: nVMX: Validate CR3 target count on nested VM-entry")
c9e9deae76b8 ("KVM: VMX: split list of shadowed VMCS field to a separate file")
ca0bde28f2ed ("kvm: nVMX: Split VMCS checks from nested_vmx_run()")
cf3215d9394a ("kvm: nVMX: Fetch VM_INSTRUCTION_ERROR from vmcs02 on vmx->fail")
ee146c1c100d ("KVM: nVMX: propagate errors from prepare_vmcs02")
v4.4.173: Failed to apply! Possible dependencies:
199b118ab3d5 ("KVM: VMX: Alphabetize the includes in vmx.c")
355f4fb1405e ("kvm: nVMX: VMCLEAR an active shadow VMCS after last use")
4f2777bc9797 ("kvm: x86: nVMX: maintain internal copy of current VMCS")
55d2375e58a6 ("KVM: nVMX: Move nested code to dedicated files")
609363cf81fc ("KVM: nVMX: Move vmcs12 code to dedicated files")
6308630bd3db ("kvm/x86: split ioapic-handled and EOI exit bitmaps")
74a497fae754 ("KVM: nVMX: track dirty state of non-shadowed VMCS fields")
75edce8a4548 ("KVM: VMX: Move eVMCS code to dedicated files")
945679e301ea ("KVM: nVMX: add enlightened VMCS state")
a821bab2d1ee ("KVM: VMX: Move VMX specific files to a "vmx" subdirectory")
a8bc284eb70f ("kvm: nVMX: Refactor handle_vmptrld()")
b8bbab928fb1 ("KVM: nVMX: implement enlightened VMPTRLD and VMCLEAR")
bb3541f175a9 ("KVM: x86: Fix typos")
d62caabb41f3 ("kvm/x86: per-vcpu apicv deactivation support")
v3.18.134: Failed to apply! Possible dependencies:
394457a928e0 ("KVM: x86: some apic broadcast modes does not work")
55d2375e58a6 ("KVM: nVMX: Move nested code to dedicated files")
6ef768fac9df ("kvm: x86: move ioapic.c and irq_comm.c back to arch/x86/")
70109e7d9d4a ("KVM: MTRR: remove mtrr_state.have_fixed")
c274e03af705 ("kvm: x86: move assigned-dev.c and iommu.c to arch/x86/")
e83d58874ba1 ("kvm/x86: move Hyper-V MSR's/hypercall code into hyperv.c file")
ff53604b40b4 ("KVM: x86: move MTRR related code to a separate file")
How should we proceed with this patch?
--
Thanks,
Sasha
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH] KVM: nVMX: unconditionally cancel preemption timer in free_nested (CVE-2019-7221)
2019-02-07 18:09 [PATCH] KVM: nVMX: unconditionally cancel preemption timer in free_nested (CVE-2019-7221) Paolo Bonzini
2019-02-11 17:26 ` Sasha Levin
@ 2019-02-11 17:26 ` Sasha Levin
1 sibling, 0 replies; 3+ messages in thread
From: Sasha Levin @ 2019-02-11 17:26 UTC (permalink / raw)
To: Sasha Levin, Paolo Bonzini, Peter Shier, linux-kernel, kvm
Cc: Peter Shier, stable, stable, stable
<20181011184646.154065-1-pshier@google.com>
References: <1549562945-5503-3-git-send-email-pbonzini@redhat.com>
<20181011184646.154065-1-pshier@google.com>
Hi,
[This is an automated email]
This commit has been processed because it contains a -stable tag.
The stable tag indicates that it's relevant for the following trees: .+
The bot has tested the following trees: v4.20.7, v4.19.20, v4.14.98, v4.9.155, v4.4.173, v3.18.134.
v4.20.7: Failed to apply! Possible dependencies:
199b118ab3d5 ("KVM: VMX: Alphabetize the includes in vmx.c")
55d2375e58a6 ("KVM: nVMX: Move nested code to dedicated files")
609363cf81fc ("KVM: nVMX: Move vmcs12 code to dedicated files")
75edce8a4548 ("KVM: VMX: Move eVMCS code to dedicated files")
a821bab2d1ee ("KVM: VMX: Move VMX specific files to a "vmx" subdirectory")
v4.19.20: Failed to apply! Possible dependencies:
09abb5e3e5e5 ("KVM: nVMX: call kvm_skip_emulated_instruction in nested_vmx_{fail,succeed}")
09abe3200266 ("KVM: nVMX: split pieces of prepare_vmcs02() to prepare_vmcs02_early()")
1438921c6dc1 ("KVM: nVMX: Flush TLB entries tagged by dest EPTP on L1<->L2 transitions")
199b118ab3d5 ("KVM: VMX: Alphabetize the includes in vmx.c")
1abf23fb42f5 ("KVM: nVMX: use vm_exit_controls_init() to write exit controls for vmcs02")
327c072187f7 ("KVM: nVMX: Flush linear and combined mappings on VPID02 related flushes")
3d5bdae8b164 ("KVM: nVMX: Use correct VPID02 when emulating L1 INVVPID")
3df5c37e55c8 ("KVM: nVMX: try to set EFER bits correctly when initializing controls")
55d2375e58a6 ("KVM: nVMX: Move nested code to dedicated files")
5b8ba41dafd7 ("KVM: nVMX: move vmcs12 EPTP consistency check to check_vmentry_prereqs()")
609363cf81fc ("KVM: nVMX: Move vmcs12 code to dedicated files")
75edce8a4548 ("KVM: VMX: Move eVMCS code to dedicated files")
7671ce21b13b ("KVM: nVMX: move check_vmentry_postreqs() call to nested_vmx_enter_non_root_mode()")
945679e301ea ("KVM: nVMX: add enlightened VMCS state")
a633e41e7362 ("KVM: nVMX: assimilate nested_vmx_entry_failure() into nested_vmx_enter_non_root_mode()")
a821bab2d1ee ("KVM: VMX: Move VMX specific files to a "vmx" subdirectory")
b8bbab928fb1 ("KVM: nVMX: implement enlightened VMPTRLD and VMCLEAR")
d63907dc7dd1 ("KVM: nVMX: rename enter_vmx_non_root_mode to nested_vmx_enter_non_root_mode")
efebf0aaec3d ("KVM: nVMX: Do not flush TLB on L1<->L2 transitions if L1 uses VPID and EPT")
v4.14.98: Failed to apply! Possible dependencies:
0234bf885236 ("KVM: x86: introduce ISA specific SMM entry/exit callbacks")
199b118ab3d5 ("KVM: VMX: Alphabetize the includes in vmx.c")
44900ba65e16 ("KVM: VMX: optimize shadow VMCS copying")
55d2375e58a6 ("KVM: nVMX: Move nested code to dedicated files")
5b15706dbf5b ("kvm: vmx: Introduce VMCS12_MAX_FIELD_INDEX")
609363cf81fc ("KVM: nVMX: Move vmcs12 code to dedicated files")
6677f3dad874 ("KVM: nVMX: introduce struct nested_vmx_msrs")
72d7b374b14d ("KVM: x86: introduce ISA specific smi_allowed callback")
72e9cbdb4338 ("KVM: nVMX: fix SMI injection in guest mode")
736fdf72518b ("KVM: VMX: rename RDSEED and RDRAND vmx ctrls to reflect exiting")
74a497fae754 ("KVM: nVMX: track dirty state of non-shadowed VMCS fields")
75edce8a4548 ("KVM: VMX: Move eVMCS code to dedicated files")
945679e301ea ("KVM: nVMX: add enlightened VMCS state")
a821bab2d1ee ("KVM: VMX: Move VMX specific files to a "vmx" subdirectory")
b8bbab928fb1 ("KVM: nVMX: implement enlightened VMPTRLD and VMCLEAR")
c5d167b27e00 ("KVM: vmx: shadow more fields that are read/written on every vmexits")
c9e9deae76b8 ("KVM: VMX: split list of shadowed VMCS field to a separate file")
f4160e459c57 ("kvm: nVMX: Add support for "VMWRITE to any supported field"")
v4.9.155: Failed to apply! Possible dependencies:
1279a6b124e4 ("KVM: nVMX: single function for switching between vmcs")
199b118ab3d5 ("KVM: VMX: Alphabetize the includes in vmx.c")
27c42a1bb867 ("KVM: nVMX: Enable VMFUNC for the L1 hypervisor")
2a499e49c2ec ("KVM: vmx: Enable VMFUNCs")
3899152ccbf4 ("KVM: nVMX: fix checks on CR{0,4} during virtual VMX operation")
44900ba65e16 ("KVM: VMX: optimize shadow VMCS copying")
4f350c6dbcb9 ("kvm: nVMX: Handle deferred early VMLAUNCH/VMRESUME failure properly")
55d2375e58a6 ("KVM: nVMX: Move nested code to dedicated files")
5a6a9748b4b4 ("KVM: nVMX: load GUEST_EFER after GUEST_CR0 during emulated VM-entry")
609363cf81fc ("KVM: nVMX: Move vmcs12 code to dedicated files")
62cc6b9dc61e ("KVM: nVMX: support restore of VMX capability MSRs")
72e9cbdb4338 ("KVM: nVMX: fix SMI injection in guest mode")
74a497fae754 ("KVM: nVMX: track dirty state of non-shadowed VMCS fields")
75edce8a4548 ("KVM: VMX: Move eVMCS code to dedicated files")
7ca29de21362 ("KVM: nVMX: fix CR3 load if L2 uses PAE paging and EPT")
858e25c06fb0 ("kvm: nVMX: Refactor nested_vmx_run()")
85fd514e2423 ("kvm: nVMX: Shadow "high" parts of shadowed 64-bit VMCS fields")
945679e301ea ("KVM: nVMX: add enlightened VMCS state")
9ed38ffad473 ("KVM: nVMX: introduce nested_vmx_load_cr3 and call it on vmentry")
a821bab2d1ee ("KVM: VMX: Move VMX specific files to a "vmx" subdirectory")
a8bc284eb70f ("kvm: nVMX: Refactor handle_vmptrld()")
b8bbab928fb1 ("KVM: nVMX: implement enlightened VMPTRLD and VMCLEAR")
c5d167b27e00 ("KVM: vmx: shadow more fields that are read/written on every vmexits")
c7c2c709b60e ("kvm: nVMX: Validate CR3 target count on nested VM-entry")
c9e9deae76b8 ("KVM: VMX: split list of shadowed VMCS field to a separate file")
ca0bde28f2ed ("kvm: nVMX: Split VMCS checks from nested_vmx_run()")
cf3215d9394a ("kvm: nVMX: Fetch VM_INSTRUCTION_ERROR from vmcs02 on vmx->fail")
ee146c1c100d ("KVM: nVMX: propagate errors from prepare_vmcs02")
v4.4.173: Failed to apply! Possible dependencies:
199b118ab3d5 ("KVM: VMX: Alphabetize the includes in vmx.c")
355f4fb1405e ("kvm: nVMX: VMCLEAR an active shadow VMCS after last use")
4f2777bc9797 ("kvm: x86: nVMX: maintain internal copy of current VMCS")
55d2375e58a6 ("KVM: nVMX: Move nested code to dedicated files")
609363cf81fc ("KVM: nVMX: Move vmcs12 code to dedicated files")
6308630bd3db ("kvm/x86: split ioapic-handled and EOI exit bitmaps")
74a497fae754 ("KVM: nVMX: track dirty state of non-shadowed VMCS fields")
75edce8a4548 ("KVM: VMX: Move eVMCS code to dedicated files")
945679e301ea ("KVM: nVMX: add enlightened VMCS state")
a821bab2d1ee ("KVM: VMX: Move VMX specific files to a "vmx" subdirectory")
a8bc284eb70f ("kvm: nVMX: Refactor handle_vmptrld()")
b8bbab928fb1 ("KVM: nVMX: implement enlightened VMPTRLD and VMCLEAR")
bb3541f175a9 ("KVM: x86: Fix typos")
d62caabb41f3 ("kvm/x86: per-vcpu apicv deactivation support")
v3.18.134: Failed to apply! Possible dependencies:
394457a928e0 ("KVM: x86: some apic broadcast modes does not work")
55d2375e58a6 ("KVM: nVMX: Move nested code to dedicated files")
6ef768fac9df ("kvm: x86: move ioapic.c and irq_comm.c back to arch/x86/")
70109e7d9d4a ("KVM: MTRR: remove mtrr_state.have_fixed")
c274e03af705 ("kvm: x86: move assigned-dev.c and iommu.c to arch/x86/")
e83d58874ba1 ("kvm/x86: move Hyper-V MSR's/hypercall code into hyperv.c file")
ff53604b40b4 ("KVM: x86: move MTRR related code to a separate file")
How should we proceed with this patch?
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2019-02-11 17:26 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-02-07 18:09 [PATCH] KVM: nVMX: unconditionally cancel preemption timer in free_nested (CVE-2019-7221) Paolo Bonzini
2019-02-11 17:26 ` Sasha Levin
2019-02-11 17:26 ` Sasha Levin
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).