Re: [lng-odp] [PATCH v2] validation: add odp_time test

2015-01-09 Thread Mike Holmes 
Any reviews to get this in 0.7.0 as planned ?

On 5 January 2015 at 17:27, Mike Holmes mike.hol...@linaro.org wrote:

 Add odp_time_x API tests

 Signed-off-by: Mike Holmes mike.hol...@linaro.org
 ---
  test/validation/.gitignore  |  1 +
  test/validation/Makefile.am |  5 ++--
  test/validation/odp_time.c  | 71
 +
  3 files changed, 75 insertions(+), 2 deletions(-)
  create mode 100644 test/validation/odp_time.c

 diff --git a/test/validation/.gitignore b/test/validation/.gitignore
 index d08db73..40ea12d 100644
 --- a/test/validation/.gitignore
 +++ b/test/validation/.gitignore
 @@ -8,3 +8,4 @@ odp_shm
  odp_system
  odp_pktio
  odp_buffer
 +odp_time
 diff --git a/test/validation/Makefile.am b/test/validation/Makefile.am
 index c0545b7..692e1a6 100644
 --- a/test/validation/Makefile.am
 +++ b/test/validation/Makefile.am
 @@ -6,9 +6,9 @@ AM_LDFLAGS += -static
  TESTS_ENVIRONMENT = ODP_PLATFORM=${with_platform}

  if ODP_CUNIT_ENABLED
 -TESTS = odp_init odp_queue odp_crypto odp_shm odp_schedule odp_pktio_run
 odp_buffer odp_system
 +TESTS = odp_init odp_queue odp_crypto odp_shm odp_schedule odp_pktio_run
 odp_buffer odp_system odp_time
  check_PROGRAMS = ${bin_PROGRAMS}
 -bin_PROGRAMS = odp_init odp_queue odp_crypto odp_shm odp_schedule
 odp_pktio odp_buffer odp_system
 +bin_PROGRAMS = odp_init odp_queue odp_crypto odp_shm odp_schedule
 odp_pktio odp_buffer odp_system odp_time
  odp_crypto_CFLAGS = $(AM_CFLAGS) -I$(srcdir)/crypto
  odp_buffer_CFLAGS = $(AM_CFLAGS) -I$(srcdir)/buffer
  endif
 @@ -27,3 +27,4 @@ dist_odp_buffer_SOURCES = buffer/odp_buffer_pool_test.c \
   buffer/odp_packet_test.c \
   odp_buffer.c common/odp_cunit_common.c
  dist_odp_system_SOURCES = odp_system.c common/odp_cunit_common.c
 +dist_odp_time_SOURCES = odp_time.c common/odp_cunit_common.c
 diff --git a/test/validation/odp_time.c b/test/validation/odp_time.c
 new file mode 100644
 index 000..b0f29cf
 --- /dev/null
 +++ b/test/validation/odp_time.c
 @@ -0,0 +1,71 @@
 +/* Copyright (c) 2014, Linaro Limited
 + * All rights reserved.
 + *
 + * SPDX-License-Identifier: BSD-3-Clause
 + */
 +
 +#include odp.h
 +#include odp_cunit_common.h
 +
 +#define TOLERANCE 1
 +#define BUSY_LOOP_CNT 100
 +
 +/* check that a cycles difference gives a reasonable result */
 +static void test_odp_cycles_diff(void)
 +{
 +   /* volatile to stop optimization of busy loop */
 +   volatile int count = 0;
 +   uint64_t diff, cycles1, cycles2;
 +
 +   cycles1 = odp_time_cycles();
 +
 +   while (count  BUSY_LOOP_CNT) {
 +   count++;
 +   };
 +
 +   cycles2 = odp_time_cycles();
 +   CU_ASSERT(cycles2  cycles1);
 +
 +   diff = odp_time_diff_cycles(cycles1, cycles2);
 +   CU_ASSERT(diff  0);
 +}
 +
 +/* check that a negative cycles difference gives a reasonable result */
 +static void test_odp_cycles_negative_diff(void)
 +{
 +   uint64_t diff, cycles1, cycles2;
 +   cycles1 = 10;
 +   cycles2 = 5;
 +   diff = odp_time_diff_cycles(cycles1, cycles2);
 +   CU_ASSERT(diff  0);
 +}
 +
 +/* check that related conversions come back to the same value */
 +static void test_odp_time_conversion(void)
 +{
 +   uint64_t ns1, ns2, cycles;
 +   uint64_t upper_limit, lower_limit;
 +   ns1 = 100;
 +   cycles = odp_time_ns_to_cycles(ns1);
 +   CU_ASSERT(cycles  0);
 +
 +   ns2 = odp_time_cycles_to_ns(cycles);
 +
 +   /* need to check within arithmetic tolerance that the same
 +* value in ns is returned after conversions */
 +   upper_limit = ns1 + TOLERANCE;
 +   lower_limit = ns1 - TOLERANCE;
 +   CU_ASSERT((ns2 = upper_limit)  (ns2 = lower_limit));
 +}
 +
 +CU_TestInfo test_odp_time[] = {
 +   {cycles diff, test_odp_cycles_diff},
 +   {negative diff, test_odp_cycles_negative_diff},
 +   {conversion, test_odp_time_conversion},
 +CU_TEST_INFO_NULL
 +};
 +
 +CU_SuiteInfo odp_testsuites[] = {
 +   {Time, NULL, NULL, NULL, NULL, test_odp_time},
 +CU_SUITE_INFO_NULL
 +};
 --
 2.1.0




