FAILED: patch "[PATCH] KVM: nVMX: vmread should not set rflags to specify success in" failed to apply to 5.4-stable tree
Subsystems (3)
[email protected]
maintainerThe patch below does not apply to the 5.4-stable tree.
If someone wants it applied there, or to any other stable or longterm tree, then please email the backport, including the original git commit id to <[email protected]>.
thanks,
greg k-h
------------------ original commit in Linus's tree ------------------
From a4d956b9390418623ae5d07933e2679c68b6f83c Mon Sep 17 00:00:00 2001
From: Miaohe Lin <[email protected]>
Date: Sat, 28 Dec 2019 14:25:24 +0800
Subject: [PATCH] KVM: nVMX: vmread should not set rflags to specify success in
case of #PF
In case writing to vmread destination operand result in a #PF, vmread should not call nested_vmx_succeed() to set rflags to specify success. Similar to as done in VMPTRST (See handle_vmptrst()).
arch/x86/kvm/vmx/nested.c +3 -1
@@ -4799,8 +4799,10 @@ static int handle_vmread(struct kvm_vcpu *vcpu) instr_info, true, len, &gva))return 1 ;/* _system ok, nested_vmx_check_permission has verified cpl=0 */ -if (kvm_write_guest_virt_system (vcpu, gva, &value, len, &e)) +if (kvm_write_guest_virt_system (vcpu, gva, &value, len, &e)) {kvm_inject_page_fault (vcpu, &e); +return 1 ; + } }return nested_vmx_succeed (vcpu);
Sasha Levin
maintainerQuoted 22 lines · On Sun, Feb 09, 2020 at 01:36:24PM +0100, [email protected] wrote:
The patch below does not apply to the 5.4-stable tree.
If someone wants it applied there, or to any other stable or longterm tree, then please email the backport, including the original git commit id to <[email protected]>.thanks,
greg k-h
------------------ original commit in Linus's tree ------------------
From
a4d956b9390418623ae5d07933e2679c68b6f83cMon Sep 17 00:00:00 2001
From: Miaohe Lin <[email protected]>
Date: Sat, 28 Dec 2019 14:25:24 +0800
Subject: [PATCH] KVM: nVMX: vmread should not set rflags to specify success in case of #PFIn case writing to vmread destination operand result in a #PF, vmread should not call
nested_vmx_succeed()to set rflags to specify success. Similar to as done in VMPTRST (Seehandle_vmptrst()).
Quoted 5 lines
Reviewed-by Liran Alon <[email protected]>Signed-off-by Miaohe Lin <[email protected]>Reviewed-by Sean Christopherson <[email protected]>Signed-off-by Paolo Bonzini <[email protected]>
Conflict because of missing c90f4d03cce1 ("kvm: nVMX: Aesthetic cleanup of handle_vmread and handle_vmwrite") on older kernels. Cleaned up and queued for 5.4-4.4.
Signature · 2 lines
Thanks, Sasha