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

Cuda UVM Allocation test broken with UVM as default space #586

Closed
crtrott opened this issue Dec 16, 2016 · 0 comments
Closed

Cuda UVM Allocation test broken with UVM as default space #586

crtrott opened this issue Dec 16, 2016 · 0 comments
Assignees
@crtrott
Labels
Bug Broken / incorrect code; it could be Kokkos' responsibility, or others’ (e.g., Trilinos)
Milestone
END 2016

Comments

@crtrott
Copy link
Member

crtrott commented Dec 16, 2016

When using UVM as default space there are more UVM allocations going on in the inner parts of Kokkos. Those need to be accounted for when expecting a throw.
@crtrott crtrott added the Bug Broken / incorrect code; it could be Kokkos' responsibility, or others’ (e.g., Trilinos) label Dec 16, 2016
@crtrott crtrott self-assigned this Dec 16, 2016
@crtrott crtrott added this to the END 2016 milestone Dec 16, 2016
crtrott added a commit that referenced this issue Dec 16, 2016
@crtrott
Fix bug for UVM test
53b504c 
addresses #586
@crtrott crtrott closed this as completed Dec 16, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@crtrott crtrott

Labels
Bug Broken / incorrect code; it could be Kokkos' responsibility, or others’ (e.g., Trilinos)
Projects
None yet
Milestone
END 2016
Development

No branches or pull requests
1 participant
@crtrott