Page MenuHomeFreeBSD

x86: Allow sharing of perfomance counter interrupts
AcceptedPublic

Authored by bnovkov on Fri, Aug 23, 4:34 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Sep 10, 11:35 PM
Unknown Object (File)
Sun, Sep 8, 7:26 AM
Unknown Object (File)
Sat, Sep 7, 4:00 PM
Unknown Object (File)
Wed, Sep 4, 3:28 PM
Unknown Object (File)
Mon, Sep 2, 3:52 PM
Unknown Object (File)
Sun, Sep 1, 9:45 AM
Unknown Object (File)
Sat, Aug 31, 9:11 AM
Unknown Object (File)
Thu, Aug 29, 4:40 AM
Subscribers

Details

Reviewers
br
kib
markj
gnn
Summary

This patch refactors the Performance Counter interrupt setup code to allow sharing the interrupt line between multiple drivers.
More specifically, Performance Counter interrupts are used by both hwpmc(4) and hwt(4)'s Intel Processor Trace backend.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

sys/x86/x86/local_apic.c
816–817

Why do we need these braces? Why not provide pcint() services regardless of hwpmc/hwt? IMO it is more logical, and slight unconditional code addition does not matter for amd64. But we reduce the ifdef maze then.

bnovkov added inline comments.
sys/x86/x86/local_apic.c
816–817

Right, the initial version was a knee-jerk reaction, what you suggested makes sense.

kib added inline comments.
sys/x86/x86/local_apic.c
39

Are both opts still needed?

This revision is now accepted and ready to land.Mon, Aug 26, 11:00 PM