-- 
*Mike Holmes*
Linaro  Sr Technical Manager
LNG - ODP
___
lng-odp mailing list
lng-odp@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/lng-odp

Re: [lng-odp] [PATCH v2] validation: add odp_time test

2015-01-09 Thread Mike Holmes 
Applied

On 9 January 2015 at 15:59, Bill Fischofer bill.fischo...@linaro.org
wrote:



 On Mon, Jan 5, 2015 at 4:27 PM, Mike Holmes mike.hol...@linaro.org
 wrote:

 Add odp_time_x API tests

 Signed-off-by: Mike Holmes mike.hol...@linaro.org


 Reviewed-by: Bill Fischofer bill.fischo...@linaro.org


 ---
  test/validation/.gitignore  |  1 +
  test/validation/Makefile.am |  5 ++--
  test/validation/odp_time.c  | 71
 +
  3 files changed, 75 insertions(+), 2 deletions(-)
  create mode 100644 test/validation/odp_time.c

 diff --git a/test/validation/.gitignore b/test/validation/.gitignore
 index d08db73..40ea12d 100644
 --- a/test/validation/.gitignore
 +++ b/test/validation/.gitignore
 @@ -8,3 +8,4 @@ odp_shm
  odp_system
  odp_pktio
  odp_buffer
 +odp_time
 diff --git a/test/validation/Makefile.am b/test/validation/Makefile.am
 index c0545b7..692e1a6 100644
 --- a/test/validation/Makefile.am
 +++ b/test/validation/Makefile.am
 @@ -6,9 +6,9 @@ AM_LDFLAGS += -static
  TESTS_ENVIRONMENT = ODP_PLATFORM=${with_platform}

  if ODP_CUNIT_ENABLED
 -TESTS = odp_init odp_queue odp_crypto odp_shm odp_schedule odp_pktio_run
 odp_buffer odp_system
 +TESTS = odp_init odp_queue odp_crypto odp_shm odp_schedule odp_pktio_run
 odp_buffer odp_system odp_time
  check_PROGRAMS = ${bin_PROGRAMS}
 -bin_PROGRAMS = odp_init odp_queue odp_crypto odp_shm odp_schedule
 odp_pktio odp_buffer odp_system
 +bin_PROGRAMS = odp_init odp_queue odp_crypto odp_shm odp_schedule
 odp_pktio odp_buffer odp_system odp_time
  odp_crypto_CFLAGS = $(AM_CFLAGS) -I$(srcdir)/crypto
  odp_buffer_CFLAGS = $(AM_CFLAGS) -I$(srcdir)/buffer
  endif
 @@ -27,3 +27,4 @@ dist_odp_buffer_SOURCES = buffer/odp_buffer_pool_test.c
 \
   buffer/odp_packet_test.c \
   odp_buffer.c common/odp_cunit_common.c
  dist_odp_system_SOURCES = odp_system.c common/odp_cunit_common.c
 +dist_odp_time_SOURCES = odp_time.c common/odp_cunit_common.c
 diff --git a/test/validation/odp_time.c b/test/validation/odp_time.c
 new file mode 100644
 index 000..b0f29cf
 --- /dev/null
 +++ b/test/validation/odp_time.c
 @@ -0,0 +1,71 @@
 +/* Copyright (c) 2014, Linaro Limited
 + * All rights reserved.
 + *
 + * SPDX-License-Identifier: BSD-3-Clause
 + */
 +
 +#include odp.h
 +#include odp_cunit_common.h
 +
 +#define TOLERANCE 1
 +#define BUSY_LOOP_CNT 100
 +
 +/* check that a cycles difference gives a reasonable result */
 +static void test_odp_cycles_diff(void)
 +{
 +   /* volatile to stop optimization of busy loop */
 +   volatile int count = 0;
 +   uint64_t diff, cycles1, cycles2;
 +
 +   cycles1 = odp_time_cycles();
 +
 +   while (count  BUSY_LOOP_CNT) {
 +   count++;
 +   };
 +
 +   cycles2 = odp_time_cycles();
 +   CU_ASSERT(cycles2  cycles1);
 +
 +   diff = odp_time_diff_cycles(cycles1, cycles2);
 +   CU_ASSERT(diff  0);
 +}
 +
 +/* check that a negative cycles difference gives a reasonable result */
 +static void test_odp_cycles_negative_diff(void)
 +{
 +   uint64_t diff, cycles1, cycles2;
 +   cycles1 = 10;
 +   cycles2 = 5;
 +   diff = odp_time_diff_cycles(cycles1, cycles2);
 +   CU_ASSERT(diff  0);
 +}
 +
 +/* check that related conversions come back to the same value */
 +static void test_odp_time_conversion(void)
 +{
 +   uint64_t ns1, ns2, cycles;
 +   uint64_t upper_limit, lower_limit;
 +   ns1 = 100;
 +   cycles = odp_time_ns_to_cycles(ns1);
 +   CU_ASSERT(cycles  0);
 +
 +   ns2 = odp_time_cycles_to_ns(cycles);
 +
 +   /* need to check within arithmetic tolerance that the same
 +* value in ns is returned after conversions */
 +   upper_limit = ns1 + TOLERANCE;
 +   lower_limit = ns1 - TOLERANCE;
 +   CU_ASSERT((ns2 = upper_limit)  (ns2 = lower_limit));
 +}
 +
 +CU_TestInfo test_odp_time[] = {
 +   {cycles diff, test_odp_cycles_diff},
 +   {negative diff, test_odp_cycles_negative_diff},
 +   {conversion, test_odp_time_conversion},
 +CU_TEST_INFO_NULL
 +};
 +
 +CU_SuiteInfo odp_testsuites[] = {
 +   {Time, NULL, NULL, NULL, NULL, test_odp_time},
 +CU_SUITE_INFO_NULL
 +};
 --
 2.1.0


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





