Page MenuHomeFreeBSD
Differential D16923

Document cpu_microcode_* in loader.conf.5.
ClosedPublic
Actions

Authored by markj on Aug 27 2018, 7:00 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Mar 30, 12:30 AM
Unknown Object (File)
Sun, Mar 29, 8:34 AM
Unknown Object (File)
Sun, Mar 29, 8:29 AM
Unknown Object (File)
Wed, Mar 25, 12:01 AM
Unknown Object (File)
Tue, Mar 24, 3:35 AM
Unknown Object (File)
Tue, Mar 24, 2:42 AM
Unknown Object (File)
Tue, Mar 24, 12:11 AM
Unknown Object (File)
Wed, Mar 18, 10:48 AM
View All Files
Subscribers
bcr
imp

Details

Reviewers
emaste
kib
jhb
bcr
Group Reviewers
manpages
Commits
rS338354: Document the cpu_microcode_* tunables.

Diff Detail

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

Event Timeline

markj created this revision.Aug 27 2018, 7:00 PM
Harbormaster completed remote builds in B19222: Diff 47357.Aug 27 2018, 7:00 PM
markj added reviewers: emaste, kib, jhb, manpages.Aug 27 2018, 7:01 PM
bcr accepted this revision.Aug 27 2018, 7:06 PM
bcr added a subscriber: bcr.

Fine from manpages. Thanks for the work!

This revision is now accepted and ready to land.Aug 27 2018, 7:06 PM
kib added a comment.Aug 27 2018, 7:35 PM

I think it would be useful to cross-reference cpucontrol(8) there. Also, when noting that the update is loaded very early, say that it is persistent over suspend/resume.

markj updated this revision to Diff 47361.Aug 27 2018, 7:45 PM
  • Reference suspend/resume and cpucontrol(8).
This revision now requires review to proceed.Aug 27 2018, 7:45 PM
Harbormaster completed remote builds in B19225: Diff 47361.Aug 27 2018, 7:45 PM
kib added inline comments.Aug 27 2018, 7:56 PM
stand/defaults/loader.conf.5
301 ↗(On Diff #47361)

Uh, I thought that it was fixed, no ?

markj added inline comments.Aug 27 2018, 7:59 PM
stand/defaults/loader.conf.5
301 ↗(On Diff #47361)

Oh, you're right, that should work now.

markj added inline comments.Aug 27 2018, 8:04 PM
stand/defaults/loader.conf.5
301 ↗(On Diff #47361)

... well, it works only for Intel CPUs for now.

markj updated this revision to Diff 47386.Aug 28 2018, 4:37 PM
  • Remove incorrect claim about cpucontrol(8)-based updates.
Harbormaster completed remote builds in B19240: Diff 47386.Aug 28 2018, 4:37 PM
kib accepted this revision.Aug 28 2018, 4:46 PM
This revision is now accepted and ready to land.Aug 28 2018, 4:46 PM
Closed by commit rS338354: Document the cpu_microcode_* tunables. (authored by markj). · Explain WhyAug 28 2018, 5:15 PM
This revision was automatically updated to reflect the committed changes.
Herald added a subscriber: imp. · View Herald TranscriptAug 28 2018, 5:15 PM

Revision Contents
Changeset List

PathSize
head/
stand/
defaults/
26 lines

Diff 47391

View Options

head/stand/defaults/loader.conf.5

Loading...