Skip to content

framework: For the sysdetect component use --enable and --disable at configure and do not show native event output for utils#588

Open
Treece-Burgess wants to merge 1 commit into
icl-utk-edu:masterfrom
Treece-Burgess:04-11-2026-sysdetect-update
Open

framework: For the sysdetect component use --enable and --disable at configure and do not show native event output for utils#588
Treece-Burgess wants to merge 1 commit into
icl-utk-edu:masterfrom
Treece-Burgess:04-11-2026-sysdetect-update

Conversation

@Treece-Burgess

@Treece-Burgess Treece-Burgess commented Apr 14, 2026

Copy link
Copy Markdown
Contributor

Pull Request Description

This PR makes it such that at configure time a user will need to use the following flags when enabling or disabling the sysdetect component:

  • --disable-sysdetect: The sysdetect component is disabled and removed from the list of compiled in components.
  • --enable-sysdetect=yes: The sysdetect component is not disabled and will be in the list of compiled in components.
  • --enable-sysdetect=no: The sysdetect component is disabled and removed from the list of compiled in components.

Testing

Testing was done on Methane at ICL (CPU: Intel Xeon Gold 6140, OS: Rocky Linux 9.6). Two groups of configures were tested, see below for more details.

The first group of configures tested were:

- ./configure --prefix=$PWD/test-install
- ./configure --prefix=$PWD/test-install --enable-sydetect
- ./configure --prefix=$PWD/test-install --enable-sysdetect=yes
- ./configure --prefix=$PWD/test-install --disable-perf-event-uncore

And the results were:

  • Configure output: ✅
checking for building sysdetect... yes
checking for components to build...  perf_event perf_event_uncore sysdetect (perf_event sysdetect in the case of --disable-perf-event-uncore)
  • PAPI build: ✅
  • papi_component_avail: ✅
    • For sysdetect, we not do not see Native: #, Preset: #, Counters: #.
  • papi_component_avail -d: ✅
    • For sysdetect, we only see the version output.
  • papi_native_avail: ✅
    • For sysdetect, the Native Events in Component header now does not output.

The second group of configures tested were:

./configure --prefix=$PWD/test-install --enable-sysdetect=no
./configure --prefix=$PWD/test-install --disable-sysdetect
./configure --prefix=$PWD/test-install --disable-cpu
./configure --prefix=$PWD/test-install --disable-perf-event

And the results were:

  • Configure output: ✅
checking for building sysdetect... no
checking for components to build... perf_event perf_event_uncore (empty in the case of --disable-cpu and --disable-perf-event)
  • PAPI build: ✅
  • papi_component_avail: ✅
    • sysdetect was not present
  • papi_native_avail: ✅
    • For sysdetect, the Native Events in Component header now does not output.

For the change of print_header.c to utils_helper.c and print_header.h to utils_helper.h, I checked the output of the utilities against the master branch.

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

…configure and do not show native event output for utils
@Treece-Burgess Treece-Burgess added component-sysdetect PRs and Issues related to the sysdetect component status-ready-for-review PR is ready to be reviewed type-maintenance Update code to keep it compatible, secure, modern. labels Apr 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component-sysdetect PRs and Issues related to the sysdetect component status-ready-for-review PR is ready to be reviewed type-maintenance Update code to keep it compatible, secure, modern.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant