Kokkos: Performance regression in DualView

Created on 4 Jan 2021  路  4Comments  路  Source: kokkos/kokkos

I'm seeing a significant performance regression in Kokkos::DualView with LAMMPS due to a bunch of new DtoH data movement. I bisected the issue to #3326, and @crtrott found the label is being pulled down for every DualView sync and modify event. Can the label only be pulled down from GPU to CPU when profiling is enabled?

picture stanmoore1

Most helpful comment

We are trying to get a fix done today.

picture crtrott on 4 Jan 2021
👍2

All 4 comments

We are trying to get a fix done today.

crtrott picture crtrott on 4 Jan 2021
👍2

Great, thanks!

stanmoore1 picture stanmoore1 on 4 Jan 2021

@stanmoore1 , yikes, sorry about that. Just pushed #3695, we're pushing to merge it into everything, starting with the release

DavidPoliakoff picture DavidPoliakoff on 4 Jan 2021
👍1

I confirm that #3701 fixes the DtoH data movement issue, thanks!

stanmoore1 picture stanmoore1 on 6 Jan 2021
Was this page helpful?
0 / 5 - 0 ratings

Related issues

ndellingwood picture ndellingwood  路  4Comments
nmhamster picture nmhamster  路  4Comments
csiefer2 picture csiefer2  路  3Comments
kyungjoo-kim picture kyungjoo-kim  路  4Comments
etphipp picture etphipp  路  5Comments