Kokkos::deep_copy must fence even if the two views are the same #1303

crtrott · 2017-12-22T21:05:53Z

Otherwise behavior changes and UVM runs may bus error out on non-nvlink systems.

Adding multiple fences which are necessary if UVM memory is used. This shoudl fix #1303 and #1302

crtrott self-assigned this Dec 22, 2017

crtrott added Bug Broken / incorrect code; it could be Kokkos' responsibility, or others’ (e.g., Trilinos) DevelopOnly labels Dec 22, 2017

crtrott added a commit that referenced this issue Dec 22, 2017

Fix some CUDA UVM issues due to new deep_copy impl

ac07b3c

Adding multiple fences which are necessary if UVM memory is used. This shoudl fix #1303 and #1302

crtrott added a commit that referenced this issue Dec 22, 2017

Fix some CUDA UVM issues due to new deep_copy impl

1f4dc10

Adding multiple fences which are necessary if UVM memory is used. This shoudl fix #1303 and #1302

crtrott added bug - fix pushed to develop branch labels Dec 23, 2017

ibaned added this to the 2018 February milestone Jan 3, 2018

ndellingwood closed this as completed Mar 7, 2018

ndellingwood mentioned this issue Mar 8, 2018

Kokkos + KokkosKernels Promotion To Version 2.6.00 trilinos/Trilinos#2351

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Kokkos::deep_copy must fence even if the two views are the same #1303

Kokkos::deep_copy must fence even if the two views are the same #1303

crtrott commented Dec 22, 2017

Kokkos::deep_copy must fence even if the two views are the same #1303

Kokkos::deep_copy must fence even if the two views are the same #1303

Comments

crtrott commented Dec 22, 2017