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

kvm: set MSR_IA32_SPEC_CTRL to the host value #10805

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

Conversation

avagin
Copy link
Collaborator

@avagin avagin commented Aug 22, 2024

Otherwise, the kernel needs to save/restore the value of this msr on each vmexit/vmenter.

Here are kvm benchmark results:
goarch: amd64
cpu: 11th Gen Intel(R) Core(TM) i7-1185G7 @ 3.00GHz Before:
BenchmarkWorldSwitchToUserRoundtrip-8 1394972 4103 ns/op
After:
BenchmarkWorldSwitchToUserRoundtrip-8 1528820 3770 ns/op

@avagin avagin added platform: kvm Issue related to the kvm platform arch: x86_64 Issue related to the x86_64 achitecture area: platform Issue related to platforms (kvm, ptrace) labels Aug 22, 2024
@avagin avagin force-pushed the kvm-msr-spec-ctrl branch 2 times, most recently from 2d9f87c to f48476b Compare August 22, 2024 00:54
Otherwise, the kernel needs to save/restore the value of this msr on
each vmexit/vmenter.

Here are kvm benchmark results:
goarch: amd64
cpu: 11th Gen Intel(R) Core(TM) i7-1185G7 @ 3.00GHz
Before:
BenchmarkWorldSwitchToUserRoundtrip-8   	 1605484	      3644 ns/op
After:
BenchmarkWorldSwitchToUserRoundtrip-8   	 1812069	      3255 ns/op

Signed-off-by: Andrei Vagin <[email protected]>
copybara-service bot pushed a commit that referenced this pull request Aug 22, 2024
Otherwise, the kernel needs to save/restore the value of this msr on each vmexit/vmenter.

Here are kvm benchmark results:
goarch: amd64
cpu: 11th Gen Intel(R) Core(TM) i7-1185G7 @ 3.00GHz Before:
Before:
BenchmarkWorldSwitchToUserRoundtrip-8            1605484              3644 ns/op
After:
BenchmarkWorldSwitchToUserRoundtrip-8            1812069              3255 ns/op
FUTURE_COPYBARA_INTEGRATE_REVIEW=#10805 from avagin:kvm-msr-spec-ctrl 7e9a33a
PiperOrigin-RevId: 666429086
copybara-service bot pushed a commit that referenced this pull request Aug 22, 2024
Otherwise, the kernel needs to save/restore the value of this msr on each vmexit/vmenter.

Here are kvm benchmark results:
goarch: amd64
cpu: 11th Gen Intel(R) Core(TM) i7-1185G7 @ 3.00GHz Before:
Before:
BenchmarkWorldSwitchToUserRoundtrip-8            1605484              3644 ns/op
After:
BenchmarkWorldSwitchToUserRoundtrip-8            1812069              3255 ns/op
FUTURE_COPYBARA_INTEGRATE_REVIEW=#10805 from avagin:kvm-msr-spec-ctrl 7e9a33a
PiperOrigin-RevId: 666429086
copybara-service bot pushed a commit that referenced this pull request Aug 22, 2024
Otherwise, the kernel needs to save/restore the value of this msr on each vmexit/vmenter.

Here are kvm benchmark results:
goarch: amd64
cpu: 11th Gen Intel(R) Core(TM) i7-1185G7 @ 3.00GHz Before:
Before:
BenchmarkWorldSwitchToUserRoundtrip-8            1605484              3644 ns/op
After:
BenchmarkWorldSwitchToUserRoundtrip-8            1812069              3255 ns/op
FUTURE_COPYBARA_INTEGRATE_REVIEW=#10805 from avagin:kvm-msr-spec-ctrl 7e9a33a
PiperOrigin-RevId: 666429086
copybara-service bot pushed a commit that referenced this pull request Aug 22, 2024
Otherwise, the kernel needs to save/restore the value of this msr on each vmexit/vmenter.

Here are kvm benchmark results:
goarch: amd64
cpu: 11th Gen Intel(R) Core(TM) i7-1185G7 @ 3.00GHz Before:
Before:
BenchmarkWorldSwitchToUserRoundtrip-8            1605484              3644 ns/op
After:
BenchmarkWorldSwitchToUserRoundtrip-8            1812069              3255 ns/op
FUTURE_COPYBARA_INTEGRATE_REVIEW=#10805 from avagin:kvm-msr-spec-ctrl 7e9a33a
PiperOrigin-RevId: 666429086
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
arch: x86_64 Issue related to the x86_64 achitecture area: platform Issue related to platforms (kvm, ptrace) platform: kvm Issue related to the kvm platform
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant