[SYCL] Add a check for interop_handle::get_backend() method (intel#258)

This patch also has NFC change that removes excessive printing to std::out

Signed-off-by: Vyacheslav N Klochkov <vyacheslav.n.klochkov@intel.com>

Author: v-klochkov

SYCL/HostInteropTask/interop-task.cpp

@@ -18,6 +18,18 @@ template <typename T> class Modifier;
 
 template <typename T> class Init;
 
+template <typename BufferT, typename ValueT>
+void checkBufferValues(BufferT Buffer, ValueT Value) {
+  auto Acc = Buffer.template get_access<mode::read>();
+  for (size_t Idx = 0; Idx < Acc.get_count(); ++Idx) {
+    if (Acc[Idx] != Value) {
+      std::cerr << "buffer[" << Idx << "] = " << Acc[Idx]
+                << ", expected val = " << Value << std::endl;
+      assert(0 && "Invalid data in the buffer");
+    }
+  }
+}
+
 template <typename DataT>
 void copy(buffer<DataT, 1> &Src, buffer<DataT, 1> &Dst, queue &Q) {
   Q.submit([&](handler &CGH) {
@@ -41,6 +53,11 @@ void copy(buffer<DataT, 1> &Src, buffer<DataT, 1> &Dst, queue &Q) {
 
       if (RC != CL_SUCCESS)
         throw runtime_error("Can't wait for event on buffer copy", RC);
+
+      if (Q.get_backend() != IH.get_backend())
+        throw runtime_error(
+            "interop_handle::get_backend() returned a wrong value",
+            CL_INVALID_VALUE);
     });
   });
 }
@@ -89,22 +106,8 @@ void test1() {
     copy(Buffer2, Buffer1, Q);
   }
 
-  {
-    auto Acc = Buffer1.get_access<mode::read>();
-
-    for (size_t Idx = 0; Idx < Acc.get_count(); ++Idx) {
-      std::cout << "First buffer [" << Idx << "] = " << Acc[Idx] << std::endl;
-      assert((Acc[Idx] == COUNT - 1) && "Invalid data in the first buffer");
-    }
-  }
-  {
-    auto Acc = Buffer2.get_access<mode::read>();
-
-    for (size_t Idx = 0; Idx < Acc.get_count(); ++Idx) {
-      std::cout << "Second buffer [" << Idx << "] = " << Acc[Idx] << std::endl;
-      assert((Acc[Idx] == COUNT - 1) && "Invalid data in the second buffer");
-    }
-  }
+  checkBufferValues(Buffer1, COUNT - 1);
+  checkBufferValues(Buffer2, COUNT - 1);
 }
 
 // Same as above, but performing each command group on a separate SYCL queue
@@ -128,23 +131,8 @@ void test2() {
     modify(Buffer2, Q);
     copy(Buffer2, Buffer1, Q);
   }
-
-  {
-    auto Acc = Buffer1.get_access<mode::read>();
-
-    for (size_t Idx = 0; Idx < Acc.get_count(); ++Idx) {
-      std::cout << "First buffer [" << Idx << "] = " << Acc[Idx] << std::endl;
-      assert((Acc[Idx] == COUNT - 1) && "Invalid data in the first buffer");
-    }
-  }
-  {
-    auto Acc = Buffer2.get_access<mode::read>();
-
-    for (size_t Idx = 0; Idx < Acc.get_count(); ++Idx) {
-      std::cout << "Second buffer [" << Idx << "] = " << Acc[Idx] << std::endl;
-      assert((Acc[Idx] == COUNT - 1) && "Invalid data in the second buffer");
-    }
-  }
+  checkBufferValues(Buffer1, COUNT - 1);
+  checkBufferValues(Buffer2, COUNT - 1);
 }
 
 // Same as above but with queue constructed out of context
@@ -168,23 +156,8 @@ void test2_1() {
     modify(Buffer2, Q);
     copy(Buffer2, Buffer1, Q);
   }
-
-  {
-    auto Acc = Buffer1.get_access<mode::read>();
-
-    for (size_t Idx = 0; Idx < Acc.get_count(); ++Idx) {
-      std::cout << "First buffer [" << Idx << "] = " << Acc[Idx] << std::endl;
-      assert((Acc[Idx] == COUNT - 1) && "Invalid data in the first buffer");
-    }
-  }
-  {
-    auto Acc = Buffer2.get_access<mode::read>();
-
-    for (size_t Idx = 0; Idx < Acc.get_count(); ++Idx) {
-      std::cout << "Second buffer [" << Idx << "] = " << Acc[Idx] << std::endl;
-      assert((Acc[Idx] == COUNT - 1) && "Invalid data in the second buffer");
-    }
-  }
+  checkBufferValues(Buffer1, COUNT - 1);
+  checkBufferValues(Buffer2, COUNT - 1);
 }
 
 // A test that does a clEnqueueWait inside the interop scope, for an event
@@ -245,14 +218,7 @@ void test5() {
 
   copy(Buffer1, Buffer2, Q);
 
-  {
-    auto Acc = Buffer2.get_access<mode::read>();
-
-    for (size_t Idx = 0; Idx < Acc.get_count(); ++Idx) {
-      std::cout << "Second buffer [" << Idx << "] = " << Acc[Idx] << std::endl;
-      assert(Acc[Idx] == 123);
-    }
-  }
+  checkBufferValues(Buffer2, static_cast<int>(123));
 }
 
 // The test checks that an exception which is thrown from host_task body
@@ -292,5 +258,6 @@ int main() {
   test4();
   test5();
   test6();
+  std::cout << "Test PASSED" << std::endl;
   return 0;
 }