Skip to content

[Backport branch/3.4.x] [libcu++] Fix the default device pool getter#9412

Open
github-actions[bot] wants to merge 1 commit into
branch/3.4.xfrom
backport-9351-to-branch/3.4.x
Open

[Backport branch/3.4.x] [libcu++] Fix the default device pool getter#9412
github-actions[bot] wants to merge 1 commit into
branch/3.4.xfrom
backport-9351-to-branch/3.4.x

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

Description

Backport of #9351 to branch/3.4.x.

* Fix the default device pool getter

* fix format

* Fix clang-tidy

* [libcu++] Move memory pool atrributes into their own file

* [libcu++] Properly initialize the default memory pool

We need to create a default memory pool per device

The best place is to create it in `physical_device` because there we already have a lot of the machinery to do it once in place

* Fix header guards

* Drop include

* Remove the physical device bits and use init_once in the pool getter

---------

Co-authored-by: Michael Schellenberger Costa <miscco@nvidia.com>
(cherry picked from commit 1028921)
@github-actions github-actions Bot requested a review from a team as a code owner June 11, 2026 18:46
@github-actions github-actions Bot requested a review from wmaxey June 11, 2026 18:46
@github-project-automation github-project-automation Bot moved this to Todo in CCCL Jun 11, 2026
@copy-pr-bot

copy-pr-bot Bot commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@pciolkosz

Copy link
Copy Markdown
Contributor

/ok to test 1c8057a

@github-project-automation github-project-automation Bot moved this from Todo to In Review in CCCL Jun 12, 2026
@github-actions

Copy link
Copy Markdown
Contributor Author

🥳 CI Workflow Results

🟩 Finished in 1h 22m: Pass: 100%/113 | Total: 2d 01h | Max: 1h 11m | Hits: 73%/458719

See results here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In Review

Development

Successfully merging this pull request may close these issues.

1 participant