hwpmc: switch in process PMCs on a thread's first run
AcceptedPublic
Actions

Authored by afscoelho_gmail.com on Sat, Oct 10, 1:22 AM.

Tags

None

Referenced Files

	F175464585: D60565.diff
	Sun, Oct 11, 1:36 AM

	F175437045: D60565.id.diff
	Sat, Oct 10, 8:43 PM

	F175410220: D60565.id189320.diff
	Sat, Oct 10, 4:16 PM

	F175409578: D60565.diff
	Sat, Oct 10, 4:09 PM

	F175399981: D60565.diff
	Sat, Oct 10, 2:20 PM

	F175372546: D60565.id189320.diff
	Sat, Oct 10, 9:12 AM

	F175371595: D60565.diff
	Sat, Oct 10, 9:02 AM

	F175357022: D60565.id189320.diff
	Sat, Oct 10, 6:26 AM

View All 10 Files

Subscribers

imp

olce

Details

Reviewers

mhorne
gnn
ali_mashtizadeh.com
kib
markj

Group Reviewers

pmc

Summary

A new thread starts in fork_exit(), not mi_switch(), so its PMCs are
not loaded until its next context switch. Call the switch-in hook
there.

Depends on D60564

Test Plan

Built amd64 GENERIC + hwpmc.ko, GENERIC without HWPMC_HOOKS, and arm64.
Ran tests/sys/pmc on AMD Zen 6 (LBR v2).

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 77938
Build 74821: arc lint + arc unit

Event Timeline

afscoelho_gmail.com created this revision.Sat, Oct 10, 1:22 AM

Herald added subscribers: olce, imp. · View Herald TranscriptSat, Oct 10, 1:22 AM

afscoelho_gmail.com requested review of this revision.Sat, Oct 10, 1:22 AM

Harbormaster completed remote builds in B77938: Diff 189320.Sat, Oct 10, 1:22 AM

afscoelho_gmail.com added a child revision: D60566: hwpmc: add the LBR capability and multipart record type.Sat, Oct 10, 1:22 AM

kib accepted this revision.Sat, Oct 10, 9:29 AM

This revision is now accepted and ready to land.Sat, Oct 10, 9:29 AM

Revision Contents
Changeset List

Path

Size

sys/

kern/

kern_fork.c

10 lines

Diff 189320

View Options

hwpmc: switch in process PMCs on a thread's first runAcceptedPublicActions