Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FEA]: Implement a memory_resource using cudaMalloc and cudaFree #1512

Closed
Tracked by #2128
miscco opened this issue Mar 7, 2024 · 1 comment · Fixed by #1578
Closed
Tracked by #2128

[FEA]: Implement a memory_resource using cudaMalloc and cudaFree #1512

miscco opened this issue Mar 7, 2024 · 1 comment · Fixed by #1578
Assignees
Labels
feature request New feature or request. libcu++ For all items related to libcu++

Comments

@miscco
Copy link
Collaborator

miscco commented Mar 7, 2024

In order to move our <memory_resource> plans forward we want to implement a memory resource that utilizes cudaMalloc and cudaFree to allocate memory.

That memory_resource should live in <cuda/memory_resource> and at least satisfy the cuda::mr::resource_with<cuda::mr::device_accessible> requirement.

Poll for the audience. Should it also satisfy cuda::mr::async_resource_with<cuda::mr::device_accessible>?

@github-project-automation github-project-automation bot moved this to Todo in CCCL Mar 7, 2024
@miscco miscco added feature request New feature or request. libcu++ For all items related to libcu++ labels Mar 7, 2024
@miscco miscco self-assigned this Mar 7, 2024
@miscco miscco changed the title cudaMalloc/cudaFree Implement a memory_resource using cudaMalloc and cudaFree Mar 7, 2024
miscco added a commit to miscco/cccl that referenced this issue Mar 7, 2024
@cccl-authenticator-app cccl-authenticator-app bot moved this from Todo to In Review in CCCL Mar 7, 2024
miscco added a commit to miscco/cccl that referenced this issue Mar 7, 2024
miscco added a commit to miscco/cccl that referenced this issue Mar 7, 2024
@miscco
Copy link
Collaborator Author

miscco commented Mar 8, 2024

I have for now removed the async interface. We do want to add a cuda_async_memory_resource anyway, so adding an essentially false promise seems bad

miscco added a commit to miscco/cccl that referenced this issue Mar 11, 2024
miscco added a commit to miscco/cccl that referenced this issue Mar 11, 2024
@miscco miscco changed the title Implement a memory_resource using cudaMalloc and cudaFree [FEA]: Implement a memory_resource using cudaMalloc and cudaFree Mar 11, 2024
miscco added a commit to miscco/cccl that referenced this issue Mar 11, 2024
miscco added a commit to miscco/cccl that referenced this issue Mar 11, 2024
@cccl-authenticator-app cccl-authenticator-app bot moved this from In Review to In Progress in CCCL Mar 12, 2024
miscco added a commit to miscco/cccl that referenced this issue Mar 12, 2024
miscco added a commit to miscco/cccl that referenced this issue Mar 12, 2024
miscco added a commit to miscco/cccl that referenced this issue Mar 12, 2024
miscco added a commit to miscco/cccl that referenced this issue Mar 12, 2024
miscco added a commit to miscco/cccl that referenced this issue Apr 2, 2024
miscco added a commit to miscco/cccl that referenced this issue Apr 2, 2024
@cccl-authenticator-app cccl-authenticator-app bot moved this from In Progress to In Review in CCCL Apr 2, 2024
miscco added a commit to miscco/cccl that referenced this issue Apr 11, 2024
miscco added a commit that referenced this issue Apr 12, 2024
* Implement `cuda::mr::cuda_memory_resource`

Fixes #1512
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request New feature or request. libcu++ For all items related to libcu++
Projects
Archived in project
1 participant