-- 
*Mike Holmes*
Linaro  Sr Technical Manager
LNG - ODP
___
lng-odp mailing list
lng-odp@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/lng-odp

Re: [lng-odp] [PATCH v2] validation: add odp_time test

2015-01-09 Thread Bill Fischofer 
On Mon, Jan 5, 2015 at 4:27 PM, Mike Holmes mike.hol...@linaro.org wrote:

 Add odp_time_x API tests

 Signed-off-by: Mike Holmes mike.hol...@linaro.org


Reviewed-by: Bill Fischofer bill.fischo...@linaro.org


 ---
  test/validation/.gitignore  |  1 +
  test/validation/Makefile.am |  5 ++--
  test/validation/odp_time.c  | 71
 +
  3 files changed, 75 insertions(+), 2 deletions(-)
  create mode 100644 test/validation/odp_time.c

 diff --git a/test/validation/.gitignore b/test/validation/.gitignore
 index d08db73..40ea12d 100644
 --- a/test/validation/.gitignore
 +++ b/test/validation/.gitignore
 @@ -8,3 +8,4 @@ odp_shm
  odp_system
  odp_pktio
  odp_buffer
 +odp_time
 diff --git a/test/validation/Makefile.am b/test/validation/Makefile.am
 index c0545b7..692e1a6 100644
 --- a/test/validation/Makefile.am
 +++ b/test/validation/Makefile.am
 @@ -6,9 +6,9 @@ AM_LDFLAGS += -static
  TESTS_ENVIRONMENT = ODP_PLATFORM=${with_platform}

  if ODP_CUNIT_ENABLED
 -TESTS = odp_init odp_queue odp_crypto odp_shm odp_schedule odp_pktio_run
 odp_buffer odp_system
 +TESTS = odp_init odp_queue odp_crypto odp_shm odp_schedule odp_pktio_run
 odp_buffer odp_system odp_time
  check_PROGRAMS = ${bin_PROGRAMS}
 -bin_PROGRAMS = odp_init odp_queue odp_crypto odp_shm odp_schedule
 odp_pktio odp_buffer odp_system
 +bin_PROGRAMS = odp_init odp_queue odp_crypto odp_shm odp_schedule
 odp_pktio odp_buffer odp_system odp_time
  odp_crypto_CFLAGS = $(AM_CFLAGS) -I$(srcdir)/crypto
  odp_buffer_CFLAGS = $(AM_CFLAGS) -I$(srcdir)/buffer
  endif
 @@ -27,3 +27,4 @@ dist_odp_buffer_SOURCES = buffer/odp_buffer_pool_test.c \
   buffer/odp_packet_test.c \
   odp_buffer.c common/odp_cunit_common.c
  dist_odp_system_SOURCES = odp_system.c common/odp_cunit_common.c
 +dist_odp_time_SOURCES = odp_time.c common/odp_cunit_common.c
 diff --git a/test/validation/odp_time.c b/test/validation/odp_time.c
 new file mode 100644
 index 000..b0f29cf
 --- /dev/null
 +++ b/test/validation/odp_time.c
 @@ -0,0 +1,71 @@
 +/* Copyright (c) 2014, Linaro Limited
 + * All rights reserved.
 + *
 + * SPDX-License-Identifier: BSD-3-Clause
 + */
 +
 +#include odp.h
 +#include odp_cunit_common.h
 +
 +#define TOLERANCE 1
 +#define BUSY_LOOP_CNT 100
 +
 +/* check that a cycles difference gives a reasonable result */
 +static void test_odp_cycles_diff(void)
 +{
 +   /* volatile to stop optimization of busy loop */
 +   volatile int count = 0;
 +   uint64_t diff, cycles1, cycles2;
 +
 +   cycles1 = odp_time_cycles();
 +
 +   while (count  BUSY_LOOP_CNT) {
 +   count++;
 +   };
 +
 +   cycles2 = odp_time_cycles();
 +   CU_ASSERT(cycles2  cycles1);
 +
 +   diff = odp_time_diff_cycles(cycles1, cycles2);
 +   CU_ASSERT(diff  0);
 +}
 +
 +/* check that a negative cycles difference gives a reasonable result */
 +static void test_odp_cycles_negative_diff(void)
 +{
 +   uint64_t diff, cycles1, cycles2;
 +   cycles1 = 10;
 +   cycles2 = 5;
 +   diff = odp_time_diff_cycles(cycles1, cycles2);
 +   CU_ASSERT(diff  0);
 +}
 +
 +/* check that related conversions come back to the same value */
 +static void test_odp_time_conversion(void)
 +{
 +   uint64_t ns1, ns2, cycles;
 +   uint64_t upper_limit, lower_limit;
 +   ns1 = 100;
 +   cycles = odp_time_ns_to_cycles(ns1);
 +   CU_ASSERT(cycles  0);
 +
 +   ns2 = odp_time_cycles_to_ns(cycles);
 +
 +   /* need to check within arithmetic tolerance that the same
 +* value in ns is returned after conversions */
 +   upper_limit = ns1 + TOLERANCE;
 +   lower_limit = ns1 - TOLERANCE;
 +   CU_ASSERT((ns2 = upper_limit)  (ns2 = lower_limit));
 +}
 +
 +CU_TestInfo test_odp_time[] = {
 +   {cycles diff, test_odp_cycles_diff},
 +   {negative diff, test_odp_cycles_negative_diff},
 +   {conversion, test_odp_time_conversion},
 +CU_TEST_INFO_NULL
 +};
 +
 +CU_SuiteInfo odp_testsuites[] = {
 +   {Time, NULL, NULL, NULL, NULL, test_odp_time},
 +CU_SUITE_INFO_NULL
 +};
 --
 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

