Page MenuHomeFreeBSD
Differential D29665

bhyve: implement rdmsr for MSR_IA32_FEATURE_CONTROL
ClosedPublic
Actions

Authored by corvink on Apr 9 2021, 5:42 AM.
Tags
Referenced Files
Unknown Object (File)
Sat, Jan 25, 7:41 PM
Unknown Object (File)
Sat, Jan 18, 10:13 PM
Unknown Object (File)
Sat, Jan 18, 10:06 PM
Unknown Object (File)
Sat, Jan 18, 5:33 PM
Unknown Object (File)
Mon, Dec 30, 1:04 PM
Unknown Object (File)
Dec 4 2024, 1:15 PM
Unknown Object (File)
Nov 28 2024, 5:29 PM
Unknown Object (File)
Nov 28 2024, 5:29 PM
View All Files
Subscribers
bcran
grehan
jhb
jlduran
rgrimes

Details

Reviewers
grehan
jhb
Group Reviewers
bhyve
Commits
rG17d214c2ca9f: bhyve: implement rdmsr for MSR_IA32_FEATURE_CONTROL
Summary

Windows guest crashes on boot. It's caused by rdmsr to MSR_IA32_FEATURE_CONTROL. Windows checks this MSR to determine enabled VMX features. Bhyve hasn't implement it yet. For that reason, bhyve injects a GP which causes Windows to crash.
This patch returns MSR_IA32_FEATURE_CONTROL with Lock Bit set and VMX disabled to tell Windows that VMX isn't available.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

corvink created this revision.Apr 9 2021, 5:42 AM
Herald added subscribers: bcran, rgrimes. · View Herald TranscriptApr 9 2021, 5:42 AM
corvink requested review of this revision.Apr 9 2021, 5:42 AM
corvink edited the summary of this revision. (Show Details)
corvink retitled this revision from bhyve: fix crash of rdmsr from windows guests to bhyve: implement rdmsr for MSR_IA32_FEATURE_CONTROL.Apr 9 2021, 5:45 AM
jlduran added a subscriber: jlduran.Apr 10 2021, 3:28 PM
jlduran added inline comments.
usr.sbin/bhyve/xmsr.c
133

I guess machine/specialreg.h already defines IA32_FEATURE_CONTROL_LOCK?

corvink updated this revision to Diff 87295.Apr 12 2021, 5:39 AM
  • Use IA32_FEATURE_CONTROL_LOCK define
corvink marked an inline comment as done.Apr 12 2021, 5:39 AM
jhb added a subscriber: jhb.Apr 12 2021, 5:54 PM

This looks ok to me with the tweaked comment. Does this permit Windows guests to boot without -w?

usr.sbin/bhyve/xmsr.c
129–131
corvink updated this revision to Diff 87361.Apr 13 2021, 5:26 AM

I'm able to boot a Windows guest without -w by applying this patch.

corvink marked an inline comment as done.Apr 13 2021, 5:27 AM
grehan accepted this revision.Apr 13 2021, 7:51 AM
grehan added a subscriber: grehan.

Looks fine, and preserves #GP on writes.

This revision is now accepted and ready to land.Apr 13 2021, 7:51 AM
jhb accepted this revision.Apr 13 2021, 3:31 PM

Do you have this already staged as a git commit in a tree Peter or I could pull from?

corvink added a comment.Apr 14 2021, 8:48 AM

This commit isn't staged in a tree.

Closed by commit rG17d214c2ca9f: bhyve: implement rdmsr for MSR_IA32_FEATURE_CONTROL (authored by corvink, committed by grehan). · Explain WhyApr 15 2021, 5:52 AM
This revision was automatically updated to reflect the committed changes.
grehan added a commit: rG17d214c2ca9f: bhyve: implement rdmsr for MSR_IA32_FEATURE_CONTROL.

Revision Contents
Changeset List

PathSize
usr.sbin/
bhyve/
8 lines

Diff 87295

View Options

usr.sbin/bhyve/xmsr.c

Loading...