Merge pull request #2284 from DARMA-tasking/2283-cleanup-performance-…

…tests #2283: Cleanup performance tests
DARMA-tasking · May 14, 2024 · 1570dcf · 1570dcf
2 parents b16eb7a + ebd5ea6
commit 1570dcf
Show file tree

Hide file tree

Showing 6 changed files with 26 additions and 26 deletions.
diff --git a/tests/perf/collection_local_send.cc b/tests/perf/collection_local_send.cc
@@ -50,7 +50,9 @@ using namespace vt::tests::perf::common;
 
 static constexpr int num_iters = 1000000;
 
-struct MyTest : PerfTestHarness { };
+struct MyTest : PerfTestHarness {
+  MyTest() { DisableGlobalTimer(); }
+};
 struct MyMsg : vt::Message {};
 
 struct NodeObj;

diff --git a/tests/perf/make_runnable_micro.cc b/tests/perf/make_runnable_micro.cc
@@ -53,7 +53,9 @@ using namespace vt::tests::perf::common;
 
 static constexpr int num_iters = 100000;
 
-struct MyTest : PerfTestHarness { };
+struct MyTest : PerfTestHarness {
+  MyTest() { DisableGlobalTimer(); }
+};
 struct MyMsg : vt::Message {};
 
 struct NodeObj;

diff --git a/tests/perf/objgroup_local_send.cc b/tests/perf/objgroup_local_send.cc
@@ -50,7 +50,9 @@ using namespace vt::tests::perf::common;
 
 static constexpr int num_iters = 1000000;
 
-struct MyTest : PerfTestHarness { };
+struct MyTest : PerfTestHarness {
+  MyTest() { DisableGlobalTimer(); }
+};
 struct MyMsg : vt::Message {};
 
 struct NodeObj;
@@ -82,9 +84,9 @@ struct NodeObj {
 
     theTerm()->disableTD();
 
-    test_obj_->StartTimer(fmt::format("colSend {}", num_iters));
+    test_obj_->StartTimer(fmt::format("ObjGroup send {}", num_iters));
     perfRunBenchmark();
-    test_obj_->StopTimer(fmt::format("colSend {}", num_iters));
+    test_obj_->StopTimer(fmt::format("ObjGroup send {}", num_iters));
 
     theTerm()->enableTD();
   }

diff --git a/tests/perf/ping_pong.cc b/tests/perf/ping_pong.cc
@@ -60,7 +60,9 @@ static constexpr NodeType const pong_node = 1;
 
 vt::EpochType the_epoch = vt::no_epoch;
 
-struct MyTest : PerfTestHarness { };
+struct MyTest : PerfTestHarness {
+  MyTest() { DisableGlobalTimer(); }
+};
 
 struct NodeObj {
   template <int64_t num_bytes>
@@ -99,11 +101,7 @@ struct NodeObj {
       constexpr auto new_num_bytes = num_bytes * 2;
       test_obj_->StartTimer(fmt::format("{} Bytes", new_num_bytes));
 
-      proxy_[pong_node]
-        .send<
-          NodeObj::PingMsg<new_num_bytes>, &NodeObj::pingPong<new_num_bytes>
-        >();
-    } else {
+      proxy_[pong_node].send<&NodeObj::pingPong<new_num_bytes>>();
     }
   }
 
@@ -115,19 +113,12 @@ struct NodeObj {
       // End of iteration for node 1
       addPerfStats(num_bytes);
 
-      proxy_[0]
-        .send<
-          NodeObj::FinishedPingMsg<num_bytes>,
-          &NodeObj::finishedPing<num_bytes>>(num_bytes);
+      proxy_[0].send<&NodeObj::finishedPing<num_bytes>>(num_bytes);
     } else {
       NodeType const next =
         theContext()->getNode() == ping_node ? pong_node : ping_node;
 
-      auto msg = vt::makeMessage<NodeObj::PingMsg<num_bytes>>(cnt + 1);
-      proxy_[next]
-        .sendMsg<NodeObj::PingMsg<num_bytes>, &NodeObj::pingPong<num_bytes>>(
-          msg
-        );
+      proxy_[next].send<&NodeObj::pingPong<num_bytes>>(cnt + 1);
     }
   }
 
@@ -155,8 +146,7 @@ VT_PERF_TEST(MyTest, test_ping_pong) {
   StartTimer(fmt::format("{} Bytes", min_bytes));
 
   if (my_node_ == 0) {
-    grp_proxy[pong_node]
-      .send<NodeObj::PingMsg<min_bytes>, &NodeObj::pingPong<min_bytes>>();
+    grp_proxy[pong_node].send<&NodeObj::pingPong<min_bytes>>();
   }
 }
 

diff --git a/tests/perf/ping_pong_am.cc b/tests/perf/ping_pong_am.cc
@@ -54,7 +54,9 @@ using namespace vt::tests::perf::common;
 static constexpr int num_iters = 10000;
 static int i = 0;
 
-struct MyTest : PerfTestHarness { };
+struct MyTest : PerfTestHarness {
+  MyTest() { DisableGlobalTimer(); }
+};
 
 struct MyMsg : vt::Message {};
 
@@ -79,14 +81,14 @@ struct NodeObj {
   }
 
   void complete() {
-    test_obj_->StopTimer(fmt::format("{} ping-pong", i));
+    test_obj_->StopTimer("ping-pong");
     if (theContext()->getNode() == 0) {
       theTerm()->enableTD();
     }
   }
 
   void perfPingPong(MyMsg* in_msg) {
-    test_obj_->StartTimer(fmt::format("{} ping-pong", i));
+    test_obj_->StartTimer("ping-pong");
     auto msg = makeMessage<MyMsg>();
     theMsg()->sendMsg<&handler>(1, msg);
   }

diff --git a/tests/perf/reduce.cc b/tests/perf/reduce.cc
@@ -52,7 +52,9 @@ using namespace vt::tests::perf::common;
 
 static constexpr int num_iters = 100;
 
-struct MyTest : PerfTestHarness { };
+struct MyTest : PerfTestHarness {
+  MyTest() { DisableGlobalTimer(); }
+};
 
 struct NodeObj {
   explicit NodeObj(MyTest* test_obj) : test_obj_(test_obj) { }