Paths

Table of Contentst

Differential D40294

hwpmc: split out PMC_OP_PMCRW
ClosedPublic
Actions

Authored by mhorne on May 26 2023, 6:40 PM.

Tags

None

Referenced Files

	F153718637: D40294.id.diff
	Thu, Apr 23, 4:45 AM

	F153715351: D40294.id.diff
	Thu, Apr 23, 4:07 AM

	Unknown Object (File)
	Tue, Apr 21, 1:32 AM

	Unknown Object (File)
	Sun, Apr 12, 9:37 PM

	Unknown Object (File)
	Sun, Apr 12, 9:28 PM

	Unknown Object (File)
	Thu, Apr 9, 11:57 PM

	Unknown Object (File)
	Sun, Apr 5, 7:45 AM

	Unknown Object (File)
	Wed, Apr 1, 3:06 AM

Subscribers

Details

Reviewers

jkoshy
emaste
gnn

Group Reviewers

Commits

rG93fe5909babb: hwpmc: split out PMC_OP_PMCRW

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 51722
Build 48613: arc lint + arc unit

Event Timeline

mhorne created this revision.May 26 2023, 6:40 PM

Herald added a reviewer: gnn. · View Herald TranscriptMay 26 2023, 6:40 PM

Herald added a subscriber: imp. · View Herald Transcript

mhorne requested review of this revision.May 26 2023, 6:40 PM

Harbormaster completed remote builds in B51722: Diff 122496.May 26 2023, 6:40 PM

mhorne added a parent revision: D40293: hwpmc: split out PMC_OP_PMCRELEASE.May 26 2023, 6:40 PM

jkoshy requested changes to this revision.May 31 2023, 6:48 AM

jkoshy added inline comments.

sys/dev/hwpmc/hwpmc_mod.c
3662	Argument `prw` is being passed by value so the `const` is redundant. How about using a `const` pointer here instead (`const struct pmc_op_pmcrw *prw`)?
4364	We could move this line closer to where 'pprw' gets used?

This revision now requires changes to proceed.May 31 2023, 6:48 AM

Handle review comments.

Harbormaster completed remote builds in B51843: Diff 122723.Jun 1 2023, 6:19 PM

jkoshy accepted this revision.Jun 2 2023, 8:53 PM

This revision is now accepted and ready to land.Jun 2 2023, 8:53 PM

Closed by commit rG93fe5909babb: hwpmc: split out PMC_OP_PMCRW (authored by mhorne). · Explain WhyJun 14 2023, 4:47 PM

This revision was automatically updated to reflect the committed changes.

mhorne added a commit: rG93fe5909babb: hwpmc: split out PMC_OP_PMCRW.

Revision Contents
Changeset List

Path

Size

sys/

dev/

hwpmc/

225 lines

Diff 122496

sys/dev/hwpmc/hwpmc_mod.c

Loading...