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

Enable ioapic #901

Closed
wants to merge 1 commit into from
Closed

Enable ioapic #901

wants to merge 1 commit into from

Conversation

jdevalk
Copy link

@jdevalk jdevalk commented Oct 4, 2017

Fixes #900

@jdevalk
Copy link
Author

jdevalk commented Oct 4, 2017

Of course this should be tested for breakage, we've tested this on a couple machines but they're all 64 bit macs :)

@swalkinshaw
Copy link
Member

Never heard of this setting before 🤔

VB docs specially say:

On any host, you should enable the I/O APIC for virtual machines that you intend to use in 64-bit mode. This is especially true for 64-bit Windows VMs. See Section 3.3.2, “"Advanced" tab”. In addition, for 64-bit Windows guests, you should make sure that the VM uses the Intel networking device, since there is no 64-bit driver support for the AMD PCNet card; see Section 6.1, “Virtual networking hardware”.

Considering we only support Ubuntu 64bit guests... seems like a good default. Would be nice for a few other people to test this as well.

@swalkinshaw
Copy link
Member

Gave this a quick test and everything seemed fine. Would still like some other people to test. Especially on Windows. @kalenjohnson @QWp6t hint hint

@swalkinshaw
Copy link
Member

swalkinshaw commented Nov 3, 2017

Thanks for this PR. Replacing it with #913 which adds a few more config options too.

@swalkinshaw swalkinshaw closed this Nov 3, 2017
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.

2 participants