Page MenuHomeFreeBSD
Differential D40544

hwpmc: add error for HWPMC_DEBUG without KTR
ClosedPublic
Actions

Authored by mhorne on Jun 14 2023, 5:58 PM.
Tags
None
Referenced Files
F167008105: D40544.diff
Tue, Aug 18, 1:29 PM
Unknown Object (File)
Fri, Aug 14, 10:34 PM
Unknown Object (File)
Wed, Aug 12, 10:02 PM
Unknown Object (File)
Wed, Aug 12, 4:45 PM
Unknown Object (File)
Wed, Aug 12, 4:43 PM
Unknown Object (File)
Sun, Aug 2, 5:57 AM
Unknown Object (File)
Sun, Aug 2, 4:01 AM
Unknown Object (File)
Sat, Aug 1, 7:33 AM
View All Files
Subscribers
imp

Details

Reviewers
jkoshy
emaste
Group Reviewers
pmc
Commits
rG3c4860d3eeed: hwpmc: add error for HWPMC_DEBUG without KTR
rG569f89b2f442: hwpmc: add error for HWPMC_DEBUG without KTR
Summary

The option is a no-op otherwise, but this is not necessarily obvious.
Failing the compile gives the user a hint.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 52057
Build 48948: arc lint + arc unit

Event Timeline

mhorne created this revision.Jun 14 2023, 5:58 PM
Herald added a subscriber: imp. · View Herald TranscriptJun 14 2023, 5:58 PM
mhorne requested review of this revision.Jun 14 2023, 5:58 PM
mhorne added a parent revision: D40543: pmc: clarify and strengthen statements about API/ABI.
Harbormaster completed remote builds in B52057: Diff 123241.Jun 14 2023, 5:58 PM
mhorne added a child revision: D40545: hwpmc(4): document debugging options.Jun 14 2023, 5:58 PM
emaste added a comment.Jun 14 2023, 8:14 PM

It might be worth putting a comment in the src as well mentioning that HWPMC_DEBUG alone will compile but have no effect.

emaste accepted this revision.Jun 14 2023, 8:14 PM
This revision is now accepted and ready to land.Jun 14 2023, 8:14 PM
jkoshy accepted this revision.Jun 16 2023, 4:05 AM
Closed by commit rG569f89b2f442: hwpmc: add error for HWPMC_DEBUG without KTR (authored by mhorne). · Explain WhyJun 16 2023, 4:45 PM
This revision was automatically updated to reflect the committed changes.
mhorne added a commit: rG569f89b2f442: hwpmc: add error for HWPMC_DEBUG without KTR.
mhorne added a commit: rG3c4860d3eeed: hwpmc: add error for HWPMC_DEBUG without KTR.Jul 17 2023, 4:49 PM

Revision Contents
Changeset List

PathSize
sys/
sys/
5 lines

Diff 123241

View Options

sys/sys/pmc.h

Loading...