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.

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 78196
Build 75079: 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

Rebase only. No code change.

This revision now requires review to proceed.Wed, Oct 14, 12:44 AM

Harbormaster completed remote builds in B78196: Diff 189844.Wed, Oct 14, 12:44 AM

Rebase-only update to add the Differential Revision trailer; no code change.

@kib, could you please re-accept?

kib accepted this revision.Wed, Oct 14, 2:23 AM

This revision is now accepted and ready to land.Wed, Oct 14, 2:23 AM

Revision Contents
Changeset List

Path

Size

sys/

kern/

kern_fork.c

10 lines

Diff 189844

View Options

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