Removed host error region

src/hyperlight_common/src/mem/mod.rs

@@ -28,11 +28,6 @@ pub struct HostFunctionDefinitions {
     pub fbHostFunctionDetails: *mut c_void,
 }
 
-#[repr(C)]
-pub struct HostException {
-    pub hostExceptionSize: u64,
-}
-
 #[repr(C)]
 pub struct GuestErrorData {
     pub guestErrorSize: u64,
@@ -89,7 +84,6 @@ pub struct GuestPanicContextData {
 pub struct HyperlightPEB {
     pub security_cookie_seed: u64,
     pub guest_function_dispatch_ptr: u64,
-    pub hostException: HostException,
     pub guestErrorData: GuestErrorData,
     pub pCode: *mut c_char,
     pub pOutb: *mut c_void,

src/hyperlight_guest/src/host_function_call.rs

@@ -28,7 +28,6 @@ use hyperlight_common::flatbuffer_wrappers::util::get_flatbuffer_result;
 use hyperlight_common::mem::RunMode;
 
 use crate::error::{HyperlightGuestError, Result};
-use crate::host_error::check_for_host_error;
 use crate::shared_input_data::try_pop_shared_input_data_into;
 use crate::shared_output_data::push_shared_output_data;
 use crate::{OUTB_PTR, OUTB_PTR_WITH_CONTEXT, P_PEB, RUNNING_MODE};
@@ -102,8 +101,6 @@ pub fn outb(port: u16, value: u8) {
                 panic!("Tried to call outb in invalid runmode");
             }
         }
-
-        check_for_host_error();
     }
 }
 

src/hyperlight_guest/src/lib.rs

@@ -38,7 +38,6 @@ pub mod guest_function_call;
 pub mod guest_function_definition;
 pub mod guest_function_register;
 
-pub mod host_error;
 pub mod host_function_call;
 
 pub(crate) mod guest_logger;

src/hyperlight_host/Cargo.toml

@@ -30,9 +30,6 @@ flatbuffers = "25.2.10"
 page_size = "0.6.0"
 termcolor = "1.2.0"
 bitflags = "2.9.0"
-lazy_static = "1.4.0"
-serde = { version = "1.0", features = ["derive"] }
-serde_json = "1.0"
 log = "0.4.27"
 tracing = { version = "0.1.41", features = ["log"] }
 tracing-log = "0.2.0"
@@ -43,7 +40,6 @@ crossbeam = "0.8.0"
 crossbeam-channel = "0.5.15"
 thiserror = "2.0.12"
 tempfile = { version = "3.19", optional = true }
-serde_yaml = "0.9"
 anyhow = "1.0"
 metrics = "0.24.2"
 
@@ -67,6 +63,7 @@ windows-result = "0.3"
 rust-embed = { version = "8.7.0", features = ["debug-embed", "include-exclude", "interpolate-folder-path"] }
 sha256 = "1.6.0"
 windows-version = "0.1"
+lazy_static = "1.4.0"
 
 [target.'cfg(unix)'.dependencies]
 gdbstub = { version = "0.7.5", optional = true }
@@ -104,6 +101,7 @@ tracing-chrome = "0.7.2"
 metrics-util = "0.19.1"
 metrics-exporter-prometheus = "0.17.0"
 tracing-tracy = "0.11.4"
