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

Windows Hypervisor support #36

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Windows Hypervisor support #36

wants to merge 2 commits into from

Conversation

JayFoxRox
Copy link
Owner

@JayFoxRox JayFoxRox commented Apr 18, 2018

This API should land in Windows 10 soon. The benefit over HAXM will be the support of non-intel CPUs (in the future.. hopefully).

I'm still open to the idea of adding HAXM for older versions of Windows (namely 7 and 8) in the future.
While this would only work on intel, it would still reach a number of users:

  • Windows 7 has Extended Support until January 14, 2020
  • Windows 8 has Extended Support until January 10, 2023

For macOS, which has a slightly different HAXM interface, we should instead use the Apple API in the future. There is no benefit in having HAXM on macOS as both, HAXM and the Apple API are only available starting with 10.10, but HAXM only works on intel, and will likely to continue to do so in the future (which could cause issues in the future). HAXM is also not allowed when shipping via the app-store.


FIXME:

  • Register functions not ported yet
  • Untested

static void nopSignalHandler() {
// We don't actually need to do anything here, but we need to interrupt
// the execution of the guest.
}
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can be removed

@JayFoxRox
Copy link
Owner Author

JayFoxRox commented May 2, 2018

Note to self: I did work on this (HAXM primarily) on my old i7-920 desktop. However, that only supports UG mode so it needs a couple of changes.

That HAXM code is in #42 and maybe also #41.

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 this pull request may close these issues.

1 participant