Initialize rollingPruningNumParts in init_plist
authorAndrey Alekseenko <al42and@gmail.com>
Wed, 3 Feb 2021 17:15:31 +0000 (17:15 +0000)
committerMark Abraham <mark.j.abraham@gmail.com>
Wed, 3 Feb 2021 17:15:31 +0000 (17:15 +0000)
commit7b33a6c67d8b32e9fe20aaaa814cca339282c768
tree4cec6482a32e5414da0884c4e785df3b1569a3b9
parente071d94ff496acb76d0df11dc6a72e2b385453b8
Initialize rollingPruningNumParts in init_plist

Previously, rollingPruningNumParts and rollingPruningPart were not
initialized, which, in the case of an empty domain, caused access
to uninitialized memory. No particular harm was being done, but it
could trigger an assertion in gpu_launch_kernel_pruneonly during
some of the regression tests.

The whole code in the preamble of gpu_launch_kernel_pruneonly is
probably due for refactoring, but since we initialize all other
plist's fields in init_plist, we better initialize these two as well.
cmake/gmxTestCompilerProblems.cmake
src/gromacs/nbnxm/gpu_types_common.h
src/gromacs/nbnxm/nbnxm_gpu_data_mgmt.cpp