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, Jun 30, 8:38 AM
Unknown Object (File)
Sun, Jun 23, 4:45 PM
Unknown Object (File)
Thu, Jun 6, 9:13 AM
Unknown Object (File)
May 28 2024, 9:20 AM
Unknown Object (File)
May 18 2024, 1:35 AM
Unknown Object (File)
May 6 2024, 7:27 AM
Unknown Object (File)
Apr 26 2024, 3:49 AM
Unknown Object (File)
Jan 23 2024, 4:06 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