Eliminate redundant GPU force reduction event dependency
authorSzilárd Páll <pall.szilard@gmail.com>
Wed, 26 May 2021 14:06:24 +0000 (14:06 +0000)
committerAndrey Alekseenko <al42and@gmail.com>
Wed, 26 May 2021 14:06:24 +0000 (14:06 +0000)
commitdb07eb0d880fee6c827418bd5d87cba1c288d5b0
tree498b10363833b38d8e8a53c951780d08c7407c91
parent4e410c60ce7755b1f7fe92010151f2a3863f5790
Eliminate redundant GPU force reduction event dependency

CPU force transfers happen on the same strem as GPU reduction, so no explicit dependency is needed.

Additionally, due to a StatePropagatorDataGpu bug, whenever the force readiness event queried is not AtomLocality::All, an incorrect event is returned leading to circular dependency on force reduction, as described in #4032.

This change however does not fix the StatePropagatorDataGpu bug, but it should help avoiding workarounds for the new SYCL backend (#3932).

Refs #4032 #3988
src/gromacs/mdlib/sim_util.cpp