Skip to content

cuda: Allow for compilation of cuda toolkit 13.1.1#558

Merged
Treece-Burgess merged 1 commit intoicl-utk-edu:masterfrom
Treece-Burgess:resolve-cuda-13-1-1-compilation-error
Feb 11, 2026
Merged

cuda: Allow for compilation of cuda toolkit 13.1.1#558
Treece-Burgess merged 1 commit intoicl-utk-edu:masterfrom
Treece-Burgess:resolve-cuda-13-1-1-compilation-error

Conversation

@Treece-Burgess
Copy link
Contributor

Pull Request Description

Compiling the cuda component on Gilgamesh at Oregon with Cuda Toolkit 13.1.1 will result in the following compilation error:

In file included from components/cuda/cupti_event_and_metric.c:15:
/packages/cuda/13.1.1/extras/CUPTI/include/cupti_profiler_target.h:558:5: error: unknown type name ‘bool’
     bool bAllowDeviceLevelCounters;                      //!< [in] if true, device level counters are included in the counter availability image

This PR resolves this behavior by including stdbool.h in cupti_profiler.c and cupti_event_and_metric.c.

Testing

Testing was done on Gilgamesh at Oregon with Cuda Toolkit 13.1.1:

  • Compilation: ✅
  • PAPI Utilities*: ✅
  • Cuda component tests: ✅

* - papi_component_avail, papi_command_line, papi_native_avail

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 type-bug Issues discussing bugs or PRs fixing bugs component-cuda PRs and Issues related to the cuda component status-ready-for-review PR is ready to be reviewed labels Feb 9, 2026
@dbarry9
Copy link
Contributor

dbarry9 commented Feb 11, 2026

I am reviewing this PR.

Copy link
Contributor

@dbarry9 dbarry9 left a comment

Choose a reason for hiding this comment

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

I tested these changes using CUDA 13.1.1; however, the system I used currently does not have an available device. These changes indeed resolve the compile-time errors, and the PAPI utilities behave as expected in the absence of a device.

@dbarry9
Copy link
Contributor

dbarry9 commented Feb 11, 2026

@Treece-Burgess Can you please rebase?

@Treece-Burgess Treece-Burgess force-pushed the resolve-cuda-13-1-1-compilation-error branch from 32a0bf0 to 6cf1f10 Compare February 11, 2026 21:02
@Treece-Burgess Treece-Burgess merged commit c60634e into icl-utk-edu:master Feb 11, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component-cuda PRs and Issues related to the cuda 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