Skip to content

Appio Component: Add a component description, as it is missing from papi_component_avail#368

Merged
Treece-Burgess merged 1 commit intoicl-utk-edu:masterfrom
Treece-Burgess:05-12-2025-appio-description
May 13, 2025
Merged

Appio Component: Add a component description, as it is missing from papi_component_avail#368
Treece-Burgess merged 1 commit intoicl-utk-edu:masterfrom
Treece-Burgess:05-12-2025-appio-description

Conversation

@Treece-Burgess
Copy link
Contributor

@Treece-Burgess Treece-Burgess commented May 12, 2025

Pull Request Description

Currently if you run ./papi_component_avail the appio component will not have a description shown:

Compiled-in components:
Name:   appio                   

Active components:
Name:   appio                   
                                Native: 45, Preset: 0, Counters: 45

--------------------------------------------------------------------------------

This PR addresses this by adding a description in the component vector of appio, now ./papi_component_avail shows:

Compiled-in components:
Name:   appio                   Linux I/O system calls


Active components:
Name:   appio                   Linux I/O system calls
                                Native: 45, Preset: 0, Counters: 45

--------------------------------------------------------------------------------

Tested this change on a machine with an Intel Xeon Gold 6140.

Author Checklist

  • Description
    Why this PR exists. Reference all relevant information, including background, issues, test failures, etc
  • Commits
    Commits are self contained and only do one thing
    Commits have a header of the form: module: short description
    Commits have a body (whenever relevant) containing a detailed description of the addressed problem and its solution
  • Tests
    The PR needs to pass all the tests

@Treece-Burgess Treece-Burgess added component-appio PRs and Issues related to the appio component type-bug Issues discussing bugs or PRs fixing bugs status-ready-for-review PR is ready to be reviewed labels May 12, 2025
@dbarry9
Copy link
Contributor

dbarry9 commented May 12, 2025

I am testing this pull request.

@dbarry9
Copy link
Contributor

dbarry9 commented May 12, 2025

I tested these changes on Frontier (AMD Zen3 architecture), and papi_component_avail now displays the information correctly.

@Treece-Burgess Treece-Burgess merged commit 2ed109c into icl-utk-edu:master May 13, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component-appio PRs and Issues related to the appio component status-ready-for-review PR is ready to be reviewed type-bug Issues discussing bugs or PRs fixing bugs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants