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.cxx11 unit test fails when force_uvm and enable_lambda are enabled #1850

Closed
ndellingwood opened this issue Oct 16, 2018 · 4 comments
Closed

cuda.cxx11 unit test fails when force_uvm and enable_lambda are enabled #1850

ndellingwood opened this issue Oct 16, 2018 · 4 comments
Assignees
@crtrott
Labels
Bug Broken / incorrect code; it could be Kokkos' responsibility, or others’ (e.g., Trilinos)
Milestone
2018 September

Comments

@ndellingwood
Copy link
Contributor

The following test failure was exposed during Trilinos integration testing:

[ RUN      ] cuda.cxx11
terminate called after throwing an instance of 'std::runtime_error'
  what():  cudaDeviceSynchronize() error( cudaErrorIllegalAddress): an illegal memory access was encountered /ascldap/users/ndellin/kokkos/core/src/Cuda/Kokkos_Cuda_Impl.cpp:119
Traceback functionality not available
@ndellingwood ndellingwood added Bug Broken / incorrect code; it could be Kokkos' responsibility, or others’ (e.g., Trilinos) Blocks Promotion Overview issue for release-blocking bugs labels Oct 16, 2018
@ndellingwood ndellingwood added this to the 2018 September milestone Oct 16, 2018
@ndellingwood
Copy link
Contributor Author

@crtrott has fix

@ndellingwood
Copy link
Contributor Author

Updated the uvm Jenkins job to also enable lambda to ensure nightly coverage.

@ndellingwood
Copy link
Contributor Author

This is a bug in the unit test, not the actual code. Exposed after parallel_reduce enhanced to support device views.

@ndellingwood ndellingwood mentioned this issue Oct 16, 2018
Closed
@ndellingwood
Copy link
Contributor Author

ndellingwood commented Oct 16, 2018 •
edited

PR #1851 issued by @crtrott

crtrott added a commit that referenced this issue Oct 16, 2018
@crtrott
Merge pull request #1851 from kokkos/issue-1850
a76b5c1 
Fix issue #1850 failing test with UVM default
@ndellingwood ndellingwood added bug - fix pushed to develop branch and removed Blocks Promotion Overview issue for release-blocking bugs labels Oct 16, 2018
@crtrott crtrott closed this as completed Nov 5, 2018
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
2018 September
Development

No branches or pull requests
2 participants
@crtrott @ndellingwood