Skip to content

Feat: Enable support for PAPI counter tarcing#343

Draft
amarathe84 wants to merge 3 commits into
llnl:developfrom
amarathe84:papi-tracing
Draft

Feat: Enable support for PAPI counter tarcing#343
amarathe84 wants to merge 3 commits into
llnl:developfrom
amarathe84:papi-tracing

Conversation

@amarathe84
Copy link
Copy Markdown
Collaborator

This PR introduces support for PAPI counter tracing.

Fixes #342.

#define DFT_YAML_FEATURES_PAPI "papi"
#define DFT_YAML_FEATURES_PAPI_ENABLE "enable"
#define DFT_YAML_FEATURES_PAPI_EVENTS "events"
#define DFT_YAML_FEATURES_PAPI_INTERVAL "interval"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should we reuse the interval similar to profiling interval already? That used by service and profiling for aggregation.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes makes sense. I've updated the code to use profiling interval

@rayandrew
Copy link
Copy Markdown
Collaborator

@amarathe84 can you push src/dftracer/core/function/papi/counters.cpp as well?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support PAPI counters

3 participants