Make stepWorkload.useGpuXBufferOps flag consistent
authorSzilárd Páll <pall.szilard@gmail.com>
Thu, 4 Nov 2021 18:14:01 +0000 (18:14 +0000)
committerSzilárd Páll <pall.szilard@gmail.com>
Thu, 4 Nov 2021 18:14:01 +0000 (18:14 +0000)
commit977d0e553d447c3a373c4dee62225922364fae27
treed052dd92f5e4fd39a9fc6878d21d1ab890f227ab
parent0da8ba6fea75b268a36e22ff9150e31133f88552
Make stepWorkload.useGpuXBufferOps flag consistent

On search steps we do not use x buffer ops, so the workload flag should
correctly reflect that.

Also slightly refactored a conditional block to clarify the scope of
workload flags.

Note that as a side-effect of this change, coordinate H2D copy will be
delayed from the beginning of do_force() to just before update on search
steps when there are no force tasks that require it (i.e. without PME).
While this is not ideal for performance, the code is easier to reason
about.

Refs #3915 #3913 #4268
src/gromacs/mdlib/sim_util.cpp