Merge branch 'main' into vm_trait

Author: ludfjig

Cargo.lock

@@ -42,8 +42,7 @@ fn main() -> hyperlight_host::Result<()> {
     // Create an uninitialized sandbox with a guest binary
     let mut uninitialized_sandbox = UninitializedSandbox::new(
         hyperlight_host::GuestBinary::FilePath(hyperlight_testing::simple_guest_as_string().unwrap()),
-        None, // default configuration
-        None, // default run options
+        None // default configuration
     )?;
 
     // Registering a host function makes it available to be called by the guest

README.md

@@ -35,7 +35,6 @@ fuzz_target!(
         let u_sbox = UninitializedSandbox::new(
             GuestBinary::FilePath(simple_guest_for_fuzzing_as_string().expect("Guest Binary Missing")),
             None,
-            None,
         )
         .unwrap();
 

fuzz/fuzz_targets/guest_call.rs

@@ -33,8 +33,7 @@ fuzz_target!(
     init: {
         let u_sbox = UninitializedSandbox::new(
             GuestBinary::FilePath(simple_guest_for_fuzzing_as_string().expect("Guest Binary Missing")),
-            None,
-            None,
+            None
         )
         .unwrap();
 

fuzz/fuzz_targets/host_call.rs

@@ -21,7 +21,6 @@ fuzz_target!(
         let u_sbox = UninitializedSandbox::new(
             GuestBinary::FilePath(simple_guest_for_fuzzing_as_string().expect("Guest Binary Missing")),
             None,
-            None,
         )
         .unwrap();
 

fuzz/fuzz_targets/host_print.rs

@@ -22,16 +22,6 @@ pub const PAGE_SIZE_USIZE: usize = 1 << 12;
 
 use core::ffi::{c_char, c_void};
 
-#[repr(u64)]
-#[derive(Debug, Copy, Clone, PartialEq, Eq)]
-pub enum RunMode {
-    None = 0,
-    Hypervisor = 1,
-    InProcessWindows = 2,
-    InProcessLinux = 3,
-    Invalid = 4,
-}
-
 #[repr(C)]
 pub struct InputData {
     pub inputDataSize: u64,
@@ -67,9 +57,6 @@ pub struct HyperlightPEB {
     pub security_cookie_seed: u64,
     pub guest_function_dispatch_ptr: u64,
     pub pCode: *mut c_char,
-    pub pOutb: *mut c_void,
-    pub pOutbContext: *mut c_void,
-    pub runMode: RunMode,
     pub inputdata: InputData,
     pub outputdata: OutputData,
     pub guestheapData: GuestHeapData,

src/hyperlight_common/src/mem.rs

@@ -15,12 +15,9 @@ limitations under the License.
 */
 
 use core::arch::global_asm;
-use core::mem::size_of;
 
-use hyperlight_common::mem::RunMode;
-
-use crate::guest_error::{set_invalid_runmode_error, set_stack_allocate_error};
-use crate::{MIN_STACK_ADDRESS, RUNNING_MODE};
+use crate::guest_error::set_stack_allocate_error;
+use crate::MIN_STACK_ADDRESS;
 
 extern "win64" {
     fn __chkstk();
@@ -35,21 +32,6 @@ global_asm!(
         push r10
         push r11
 
-        /* Load run_mode into r10 */
-        mov r10, qword ptr [rip+{run_mode}]
-
-        cmp r10, 0
-        je handle_none
-        cmp r10, 1
-        je handle_hypervisor
-        cmp r10, 2
-        je handle_inproc_windows
-        cmp r10, 3
-        je handle_inproc_linux
-        /* run_mode > 3 (invalid), so treat like handle_none */
-        jmp handle_invalid
-
-    handle_hypervisor:
         /* Load the minimum stack address from the PEB */
         mov r11, [rip+{min_stack_addr}]  
 
@@ -72,49 +54,11 @@ global_asm!(
         call {set_error}
         hlt
     
-    handle_inproc_windows:
-        /* Get the current stack pointer */
-        lea r10, [rsp + 0x18]
-
-        /* Calculate what the new stack pointer will be */
-        sub r10, rax
-        cmovb r10, r11
-        mov r11, qword ptr gs:[0x0000000000000010]
-        cmp r10, r11
-        jae cs_ret
-        and r10w, 0x0F000
-    csip_stackprobe:
-        lea r11, [r11 + 0x0FFFFFFFFFFFFF000]
-        mov byte ptr [r11], 0
-        cmp r10, r11
-        jne csip_stackprobe
     cs_ret:
         /* Restore RAX, R11 */
         pop r11
         pop r10
-        ret
-    handle_inproc_linux:
-        /* no-op */
-        jmp cs_ret
-    handle_none:
-        /* no-op. This can entrypoint has a large stack allocation
-            before RunMode variable is set */
-        jmp cs_ret
-    handle_invalid:
-        call {invalid_runmode}",
-    run_mode = sym RUNNING_MODE,
+        ret",
     min_stack_addr = sym MIN_STACK_ADDRESS,
     set_error = sym set_stack_allocate_error,
-    invalid_runmode = sym set_invalid_runmode_error
 );
-
-// Assumptions made in implementation above. If these are no longer true, compilation will fail
-// and the developer will need to update the assembly code.
-const _: () = {
-    assert!(size_of::<RunMode>() == size_of::<u64>());
-    assert!(RunMode::None as u64 == 0);
-    assert!(RunMode::Hypervisor as u64 == 1);
-    assert!(RunMode::InProcessWindows as u64 == 2);
-    assert!(RunMode::InProcessLinux as u64 == 3);
-    assert!(RunMode::Invalid as u64 == 4);
-};