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)
Jan 23 2024, 4:06 PM
Unknown Object (File)
Dec 26 2023, 1:01 AM
Unknown Object (File)
Dec 20 2023, 1:46 AM
Unknown Object (File)
Dec 14 2023, 11:28 PM
Unknown Object (File)
Oct 19 2023, 10:21 AM
Unknown Object (File)
Sep 21 2023, 4:43 PM
Unknown Object (File)
Sep 3 2023, 6:12 AM
Unknown Object (File)
Aug 26 2023, 1:21 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