hypervisor detection

[wenzel]

hypervisor detection

[rageagainsthepc]

rust-lang/cargo#1197

:(

[Wenzel]

We should rename this feature as hypervisor test, and test whether a given domain name is available on for a specific hypervisor.

Implementing the same logic that LibVMI's driver_init_mode function does.

Overview:

• xen_test
  • xen_init
    • xen_get_version
      • reading /sys/hypervisor/major/minor
    • init libxc_wrapper
    • init libxs_wrapper
  • test converting domain name to domain id
    • via xenstore directories
• kvm_test
  • kvm_init
    • init libvirt_wrapper
    • connect to qemu:///system

[Wenzel]

Thanks for finding this.
At least this should be available in nightly already, according to:

Target-specific features for dependencies has been implemented and is available as a nightly-only feature on the latest nightly 2020-02-23. See the tracking issue at #7914.

In the meantime, we can assume linux by default.

[Wenzel]

I was looking at adding a new function in the Introspectable trait:

fn detect() -> Result<bool,Box<Error>> where Self: Sized {

And then you could loop over the DriverType enum, and pick the only hypervisor that will return true

[Wenzel]

Next here would be to enable a set of features by default.
Of course, we have to take the platform into account, Linux or Windows.

I tried to play with Cargo conditional compilation, but I can't get this thing working as I expect:

Im getting

warning: unused manifest key: target.cfg(target_os="linux").features
warning: unused manifest key: target.cfg(target_os="windows").features
    Finished dev [unoptimized + debuginfo] target(s) in 0.02s

I saw the syntax in a SO post:
https://stackoverflow.com/questions/39709542/why-does-the-last-platform-specific-dependency-take-precedence-in-cargo

of course, if this isn't supported for the features section, we can ditch the hyper-v driver and set the default linux drivers