Replace log with tracing in guest code and update Cargo.toml features

Co-authored-by: dblnz <13020154+dblnz@users.noreply.github.com>

Author: Copilot

Cargo.lock

@@ -17,7 +17,7 @@ workspace = true
 [dependencies]
 flatbuffers = { version = "25.9.23", default-features = false }
 anyhow = { version = "1.0.100", default-features = false }
-tracing = { version = "0.1.41" }
+tracing = { version = "0.1.41", default-features = false, features = [ "attributes", "log" ] }
 arbitrary = {version = "1.4.2", optional = true, features = ["derive"]}
 spin = "0.10.0"
 

src/hyperlight_common/Cargo.toml

@@ -110,3 +110,15 @@ impl From<Level> for LogLevel {
         }
     }
 }
+
+impl From<tracing::log::Level> for LogLevel {
+    fn from(val: tracing::log::Level) -> LogLevel {
+        match val {
+            tracing::log::Level::Trace => LogLevel::Trace,
+            tracing::log::Level::Debug => LogLevel::Debug,
+            tracing::log::Level::Info => LogLevel::Information,
+            tracing::log::Level::Warn => LogLevel::Warning,
+            tracing::log::Level::Error => LogLevel::Error,
+        }
+    }
+}

src/hyperlight_common/src/flatbuffer_wrappers/guest_log_level.rs

@@ -25,10 +25,9 @@ hyperlight-guest = { workspace = true, default-features = false }
 hyperlight-common = { workspace = true, default-features = false }
 hyperlight-guest-tracing = { workspace = true, default-features = false }
 buddy_system_allocator = "0.11.0"
-log = { version = "0.4", default-features = false }
 spin = "0.10.0"
 flatbuffers = { version = "25.2.10", default-features = false }
-tracing = { version = "0.1.41", default-features = false, features = ["attributes"] }
+tracing = { version = "0.1.41", default-features = false, features = ["attributes", "log"] }
 
 [lints]
 workspace = true

src/hyperlight_guest_bin/Cargo.toml

@@ -90,7 +90,7 @@ fn internal_dispatch_function() {
     let handle = unsafe { GUEST_HANDLE };
 
     #[cfg(debug_assertions)]
-    log::trace!("internal_dispatch_function");
+    tracing::trace!("internal_dispatch_function");
 
     let function_call = handle
         .try_pop_shared_input_data_into::<FunctionCall>()

src/hyperlight_guest_bin/src/guest_function/call.rs

@@ -17,7 +17,7 @@ limitations under the License.
 use alloc::format;
 
 use hyperlight_common::flatbuffer_wrappers::guest_log_level::LogLevel;
-use log::{LevelFilter, Metadata, Record};
+use tracing::log::{LevelFilter, Metadata, Record};
 
 use crate::GUEST_HANDLE;
 
@@ -27,11 +27,11 @@ struct GuestLogger {}
 pub(crate) fn init_logger(level: LevelFilter) {
     // if this `expect` fails we have no way to recover anyway, so we actually prefer a panic here
     // below temporary guest logger is promoted to static by the compiler.
-    log::set_logger(&GuestLogger {}).expect("unable to setup guest logger");
-    log::set_max_level(level);
+    tracing::log::set_logger(&GuestLogger {}).expect("unable to setup guest logger");
+    tracing::log::set_max_level(level);
 }
 
-impl log::Log for GuestLogger {
+impl tracing::log::Log for GuestLogger {
     // The various macros like `info!` and `error!` will call the global log::max_level()
     // before calling our `log`. This means that we should log every message we get, because
     // we won't even see the ones that are above the set max level.

src/hyperlight_guest_bin/src/guest_logger.rs

@@ -32,8 +32,8 @@ use hyperlight_common::mem::HyperlightPEB;
 use hyperlight_common::outb::OutBAction;
 use hyperlight_guest::exit::{halt, write_abort};
 use hyperlight_guest::guest_handle::handle::GuestHandle;
-use log::LevelFilter;
 use spin::Once;
+use tracing::log::LevelFilter;
 
 // === Modules ===
 #[cfg(target_arch = "x86_64")]

src/hyperlight_guest_bin/src/lib.rs

@@ -17,7 +17,7 @@ hyperlight-guest-bin = { workspace = true, default-features = true }
 hyperlight-common = { workspace = true, default-features = false }
 
 flatbuffers = { version = "25.2.10", default-features = false }
-log = { version = "0.4", default-features = false }
+tracing = { version = "0.1.41", default-features = false, features = ["log"] }
 
 [build-dependencies]
 cbindgen = "0.29.2"

src/hyperlight_guest_capi/Cargo.toml

@@ -16,21 +16,41 @@ limitations under the License.
 
 use core::ffi::c_char;
 
+/// C-compatible log level enum
+#[repr(C)]
+#[derive(Debug, Copy, Clone, PartialEq, Eq)]
+pub enum Level {
+    Error = 1,
+    Warn = 2,
+    Info = 3,
+    Debug = 4,
+    Trace = 5,
+}
+
+impl From<Level> for tracing::log::Level {
+    fn from(level: Level) -> Self {
+        match level {
+            Level::Error => tracing::log::Level::Error,
+            Level::Warn => tracing::log::Level::Warn,
+            Level::Info => tracing::log::Level::Info,
+            Level::Debug => tracing::log::Level::Debug,
+            Level::Trace => tracing::log::Level::Trace,
+        }
+    }
+}
+
 #[unsafe(no_mangle)]
-pub extern "C" fn hl_log(
-    level: log::Level,
-    message: *const c_char,
-    line: i32,
-    file: *const c_char,
-) {
-    if log::log_enabled!(level) {
+pub extern "C" fn hl_log(level: Level, message: *const c_char, line: i32, file: *const c_char) {
+    let log_level: tracing::log::Level = level.into();
+
+    if tracing::log::log_enabled!(log_level) {
         let message = unsafe { core::ffi::CStr::from_ptr(message).to_string_lossy() };
         let file = unsafe { core::ffi::CStr::from_ptr(file).to_string_lossy() };
 
-        log::logger().log(
-            &log::RecordBuilder::new()
-                .args(format_args!("{}: {}", level, message))
-                .level(level)
+        tracing::log::logger().log(
+            &tracing::log::RecordBuilder::new()
+                .args(format_args!("{}: {}", log_level, message))
+                .level(log_level)
                 .line(Some(line as u32))
                 .file(Some(&file))
                 .build(),