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

cuda.scatterview unit test causes "Bus error" when force_uvm and enable_lambda are enabled #1852

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

Comments

@ndellingwood
Copy link
Contributor

Tested with SHAs 21e5f5b and a76b5c1

Tested on White on the Kepler queue.

Sample error output:

[ RUN      ] cuda.scatterview
Bus error (core dumped)
bash-4.2$ 

generate_makefile reproducer:
../../generate_makefile.bash --with-cuda --with-serial --compiler=${HOME}/kokkos/bin/nvcc_wrapper --with-cuda-options="force_uvm,enable_lambda" --arch="Power8,Kepler37"

@ndellingwood ndellingwood added Bug Broken / incorrect code; it could be Kokkos' responsibility, or others’ (e.g., Trilinos) Question For Kokkos internal and external contributors and users labels Oct 16, 2018
@ndellingwood
Copy link
Contributor Author

Running the scatterview test with uvm and lambda require setting CUDA_LAUNCH_BLOCKING=1 and CUDA_MANAGED_FORCE_DEVICE_ALLOC=1 allows the test to pass.
@ibaned @crtrott are these environment variables supposed to be required, or does needing these to make the test pass an indication of an issue?

@ndellingwood ndellingwood removed the Bug Broken / incorrect code; it could be Kokkos' responsibility, or others’ (e.g., Trilinos) label Oct 16, 2018
@crtrott
Copy link
Member

crtrott commented Oct 16, 2018

This is a bug basically there is a missing Kokkos::fence()

@crtrott crtrott added Bug Broken / incorrect code; it could be Kokkos' responsibility, or others’ (e.g., Trilinos) and removed Question For Kokkos internal and external contributors and users labels Oct 16, 2018
@crtrott
Copy link
Member

crtrott commented Oct 16, 2018

Bug is in the test not in the code.

@crtrott
Copy link
Member

crtrott commented Oct 17, 2018

PR #1854

@ndellingwood
Copy link
Contributor Author

See PR #1854.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Broken / incorrect code; it could be Kokkos' responsibility, or others’ (e.g., Trilinos)
Projects
None yet
Development

No branches or pull requests

2 participants