target/i386/hvf/vmx.h | 1 + 1 file changed, 1 insertion(+)
Removal of register reset omitted initialization of CR4 guest/host mask.
x86_64 guests aren't booting without it.
Fixes: 5009ef22c6bb2 ("i386: hvf: Don't duplicate register reset")
Signed-off-by: Roman Bolshakov <r.bolshakov@yadro.com>
---
target/i386/hvf/vmx.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/target/i386/hvf/vmx.h b/target/i386/hvf/vmx.h
index 75ba1e2a5f..587b1b8375 100644
--- a/target/i386/hvf/vmx.h
+++ b/target/i386/hvf/vmx.h
@@ -166,6 +166,7 @@ static inline void macvm_set_cr4(hv_vcpuid_t vcpu, uint64_t cr4)
wvmcs(vcpu, VMCS_GUEST_CR4, guest_cr4);
wvmcs(vcpu, VMCS_CR4_SHADOW, cr4);
+ wvmcs(vcpu, VMCS_CR4_MASK, CR4_VMXE);
hv_vcpu_invalidate_tlb(vcpu);
hv_vcpu_flush(vcpu);
--
2.26.1
On Tue, Jul 14, 2020 at 12:07:27PM +0300, Roman Bolshakov wrote: > Removal of register reset omitted initialization of CR4 guest/host mask. > x86_64 guests aren't booting without it. > > Fixes: 5009ef22c6bb2 ("i386: hvf: Don't duplicate register reset") > Signed-off-by: Roman Bolshakov <r.bolshakov@yadro.com> > If one has a chance to test or review it, it'd be very helpful. That'd allow to include it in RC0. Thanks, Roman
Hi Roman, please ask Peter to apply it directly because I won't be able to send a pull request in the next couple of weeks. Paolo Il mar 14 lug 2020, 12:39 Roman Bolshakov <r.bolshakov@yadro.com> ha scritto: > On Tue, Jul 14, 2020 at 12:07:27PM +0300, Roman Bolshakov wrote: > > Removal of register reset omitted initialization of CR4 guest/host mask. > > x86_64 guests aren't booting without it. > > > > Fixes: 5009ef22c6bb2 ("i386: hvf: Don't duplicate register reset") > > Signed-off-by: Roman Bolshakov <r.bolshakov@yadro.com> > > > > If one has a chance to test or review it, it'd be very helpful. That'd > allow to include it in RC0. > > Thanks, > Roman > >
On Tue, Jul 14, 2020 at 08:20:04PM +0200, Paolo Bonzini wrote: > Hi Roman, please ask Peter to apply it directly because I won't be able to > send a pull request in the next couple of weeks. > > Paolo > > Il mar 14 lug 2020, 12:39 Roman Bolshakov <r.bolshakov@yadro.com> ha > scritto: > > > On Tue, Jul 14, 2020 at 12:07:27PM +0300, Roman Bolshakov wrote: > > > Removal of register reset omitted initialization of CR4 guest/host mask. > > > x86_64 guests aren't booting without it. > > > > > > Fixes: 5009ef22c6bb2 ("i386: hvf: Don't duplicate register reset") > > > Signed-off-by: Roman Bolshakov <r.bolshakov@yadro.com> > > > > > > > If one has a chance to test or review it, it'd be very helpful. That'd > > allow to include it in RC0. > > I'll queue it for my -rc1 pull request. -- Eduardo
On Thu, Jul 16, 2020 at 02:14:57PM -0400, Eduardo Habkost wrote: > On Tue, Jul 14, 2020 at 08:20:04PM +0200, Paolo Bonzini wrote: > > Hi Roman, please ask Peter to apply it directly because I won't be able to > > send a pull request in the next couple of weeks. > > > > Paolo > > > > Il mar 14 lug 2020, 12:39 Roman Bolshakov <r.bolshakov@yadro.com> ha > > scritto: > > > > > On Tue, Jul 14, 2020 at 12:07:27PM +0300, Roman Bolshakov wrote: > > > > Removal of register reset omitted initialization of CR4 guest/host mask. > > > > x86_64 guests aren't booting without it. > > > > > > > > Fixes: 5009ef22c6bb2 ("i386: hvf: Don't duplicate register reset") > > > > Signed-off-by: Roman Bolshakov <r.bolshakov@yadro.com> > > > > > > > > > > If one has a chance to test or review it, it'd be very helpful. That'd > > > allow to include it in RC0. > > > > > I'll queue it for my -rc1 pull request. > Thanks! -Roman
© 2016 - 2024 Red Hat, Inc.