Skip to content

framework: Properly handle the case of no components configured#589

Open
Treece-Burgess wants to merge 1 commit into
icl-utk-edu:masterfrom
Treece-Burgess:04-14-2026-no-components
Open

framework: Properly handle the case of no components configured#589
Treece-Burgess wants to merge 1 commit into
icl-utk-edu:masterfrom
Treece-Burgess:04-14-2026-no-components

Conversation

@Treece-Burgess

Copy link
Copy Markdown
Contributor

Pull Request Description

In the master branch, if you configure PAPI as follows:

./configure --prefix=$PWD/test-install --disable-cpu

No components will be configured and this is indeed correct behavior. However, configure does not output anything for "checking for components to build..." and papi_component_avail shows Native: 0, Preset: 0, Counters: 0, i.e.

configure:

checking for components to build...

papi_component_avail:

Compiled-in components:
Name:   No Components Configured.  

Active components:
Name:   No Components Configured.  
                                Native: 0, Preset: 0, Counters: 0


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

This PR updates this behavior as follows:

  1. If no components are configured, configure will now output:
checking for components to build... No components configured
  1. If no components are configured, papi_component_avail will output:
Compiled-in components:
Name:   No Components Configured. 

Active components:
Name:   No Components Configured. 

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

Testing

These changes were verified on Methane at ICL (CPU: Intel Xeon Gold 6140, OS: Rocky Linux 9.6 )

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 requested a review from dbarry9 April 14, 2026 23:20
@Treece-Burgess Treece-Burgess added status-ready-for-review PR is ready to be reviewed type-maintenance Update code to keep it compatible, secure, modern. update-papi-utilities Updates that pertain to the PAPI utilities such as papi_component_avail.c or papi_native_avail.c. labels Apr 14, 2026
@Treece-Burgess Treece-Burgess changed the title framework: properly handle the case of no components configured framework: Properly handle the case of no components configured Apr 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status-ready-for-review PR is ready to be reviewed type-maintenance Update code to keep it compatible, secure, modern. update-papi-utilities Updates that pertain to the PAPI utilities such as papi_component_avail.c or papi_native_avail.c.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant