Fix exiting on CR0 write for unrestricted guests (#883)

vcpu::set_eptp() didn't disable exiting on writes to CR0. On the first
run (BSP) handlers were set as if there was no support for unrestricted
guests. Default handler for CR0 writes called emulate_ia_32e_mode_switch(),
which doesn't allow for switches between all modes.

Signed-off-by: Krystian Hebel <[email protected]>

Author: krystian-hebel

bfvmm/src/hve/arch/intel_x64/vcpu.cpp

@@ -891,15 +891,33 @@ vcpu::disable_preemption_timer()
 void
 vcpu::set_eptp(ept::mmap &map)
 {
+    using namespace ::intel_x64::vmcs;
+    using namespace ::intel_x64::cr0;
+
     m_ept_handler.set_eptp(&map);
     m_mmap = ↦
+
+    // Disable exiting on changes to CR0.PG and CR0.PE.
+    //
+    // They were enabled by vCPU constructor for systems not using unrestricted
+    // guests. This led to different treatment of mode changes on BSP than APs,
+    // and as a result to inability to use some modes on BSP (e.g. 32b mode
+    // without PAE).
+    cr0_guest_host_mask::set(cr0_guest_host_mask::get() & ~(paging::mask | protection_enable::mask));
+    m_control_register_handler.enable_wrcr0_exiting(cr0_guest_host_mask::get());
 }
 
 void
 vcpu::disable_ept()
 {
+    using namespace ::intel_x64::vmcs;
+    using namespace ::intel_x64::cr0;
+
     m_ept_handler.set_eptp(nullptr);
     m_mmap = nullptr;
+
+    cr0_guest_host_mask::set(cr0_guest_host_mask::get() | (paging::mask | protection_enable::mask));
+    m_control_register_handler.enable_wrcr0_exiting(cr0_guest_host_mask::get());
 }
 
 //==========================================================================