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

Add a workaround to Linux 6.1's sev-guest lockout #41

Merged
merged 1 commit into from
May 2, 2023

Conversation

deeglaze
Copy link
Collaborator

The IV reuse fix 47894e0fa6a5 works against the host's throttling by making the sev-guest device useless when throttled. The x86/urgent fix 72f7754dcf31 that merged to tip on March 16 still hasn't made it to some distros, so this workaround is still needed temporarily.

Issue #40 is to revert this change at the appropriate time. The same fix for #40 includes the fix for #5.

The IV reuse fix 47894e0fa6a5 works against the host's throttling by
making the sev-guest device useless when throttled. The x86/urgent fix
72f7754dcf31 that merged to tip on March 16 still hasn't made it to
some distros, so this workaround is still needed temporarily.

Issue google#40 is to revert this change at the appropriate time. The same
fix for google#40 includes the fix for google#5.

Signed-off-by: Dionna Glaze <[email protected]>
@deeglaze
Copy link
Collaborator Author

deeglaze commented May 2, 2023

SLES-15 still doesn't have the required patches, so we should merge this for a better user experience.

const (
// defaultSevGuestDevicePath is the platform's usual device path to the SEV guest.
defaultSevGuestDevicePath = "/dev/sev-guest"
throttleDuration = 2 * time.Second
Copy link
Collaborator

Choose a reason for hiding this comment

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

Maybe a comment on how the 2 second throttleDuration was picked?

@deeglaze deeglaze merged commit 8ac7f5f into google:main May 2, 2023
@deeglaze deeglaze deleted the lk61 branch February 8, 2024 16:49
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