Remove mshv2 feature from Hyperlight

- This removes CI builds and tests for mshv2 feature
- Removes mshv2 special handling of VMs

Signed-off-by: Doru Blânzeanu <[email protected]>

Author: dblnz

.github/workflows/Benchmarks.yml

@@ -18,7 +18,7 @@ jobs:
     strategy:
       fail-fast: true
       matrix:
-        hypervisor: [hyperv, 'hyperv-ws2025', mshv, mshv3, kvm] # hyperv,hyperv-ws2025 are windows, mshv, mshv3 and kvm are linux
+        hypervisor: [hyperv, 'hyperv-ws2025', mshv3, kvm] # hyperv,hyperv-ws2025 are windows, mshv3 and kvm are linux
         cpu: [amd, intel]
         config: [release]
     runs-on: ${{ fromJson(
@@ -66,7 +66,7 @@ jobs:
           GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
 
       - name: Run Benchmarks
-        run: just bench-ci main ${{ matrix.hypervisor == 'mshv' && 'mshv2' || ''}}
+        run: just bench-ci main
 
       - uses: actions/upload-artifact@v4
         with:

.github/workflows/CreateRelease.yml

@@ -146,9 +146,6 @@ jobs:
           # kvm
           tar -zcvf benchmarks_Linux_kvm_amd.tar.gz benchmarks_Linux_kvm_amd
           tar -zcvf benchmarks_Linux_kvm_intel.tar.gz benchmarks_Linux_kvm_intel
-          # mshv2
-          tar -zcvf benchmarks_Linux_mshv_intel.tar.gz benchmarks_Linux_mshv_intel
-          tar -zcvf benchmarks_Linux_mshv_amd.tar.gz benchmarks_Linux_mshv_amd
           # mshv3
           tar -zcvf benchmarks_Linux_mshv3_amd.tar.gz benchmarks_Linux_mshv3_amd
           tar -zcvf benchmarks_Linux_mshv3_intel.tar.gz benchmarks_Linux_mshv3_intel
@@ -173,8 +170,6 @@ jobs:
             benchmarks_Windows_hyperv_intel.tar.gz \
             benchmarks_Linux_kvm_amd.tar.gz \
             benchmarks_Linux_kvm_intel.tar.gz \
-            benchmarks_Linux_mshv_intel.tar.gz \
-            benchmarks_Linux_mshv_amd.tar.gz \
             benchmarks_Linux_mshv3_amd.tar.gz \
             benchmarks_Linux_mshv3_intel.tar.gz \
             hyperlight-guest-c-api-linux.tar.gz \
@@ -191,8 +186,6 @@ jobs:
             benchmarks_Windows_hyperv_intel.tar.gz \
             benchmarks_Linux_kvm_amd.tar.gz \
             benchmarks_Linux_kvm_intel.tar.gz \
-            benchmarks_Linux_mshv_intel.tar.gz \
-            benchmarks_Linux_mshv_amd.tar.gz \
             benchmarks_Linux_mshv3_amd.tar.gz \
             benchmarks_Linux_mshv3_intel.tar.gz \
             hyperlight-guest-c-api-linux.tar.gz \

.github/workflows/ValidatePullRequest.yml

@@ -47,7 +47,7 @@ jobs:
     strategy:
       fail-fast: true
       matrix:
-        hypervisor: [hyperv, 'hyperv-ws2025', mshv, mshv3, kvm]
+        hypervisor: [hyperv, 'hyperv-ws2025', mshv3, kvm]
         cpu: [amd, intel]
         config: [debug, release]
     uses: ./.github/workflows/dep_rust.yml

.github/workflows/dep_rust.yml

@@ -132,7 +132,7 @@ jobs:
           # use these commands in favor of build-and-move-c-guests to avoid building both configs
           just build-c-guests ${{ inputs.config }}
           just move-c-guests ${{ inputs.config }}
-          
+
       - name: Build
         run: just build ${{ inputs.config }}
         env: 
@@ -144,10 +144,10 @@ jobs:
           TARGET_TRIPLE: ${{ inputs.target_triple }}
         run: |
           # with default features
-          just test ${{ inputs.config }} ${{ inputs.hypervisor == 'mshv' && 'mshv2' || '""'}}
+          just test ${{ inputs.config }}
 
           # with only one driver enabled (driver mshv/kvm feature is ignored on windows)
-          just test ${{ inputs.config }} ${{ inputs.hypervisor == 'mshv' && 'mshv2' || inputs.hypervisor == 'mshv3' && 'mshv3' || 'kvm' }}
+          just test ${{ inputs.config }} ${{ inputs.hypervisor == 'mshv3' && 'mshv3' || 'kvm' }}
 
           # make sure certain cargo features compile
           just check
@@ -171,29 +171,29 @@ jobs:
           CARGO_TERM_COLOR: always
           RUST_LOG: debug
           TARGET_TRIPLE: ${{ inputs.target_triple }}
-        run: just run-rust-examples-linux ${{ inputs.config }} ${{ inputs.hypervisor == 'mshv' && 'mshv2' || '""'}}
+        run: just run-rust-examples-linux ${{ inputs.config }}
 
       - name: Run Rust Gdb tests
         env:
           CARGO_TERM_COLOR: always
           RUST_LOG: debug
           TARGET_TRIPLE: ${{ inputs.target_triple }}
-        run: just test-rust-gdb-debugging ${{ inputs.config }} ${{ inputs.hypervisor == 'mshv' && 'mshv2' || '""'}}
+        run: just test-rust-gdb-debugging ${{ inputs.config }}
 
       - name: Run Rust Crashdump tests
         env:
           CARGO_TERM_COLOR: always
           RUST_LOG: debug
           TARGET_TRIPLE: ${{ inputs.target_triple }}
-        run: just test-rust-crashdump ${{ inputs.config }} ${{ inputs.hypervisor == 'mshv' && 'mshv2' || '""'}}
+        run: just test-rust-crashdump ${{ inputs.config }}
 
       - name: Run Rust Tracing tests - linux
         if: runner.os == 'Linux'
         env:
           CARGO_TERM_COLOR: always
           RUST_LOG: debug
           TARGET_TRIPLE: ${{ inputs.target_triple }}
-        run: just test-rust-tracing ${{ inputs.config }} ${{ inputs.hypervisor == 'mshv' && 'mshv2' || '""'}}
+        run: just test-rust-tracing ${{ inputs.config }}
 
       - name: Download benchmarks from "latest"
         run: just bench-download ${{ runner.os }} ${{ inputs.hypervisor }} ${{ inputs.cpu}} dev-latest # compare to prerelease
@@ -204,5 +204,5 @@ jobs:
 
       - name: Run benchmarks
         run: |
-          just bench-ci main ${{ inputs.hypervisor == 'mshv' && 'mshv2' || ''}}
+          just bench-ci main
         if: ${{ inputs.config == 'release' &&  inputs.target_triple == '' }}

Cargo.lock

@@ -78,10 +78,10 @@ clean-rust:
 # convenience recipe to run all tests with the given target and features (similar to CI)
 test-like-ci config=default-target hypervisor="kvm":
     @# with default features
-    just test {{config}} {{ if hypervisor == "mshv" {"mshv2"} else {""} }}
+    just test {{config}}
 
     @# with only one driver enabled + build-metadata + init-paging
-    just test {{config}} build-metadata,init-paging,{{ if hypervisor == "mshv" {"mshv2"} else if hypervisor == "mshv3" {"mshv3"} else {"kvm"} }}
+    just test {{config}} build-metadata,init-paging,{{ if hypervisor == "mshv3" {"mshv3"} else {"kvm"} }}
 
     @# make sure certain cargo features compile
     just check
@@ -93,7 +93,7 @@ test-like-ci config=default-target hypervisor="kvm":
     just test-rust-crashdump {{config}}
 
     @# test the tracing related features
-    {{ if os() == "linux" { "just test-rust-tracing " + config + " " + if hypervisor == "mshv" { "mshv2" } else if hypervisor == "mshv3" { "mshv3" } else { "kvm" } } else { "" } }}
+    {{ if os() == "linux" { "just test-rust-tracing " + config + " " + if hypervisor == "mshv3" { "mshv3" } else { "kvm" } } else { "" } }}
 
 like-ci config=default-target hypervisor="kvm":
     @# Ensure up-to-date Cargo.lock
@@ -130,19 +130,19 @@ like-ci config=default-target hypervisor="kvm":
     {{ if os() == "windows" { "just run-rust-examples " + config } else { "" } }}
 
     @# Run Rust examples - linux
-    {{ if os() == "linux" { "just run-rust-examples-linux " + config + " " + if hypervisor == "mshv" { "mshv2" } else if hypervisor == "mshv3" { "mshv3" } else { "kvm" } } else { "" } }}
+    {{ if os() == "linux" { "just run-rust-examples-linux " + config + " " + if hypervisor == "mshv3" { "mshv3" } else { "kvm" } } else { "" } }}
 
     @# Run Rust Gdb tests
-    just test-rust-gdb-debugging {{ config }} {{ if hypervisor == "mshv" { "mshv2" } else if hypervisor == "mshv3" { "mshv3" } else { "kvm" } }} 
+    just test-rust-gdb-debugging {{ config }} {{ if hypervisor == "mshv3" { "mshv3" } else { "kvm" } }} 
 
     @# Run Rust Crashdump tests
-    just test-rust-crashdump {{config}} {{ if hypervisor == "mshv" { "mshv2" } else if hypervisor == "mshv3" { "mshv3" } else { "kvm" } }}
+    just test-rust-crashdump {{config}} {{ if hypervisor == "mshv3" { "mshv3" } else { "kvm" } }}
 
     @# Run Rust Tracing tests - linux
-    {{ if os() == "linux" { "just test-rust-tracing " + config + " " + if hypervisor == "mshv" { "mshv2" } else if hypervisor == "mshv3" { "mshv3" } else { "kvm" } } else { "" } }}
+    {{ if os() == "linux" { "just test-rust-tracing " + config + " " + if hypervisor == "mshv3" { "mshv3" } else { "kvm" } } else { "" } }}
 
     @# Run benchmarks
-    {{ if config == "release" { "just bench-ci main " + if hypervisor == "mshv" { "mshv2" } else if hypervisor == "mshv3" { "mshv3" } else { "kvm" } } else { "" } }}
+    {{ if config == "release" { "just bench-ci main " + if hypervisor == "mshv3" { "mshv3" } else { "kvm" } } else { "" } }}
 
 # runs all tests
 test target=default-target features="": (test-unit target features) (test-isolated target features) (test-integration "rust" target features) (test-integration "c" target features) (test-doc target features)
@@ -178,7 +178,6 @@ test-compilation-no-default-features target=default-target:
     {{ if os() == "windows" { cargo-cmd + " check -p hyperlight-host --no-default-features" } else { "" } }}
     @# Linux should succeed with a hypervisor driver but without init-paging
     {{ if os() == "linux" { cargo-cmd + " check -p hyperlight-host --no-default-features --features kvm" } else { "" } }}  {{ target-triple-flag }}
-    {{ if os() == "linux" { cargo-cmd + " check -p hyperlight-host --no-default-features --features mshv2" } else { "" } }}  {{ target-triple-flag }}
     {{ if os() == "linux" { cargo-cmd + " check -p hyperlight-host --no-default-features --features mshv3" } else { "" } }}  {{ target-triple-flag }}
 
 # runs tests that exercise gdb debugging

Justfile

@@ -77,8 +77,6 @@ lazy_static = "1.4.0"
 [target.'cfg(unix)'.dependencies]
 kvm-bindings = { version = "0.14", features = ["fam-wrappers"], optional = true }
 kvm-ioctls = { version = "0.24", optional = true }
-mshv-bindings2 = { package="mshv-bindings", version = "=0.2.1", optional = true }
-mshv-ioctls2 = { package="mshv-ioctls",  version = "=0.2.1", optional = true}
 mshv-bindings3 = { package="mshv-bindings", version = "0.6.1", optional = true }
 mshv-ioctls3 = { package="mshv-ioctls",  version = "0.6.1", optional = true}
 
@@ -135,8 +133,6 @@ crashdump = ["dep:chrono"]
 trace_guest = ["dep:opentelemetry", "dep:tracing-opentelemetry", "dep:hyperlight-guest-tracing", "hyperlight-common/trace_guest"]
 mem_profile = [ "trace_guest", "dep:framehop", "dep:fallible-iterator", "hyperlight-common/mem_profile" ]
 kvm = ["dep:kvm-bindings", "dep:kvm-ioctls"]
-# This feature is deprecated in favor of mshv3
-mshv2 = ["dep:mshv-bindings2", "dep:mshv-ioctls2"]
 mshv3 = ["dep:mshv-bindings3", "dep:mshv-ioctls3"]
 # This enables easy debug in the guest
 gdb = ["dep:gdbstub", "dep:gdbstub_arch"]

src/hyperlight_host/Cargo.toml

@@ -86,21 +86,17 @@ fn main() -> Result<()> {
     }
 
     // Makes #[cfg(kvm)] == #[cfg(all(feature = "kvm", target_os = "linux"))]
-    // and #[cfg(mshv)] == #[cfg(all(any(feature = "mshv2", feature = "mshv3"), target_os = "linux"))].
+    // and #[cfg(mshv)] == #[cfg(all(feature = "mshv3", target_os = "linux"))].
     // Essentially the kvm and mshv features are ignored on windows as long as you use #[cfg(kvm)] and not #[cfg(feature = "kvm")].
     // You should never use #[cfg(feature = "kvm")] or #[cfg(feature = "mshv")] in the codebase.
     cfg_aliases::cfg_aliases! {
         gdb: { all(feature = "gdb", debug_assertions) },
         kvm: { all(feature = "kvm", target_os = "linux") },
-        mshv: { all(any(feature = "mshv2", feature = "mshv3"), target_os = "linux") },
+        mshv: { all(feature = "mshv3", target_os = "linux") },
         crashdump: { all(feature = "crashdump") },
         // print_debug feature is aliased with debug_assertions to make it only available in debug-builds.
         print_debug: { all(feature = "print_debug", debug_assertions) },
-        // the following features are mutually exclusive but rather than enforcing that here we are enabling mshv2 to override mshv3 when both are enabled
-        // because mshv3 is in the default feature set we want to allow users to enable mshv2 without having to set --no-default-features and the re-enable
-        // the other features they want.
-        mshv2: { all(feature = "mshv2", target_os = "linux") },
-        mshv3: { all(feature = "mshv3", not(feature="mshv2"), target_os = "linux") },
+        mshv3: { all(feature = "mshv3", target_os = "linux") },
     }
 
     #[cfg(feature = "build-metadata")]

src/hyperlight_host/build.rs

@@ -136,9 +136,6 @@ mod tests {
 
         write_cmds_file(&cmd_file_path, cmd).expect("Failed to write gdb commands to file");
 
-        #[cfg(mshv2)] // mshv3 is a default feature is mutually exclusive with the mshv2 feature
-        let features = "gdb,mshv2";
-        #[cfg(not(mshv2))]
         let features = "gdb";
 
         // build it before running to avoid a race condition below

src/hyperlight_host/examples/guest-debugging/main.rs

@@ -14,9 +14,6 @@ See the License for the specific language governing permissions and
 limitations under the License.
 */
 
-#[cfg(mshv2)]
-extern crate mshv_ioctls2 as mshv_ioctls;
-
 #[cfg(mshv3)]
 extern crate mshv_ioctls3 as mshv_ioctls;