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.

Sign up for GitHub

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

Jump to bottom

Stream callbacks should use CUDA_CB, not CUDART_CB #451

Closed
eyalroz opened this issue Jan 14, 2023 · 0 comments
Closed

Stream callbacks should use CUDA_CB, not CUDART_CB #451

eyalroz opened this issue Jan 14, 2023 · 0 comments
Assignees
@eyalroz
Labels
resolved-on-development task

Comments

@eyalroz
Copy link
Owner

eyalroz commented Jan 14, 2023

A change we should have with the adoption of the driver API is using the driver's notion of a host-side callback function, CUDA_CB, rather than the runtime's notion of it CUDART_CB. It's only a nitpick - since it's the same declaration - but still.
@eyalroz eyalroz added the task label Jan 14, 2023
@eyalroz eyalroz self-assigned this Jan 14, 2023
eyalroz added a commit that referenced this issue Jan 14, 2023
@eyalroz
Fixes #451: Using CUDA_CB instead of CUDART_CB for callbacks
a25f4b1
eyalroz added a commit that referenced this issue Jan 14, 2023
@eyalroz
Fixes #451: Using CUDA_CB instead of CUDART_CB for callbacks
37b80aa
eyalroz added a commit that referenced this issue Jan 14, 2023
@eyalroz
Fixes #451: Using CUDA_CB instead of CUDART_CB for callbacks
974a55f
eyalroz added a commit that referenced this issue Feb 10, 2023
@eyalroz
Fixes #451: Using CUDA_CB instead of CUDART_CB for callbacks
240c5d3
eyalroz added a commit that referenced this issue Mar 9, 2023
@eyalroz
Fixes #451: Using CUDA_CB instead of CUDART_CB for callbacks
fea0431
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@eyalroz eyalroz

Labels
resolved-on-development task
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@eyalroz