Skip to content

cuda: Remove unformatted args from SUBDBG calls#593

Open
Treece-Burgess wants to merge 1 commit into
icl-utk-edu:masterfrom
Treece-Burgess:04-20-2026-cuda-remove-arg
Open

cuda: Remove unformatted args from SUBDBG calls#593
Treece-Burgess wants to merge 1 commit into
icl-utk-edu:masterfrom
Treece-Burgess:04-20-2026-cuda-remove-arg

Conversation

@Treece-Burgess

Copy link
Copy Markdown
Contributor

Pull Request Description

PR #585 updated the handling of PAPI_CUDA_RUNTIME, PAPI_CUDA_PERFWORKS, and PAPI_CUDA_CUPTI. With the update more SUBDBG messages were added. Specifically for each of the aforementioned env's a SUBDBG was added in the format of:

SUBDBG("PAPI_CUDA_PERFWORKS was set, but did not result in successfully loading the libnvperf_host shared object."
               " Set PAPI_CUDA_PERFWORKS to a valid libnvperf_host shared object.\n", papi_cuda_perfworks);

There is an issue with the above SUBDBG which is the second argument papi_cuda_perfworks as it needs a format specifier that is not present. To rectify this, the second argument for all three of the aforementioned env's has been removed in this PR.

Testing

Testing was done on Hopper1 at Oregon (1 * GH200) with Cuda Toolkit 12.8.1.

  • PAPI Build: ✅
  • PAPI Utilities*: ✅
  • Cuda tests: ✅

* - papi_component_avail, papi_native_avail, and papi_command_line

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 21, 2026 00:16
@Treece-Burgess Treece-Burgess added component-cuda PRs and Issues related to the cuda component status-ready-for-review PR is ready to be reviewed type-maintenance Update code to keep it compatible, secure, modern. labels Apr 21, 2026
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-maintenance Update code to keep it compatible, secure, modern.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant