Differential D36401

libpmc: pmc_init() should avoid overflowing pmc_class_table
ClosedPublic
Actions

Authored by tsoome on Aug 31 2022, 11:14 AM.

Tags

None

Referenced Files

	F97574023: D36401.diff
	Mon, Sep 30, 3:37 AM

	Unknown Object (File)
	Sat, Sep 28, 10:29 PM

	Unknown Object (File)
	Fri, Sep 27, 10:42 AM

	Unknown Object (File)
	Thu, Sep 26, 10:47 AM

	Unknown Object (File)
	Mon, Sep 23, 8:23 PM

	Unknown Object (File)
	Fri, Sep 13, 7:49 AM

	Unknown Object (File)
	Wed, Sep 4, 10:13 AM

	Unknown Object (File)
	Mon, Sep 2, 12:45 PM

View All 90 Files

Subscribers

Details

Reviewers: None
Commits: rG1f5679569dd4: libpmc: pmc_init() should avoid overflowing pmc_class_table

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 47206
Build 44093: arc lint + arc unit

Event Timeline

tsoome created this revision.Aug 31 2022, 11:14 AM

Herald added a subscriber: imp. · View Herald TranscriptAug 31 2022, 11:14 AM

tsoome requested review of this revision.Aug 31 2022, 11:14 AM

Harbormaster completed remote builds in B47196: Diff 110009.Aug 31 2022, 11:14 AM

commit message update

Harbormaster completed remote builds in B47198: Diff 110011.Aug 31 2022, 11:30 AM

50 enumeration values not handled in switch

Harbormaster completed remote builds in B47199: Diff 110012.Aug 31 2022, 11:44 AM

jrtc27 added a subscriber: jrtc27.Aug 31 2022, 2:39 PM

jrtc27 added inline comments.

lib/libpmc/libpmc.c
1476	Presumably this was a hacky alternative way to write that pm_classes contained a PMC_CLASS_TSC? I imagine this check is now redundant.
1607	Should we have an equivalent of this for the classes case? Also for the ARMV7/8 switches we now have? I guess continuing with the subset libpmc has been taught about is helpful, but silently not getting a set of counters also isn't...

review feedback.

Harbormaster completed remote builds in B47206: Diff 110021.Aug 31 2022, 4:15 PM

tsoome marked an inline comment as done.Aug 31 2022, 4:16 PM

tsoome added inline comments.

lib/libpmc/libpmc.c
1476	I have no idea about the history there, but yea, if the kernel does know about PMC_CLASS_TSC, it makes little sense to filter it out.
1607	Yes, good point.

This revision was not accepted when it landed; it landed in state Needs Review.Sep 5 2022, 12:39 PM

Closed by commit rG1f5679569dd4: libpmc: pmc_init() should avoid overflowing pmc_class_table (authored by tsoome). · Explain Why

This revision was automatically updated to reflect the committed changes.

tsoome marked an inline comment as done.

tsoome added a commit: rG1f5679569dd4: libpmc: pmc_init() should avoid overflowing pmc_class_table.

Revision Contents
Changeset List

Path

Size

lib/

libpmc/

2 lines

108 lines

Diff 110021

lib/libpmc/Makefile

Loading...

lib/libpmc/libpmc.c

Loading...