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: Enable RDRAND, RDSEED for x86 TCG #1459

Merged

Conversation

skuenzer
Copy link
Member

QEMU's TCG for x86 can emulate rdrand and rdseed instructions as long as specified via -cpu parameter and QEMU 8.1.3 or newer is used. Due to recent requirements introduced with lib/ukrandom (see PR #1008, PR #1451), we request rdseed and rdrand when using TCG.

QEMU's TCG for x86 can emulate `rdrand` and `rdseed` instructions as
long as specified via `-cpu` parameter and QEMU 8.1.3 or newer is used.
Due to recent requirements introduced with `lib/ukrandom` (see PR unikraft#1008,
PR unikraft#1451), we request `rdseed` and `rdrand` when using TCG.

Signed-off-by: Simon Kuenzer <simon@unikraft.io>
@skuenzer skuenzer requested a review from michpappas June 14, 2024 22:51
@skuenzer skuenzer requested a review from a team as a code owner June 14, 2024 22:51
@github-actions github-actions bot added the area/support Support scripts, tools, services. label Jun 14, 2024
Copy link
Member

@michpappas michpappas left a comment

Choose a reason for hiding this comment

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

Looks good, thanks.

Reviewed-by: Michalis Pappas michalis@unikraft.io

@razvand razvand removed the request for review from a team June 15, 2024 07:13
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

@razvand razvand added the merge Label to trigger merge action label Jun 15, 2024
@unikraft-bot unikraft-bot changed the base branch from staging to staging-1459 June 15, 2024 07:30
@unikraft-bot unikraft-bot merged commit bb29779 into unikraft:staging-1459 Jun 15, 2024
15 checks passed
unikraft-bot pushed a commit that referenced this pull request Jun 15, 2024
QEMU's TCG for x86 can emulate `rdrand` and `rdseed` instructions as
long as specified via `-cpu` parameter and QEMU 8.1.3 or newer is used.
Due to recent requirements introduced with `lib/ukrandom` (see PR #1008,
PR #1451), we request `rdseed` and `rdrand` when using TCG.

Signed-off-by: Simon Kuenzer <simon@unikraft.io>
Reviewed-by: Michalis Pappas <michalis@unikraft.io>
Approved-by: Razvan Deaconescu <razvand@unikraft.io>
GitHub-Closes: #1459
@unikraft-bot unikraft-bot added ci/merged Merged by CI and removed merge Label to trigger merge action labels Jun 15, 2024
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
Projects
Status: Done!
Development

Successfully merging this pull request may close these issues.

None yet

4 participants