[lng-odp] [PATCH v2] validation: add odp_time test

2015-01-05 Thread Mike Holmes 
Add odp_time_x API tests

Signed-off-by: Mike Holmes mike.hol...@linaro.org
---
 test/validation/.gitignore  |  1 +
 test/validation/Makefile.am |  5 ++--
 test/validation/odp_time.c  | 71 +
 3 files changed, 75 insertions(+), 2 deletions(-)
 create mode 100644 test/validation/odp_time.c

diff --git a/test/validation/.gitignore b/test/validation/.gitignore
index d08db73..40ea12d 100644
--- a/test/validation/.gitignore
+++ b/test/validation/.gitignore
@@ -8,3 +8,4 @@ odp_shm
 odp_system
 odp_pktio
 odp_buffer
+odp_time
diff --git a/test/validation/Makefile.am b/test/validation/Makefile.am
index c0545b7..692e1a6 100644
--- a/test/validation/Makefile.am
+++ b/test/validation/Makefile.am
@@ -6,9 +6,9 @@ AM_LDFLAGS += -static
 TESTS_ENVIRONMENT = ODP_PLATFORM=${with_platform}
 
 if ODP_CUNIT_ENABLED
-TESTS = odp_init odp_queue odp_crypto odp_shm odp_schedule odp_pktio_run 
odp_buffer odp_system
+TESTS = odp_init odp_queue odp_crypto odp_shm odp_schedule odp_pktio_run 
odp_buffer odp_system odp_time
 check_PROGRAMS = ${bin_PROGRAMS}
