Deleted odph_pause() and replaced usage with odp_cpu_pause() calls. Signed-off-by: Petri Savolainen <petri.savolai...@nokia.com> --- helper/Makefile.am | 1 - helper/odph_pause.h | 54 ------------------------------------------------ helper/ring.c | 10 +++------ helper/test/Makefile.am | 2 -- helper/test/odph_pause.c | 14 ------------- 5 files changed, 3 insertions(+), 78 deletions(-) delete mode 100644 helper/odph_pause.h delete mode 100644 helper/test/odph_pause.c
diff --git a/helper/Makefile.am b/helper/Makefile.am index 876e541..a8665b2 100644 --- a/helper/Makefile.am +++ b/helper/Makefile.am @@ -24,7 +24,6 @@ helperinclude_HEADERS = \ noinst_HEADERS = \ $(srcdir)/odph_debug.h \ - $(srcdir)/odph_pause.h \ $(srcdir)/odph_hashtable.h \ $(srcdir)/odph_lineartable.h \ $(srcdir)/odph_list_internal.h diff --git a/helper/odph_pause.h b/helper/odph_pause.h deleted file mode 100644 index 5618f1f..0000000 --- a/helper/odph_pause.h +++ /dev/null @@ -1,54 +0,0 @@ -/* Copyright (c) 2015, Linaro Limited - * All rights reserved. - * - * SPDX-License-Identifier: BSD-3-Clause - */ - -#ifndef ODPH_PAUSE_H_ -#define ODPH_PAUSE_H_ - -#ifdef __cplusplus -extern "C" { -#endif - -/** - * Spin loop for helper internal use - */ -static inline void odph_pause(void) -{ -#if defined __x86_64__ || defined __i386__ - -#ifdef __SSE2__ - __asm__ __volatile__ ("pause"); -#else - __asm__ __volatile__ ("rep; nop"); -#endif - -#elif defined __arm__ - -#if __ARM_ARCH == 7 - __asm__ __volatile__ ("nop"); - __asm__ __volatile__ ("nop"); - __asm__ __volatile__ ("nop"); - __asm__ __volatile__ ("nop"); -#endif - -#elif defined __OCTEON__ - - __asm__ __volatile__ ("nop"); - __asm__ __volatile__ ("nop"); - __asm__ __volatile__ ("nop"); - __asm__ __volatile__ ("nop"); - __asm__ __volatile__ ("nop"); - __asm__ __volatile__ ("nop"); - __asm__ __volatile__ ("nop"); - __asm__ __volatile__ ("nop"); - -#endif -} - -#ifdef __cplusplus -} -#endif - -#endif diff --git a/helper/ring.c b/helper/ring.c index 3122173..6699186 100644 --- a/helper/ring.c +++ b/helper/ring.c @@ -69,15 +69,11 @@ * ***************************************************************************/ -#include <odp/shared_memory.h> -#include <odp/spinlock.h> -#include "odph_pause.h" -#include <odp/align.h> +#include <odp.h> #include <fcntl.h> #include <stdio.h> #include <string.h> #include "odph_debug.h" -#include <odp/rwlock.h> #include <odp/helper/ring.h> static TAILQ_HEAD(, odph_ring) odp_ring_list; @@ -283,7 +279,7 @@ int __odph_ring_mp_do_enqueue(odph_ring_t *r, void * const *obj_table, * we need to wait for them to complete */ while (odp_unlikely(r->prod.tail != prod_head)) - odph_pause(); + odp_cpu_pause(); /* Release our entries and the memory they refer to */ __atomic_thread_fence(__ATOMIC_RELEASE); @@ -400,7 +396,7 @@ int __odph_ring_mc_do_dequeue(odph_ring_t *r, void **obj_table, * we need to wait for them to complete */ while (odp_unlikely(r->cons.tail != cons_head)) - odph_pause(); + odp_cpu_pause(); /* Release our entries and the memory they refer to */ __atomic_thread_fence(__ATOMIC_RELEASE); diff --git a/helper/test/Makefile.am b/helper/test/Makefile.am index d6820e1..bbad2a5 100644 --- a/helper/test/Makefile.am +++ b/helper/test/Makefile.am @@ -8,7 +8,6 @@ TESTS_ENVIRONMENT += TEST_DIR=${builddir} EXECUTABLES = odp_chksum$(EXEEXT) \ odp_thread$(EXEEXT) \ odp_process$(EXEEXT)\ - odph_pause$(EXEEXT)\ odp_table$(EXEEXT) COMPILE_ONLY = @@ -29,5 +28,4 @@ dist_odp_thread_SOURCES = odp_thread.c odp_thread_LDADD = $(LIB)/libodphelper.la $(LIB)/libodp.la dist_odp_process_SOURCES = odp_process.c odp_process_LDADD = $(LIB)/libodphelper.la $(LIB)/libodp.la -odph_pause_SOURCES = odph_pause.c dist_odp_table_SOURCES = odp_table.c diff --git a/helper/test/odph_pause.c b/helper/test/odph_pause.c deleted file mode 100644 index f5f5da3..0000000 --- a/helper/test/odph_pause.c +++ /dev/null @@ -1,14 +0,0 @@ -/* Copyright (c) 2015, Linaro Limited - * All rights reserved. - * - * SPDX-License-Identifier: BSD-3-Clause - */ - -#include <test_debug.h> -#include "../odph_pause.h" - -int main(int argc TEST_UNUSED, char *argv[] TEST_UNUSED) -{ - odph_pause(); - return 0; -} -- 2.6.3 _______________________________________________ lng-odp mailing list lng-odp@lists.linaro.org https://lists.linaro.org/mailman/listinfo/lng-odp