Reorder citest script for module start/complete.

    Changed the citest script to read g_ModulesCompleted before
    g_ModulesStarted.  Otherwise, g_ModulesStarted value can
    be stale relative to the g_ModulesCompleted, so they appear
    to have the same value which is the indication that the test
    cases have completed.  Instead read g_ModulesCompleted first,
    which will be smaller than g_ModulesStarted unless we really
    are complete.

Change-Id: Ifeaf7cefbcfe481fab41804308af0e7c5dccf293
Reviewed-on: http://gfw160.austin.ibm.com:8080/gerrit/898
Reviewed-by: ADAM R. MUHLE <armuhle@us.ibm.com>
Tested-by: Jenkins Server
Reviewed-by: Mark W. Wenning <wenning@us.ibm.com>
Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>

1 files changed, 2 insertions, 2 deletions

diff --git a/src/build/citest/autocitest b/src/build/citest/autocitest
index 9d952bbc3..bd41cfccd 100755
--- a/src/build/citest/autocitest
+++ b/src/build/citest/autocitest
@@ -326,10 +326,10 @@ loopcount=0
 while [ $(($modsstarted)) -lt 1 -o $(($modsstarted)) -ne $(($modscompleted)) ]; do
     sleep 5
     ((loopcount++))     # increment loopcount
+        autosim $NOWIN --simcmd   "phys_mem.read 0x$mods_completed_addr 0x08"     1> $SBXHOME/modscompleted.log  2> /dev/null
+    modscompleted=`cat $SBXHOME/modscompleted.log | xargs echo -n`
     autosim $NOWIN --simcmd   "phys_mem.read 0x$mods_started_addr 0x08"     1> $SBXHOME/modsstarted.log  2> /dev/null
     modsstarted=`cat $SBXHOME/modsstarted.log | xargs echo -n`
-    autosim $NOWIN --simcmd   "phys_mem.read 0x$mods_completed_addr 0x08"     1> $SBXHOME/modscompleted.log  2> /dev/null
-    modscompleted=`cat $SBXHOME/modscompleted.log | xargs echo -n`
 
     echo "ModulesStarted:ModulesCompleted  => $modsstarted:$modscompleted"