[Mesa-dev,09/10] gallium, clover: add OpenCL interoperability support for CL events

Submitted by Marek Olšák on April 27, 2015, 12:19 a.m.

Details

Message ID 1430093959-28988-1-git-send-email-maraeo@gmail.com
State New
Headers show

Not browsing as part of any series.

Commit Message

Marek Olšák April 27, 2015, 12:19 a.m.
From: Marek Olšák <marek.olsak@amd.com>

v2: - move interop.cpp to clover/api
    - change intptr_t to void* in the interface
    - add a virtual function fence() to simplify some code
---
 src/gallium/include/state_tracker/opencl_interop.h | 42 +++++++++++++++
 src/gallium/state_trackers/clover/Makefile.sources |  1 +
 src/gallium/state_trackers/clover/api/interop.cpp  | 60 ++++++++++++++++++++++
 src/gallium/state_trackers/clover/core/event.hpp   |  8 +++
 src/gallium/targets/opencl/opencl.sym              |  1 +
 5 files changed, 112 insertions(+)
 create mode 100644 src/gallium/include/state_tracker/opencl_interop.h
 create mode 100644 src/gallium/state_trackers/clover/api/interop.cpp

Patch hide | download patch | download mbox

diff --git a/src/gallium/include/state_tracker/opencl_interop.h b/src/gallium/include/state_tracker/opencl_interop.h
new file mode 100644
index 0000000..0c9f34a
--- /dev/null
+++ b/src/gallium/include/state_tracker/opencl_interop.h
@@ -0,0 +1,42 @@ 
+/**************************************************************************
+ *
+ * Copyright 2015 Advanced Micro Devices, Inc.
+ * All Rights Reserved.
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining a
+ * copy of this software and associated documentation files (the
+ * "Software"), to deal in the Software without restriction, including
+ * without limitation the rights to use, copy, modify, merge, publish,
+ * distribute, sub license, and/or sell copies of the Software, and to
+ * permit persons to whom the Software is furnished to do so, subject to
+ * the following conditions:
+ *
+ * The above copyright notice and this permission notice (including the
+ * next paragraph) shall be included in all copies or substantial portions
+ * of the Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS
+ * OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
+ * MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NON-INFRINGEMENT.
+ * IN NO EVENT SHALL THE COPYRIGHT HOLDER(S) OR AUTHOR(S) BE LIABLE FOR
+ * ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT,
+ * TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE
+ * SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
+ *
+ **************************************************************************/
+
+#ifndef OPENCL_INTEROP_H
+#define OPENCL_INTEROP_H
+
+/* dlsym these without the "_t" suffix. You should get the correct symbols
+ * if the OpenCL driver is loaded.
+ *
+ * All functions return non-zero on success.
+ */
+
+typedef int (*opencl_dri_event_add_ref_t)(void *cl_event);
+typedef int (*opencl_dri_event_release_t)(void *cl_event);
+typedef int (*opencl_dri_event_wait_t)(void *cl_event, uint64_t timeout);
+typedef struct pipe_fence_handle *(*opencl_dri_event_get_fence_t)(void *cl_event);
+
+#endif /* OPENCL_INTEROP_H */
diff --git a/src/gallium/state_trackers/clover/Makefile.sources b/src/gallium/state_trackers/clover/Makefile.sources
index 5b3344c..9e30c69 100644
--- a/src/gallium/state_trackers/clover/Makefile.sources
+++ b/src/gallium/state_trackers/clover/Makefile.sources
@@ -4,6 +4,7 @@  CPP_SOURCES := \
 	api/dispatch.cpp \
 	api/dispatch.hpp \
 	api/event.cpp \
+	api/interop.cpp \
 	api/kernel.cpp \
 	api/memory.cpp \
 	api/platform.cpp \
diff --git a/src/gallium/state_trackers/clover/api/interop.cpp b/src/gallium/state_trackers/clover/api/interop.cpp
new file mode 100644
index 0000000..2bec839
--- /dev/null
+++ b/src/gallium/state_trackers/clover/api/interop.cpp
@@ -0,0 +1,60 @@ 
+//
+// Copyright 2015 Advanced Micro Devices, Inc.
+// All Rights Reserved.
+//
+// Permission is hereby granted, free of charge, to any person obtaining a
+// copy of this software and associated documentation files (the "Software"),
+// to deal in the Software without restriction, including without limitation
+// the rights to use, copy, modify, merge, publish, distribute, sublicense,
+// and/or sell copies of the Software, and to permit persons to whom the
+// Software is furnished to do so, subject to the following conditions:
+//
+// The above copyright notice and this permission notice shall be included in
+// all copies or substantial portions of the Software.
+//
+// THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+// IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+// FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
+// THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR
+// OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE,
+// ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
+// OTHER DEALINGS IN THE SOFTWARE.
+//
+
+#include "core/event.hpp"
+#include "api/util.hpp"
+
+using namespace clover;
+
+extern "C" {
+
+PUBLIC int
+opencl_dri_event_add_ref(cl_event _event)
+{
+   return clRetainEvent(_event) == CL_SUCCESS;
+}
+
+PUBLIC int
+opencl_dri_event_release(cl_event _event)
+{
+   return clReleaseEvent(_event) == CL_SUCCESS;
+}
+
+PUBLIC int
+opencl_dri_event_wait(cl_event _event, uint64_t timeout)
+{
+   if (!timeout) {
+      return obj(_event).status() == CL_COMPLETE;
+   }
+
+   obj(_event).wait();
+   return true;
+}
+
+PUBLIC struct pipe_fence_handle *
+opencl_dri_event_get_fence(cl_event _event)
+{
+   return obj(_event).fence();
+}
+
+}
diff --git a/src/gallium/state_trackers/clover/core/event.hpp b/src/gallium/state_trackers/clover/core/event.hpp
index 0e1359a..d407c80 100644
--- a/src/gallium/state_trackers/clover/core/event.hpp
+++ b/src/gallium/state_trackers/clover/core/event.hpp
@@ -70,6 +70,10 @@  namespace clover {
       virtual cl_command_type command() const = 0;
       virtual void wait() const = 0;
 
+      virtual struct pipe_fence_handle *fence() const {
+         return NULL;
+      }
+
       const intrusive_ref<clover::context> context;
 
    protected:
@@ -116,6 +120,10 @@  namespace clover {
 
       friend class command_queue;
 
+      virtual struct pipe_fence_handle *fence() const {
+         return _fence;
+      }
+
    private:
       virtual void fence(pipe_fence_handle *fence);
       action profile(command_queue &q, const action &action) const;
diff --git a/src/gallium/targets/opencl/opencl.sym b/src/gallium/targets/opencl/opencl.sym
index ee8aacf..9fcc576 100644
--- a/src/gallium/targets/opencl/opencl.sym
+++ b/src/gallium/targets/opencl/opencl.sym
@@ -1,6 +1,7 @@ 
 {
 	global:
 		cl*;
+		opencl_dri_*;
 	local:
 		*;
 };