-bin_PROGRAMS = odp_init odp_queue odp_crypto odp_shm odp_schedule odp_pktio 
odp_buffer odp_system
+bin_PROGRAMS = odp_init odp_queue odp_crypto odp_shm odp_schedule odp_pktio 
odp_buffer odp_system odp_time
 odp_crypto_CFLAGS = $(AM_CFLAGS) -I$(srcdir)/crypto
 odp_buffer_CFLAGS = $(AM_CFLAGS) -I$(srcdir)/buffer
 endif
@@ -27,3 +27,4 @@ dist_odp_buffer_SOURCES = buffer/odp_buffer_pool_test.c \
  buffer/odp_packet_test.c \
  odp_buffer.c common/odp_cunit_common.c
 dist_odp_system_SOURCES = odp_system.c common/odp_cunit_common.c
+dist_odp_time_SOURCES = odp_time.c common/odp_cunit_common.c
diff --git a/test/validation/odp_time.c b/test/validation/odp_time.c
new file mode 100644
index 000..b0f29cf
--- /dev/null
+++ b/test/validation/odp_time.c
@@ -0,0 +1,71 @@
+/* Copyright (c) 2014, Linaro Limited
+ * All rights reserved.
+ *
+ * SPDX-License-Identifier: BSD-3-Clause
+ */
+
+#include odp.h
+#include odp_cunit_common.h
+
+#define TOLERANCE 1
+#define BUSY_LOOP_CNT 100
+
+/* check that a cycles difference gives a reasonable result */
+static void test_odp_cycles_diff(void)
+{
+   /* volatile to stop optimization of busy loop */
+   volatile int count = 0;
+   uint64_t diff, cycles1, cycles2;
+
+   cycles1 = odp_time_cycles();
+
+   while (count  BUSY_LOOP_CNT) {
+   count++;
+   };
+
+   cycles2 = odp_time_cycles();
+   CU_ASSERT(cycles2  cycles1);
+
+   diff = odp_time_diff_cycles(cycles1, cycles2);
+   CU_ASSERT(diff  0);
+}
+
+/* check that a negative cycles difference gives a reasonable result */
+static void test_odp_cycles_negative_diff(void)
+{
+   uint64_t diff, cycles1, cycles2;
+   cycles1 = 10;
+   cycles2 = 5;
+   diff = odp_time_diff_cycles(cycles1, cycles2);
+   CU_ASSERT(diff  0);
+}
+
+/* check that related conversions come back to the same value */
+static void test_odp_time_conversion(void)
+{
+   uint64_t ns1, ns2, cycles;
+   uint64_t upper_limit, lower_limit;
+   ns1 = 100;
+   cycles = odp_time_ns_to_cycles(ns1);
+   CU_ASSERT(cycles  0);
+
+   ns2 = odp_time_cycles_to_ns(cycles);
+
+   /* need to check within arithmetic tolerance that the same
+* value in ns is returned after conversions */
+   upper_limit = ns1 + TOLERANCE;
+   lower_limit = ns1 - TOLERANCE;
+   CU_ASSERT((ns2 = upper_limit)  (ns2 = lower_limit));
+}
+
+CU_TestInfo test_odp_time[] = {
+   {cycles diff, test_odp_cycles_diff},
+   {negative diff, test_odp_cycles_negative_diff},
+   {conversion, test_odp_time_conversion},
+CU_TEST_INFO_NULL
+};
+
+CU_SuiteInfo odp_testsuites[] = {
+   {Time, NULL, NULL, NULL, NULL, test_odp_time},
+CU_SUITE_INFO_NULL
+};
-- 
2.1.0


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

