i386: hvf: Make long mode enter and exit clearer
Intel SDM "9.8.5 Initializing IA-32e Mode" and "9.8.5.4 Switching Out of IA-32e Mode Operation" define activation and deactivation of long mode only upon a change of CR0.PG but current code invokes exit_long_mode() unconditionally until LME is cleared. Signed-off-by: Cameron Esfahani <dirty@apple.com> Signed-off-by: Roman Bolshakov <r.bolshakov@yadro.com> Message-Id: <20200630102824.77604-6-r.bolshakov@yadro.com> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
This commit is contained in:
parent
5536c98e44
commit
a4e26fa858
@ -121,6 +121,7 @@ static inline void macvm_set_cr0(hv_vcpuid_t vcpu, uint64_t cr0)
|
|||||||
uint64_t pdpte[4] = {0, 0, 0, 0};
|
uint64_t pdpte[4] = {0, 0, 0, 0};
|
||||||
uint64_t efer = rvmcs(vcpu, VMCS_GUEST_IA32_EFER);
|
uint64_t efer = rvmcs(vcpu, VMCS_GUEST_IA32_EFER);
|
||||||
uint64_t old_cr0 = rvmcs(vcpu, VMCS_GUEST_CR0);
|
uint64_t old_cr0 = rvmcs(vcpu, VMCS_GUEST_CR0);
|
||||||
|
uint64_t changed_cr0 = old_cr0 ^ cr0;
|
||||||
uint64_t mask = CR0_PG | CR0_CD | CR0_NW | CR0_NE | CR0_ET;
|
uint64_t mask = CR0_PG | CR0_CD | CR0_NW | CR0_NE | CR0_ET;
|
||||||
|
|
||||||
if ((cr0 & CR0_PG) && (rvmcs(vcpu, VMCS_GUEST_CR4) & CR4_PAE) &&
|
if ((cr0 & CR0_PG) && (rvmcs(vcpu, VMCS_GUEST_CR4) & CR4_PAE) &&
|
||||||
@ -138,13 +139,14 @@ static inline void macvm_set_cr0(hv_vcpuid_t vcpu, uint64_t cr0)
|
|||||||
wvmcs(vcpu, VMCS_CR0_SHADOW, cr0);
|
wvmcs(vcpu, VMCS_CR0_SHADOW, cr0);
|
||||||
|
|
||||||
if (efer & MSR_EFER_LME) {
|
if (efer & MSR_EFER_LME) {
|
||||||
if (!(old_cr0 & CR0_PG) && (cr0 & CR0_PG)) {
|
if (changed_cr0 & CR0_PG) {
|
||||||
|
if (cr0 & CR0_PG) {
|
||||||
enter_long_mode(vcpu, cr0, efer);
|
enter_long_mode(vcpu, cr0, efer);
|
||||||
}
|
} else {
|
||||||
if (/*(old_cr0 & CR0_PG) &&*/ !(cr0 & CR0_PG)) {
|
|
||||||
exit_long_mode(vcpu, cr0, efer);
|
exit_long_mode(vcpu, cr0, efer);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
}
|
||||||
|
|
||||||
/* Filter new CR0 after we are finished examining it above. */
|
/* Filter new CR0 after we are finished examining it above. */
|
||||||
cr0 = (cr0 & ~(mask & ~CR0_PG));
|
cr0 = (cr0 & ~(mask & ~CR0_PG));
|
||||||
|
Loading…
Reference in New Issue
Block a user