[lng-odp] [RFC v2 4/4] barrier: allow platform to override odp_barrier_t

2015-03-17 Thread Jerin Jacob
Signed-off-by: Jerin Jacob jerin.ja...@caviumnetworks.com
---
 include/odp/api/barrier.h  |  7 +---
 platform/linux-generic/include/odp/barrier.h   |  1 +
 .../linux-generic/include/odp/plat/barrier_types.h | 47 ++
 3 files changed, 49 insertions(+), 6 deletions(-)
 create mode 100644 platform/linux-generic/include/odp/plat/barrier_types.h

diff --git a/include/odp/api/barrier.h b/include/odp/api/barrier.h
index 35f0981..28310ba 100644
--- a/include/odp/api/barrier.h
+++ b/include/odp/api/barrier.h
@@ -18,20 +18,15 @@
 extern C {
 #endif
 
-
 /** @addtogroup odp_synchronizers
  *  Synchronize threads.
  *  @{
  */
 
 /**
+ * @typedef odp_barrier_t
  * ODP thread synchronization barrier
  */
-typedef struct odp_barrier_t {
-   uint32_t count;  /** @private Thread count */
-   odp_atomic_u32_t bar;/** @private Barrier counter */
-} odp_barrier_t;
-
 
 /**
  * Initialize barrier with thread count.
diff --git a/platform/linux-generic/include/odp/barrier.h 
b/platform/linux-generic/include/odp/barrier.h
index d63c219..7ea5a6b 100644
--- a/platform/linux-generic/include/odp/barrier.h
+++ b/platform/linux-generic/include/odp/barrier.h
@@ -19,6 +19,7 @@ extern C {
 
 #include odp/atomic.h
 #include odp/plat/shared_memory_types.h
+#include odp/plat/barrier_types.h
 
 /** @ingroup odp_synchronizers
  *  @{
diff --git a/platform/linux-generic/include/odp/plat/barrier_types.h 
b/platform/linux-generic/include/odp/plat/barrier_types.h
new file mode 100644
index 000..c8c978d
--- /dev/null
+++ b/platform/linux-generic/include/odp/plat/barrier_types.h
@@ -0,0 +1,47 @@
+/* Copyright (c) 2015, Linaro Limited
+ * All rights reserved.
+ *
+ * SPDX-License-Identifier: BSD-3-Clause
+ */
+
+
+/**
+ * @file
+ *
+ * ODP barrier
+ */
+
+#ifndef ODP_BARRIER_TYPES_H_
+#define ODP_BARRIER_TYPES_H_
+
+#ifdef __cplusplus
+extern C {
+#endif
+
+#include odp/std_types.h
+#include odp/atomic.h
+
+/**
+ * @internal
+ * ODP thread synchronization barrier
+ */
+struct odp_barrier_s {
+   uint32_t count;  /** Thread count */
+   odp_atomic_u32_t bar;/** Barrier counter */
+};
+
+/** @addtogroup odp_synchronizers
+ *  @{
+ */
+
+typedef struct odp_barrier_s odp_barrier_t;
+
+/**
+ * @}
+ */
+
+#ifdef __cplusplus
+}
+#endif
+
+#endif
-- 
2.1.0


___
lng-odp mailing list
lng-odp@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/lng-odp


Re: [lng-odp] [RFC v2 4/4] barrier: allow platform to override odp_barrier_t

2015-03-17 Thread Bill Fischofer
This part fails make distcheck with error:

  CC   odp_barrier.lo
In file included from ../../../platform/linux-generic/odp_barrier.c:7:0:
../../../platform/linux-generic/include/odp/barrier.h:22:36: fatal error:
odp/plat/barrier_types.h: No such file or directory
 #include odp/plat/barrier_types.h
^
compilation terminated.
Makefile:665: recipe for target 'odp_barrier.lo' failed

On Tue, Mar 17, 2015 at 5:11 AM, Jerin Jacob jerin.ja...@caviumnetworks.com
 wrote:

 Signed-off-by: Jerin Jacob jerin.ja...@caviumnetworks.com
 ---
  include/odp/api/barrier.h  |  7 +---
  platform/linux-generic/include/odp/barrier.h   |  1 +
  .../linux-generic/include/odp/plat/barrier_types.h | 47
 ++
  3 files changed, 49 insertions(+), 6 deletions(-)
  create mode 100644 platform/linux-generic/include/odp/plat/barrier_types.h

 diff --git a/include/odp/api/barrier.h b/include/odp/api/barrier.h
 index 35f0981..28310ba 100644
 --- a/include/odp/api/barrier.h
 +++ b/include/odp/api/barrier.h
 @@ -18,20 +18,15 @@
  extern C {
  #endif

 -
  /** @addtogroup odp_synchronizers
   *  Synchronize threads.
   *  @{
   */

  /**
 + * @typedef odp_barrier_t
   * ODP thread synchronization barrier
   */
 -typedef struct odp_barrier_t {
 -   uint32_t count;  /** @private Thread count */
 -   odp_atomic_u32_t bar;/** @private Barrier counter */
 -} odp_barrier_t;
 -

  /**
   * Initialize barrier with thread count.
 diff --git a/platform/linux-generic/include/odp/barrier.h
 b/platform/linux-generic/include/odp/barrier.h
 index d63c219..7ea5a6b 100644
 --- a/platform/linux-generic/include/odp/barrier.h
 +++ b/platform/linux-generic/include/odp/barrier.h
 @@ -19,6 +19,7 @@ extern C {

  #include odp/atomic.h
  #include odp/plat/shared_memory_types.h
 +#include odp/plat/barrier_types.h

  /** @ingroup odp_synchronizers
   *  @{
 diff --git a/platform/linux-generic/include/odp/plat/barrier_types.h
 b/platform/linux-generic/include/odp/plat/barrier_types.h
 new file mode 100644
 index 000..c8c978d
 --- /dev/null
 +++ b/platform/linux-generic/include/odp/plat/barrier_types.h
 @@ -0,0 +1,47 @@
 +/* Copyright (c) 2015, Linaro Limited
 + * All rights reserved.
 + *
 + * SPDX-License-Identifier: BSD-3-Clause
 + */
 +
 +
 +/**
 + * @file
 + *
 + * ODP barrier
 + */
 +
 +#ifndef ODP_BARRIER_TYPES_H_
 +#define ODP_BARRIER_TYPES_H_
 +
 +#ifdef __cplusplus
 +extern C {
 +#endif
 +
 +#include odp/std_types.h
 +#include odp/atomic.h
 +
 +/**
 + * @internal
 + * ODP thread synchronization barrier
 + */
 +struct odp_barrier_s {
 +   uint32_t count;  /** Thread count */
 +   odp_atomic_u32_t bar;/** Barrier counter */
 +};
 +
 +/** @addtogroup odp_synchronizers
 + *  @{
 + */
 +
 +typedef struct odp_barrier_s odp_barrier_t;
 +
 +/**
 + * @}
 + */
 +
 +#ifdef __cplusplus
 +}
 +#endif
 +
 +#endif
 --
 2.1.0


 ___
 lng-odp mailing list
 lng-odp@lists.linaro.org
 http://lists.linaro.org/mailman/listinfo/lng-odp

___
lng-odp mailing list
lng-odp@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/lng-odp