Re: [lng-odp] [PATCH v2] validation: add odp_time test

2015-01-03 Thread Jerin Jacob 
On Mon, Dec 29, 2014 at 11:08:58AM -0500, Mike Holmes wrote:
 Add odp_time_x API tests
 
 Signed-off-by: Mike Holmes mike.hol...@linaro.org
 ---
 v2:
 fix bad auto merge of Makefile.am
 clean up names
 
  test/validation/.gitignore  |  1 +
  test/validation/Makefile.am |  5 ++--
  test/validation/odp_time.c  | 71 
 +
  3 files changed, 75 insertions(+), 2 deletions(-)
  create mode 100644 test/validation/odp_time.c
 
 diff --git a/test/validation/.gitignore b/test/validation/.gitignore
 index d08db73..40ea12d 100644
 --- a/test/validation/.gitignore
 +++ b/test/validation/.gitignore
 @@ -8,3 +8,4 @@ odp_shm
  odp_system
  odp_pktio
  odp_buffer
 +odp_time
 diff --git a/test/validation/Makefile.am b/test/validation/Makefile.am
 index c0545b7..692e1a6 100644
 --- a/test/validation/Makefile.am
 +++ b/test/validation/Makefile.am
 @@ -6,9 +6,9 @@ AM_LDFLAGS += -static
  TESTS_ENVIRONMENT = ODP_PLATFORM=${with_platform}
  
  if ODP_CUNIT_ENABLED
 -TESTS = odp_init odp_queue odp_crypto odp_shm odp_schedule odp_pktio_run 
 odp_buffer odp_system
 +TESTS = odp_init odp_queue odp_crypto odp_shm odp_schedule odp_pktio_run 
 odp_buffer odp_system odp_time
  check_PROGRAMS = ${bin_PROGRAMS}
 -bin_PROGRAMS = odp_init odp_queue odp_crypto odp_shm odp_schedule odp_pktio 
 odp_buffer odp_system
 +bin_PROGRAMS = odp_init odp_queue odp_crypto odp_shm odp_schedule odp_pktio 
 odp_buffer odp_system odp_time
  odp_crypto_CFLAGS = $(AM_CFLAGS) -I$(srcdir)/crypto
  odp_buffer_CFLAGS = $(AM_CFLAGS) -I$(srcdir)/buffer
  endif
 @@ -27,3 +27,4 @@ dist_odp_buffer_SOURCES = buffer/odp_buffer_pool_test.c \
 buffer/odp_packet_test.c \
 odp_buffer.c common/odp_cunit_common.c
  dist_odp_system_SOURCES = odp_system.c common/odp_cunit_common.c
 +dist_odp_time_SOURCES = odp_time.c common/odp_cunit_common.c
 diff --git a/test/validation/odp_time.c b/test/validation/odp_time.c
 new file mode 100644
 index 000..75be1f0
 --- /dev/null
 +++ b/test/validation/odp_time.c
 @@ -0,0 +1,71 @@
 +/* Copyright (c) 2014, Linaro Limited
 + * All rights reserved.
 + *
 + * SPDX-License-Identifier: BSD-3-Clause
 + */
 +
 +#include odp.h
 +#include odp_cunit_common.h
 +
 +#define TOLERANCE 1
 +#define BUSY_LOOP_CNT 100
 +
 +/* check that a cycles difference gives a reasonable result */
 +static void test_odp_cycles_diff(void)
 +{
 + /* volatile to stop optimization of busy loop */
 + volatile int count;
 + uint64_t diff, cycles1, cycles2;
 +
 + cycles1 = odp_time_cycles();
 +
 + while (count  BUSY_LOOP_CNT) {

local variable count is uninitialized.

 + count++;
 + };
 +
 + cycles2 = odp_time_cycles();
 + CU_ASSERT(cycles2  cycles1);
 +
 + diff = odp_time_diff_cycles(cycles1, cycles2);
 + CU_ASSERT(diff  0);
 +}
 +
 +/* check that a negative cycles difference gives a reasonable result */
 +static void test_odp_cycles_negative_diff(void)
 +{
 + uint64_t diff, cycles1, cycles2;
 + cycles1 = 10;
 + cycles2 = 5;
 + diff = odp_time_diff_cycles(cycles1, cycles2);
 + CU_ASSERT(diff  0);
 +}
 +
 +/* check that related conversions come back to the same value */
 +static void test_odp_time_conversion(void)
 +{
 + uint64_t ns1, ns2, cycles;
 + uint64_t upper_limit, lower_limit;
 + ns1 = 100;
 + cycles = odp_time_ns_to_cycles(ns1);
 + CU_ASSERT(cycles  0);
 +
 + ns2 = odp_time_cycles_to_ns(cycles);
 +
 + /* need to check within arithmetic tolerance that the same
 +  * value in ns is returned after conversions */
 + upper_limit = ns1 + TOLERANCE;
 + lower_limit = ns1 - TOLERANCE;
 + CU_ASSERT((ns2 = upper_limit)  (ns2 = lower_limit));
 +}
 +
 +CU_TestInfo test_odp_time[] = {
 + {cycles diff, test_odp_cycles_diff},
 + {negative diff, test_odp_cycles_negative_diff},
 + {conversion, test_odp_time_conversion},
 +  CU_TEST_INFO_NULL
 +};
 +
 +CU_SuiteInfo odp_testsuites[] = {
 + {Time, NULL, NULL, NULL, NULL, test_odp_time},
 +  CU_SUITE_INFO_NULL
 +};
 -- 
 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

