Skip to content

Commit

Permalink
Merge pull request #19518 from arixmkii/qemu_win_settings_qemu64
Browse files Browse the repository at this point in the history
Change default QEMU CPU level to `qemu64` on Windows amd64
  • Loading branch information
openshift-merge-bot[bot] authored Dec 4, 2023
2 parents 093868b + 4c249a2 commit cbb3e4d
Showing 1 changed file with 3 additions and 5 deletions.
8 changes: 3 additions & 5 deletions pkg/machine/qemu/options_windows_amd64.go
Original file line number Diff line number Diff line change
Expand Up @@ -5,11 +5,9 @@ var (
)

func (v *MachineVM) addArchOptions(_ *setNewMachineCMDOpts) []string {
// "max" level is used, because "host" is not supported with "whpx" acceleration
// "vmx=off" disabled nested virtualization (not needed for podman)
// QEMU issue to track nested virtualization: https://gitlab.com/qemu-project/qemu/-/issues/628
// "monitor=off" needed to support hosts, which have mwait calls disabled in BIOS/UEFI
opts := []string{"-machine", "q35,accel=whpx:tcg", "-cpu", "max,vmx=off,monitor=off"}
// "qemu64" level is used, because "host" is not supported with "whpx" acceleration.
// It is a stable choice for running on bare metal and inside Hyper-V machine with nested virtualization.
opts := []string{"-machine", "q35,accel=whpx:tcg", "-cpu", "qemu64"}
return opts
}

Expand Down

0 comments on commit cbb3e4d

Please sign in to comment.