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)
Thu, Dec 12, 4:42 AM
Unknown Object (File)
Sun, Dec 1, 12:11 AM
Unknown Object (File)
Nov 12 2024, 2:58 PM
Unknown Object (File)
Nov 9 2024, 1:06 PM
Unknown Object (File)
Oct 18 2024, 7:48 AM
Unknown Object (File)
Oct 10 2024, 4:52 AM
Unknown Object (File)
Oct 6 2024, 12:26 AM
Unknown Object (File)
Oct 5 2024, 7:03 AM
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