Fixed bug reporting initial DLB state
authorKevin Boyd <kevin.boyd@uconn.edu>
Sat, 1 Sep 2018 17:25:43 +0000 (13:25 -0400)
committerKevin Boyd <kevin.boyd@uconn.edu>
Sun, 2 Sep 2018 14:36:23 +0000 (16:36 +0200)
The issue was a size/mapping mismatch between the enum representing
DLB states and the array with the corresponding state string

fixes #2631

Change-Id: I0be35d020d91c30c9770e83e8f57560b75a71b1a

docs/release-notes/2018/2018.4.rst
src/gromacs/domdec/domdec.cpp

index dd0c3345fcd19871a9658dbf9f123da692d8335b..a2eeb1d1d245e30add7a0d72c946b65af98aae76 100644 (file)
@@ -17,3 +17,8 @@ Fixes to improve portability
 
 Miscellaneous
 ^^^^^^^^^^^^^
+
+Fixed an issue where the log file could sometimes report an incorrect
+initial dynamic load balancing state
+
+:issue:`2631`
index 913c2488de8e6237221461b2fd2bc442ade3d62c..e2ebc5f256a939175cfc1f3042de39c35d44969e 100644 (file)
@@ -130,7 +130,7 @@ struct gmx_domdec_master_t
 
 #define DD_NLOAD_MAX 9
 
-const char *edlbs_names[edlbsNR] = { "off", "auto", "locked", "on", "on" };
+static const char *edlbs_names[edlbsNR] = { "off", "off", "off", "locked", "on", "on" };
 
 /* The size per charge group of the cggl_flag buffer in gmx_domdec_comm_t */
 #define DD_CGIBS 2