Undo testing changes

Signed-off-by: Ludvig Liljenberg <[email protected]>

Author: ludfjig

.github/workflows/dep_rust.yml

@@ -34,7 +34,7 @@ jobs:
   build:
     if: ${{ inputs.docs_only == 'false' }}
     strategy:
-      fail-fast: false
+      fail-fast: true
       matrix:
         hypervisor: [hyperv, mshv, mshv3, kvm] # hyperv is windows, mshv and kvm are linux
         cpu: [amd, intel]

Justfile

@@ -77,7 +77,7 @@ test target=default-target features="": (test-unit target features) (test-isolat
 
 # runs unit tests
 test-unit target=default-target features="":
-    cargo test {{ if features =="" {''} else if features=="no-default-features" {"--no-default-features" } else {"--no-default-features -F " + features } }} --profile={{ if target == "debug" { "dev" } else { target } }} --lib -- --nocapture --test-threads=1
+    cargo test {{ if features =="" {''} else if features=="no-default-features" {"--no-default-features" } else {"--no-default-features -F " + features } }} --profile={{ if target == "debug" { "dev" } else { target } }} --lib
 
 # runs tests that requires being run separately, for example due to global state
 test-isolated target=default-target features="":

src/hyperlight_host/src/hypervisor/gdb/arch.rs

@@ -67,7 +67,6 @@ pub(crate) fn vcpu_stop_reason(
         // Check page 19-4 Vol. 3B of Intel 64 and IA-32
         // Architectures Software Developer's Manual
         if dr6 & DR6_BS_FLAG_MASK != 0 {
-            log::info!("Done Step stop reason");
             return Ok(VcpuStopReason::DoneStep);
         }
 
@@ -77,17 +76,14 @@ pub(crate) fn vcpu_stop_reason(
         // Architectures Software Developer's Manual
         if DR6_HW_BP_FLAGS_MASK & dr6 != 0 {
             if rip == entrypoint {
-                log::info!("EntryPoint stop reason");
                 vm.remove_hw_breakpoint(entrypoint)?;
                 return Ok(VcpuStopReason::EntryPointBp);
             }
-            log::info!("Hardware breakpoint stop reason");
             return Ok(VcpuStopReason::HwBp);
         }
     }
 
     if BP_EX_ID == exception {
-        log::info!("Software breakpoint stop reason");
         return Ok(VcpuStopReason::SwBp);
     }
 

src/hyperlight_host/src/hypervisor/whp.rs

@@ -170,20 +170,8 @@ impl Vm for WhpVm {
     }
 
     fn get_sregs(&self) -> Result<CommonSpecialRegisters> {
-        println!("whp get_sregs");
         let mut whp_sregs_values: [WHV_REGISTER_VALUE; WHP_SREGS_NAMES_LEN] =
             unsafe { std::mem::zeroed() };
-        println!("whp get_sregs2");
-
-        // println!("partition: {:?}", self.partition);
-        // println!("whp_sregs_names: {:?}", WHP_SREGS_NAMES);
-        // println!("ptr : {:?}", WHP_SREGS_NAMES.as_ptr());
-        // println!("first value: {:?}", unsafe {
-        //     WHP_SREGS_NAMES.as_ptr().read()
-        // });
-        // println!("last value: {:?}", unsafe {
-        //     WHP_SREGS_NAMES.as_ptr().add(WHP_SREGS_NAMES_LEN - 1).read()
-        // });
 
         unsafe {
             WHvGetVirtualProcessorRegisters(
@@ -194,9 +182,8 @@ impl Vm for WhpVm {
                 whp_sregs_values.as_mut_ptr(),
             )?;
         }
-        println!("whp get_sregs3");
 
-        let res = WHP_SREGS_NAMES
+        WHP_SREGS_NAMES
             .into_iter()
             .zip(whp_sregs_values)
             .collect::<Vec<(WHV_REGISTER_NAME, WHV_REGISTER_VALUE)>>()
@@ -207,10 +194,7 @@ impl Vm for WhpVm {
                     "Failed to convert WHP registers to CommonSpecialRegisters: {:?}",
                     e
                 )
-            });
-
-        println!("whp get_sregs");
-        res
+            })
     }
 
     fn set_sregs(&self, sregs: &CommonSpecialRegisters) -> Result<()> {
@@ -348,8 +332,6 @@ impl Vm for WhpVm {
             WHvRunVpExitReasonMemoryAccess => {
                 let gpa = unsafe { exit_context.Anonymous.MemoryAccess.Gpa };
                 let rip = exit_context.VpContext.Rip;
-                println!("MemoryAccess: gpa: {:#x}", gpa);
-                println!("MemoryAccess: rip: {:#x}", rip);
                 let access_info = unsafe {
                     WHV_MEMORY_ACCESS_TYPE(
                         // 2 first bits are the access type, see https://learn.microsoft.com/en-us/virtualization/api/hypervisor-platform/funcs/memoryaccess#syntax