Paths

Table of Contentst

Differential D55001

hwpstate_amd(4): Assert PSTATE_CPPC is on in corresponding sysctl handlers
ClosedPublic
Actions

Authored by olce on Jan 31 2026, 11:46 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Thu, Jul 23, 3:21 PM

	Unknown Object (File)
	Thu, Jul 23, 4:33 AM

	Unknown Object (File)
	Wed, Jul 22, 6:32 PM

	Unknown Object (File)
	Tue, Jul 21, 4:00 PM

	Unknown Object (File)
	Tue, Jul 21, 1:16 AM

	Unknown Object (File)
	Sun, Jul 19, 4:21 PM

	Unknown Object (File)
	Sat, Jul 18, 7:58 PM

	Unknown Object (File)
	Fri, Jul 17, 11:37 PM

View All 50 Files

Subscribers

Details

Reviewers

aokblast
lwhsu
emaste

Commits

rGfa0512f7156d: hwpstate_amd(4): Assert PSTATE_CPPC is on in corresponding sysctl handlers

Summary

No functional change intended.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

olce created this revision.Jan 31 2026, 11:46 AM

Herald added a subscriber: imp. · View Herald TranscriptJan 31 2026, 11:46 AM

olce requested review of this revision.Jan 31 2026, 11:46 AM

Harbormaster completed remote builds in B70273: Diff 170836.Jan 31 2026, 11:46 AM

olce added a parent revision: D55000: hwpstate_amd(4): Rename register dump sysctl handler.Jan 31 2026, 1:25 PM

olce added a child revision: D55002: hwpstate_amd(4): Rename EPP handler, convert to unsigned int.

aokblast added inline comments.Feb 1 2026, 8:01 AM

sys/x86/cpufreq/hwpstate_amd.c
206	Can it just be a macro? I don't think inline always work. Or always inline?

olce added inline comments.Feb 2 2026, 4:51 PM

sys/x86/cpufreq/hwpstate_amd.c
206	I think we should use functions unless we have a good reason to use macros, for type safety, single evaluations of arguments, etc. The `inline` here is anyway cosmetic, `clang` always inlines such small functions even if there's no explicit hint.

This revision was not accepted when it landed; it landed in state Needs Review.Feb 6 2026, 4:29 PM

Closed by commit rGfa0512f7156d: hwpstate_amd(4): Assert PSTATE_CPPC is on in corresponding sysctl handlers (authored by olce). · Explain Why

This revision was automatically updated to reflect the committed changes.

olce added a commit: rGfa0512f7156d: hwpstate_amd(4): Assert PSTATE_CPPC is on in corresponding sysctl handlers.

Revision Contents
Changeset List

Path

Size

sys/

x86/

cpufreq/

15 lines

Diff 171349

sys/x86/cpufreq/hwpstate_amd.c

Loading...