[lng-odp] [PATCH v2] validation: add odp_time test

2014-12-28 Thread Mike Holmes 
Add odp_time_x API tests

Signed-off-by: Mike Holmes mike.hol...@linaro.org
---
 test/validation/.gitignore  |  1 +
 test/validation/Makefile.am | 19 ++--
 test/validation/odp_time.c  | 70 +
 3 files changed, 87 insertions(+), 3 deletions(-)
 create mode 100644 test/validation/odp_time.c

diff --git a/test/validation/.gitignore b/test/validation/.gitignore
index d08db73..40ea12d 100644
--- a/test/validation/.gitignore
+++ b/test/validation/.gitignore
@@ -8,3 +8,4 @@ odp_shm
 odp_system
 odp_pktio
 odp_buffer
+odp_time
diff --git a/test/validation/Makefile.am b/test/validation/Makefile.am
index c0545b7..3de5bdb 100644
--- a/test/validation/Makefile.am
+++ b/test/validation/Makefile.am
@@ -6,11 +6,19 @@ AM_LDFLAGS += -static
 TESTS_ENVIRONMENT = ODP_PLATFORM=${with_platform}
 
 if ODP_CUNIT_ENABLED
-TESTS = odp_init odp_queue odp_crypto odp_shm odp_schedule odp_pktio_run 
odp_buffer odp_system
+TESTS = odp_init odp_queue odp_crypto odp_shm odp_schedule odp_pktio_run 
odp_buffer odp_system odp_time
 check_PROGRAMS = ${bin_PROGRAMS}
-bin_PROGRAMS = odp_init odp_queue odp_crypto odp_shm odp_schedule odp_pktio 
odp_buffer odp_system
-odp_crypto_CFLAGS = $(AM_CFLAGS) -I$(srcdir)/crypto
+bin_PROGRAMS = odp_init odp_queue odp_crypto odp_shm odp_schedule odp_pktio 
odp_buffer odp_system odp_time
 odp_buffer_CFLAGS = $(AM_CFLAGS) -I$(srcdir)/buffer
