TEST/IB: Re-enable ODP tests, and flush sender iface to fix err syndrome

The test used to fail because we didn't wait for all sends to complete before releasing the buffers.
openucx · Oct 25, 2018 · f64dcc9 · f64dcc9
1 parent fee17d5
commit f64dcc9
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 6 deletions.
diff --git a/test/gtest/uct/ib/test_ib_xfer.cc b/test/gtest/uct/ib/test_ib_xfer.cc
@@ -48,9 +48,6 @@ class uct_p2p_rma_test_alloc_methods : public uct_p2p_rma_test {
 UCS_TEST_P(uct_p2p_rma_test_alloc_methods, xfer_reg_odp,
            "REG_METHODS=odp,direct")
 {
-#if defined(__powerpc64__) || defined(__aarch64__)
-    UCS_TEST_SKIP_R("odp on ppc64/aarch64");
-#endif
     test_put_zcopy();
     test_get_zcopy();
 }
@@ -77,9 +74,6 @@ class uct_p2p_mix_test_alloc_methods : public uct_p2p_mix_test {};
 UCS_TEST_P(uct_p2p_mix_test_alloc_methods, mix1000_odp,
            "REG_METHODS=odp,direct")
 {
-#if defined(__powerpc64__) || defined(__aarch64__)
-    UCS_TEST_SKIP_R("odp on ppc64");
-#endif
     run(1000);
 }
 

diff --git a/test/gtest/uct/test_p2p_mix.cc b/test/gtest/uct/test_p2p_mix.cc
@@ -155,6 +155,8 @@ void uct_p2p_mix_test::run(unsigned count) {
     for (unsigned i = 0; i < count; ++i) {
         random_op(sendbuf, recvbuf);
     }
+
+    sender().flush();
 }
 
 void uct_p2p_mix_test::init() {

diff --git a/test/gtest/uct/uct_p2p_test.cc b/test/gtest/uct/uct_p2p_test.cc
@@ -166,6 +166,7 @@ void uct_p2p_test::test_xfer_multi(send_func_t send, size_t min_length,
                                  (uct_memory_type_t) mem_type);
     }
 }
+
 void uct_p2p_test::test_xfer_multi_mem_type(send_func_t send, size_t min_length,
                                             size_t max_length, unsigned flags,
                                             uct_memory_type_t mem_type) {