Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Running uhyve in a Linux VirtualBox VM (with nested virtualization enabled) panics in detect_freq_from_cpuid #537

Closed
ColinFinck opened this issue Jul 7, 2023 · 4 comments
Assignees

Comments

@ColinFinck
Copy link
Member

During the unikernel meet-up, I just tried uhyve in a Linux VirtualBox VM (with nested virtualization enabled) and I get

cfinck@kubuntuvm:~/unikernel-meetup/hands-on-session/hello-hermit$ RUST_BACKTRACE=1 uhyve -v target/x86_64-unknown-hermit/debug/hello-hermit
thread 'main' panicked at 'attempt to divide by zero', /home/cfinck/.cargo/registry/src/index.crates.io-6f17d22bba15001f/uhyve-0.2.2/src/arch/x86_64/mod.rs:47:25
stack backtrace:
   0: rust_begin_unwind
             at /rustc/90c541806f23a127002de5b4038be731ba1458ca/library/std/src/panicking.rs:578:5
   1: core::panicking::panic_fmt
             at /rustc/90c541806f23a127002de5b4038be731ba1458ca/library/core/src/panicking.rs:67:14
   2: core::panicking::panic
             at /rustc/90c541806f23a127002de5b4038be731ba1458ca/library/core/src/panicking.rs:117:5
   3: uhyvelib::arch::x86_64::detect_freq_from_cpuid
   4: uhyvelib::vm::detect_cpu_freq
   5: uhyvelib::linux::<impl uhyvelib::linux::uhyve::Uhyve>::run
   6: uhyve::run_uhyve
   7: uhyve::main
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.

CC @mkroening

@mkroening
Copy link
Member

@charlottestinson
Copy link
Contributor

Hello, this is the same team from UT! We are interested in working on this issue as well. Would that be alright?

@jounathaen
Copy link
Member

No need to ask, PRs are always welcome

@n0toose
Copy link
Member

n0toose commented Sep 23, 2024

I believe that this issue can be closed because of #688, #701.

I tested it myself just to be sure. The kernel still panics later down the line, but adding detect_cpu_freq() with the detect_freq_from_sysinfo() call removed does not immediately panic, meaning that detect_freq_from_cpuid is sound and that the case is being handled properly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants