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

Change Penryn to max #312

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Change Penryn to max #312

wants to merge 1 commit into from

Conversation

sickcodes
Copy link
Owner

As stated in QEMU 6

Can test currently with:

-e CPU=max \

[user@hostname ~]$ qemu-system-x86_64 -cpu help
x86 Penryn                (alias configured by machine type)                        
x86 Penryn-v1             Intel Core 2 Duo P9xxx (Penryn Class Core 2)              
...
x86 base                  base CPU model type with no features enabled              
x86 host                  KVM processor with all supported host features            
x86 max                   Enables all features supported by the accelerator in the current host

See #304

As stated in QEMU 6 

Can test currently with:

`-e CPU=max \`

```
[user@hostname ~]$ qemu-system-x86_64 -cpu help
x86 Penryn                (alias configured by machine type)                        
x86 Penryn-v1             Intel Core 2 Duo P9xxx (Penryn Class Core 2)              
...
x86 base                  base CPU model type with no features enabled              
x86 host                  KVM processor with all supported host features            
x86 max                   Enables all features supported by the accelerator in the current host
```

See #304
@sickcodes
Copy link
Owner Author

@sickcodes
Copy link
Owner Author

    -cpu max,vendor=GenuineIntel \
    -machine q35,${KVM-"accel=kvm:tcg"} \

I am considering the following changes.

VMX doesn't show up, but apparently it should work on Intel.

I will give it a test on Digital Ocean, which lets you select AMD or Intel, unless someone wants to.

@rationalthinker1
Copy link

Hey sickcodes. Is there a reason why this isn't merged?

@sickcodes
Copy link
Owner Author

Hey sickcodes. Is there a reason why this isn't merged?

Haven't had much time to be able to test all the images with the maxed out CPU. I'll give it a shot today

@sickcodes
Copy link
Owner Author

This isn't 100% required, however if the following provides a performance boost, I'll merge it:

-e CPU=max \
-e CPUID_FLAGS='vendor=GenuineIntel' \

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