Page MenuHomeFreeBSD

libpmc: remove PMC_MDEP_TABLE logic
ClosedPublic

Authored by mhorne on May 10 2021, 8:14 PM.
Tags
None
Referenced Files
F103102811: D30196.id88986.diff
Thu, Nov 21, 12:52 AM
Unknown Object (File)
Wed, Nov 13, 7:14 PM
Unknown Object (File)
Oct 19 2024, 7:11 AM
Unknown Object (File)
Oct 4 2024, 11:36 PM
Unknown Object (File)
Oct 4 2024, 11:11 AM
Unknown Object (File)
Oct 4 2024, 2:15 AM
Unknown Object (File)
Oct 2 2024, 5:45 AM
Unknown Object (File)
Oct 1 2024, 11:21 PM
Subscribers

Details

Summary

This logic was added for handling some of the complicated relationships
between events and x86 CPU models. Since that logic has been mostly
removed from libpmc(3) in favor of pmu-events, this no longer serves
much of a purpose. Mapping CPU types to event tables is already handled
by the switch statement in pmc_init().

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

So in effect this basically just repeated existing tests that exist elsewhere?

This revision is now accepted and ready to land.May 10 2021, 8:20 PM

So in effect this basically just repeated existing tests that exist elsewhere?

Pretty much. pmc_init() adds only supported classes to the pmc_class_table array, so pmc_mdep_is_supported_class() only duplicates work by checking those classes against its own tables.

I like it.
Thanks!

This revision was automatically updated to reflect the committed changes.