This includes new code for parsing microcode files as well as
the kernel-side change to apply the update on all processors
at the same time.
Developed with help from Borislav Petkov, formerly bp@amd64.org.
Details
Details
- Reviewers
kib stas - Commits
- rS308218: Add support for microcode update on newer AMD CPUs (10h+)
Tested using Athlon II X2 processor on a system where BIOS does
not have the latest microcode version:
/boot/firmware/microcode_amd.bin: updating cpu /dev/cpuctl0 to revision 0x10000c7... done.
The microcode file is taken from here:
https://web.archive.org/web/20160528230514/http://www.amd64.org/microcode.html
(note that the site seems to be down at the moment)
It can also be found here:
https://git.kernel.org/cgit/linux/kernel/git/firmware/linux-firmware.git/tree/amd-ucode
Diff Detail
Diff Detail
- Repository
- rS FreeBSD src repository - subversion
- Lint
Lint Passed - Unit
No Test Coverage - Build Status
Buildable 5822 Build 6111: arc lint + arc unit
Event Timeline
| sys/dev/cpuctl/cpuctl.c | ||
|---|---|---|
| 60 | This is unrelated, commit separately (now ?). | |
| 401 | Why is this required ? I do not see any harm from allowing the update to run from arbitrary /dev/cpuctlN. | |
| 424 | Check for ptr != NULL is not needed. Use plain free(9). | |
| usr.sbin/cpucontrol/amd10h.c | ||
| 3 | Copyright line should be updated. | |
Comment Actions
Looks good besides what kib@ already mentioned.
Thanks!
| sys/dev/cpuctl/cpuctl.c | ||
|---|---|---|
| 401 | I agree. Unless there are restrictions on which cores the update can be performed on... But in that case the check will have to be a bit more sophisticated to account for multi-socket configurations. | |
| usr.sbin/cpucontrol/amd10h.c | ||
| 175 | Double space. | |
| sys/dev/cpuctl/cpuctl.c | ||
|---|---|---|
| 401 | There is really no restriction. I just wanted to skip unnecessary writes to the update MSR in the default configuration with microcode_cpus="ALL". But that was not needed anyway, because the first invocation of cpucontrol would update all CPUs and subsequent invocations would see that the CPUs have the latest version. | |
Comment Actions
Just as a data point:
$ cpucontrol -vvv -d /boot/firmware -u /dev/cpuctl0 cpucontrol: skipping /boot/firmware/.: is a directory cpucontrol: skipping /boot/firmware/..: is a directory cpucontrol: found cpu family 0xf model 0x6 stepping 0x2 extfamily 0x1 extmodel 0. cpucontrol: microcode revision 0x1000098 cpucontrol: equiv_id: 1062 cpucontrol: selecting revision: 10000c7 cpucontrol: selected ucode size is 960 /boot/firmware/microcode_amd.bin: updating cpu /dev/cpuctl0 to revision 0x10000c7... done.
$ cpucontrol -vvv -d /boot/firmware -u /dev/cpuctl1 cpucontrol: skipping /boot/firmware/.: is a directory cpucontrol: skipping /boot/firmware/..: is a directory cpucontrol: found cpu family 0xf model 0x6 stepping 0x2 extfamily 0x1 extmodel 0. cpucontrol: microcode revision 0x10000c7 cpucontrol: equiv_id: 1062