Page MenuHomeFreeBSD

AMD - emulate Extended CPUID Features override.
ClosedPublic

Authored by araujo on Oct 26 2018, 11:42 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, May 17, 7:52 PM
Unknown Object (File)
Tue, May 12, 9:17 AM
Unknown Object (File)
Sun, May 10, 12:20 PM
Unknown Object (File)
Sun, May 10, 12:20 PM
Unknown Object (File)
Thu, Apr 23, 2:09 AM
Unknown Object (File)
Apr 11 2026, 5:31 AM
Unknown Object (File)
Apr 6 2026, 7:56 AM
Unknown Object (File)
Apr 5 2026, 4:13 PM
Subscribers

Details

Summary

This patch comes from a PR: https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=224476
It looks right for me, but unfortunately I don't have any AMD FX now for testing.

Test Plan

Smoke test: build kernel on AMD Ryzen.
Run: Linux Fedora and FreeBSD HEAD as guest.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

I am good with this too, other than it is not emulating it, it is just ignoring it by returning a 0, which may or may not have future implications.

This revision is now accepted and ready to land.Oct 26 2018, 1:06 PM
rgrimes added a reviewer: bhyve.

Sorry for the extra noise here, I did not mean to accept it as bhvye, only me personally, so did delete/add bhyve umbrella to clear that bit.

This revision was automatically updated to reflect the committed changes.
anish added inline comments.
head/sys/amd64/vmm/amd/svm_msr.c
130

This cases could have been merged with upper block, something like

 case MSR_MTRR64kBase:
 case MSR_SYSCFG:
 case MSR_AMDK8_IPM:
case MSR_EXTFEATURES:
         *result = 0;
         break;
araujo added inline comments.
head/sys/amd64/vmm/amd/svm_msr.c
130