+serde_json = "1.0"
 
 [target.'cfg(windows)'.dev-dependencies]
 windows = { version = "0.61", features = [

src/hyperlight_host/src/error.rs

@@ -25,7 +25,6 @@ use std::cell::{BorrowError, BorrowMutError};
 use std::convert::Infallible;
 use std::error::Error;
 use std::num::TryFromIntError;
-use std::str::Utf8Error;
 use std::string::FromUtf8Error;
 use std::sync::{MutexGuard, PoisonError};
 use std::time::SystemTimeError;
@@ -35,21 +34,13 @@ use crossbeam_channel::{RecvError, SendError};
 use flatbuffers::InvalidFlatbuffer;
 use hyperlight_common::flatbuffer_wrappers::function_types::{ParameterValue, ReturnValue};
 use hyperlight_common::flatbuffer_wrappers::guest_error::ErrorCode;
-use serde::{Deserialize, Serialize};
-use serde_yaml;
 use thiserror::Error;
 
 #[cfg(target_os = "windows")]
 use crate::hypervisor::wrappers::HandleWrapper;
 use crate::mem::memory_region::MemoryRegionFlags;
 use crate::mem::ptr::RawPtr;
 
-#[derive(Serialize, Deserialize, Debug, PartialEq, Eq)]
-pub(crate) struct HyperlightHostError {
-    pub(crate) message: String,
-    pub(crate) source: String,
-}
-
 /// The error type for Hyperlight operations
 #[derive(Error, Debug)]
 pub enum HyperlightError {
@@ -87,14 +78,6 @@ pub enum HyperlightError {
     #[error("{0}")]
     Error(String),
 
-    /// Exception Data Length is incorrect
-    #[error("Exception Data Length is incorrect. Expected: {0}, Actual: {1}")]
-    ExceptionDataLengthIncorrect(i32, usize),
-
-    /// Exception Message is too big
-    #[error("Exception Message is too big. Max Size: {0}, Actual: {1}")]
-    ExceptionMessageTooBig(usize, usize),
-
     /// Execution violation
     #[error("Non-executable address {0:#x} tried to be executed")]
     ExecutionAccessViolation(u64),
@@ -171,10 +154,6 @@ pub enum HyperlightError {
     #[error("The flatbuffer is invalid")]
     InvalidFlatBuffer(#[from] InvalidFlatbuffer),
 
-    /// Conversion of str to Json failed
-    #[error("Conversion of str data to json failed")]
-    JsonConversionFailure(#[from] serde_json::Error),
-
     /// Error occurred in KVM Operation
     #[error("KVM Error {0:?}")]
     #[cfg(kvm)]
@@ -225,10 +204,6 @@ pub enum HyperlightError {
     #[error("Restore_state called with no valid snapshot")]
     NoMemorySnapshot,
 
-    /// An error occurred handling an outb message
-    #[error("An error occurred handling an outb message {0:?}: {1}")]
-    OutBHandlingError(String, String),
-
     /// Failed to get value from parameter value
     #[error("Failed To Convert Parameter Value {0:?} to {1:?}")]
     ParameterValueConversionFailure(ParameterValue, &'static str),
@@ -292,10 +267,6 @@ pub enum HyperlightError {
     #[error("The return value type is unexpected got {0:?} expected {1:?}")]
     UnexpectedReturnValueType(ReturnValue, String),
 
-    /// Slice conversion to UTF8 failed
-    #[error("Slice Conversion of UTF8 data to str failed")]
-    UTF8SliceConversionFailure(#[from] Utf8Error),
-
     /// Slice conversion to UTF8 failed
     #[error("String Conversion of UTF8 data to str failed")]
     UTF8StringConversionFailure(#[from] FromUtf8Error),
@@ -315,10 +286,6 @@ pub enum HyperlightError {
     #[cfg(target_os = "windows")]
     #[error("Windows API Error Result {0:?}")]
     WindowsAPIError(#[from] windows_result::Error),
-
-    /// Conversion of str to YAML failed
-    #[error("Conversion of str data to yaml failed")]
-    YamlConversionFailure(#[from] serde_yaml::Error),
 }
 
 impl From<Infallible> for HyperlightError {

src/hyperlight_host/src/func/guest_err.rs

@@ -16,7 +16,7 @@ limitations under the License.
 
 use hyperlight_common::flatbuffer_wrappers::guest_error::ErrorCode;
 
-use crate::error::HyperlightError::{GuestError, OutBHandlingError, StackOverflow};
+use crate::error::HyperlightError::{GuestError, StackOverflow};
 use crate::mem::shared_mem::HostSharedMemory;
 use crate::metrics::{METRIC_GUEST_ERROR, METRIC_GUEST_ERROR_LABEL_CODE};
 use crate::sandbox::mem_mgr::MemMgrWrapper;
@@ -27,18 +27,6 @@ pub(crate) fn check_for_guest_error(mgr: &MemMgrWrapper<HostSharedMemory>) -> Re
     let guest_err = mgr.as_ref().get_guest_error()?;
     match guest_err.code {
         ErrorCode::NoError => Ok(()),
-        ErrorCode::OutbError => match mgr.as_ref().get_host_error()? {
-            Some(host_err) => {
-                metrics::counter!(METRIC_GUEST_ERROR, METRIC_GUEST_ERROR_LABEL_CODE => (guest_err.code as u64).to_string()).increment(1);
-
-                log_then_return!(OutBHandlingError(
-                    host_err.source.clone(),
-                    guest_err.message.clone()
-                ));
-            }
-            // TODO: Not sure this is correct behavior. We should probably return error here
-            None => Ok(()),
-        },
         ErrorCode::StackOverflow => {
             metrics::counter!(METRIC_GUEST_ERROR, METRIC_GUEST_ERROR_LABEL_CODE => (guest_err.code as u64).to_string()).increment(1);
             log_then_return!(StackOverflow());