Remove superfluous CkQdMsg parameters in Node 27/4427/2
authorRonak Buch <rabuch2@illinois.edu>
Wed, 1 Aug 2018 19:50:12 +0000 (14:50 -0500)
committerRonak Buch <rabuch2@illinois.edu>
Wed, 1 Aug 2018 21:22:21 +0000 (16:22 -0500)
Change-Id: Ida651555cb13f939609535d091f5a88baa288ae3

src/BackEnd.C
src/Node.C
src/Node.ci
src/Node.h

index de8e892..b834ac8 100644 (file)
@@ -214,7 +214,7 @@ void master_init(int argc, char **argv){
   msg->group = group;
   CkGroupID node = CProxy_Node::ckNew(msg);
  
-  CkStartQD(CkCallback(CkIndex_Node::exitScheduler((CkQdMsg*)0), CkMyPe(), node));
+  CkStartQD(CkCallback(CkIndex_Node::exitScheduler(), CkMyPe(), node));
   CsdScheduler(-1);
 }
 
index 57ffe3c..9c79527 100644 (file)
@@ -401,8 +401,7 @@ void Node::mallocTest(int step) {
   memset(foo,0,size*MB*sizeof(char));
 }
 
-void Node::mallocTestQd(CkQdMsg *qmsg) {
-  delete qmsg;
+void Node::mallocTestQd() {
   if ( mallocTest_size ) {
     CkPrintf("All PEs successfully allocated %d MB.\n", 100*mallocTest_size);
   } else {
@@ -410,7 +409,7 @@ void Node::mallocTestQd(CkQdMsg *qmsg) {
   }
   fflush(stdout);
   ++mallocTest_size;
-  CkStartQD(CkIndex_Node::mallocTestQd((CkQdMsg*)0),&thishandle);
+  CkStartQD(CkIndex_Node::mallocTestQd(), &thishandle);
   (CProxy_Node(CkpvAccess(BOCclass_group).node)).mallocTest(mallocTest_size);
 }
 
@@ -421,11 +420,6 @@ void Node::messageStartUp() {
   (CProxy_Node(CkpvAccess(BOCclass_group).node)).startup();
 }
 
-void Node::startUp(CkQdMsg *qmsg) {
-  delete qmsg;
-  (CProxy_Node(CkpvAccess(BOCclass_group).node)).startup();
-}
-
 SimParameters *node_simParameters;
 Parameters *node_parameters;
 Molecule *node_molecule;
@@ -488,7 +482,7 @@ void Node::startup() {
     if ( simParameters->mallocTest ) {
       if (!CkMyPe()) {
         mallocTest_size = 0;
-        CkStartQD(CkIndex_Node::mallocTestQd((CkQdMsg*)0),&thishandle);
+        CkStartQD(CkIndex_Node::mallocTestQd(), &thishandle);
       }
       return;
     }
@@ -932,7 +926,7 @@ void Node::startup() {
   startupPhase++;
   if (!CkMyPe()) {
     if (!gotoRun) {
-      CkStartQD(CkIndex_Node::startUp((CkQdMsg*)0),&thishandle);
+      CkStartQD(CkCallback(CkIndex_Node::startup(), thisgroup));
     } else {
       Node::messageRun();
     }
@@ -1157,11 +1151,10 @@ void Node::run()
 //-----------------------------------------------------------------------
 
 void Node::enableScriptBarrier() {
-  CkStartQD(CkIndex_Node::scriptBarrier((CkQdMsg*)0),&thishandle);
+  CkStartQD(CkIndex_Node::scriptBarrier(), &thishandle);
 }
 
-void Node::scriptBarrier(CkQdMsg *qmsg) {
-  delete qmsg;
+void Node::scriptBarrier() {
   //script->awaken();
 }
 
@@ -1351,14 +1344,12 @@ void Node::recvCheckpointAck(CheckpointMsg *msg) {
 
 void Node::sendEnableExitScheduler(void) {
   //CmiPrintf("sendEnableExitScheduler\n");
-  CkQdMsg *msg = new CkQdMsg;
   CProxy_Node nodeProxy(thisgroup);
-  nodeProxy[0].recvEnableExitScheduler(msg);
+  nodeProxy[0].recvEnableExitScheduler();
 }
 
-void Node::recvEnableExitScheduler(CkQdMsg *msg) {
+void Node::recvEnableExitScheduler(void) {
   //CmiPrintf("recvEnableExitScheduler\n");
-  delete msg;
   enableExitScheduler();
 }
 
@@ -1366,24 +1357,21 @@ void Node::enableExitScheduler(void) {
   if ( CkMyPe() ) {
     sendEnableExitScheduler();
   } else {
-    CkStartQD(CkIndex_Node::exitScheduler((CkQdMsg*)0),&thishandle);
+    CkStartQD(CkIndex_Node::exitScheduler(), &thishandle);
   }
 }
 
-void Node::exitScheduler(CkQdMsg *msg) {
+void Node::exitScheduler(void) {
   //CmiPrintf("exitScheduler %d\n",CkMyPe());
   CsdExitScheduler();
-  delete msg;
 }
 
 void Node::sendEnableEarlyExit(void) {
-  CkQdMsg *msg = new CkQdMsg;
   CProxy_Node nodeProxy(thisgroup);
-  nodeProxy[0].recvEnableEarlyExit(msg);
+  nodeProxy[0].recvEnableEarlyExit();
 }
 
-void Node::recvEnableEarlyExit(CkQdMsg *msg) {
-  delete msg;
+void Node::recvEnableEarlyExit() {
   enableEarlyExit();
 }
 
@@ -1391,15 +1379,14 @@ void Node::enableEarlyExit(void) {
   if ( CkMyPe() ) {
     sendEnableEarlyExit();
   } else {
-    CkStartQD(CkIndex_Node::earlyExit((CkQdMsg*)0),&thishandle);
+    CkStartQD(CkIndex_Node::earlyExit(),&thishandle);
   }
 }
 
-void Node::earlyExit(CkQdMsg *msg) {
+void Node::earlyExit(void) {
   iout << iERROR << "Exiting prematurely; see error messages above.\n" << endi;
   if ( CmiNumPartitions() > 1 ) NAMD_quit("Exiting prematurely; see error messages above.");
   BackEnd::exit();
-  delete msg;
 }
 
 
index 3da13a8..5522aba 100644 (file)
@@ -20,17 +20,16 @@ module Node {
   group Node
   {
     entry Node(GroupInitMsg *);
-    entry void startUp(CkQdMsg *);
     entry void startup(void);
 
     entry void mallocTest(int);
-    entry void mallocTestQd(CkQdMsg *);
+    entry void mallocTestQd(void);
 
     // after startup barriers - run simulation
     entry void run(void);
 
     // used to change parameters in mid-run
-    entry void scriptBarrier(CkQdMsg *);
+    entry void scriptBarrier(void);
     entry void scriptParam(ScriptParamMsg *);
 
     // used to change Molecule fields in mid-run
@@ -47,12 +46,12 @@ module Node {
     entry void recvCheckpointAck(CheckpointMsg*);
 
     // exit scheduler to return to front end
-    entry void recvEnableExitScheduler(CkQdMsg *);
-    entry void exitScheduler(CkQdMsg *);
+    entry void recvEnableExitScheduler(void);
+    entry void exitScheduler(void);
 
     // clean early exit after quiescence
-    entry void recvEnableEarlyExit(CkQdMsg *);
-    entry void earlyExit(CkQdMsg *);
+    entry void recvEnableEarlyExit(void);
+    entry void earlyExit(void);
 
     entry void startHPM();
     entry void stopHPM();
index 8b969b9..f254e1f 100644 (file)
@@ -91,7 +91,7 @@ public:
 
   // Change parameters in mid-run
   void enableScriptBarrier();  
-  void scriptBarrier(CkQdMsg *);  
+  void scriptBarrier(void);
   void scriptParam(ScriptParamMsg *);
 
   void reloadCharges(const char *filename);
@@ -111,22 +111,21 @@ public:
   void recvCheckpointAck(CheckpointMsg*);
   
   void sendEnableExitScheduler(void);
-  void recvEnableExitScheduler(CkQdMsg *);
+  void recvEnableExitScheduler(void);
   void enableExitScheduler(void);
-  void exitScheduler(CkQdMsg *);
+  void exitScheduler(void);
 
   void sendEnableEarlyExit(void);
-  void recvEnableEarlyExit(CkQdMsg *);
+  void recvEnableEarlyExit(void);
   void enableEarlyExit(void);
-  void earlyExit(CkQdMsg *);
+  void earlyExit(void);
 
   // Charm Entry point - Read in system data, get all ready to simulate
   static void messageStartUp();
-  void startup();  
-  void startUp(CkQdMsg *);
+  void startup();
 
   void mallocTest(int);
-  void mallocTestQd(CkQdMsg *);
+  void mallocTestQd(void);
   int mallocTest_size;
   
 #ifdef MEM_OPT_VERSION