+odp_init_LDFLAGS = $(AM_LDFLAGS)
+odp_queue_LDFLAGS = $(AM_LDFLAGS)
+odp_crypto_CFLAGS = $(AM_CFLAGS) -I$(srcdir)/crypto
+odp_crypto_LDFLAGS = $(AM_LDFLAGS)
+odp_shm_CFLAGS = $(AM_CFLAGS)
+odp_shm_LDFLAGS = $(AM_LDFLAGS)
+odp_schedule_CFLAGS = $(AM_CFLAGS)
+odp_schedule_LDFLAGS = $(AM_LDFLAGS)
+odp_time_LDFLAGS = $(AM_LDFLAGS)
 endif
 
 dist_odp_init_SOURCES = odp_init.c
@@ -27,3 +35,8 @@ dist_odp_buffer_SOURCES = buffer/odp_buffer_pool_test.c \
  buffer/odp_packet_test.c \
  odp_buffer.c common/odp_cunit_common.c
 dist_odp_system_SOURCES = odp_system.c common/odp_cunit_common.c
+dist_odp_time_SOURCES = odp_time.c common/odp_cunit_common.c
+
+#For Linux generic the unimplemented crypto API functions break the
+#regression TODO: https://bugs.linaro.org/show_bug.cgi?id=975
+XFAIL_TESTS=odp_crypto
diff --git a/test/validation/odp_time.c b/test/validation/odp_time.c
new file mode 100644
index 000..1198bea
--- /dev/null
+++ b/test/validation/odp_time.c
@@ -0,0 +1,70 @@
+/* Copyright (c) 2014, Linaro Limited
+ * All rights reserved.
+ *
+ * SPDX-License-Identifier: BSD-3-Clause
+ */
+
+#include odp.h
+#include odp_cunit_common.h
+
+#define TOLERANCE 1
+#define BUSY_LOOP_CNT 100
+
+/* check that a cycles difference gives a reasonable result */
+static void test_odp_time_diff(void)
+{
+   /* volatile to stop optimization of busy loop */
+   volatile int count;
+   uint64_t diff, cycles1, cycles2;
+
+   cycles1 = odp_time_cycles();
+
+   while (count  BUSY_LOOP_CNT) {
+   count++;
+   };
+
+   cycles2 = odp_time_cycles();
+   CU_ASSERT(cycles2  cycles1);
+
+   diff = odp_time_diff_cycles(cycles1, cycles2);
+   CU_ASSERT(diff  0);
+}
+
+/* check that a negative cycles difference gives a reasonable result */
+static void test_odp_time_negative_diff(void)
+{
+   uint64_t diff, cycles1, cycles2;
+   cycles1 = 10;
+   cycles2 = 5;
+   diff = odp_time_diff_cycles(cycles1, cycles2);
+   CU_ASSERT(diff  0);
+}
+/* check that related conversions come back to the same value */
+static void test_odp_time_conversion(void)
+{
+   uint64_t ns1, ns2, cycles1;
+   uint64_t upper_limit, lower_limit;
+   ns1 = 100;
+   cycles1 = odp_time_ns_to_cycles(ns1);
+   CU_ASSERT(cycles1  0);
+
+   ns2 = odp_time_cycles_to_ns(cycles1);
+
+   /* need to check within arithmetic tolerance that the same
+* value in ns is returned after conversions */
+   upper_limit = ns1 + TOLERANCE;
+   lower_limit = ns1 - TOLERANCE;
+   CU_ASSERT((ns2 = upper_limit)  (ns2 = lower_limit));
+}
+
+CU_TestInfo test_odp_time[] = {
+   {diff, test_odp_time_diff},
+   {negative diff, test_odp_time_negative_diff},
+   {conversion, test_odp_time_conversion},
+CU_TEST_INFO_NULL
+};
+
+CU_SuiteInfo odp_testsuites[] = {
+   {Time, NULL, NULL, NULL, NULL, test_odp_time},
+CU_SUITE_INFO_NULL
+};
-- 
2.1.0


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