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

support/qemu-guest: Workaround for TCG/x86 and Unikraft paging #1041

Conversation

skuenzer
Copy link
Member

Limits QEMU PC machine model up to version 7.0 (pc-i440fx-7.0) for x86. This is primarily done to keep Unikraft compatibility with TCG.

Related GitHub issue: #1040

@razvand razvand requested review from StefanJum and removed request for dragosargint August 14, 2023 19:31
@razvand razvand assigned razvand and unassigned nderjung Aug 14, 2023
@razvand razvand added kind/quick-fix Issue is a quick fix area/support Support scripts, tools, services. labels Aug 14, 2023
@razvand razvand added this to the v0.14.0 (Prometheus) milestone Aug 14, 2023
Copy link
Member

@StefanJum StefanJum left a comment

Choose a reason for hiding this comment

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

All good, thanks.
Reviewed-by: Stefan Jumarea stefanjumarea02@gmail.com

Limits QEMU PC machine model up to version 7.0 (pc-i440fx-7.0) for
x86. This is primarily done to keep Unikraft compatibility with TCG.

GitHub-Issue: unikraft#1040
Signed-off-by: Simon Kuenzer <simon@unikraft.io>
@skuenzer skuenzer force-pushed the skuenzer/workaround-qemu-guest-tcg7.0 branch from 9d43167 to 45e0cee Compare August 14, 2023 20:04
@skuenzer
Copy link
Member Author

I pushed the updated.

@unikraft-bot
Copy link
Member

Checkpatch passed

Beep boop! I ran Unikraft's checkpatch.pl support script on your pull request and it all looks good!

SHA commit checkpatch
45e0cee support/qemu-guest: Workaround for TCG/x86 and Unikraft paging

@razvand razvand requested a review from nderjung August 15, 2023 19:49
Copy link
Contributor

@razvand razvand left a comment

Choose a reason for hiding this comment

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

Approved-by: Razvan Deaconescu razvand@unikraft.io

@unikraft-bot unikraft-bot added the ci/merged Merged by CI label Aug 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/support Support scripts, tools, services. ci/merged Merged by CI kind/quick-fix Issue is a quick fix
Projects
Status: Done!
Development

Successfully merging this pull request may close these issues.

None yet

5 participants