From patchwork Thu Aug 4 07:59:55 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Christophe Milard X-Patchwork-Id: 73242 Delivered-To: patch@linaro.org Received: by 10.140.29.52 with SMTP id a49csp1258251qga; Thu, 4 Aug 2016 00:59:58 -0700 (PDT) X-Received: by 10.55.142.4 with SMTP id q4mr4663684qkd.117.1470297598188; Thu, 04 Aug 2016 00:59:58 -0700 (PDT) Return-Path: <010001565490dd66-3d1cd3fc-e9e7-44f0-b15b-7807dca994db-000000@amazonses.com> Received: from a8-71.smtp-out.amazonses.com (a8-71.smtp-out.amazonses.com. [54.240.8.71]) by mx.google.com with ESMTPS id b1si7486995qte.69.2016.08.04.00.59.55 (version=TLS1 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Thu, 04 Aug 2016 00:59:58 -0700 (PDT) Received-SPF: pass (google.com: domain of 010001565490dd66-3d1cd3fc-e9e7-44f0-b15b-7807dca994db-000000@amazonses.com designates 54.240.8.71 as permitted sender) client-ip=54.240.8.71; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org; dkim=pass header.i=@amazonses.com; spf=pass (google.com: domain of 010001565490dd66-3d1cd3fc-e9e7-44f0-b15b-7807dca994db-000000@amazonses.com designates 54.240.8.71 as permitted sender) smtp.mailfrom=010001565490dd66-3d1cd3fc-e9e7-44f0-b15b-7807dca994db-000000@amazonses.com; dmarc=pass (p=NONE dis=NONE) header.from=linaro.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/simple; s=6ljdmj5fma5i5lszv46ktqgvm52mzzyo; d=linaro.org; t=1470297595; h=From:To:Date:Message-Id:Subject:List-Id:List-Unsubscribe:List-Archive:List-Post:List-Help:List-Subscribe:Sender; bh=fWJfy2DX+FmK4ldMXYirIQCgeuP9W9dygMEMc2iYTsg=; b=iLiEILiyL4mgxOyQRHmKqXoQY+J2ZP/7fT9kQdbNBud4U1ABjx8Eviuh13Wk0YAF 5rmIFXUjqhqmpRtmnlgo+l1obhfsrlddQ4rKAE6Z0uVAhUWRjAPPbXRcV5/P8YMjOMZ fmblWcnjmpGJ4CqQ6xiSXtU8XQqKa1vRzyK5xlAY= DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/simple; s=6gbrjpgwjskckoa6a5zn6fwqkn67xbtw; d=amazonses.com; t=1470297595; h=From:To:Date:Message-Id:Subject:List-Id:List-Unsubscribe:List-Archive:List-Post:List-Help:List-Subscribe:Sender:Feedback-ID; bh=fWJfy2DX+FmK4ldMXYirIQCgeuP9W9dygMEMc2iYTsg=; b=YEBULxwKDmgW+KqUkc4g3h0z4U9oM1j3+UE4IkujSZ4vjryzd8w7XRn+QdH5xs/R BQjszCAWIprU7hsLxGPoJPVWCqAP3/kOdliMFVtibn8iiz25rgdaWmRxquNJpWMJaCh OMG67IvlwdB3As/RrnwYOY3Kugkz2OTV4H7CDCwc= X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on ip-10-142-244-252 X-Spam-Level: X-Spam-Status: No, score=-2.6 required=5.0 tests=BAYES_00, RCVD_IN_DNSWL_LOW, RCVD_IN_MSPIKE_H3, RCVD_IN_MSPIKE_WL, URIBL_BLOCKED autolearn=disabled version=3.4.0 X-Original-To: lng-odp@lists.linaro.org Delivered-To: lng-odp@lists.linaro.org X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:date:message-id; bh=4KUJqV1/J3yLuQmkHEV/XDWAhTW+9GtAj+XO3Tf1hkY=; b=MXgs6fmb1JpXMm7DukvK8ubv7pTCDtTp7wBZqyJdk+TSVc9riOjCPCuuW7cpTUQN6F b1vzrkAeCZKOSGNE74kuJKF2LN5G28MWgTnf3aeIzeiwxFtR6aH3XHV8Rq9iAe95Clnx JdWtBOLLuGwqZa3bITu1CZNZLEC45Buci47TWMDls/DnaXAymoznxa4rVwlT80oSjZy0 R8EWqWKSaj7jk8m3eTD1GICoPWzSPOfGLPf3WQbKzNJyx9FZEf9E+CLR6kMMEIFMHXFG s41ilcPVtodXnJfoVxWtzpjpN7rX5+mRLO84YsYQxs3sTCEsgR1bq5Rebscqs6egPQxa AtKQ== X-Gm-Message-State: AEkooutXnoOhpjfPKcX8HGU+1EtkFdlewXDMqPQBQzKQu5FtSVcPHAVWbb8KBj1uaY1Rt9MZ8ys= X-Received: by 10.25.76.137 with SMTP id z131mr19525084lfa.49.1470297580210; Thu, 04 Aug 2016 00:59:40 -0700 (PDT) From: Christophe Milard To: mike.holmes@linaro.org, balakrishna.garapati@linaro.org, bill.fischofer@linaro.org, lng-odp@lists.linaro.org Date: Thu, 4 Aug 2016 07:59:55 +0000 Message-ID: <010001565490dd66-3d1cd3fc-e9e7-44f0-b15b-7807dca994db-000000@email.amazonses.com> X-Mailer: git-send-email 2.7.4 X-Topics: patch Subject: [lng-odp] [API-NEXT PATCH] test: drv: atomic renamed to drvatomic X-BeenThere: lng-odp@lists.linaro.org X-Mailman-Version: 2.1.16 Precedence: list List-Id: "The OpenDataPlane \(ODP\) List" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: lng-odp-bounces@lists.linaro.org Sender: "lng-odp" X-SES-Outgoing: 2016.08.04-54.240.8.71 Feedback-ID: 1.us-east-1.Y2eXJruXwxqfBntQfC4/6O4zDrCY0XKEghZ/yI2DADM=:AmazonSES At installation time (make install) all tests executables are copied into a single directory (lib/odp/tests). Having two modules with identical names on the two interfaces (api and drv) leads to the same test executable name ('atomic_main' in this case). As a consequence one of the test overwrites the other at copy time, and is run twice at make check install (successfully, so no error is shown). We therefore have to guarantee module name unicity over ODP: we cannot have an atomic module on the drv interface if such a module name already exists on the api interface. The atomic module from the drv interface is hence renamed drvatomic. Signed-off-by: Christophe Milard --- test/common_plat/m4/configure.m4 | 2 +- test/common_plat/validation/drv/Makefile.am | 2 +- test/common_plat/validation/drv/atomic/.gitignore | 1 - test/common_plat/validation/drv/atomic/Makefile.am | 10 ---- test/common_plat/validation/drv/atomic/atomic.h | 38 --------------- .../validation/drv/drvatomic/.gitignore | 1 + .../validation/drv/drvatomic/Makefile.am | 10 ++++ .../drv/{atomic/atomic.c => drvatomic/drvatomic.c} | 54 +++++++++++----------- .../validation/drv/drvatomic/drvatomic.h | 35 ++++++++++++++ .../atomic_main.c => drvatomic/drvatomic_main.c} | 4 +- test/linux-generic/Makefile.am | 2 +- 11 files changed, 78 insertions(+), 81 deletions(-) delete mode 100644 test/common_plat/validation/drv/atomic/.gitignore delete mode 100644 test/common_plat/validation/drv/atomic/Makefile.am delete mode 100644 test/common_plat/validation/drv/atomic/atomic.h create mode 100644 test/common_plat/validation/drv/drvatomic/.gitignore create mode 100644 test/common_plat/validation/drv/drvatomic/Makefile.am rename test/common_plat/validation/drv/{atomic/atomic.c => drvatomic/drvatomic.c} (94%) create mode 100644 test/common_plat/validation/drv/drvatomic/drvatomic.h rename test/common_plat/validation/drv/{atomic/atomic_main.c => drvatomic/drvatomic_main.c} (72%) -- 2.7.4 diff --git a/test/common_plat/m4/configure.m4 b/test/common_plat/m4/configure.m4 index bfde6cb..20c3a3b 100644 --- a/test/common_plat/m4/configure.m4 +++ b/test/common_plat/m4/configure.m4 @@ -32,4 +32,4 @@ AC_CONFIG_FILES([test/common_plat/Makefile test/common_plat/validation/api/timer/Makefile test/common_plat/validation/api/traffic_mngr/Makefile test/common_plat/validation/drv/Makefile - test/common_plat/validation/drv/atomic/Makefile]) + test/common_plat/validation/drv/drvatomic/Makefile]) diff --git a/test/common_plat/validation/drv/Makefile.am b/test/common_plat/validation/drv/Makefile.am index 0dae3c3..eedbad5 100644 --- a/test/common_plat/validation/drv/Makefile.am +++ b/test/common_plat/validation/drv/Makefile.am @@ -1,4 +1,4 @@ -ODPDRV_MODULES = atomic +ODPDRV_MODULES = drvatomic SUBDIRS = $(ODPDRV_MODULES) diff --git a/test/common_plat/validation/drv/atomic/.gitignore b/test/common_plat/validation/drv/atomic/.gitignore deleted file mode 100644 index 610ffea..0000000 --- a/test/common_plat/validation/drv/atomic/.gitignore +++ /dev/null @@ -1 +0,0 @@ -atomic_main diff --git a/test/common_plat/validation/drv/atomic/Makefile.am b/test/common_plat/validation/drv/atomic/Makefile.am deleted file mode 100644 index 9b6bd63..0000000 --- a/test/common_plat/validation/drv/atomic/Makefile.am +++ /dev/null @@ -1,10 +0,0 @@ -include ../Makefile.inc - -noinst_LTLIBRARIES = libtestatomic.la -libtestatomic_la_SOURCES = atomic.c - -test_PROGRAMS = atomic_main$(EXEEXT) -dist_atomic_main_SOURCES = atomic_main.c -atomic_main_LDADD = libtestatomic.la $(LIBCUNIT_COMMON) $(LIBODP) - -EXTRA_DIST = atomic.h diff --git a/test/common_plat/validation/drv/atomic/atomic.h b/test/common_plat/validation/drv/atomic/atomic.h deleted file mode 100644 index 438c0a3..0000000 --- a/test/common_plat/validation/drv/atomic/atomic.h +++ /dev/null @@ -1,38 +0,0 @@ -/* Copyright (c) 2016, Linaro Limited - * All rights reserved. - * - * SPDX-License-Identifier: BSD-3-Clause - */ - -#ifndef _ODP_TEST_ATOMIC_H_ -#define _ODP_TEST_ATOMIC_H_ - -#include - -/* test functions: */ -void atomic_test_atomic_inc_dec(void); -void atomic_test_atomic_add_sub(void); -void atomic_test_atomic_fetch_inc_dec(void); -void atomic_test_atomic_fetch_add_sub(void); -void atomic_test_atomic_max_min(void); -void atomic_test_atomic_cas_inc_dec(void); -void atomic_test_atomic_xchg(void); -void atomic_test_atomic_non_relaxed(void); -void atomic_test_atomic_op_lock_free(void); - -/* test arrays: */ -extern odp_testinfo_t atomic_suite_atomic[]; - -/* test array init/term functions: */ -int atomic_suite_init(void); - -/* test registry: */ -extern odp_suiteinfo_t atomic_suites[]; - -/* executable init/term functions: */ -int atomic_init(odp_instance_t *inst); - -/* main test program: */ -int atomic_main(int argc, char *argv[]); - -#endif diff --git a/test/common_plat/validation/drv/drvatomic/.gitignore b/test/common_plat/validation/drv/drvatomic/.gitignore new file mode 100644 index 0000000..3b5e510 --- /dev/null +++ b/test/common_plat/validation/drv/drvatomic/.gitignore @@ -0,0 +1 @@ +drvatomic_main diff --git a/test/common_plat/validation/drv/drvatomic/Makefile.am b/test/common_plat/validation/drv/drvatomic/Makefile.am new file mode 100644 index 0000000..53e971c --- /dev/null +++ b/test/common_plat/validation/drv/drvatomic/Makefile.am @@ -0,0 +1,10 @@ +include ../Makefile.inc + +noinst_LTLIBRARIES = libtestdrvatomic.la +libtestdrvatomic_la_SOURCES = drvatomic.c + +test_PROGRAMS = drvatomic_main$(EXEEXT) +dist_drvatomic_main_SOURCES = drvatomic_main.c +drvatomic_main_LDADD = libtestdrvatomic.la $(LIBCUNIT_COMMON) $(LIBODP) + +EXTRA_DIST = drvatomic.h diff --git a/test/common_plat/validation/drv/atomic/atomic.c b/test/common_plat/validation/drv/drvatomic/drvatomic.c similarity index 94% rename from test/common_plat/validation/drv/atomic/atomic.c rename to test/common_plat/validation/drv/drvatomic/drvatomic.c index 162f6e4..cc5e441 100644 --- a/test/common_plat/validation/drv/atomic/atomic.c +++ b/test/common_plat/validation/drv/drvatomic/drvatomic.c @@ -10,7 +10,7 @@ #include #include #include -#include "atomic.h" +#include "drvatomic.h" #define VERBOSE 0 #define MAX_ITERATIONS 1000 @@ -541,7 +541,7 @@ static void test_atomic_validate(int check) } } -int atomic_init(odp_instance_t *inst) +int drvatomic_init(odp_instance_t *inst) { uint32_t workers_count, max_threads; int ret = 0; @@ -707,48 +707,48 @@ static void test_atomic_functional(int func_ptr(void *), int check) test_atomic_validate(check); } -void atomic_test_atomic_inc_dec(void) +void drvatomic_test_atomic_inc_dec(void) { test_atomic_functional(test_atomic_inc_dec_thread, 0); } -void atomic_test_atomic_add_sub(void) +void drvatomic_test_atomic_add_sub(void) { test_atomic_functional(test_atomic_add_sub_thread, 0); } -void atomic_test_atomic_fetch_inc_dec(void) +void drvatomic_test_atomic_fetch_inc_dec(void) { test_atomic_functional(test_atomic_fetch_inc_dec_thread, 0); } -void atomic_test_atomic_fetch_add_sub(void) +void drvatomic_test_atomic_fetch_add_sub(void) { test_atomic_functional(test_atomic_fetch_add_sub_thread, 0); } -void atomic_test_atomic_max_min(void) +void drvatomic_test_atomic_max_min(void) { test_atomic_functional(test_atomic_max_min_thread, CHECK_MAX_MIN); } -void atomic_test_atomic_cas_inc_dec(void) +void drvatomic_test_atomic_cas_inc_dec(void) { test_atomic_functional(test_atomic_cas_inc_dec_thread, 0); } -void atomic_test_atomic_xchg(void) +void drvatomic_test_atomic_xchg(void) { test_atomic_functional(test_atomic_xchg_thread, CHECK_XCHG); } -void atomic_test_atomic_non_relaxed(void) +void drvatomic_test_atomic_non_relaxed(void) { test_atomic_functional(test_atomic_non_relaxed_thread, CHECK_MAX_MIN | CHECK_XCHG); } -void atomic_test_atomic_op_lock_free(void) +void drvatomic_test_atomic_op_lock_free(void) { odpdrv_atomic_op_t atomic_op; int ret_null, ret; @@ -854,26 +854,26 @@ void atomic_test_atomic_op_lock_free(void) } } -odp_testinfo_t atomic_suite_atomic[] = { - ODP_TEST_INFO(atomic_test_atomic_inc_dec), - ODP_TEST_INFO(atomic_test_atomic_add_sub), - ODP_TEST_INFO(atomic_test_atomic_fetch_inc_dec), - ODP_TEST_INFO(atomic_test_atomic_fetch_add_sub), - ODP_TEST_INFO(atomic_test_atomic_max_min), - ODP_TEST_INFO(atomic_test_atomic_cas_inc_dec), - ODP_TEST_INFO(atomic_test_atomic_xchg), - ODP_TEST_INFO(atomic_test_atomic_non_relaxed), - ODP_TEST_INFO(atomic_test_atomic_op_lock_free), +odp_testinfo_t drvatomic_suite_atomic[] = { + ODP_TEST_INFO(drvatomic_test_atomic_inc_dec), + ODP_TEST_INFO(drvatomic_test_atomic_add_sub), + ODP_TEST_INFO(drvatomic_test_atomic_fetch_inc_dec), + ODP_TEST_INFO(drvatomic_test_atomic_fetch_add_sub), + ODP_TEST_INFO(drvatomic_test_atomic_max_min), + ODP_TEST_INFO(drvatomic_test_atomic_cas_inc_dec), + ODP_TEST_INFO(drvatomic_test_atomic_xchg), + ODP_TEST_INFO(drvatomic_test_atomic_non_relaxed), + ODP_TEST_INFO(drvatomic_test_atomic_op_lock_free), ODP_TEST_INFO_NULL, }; -odp_suiteinfo_t atomic_suites[] = { - {"atomic", NULL, NULL, - atomic_suite_atomic}, +odp_suiteinfo_t drvatomic_suites[] = { + {"drvatomic", NULL, NULL, + drvatomic_suite_atomic}, ODP_SUITE_INFO_NULL }; -int atomic_main(int argc, char *argv[]) +int drvatomic_main(int argc, char *argv[]) { int ret; @@ -881,9 +881,9 @@ int atomic_main(int argc, char *argv[]) if (odp_cunit_parse_options(argc, argv)) return -1; - odp_cunit_register_global_init(atomic_init); + odp_cunit_register_global_init(drvatomic_init); - ret = odp_cunit_register(atomic_suites); + ret = odp_cunit_register(drvatomic_suites); if (ret == 0) ret = odp_cunit_run(); diff --git a/test/common_plat/validation/drv/drvatomic/drvatomic.h b/test/common_plat/validation/drv/drvatomic/drvatomic.h new file mode 100644 index 0000000..02ce14a --- /dev/null +++ b/test/common_plat/validation/drv/drvatomic/drvatomic.h @@ -0,0 +1,35 @@ +/* Copyright (c) 2016, Linaro Limited + * All rights reserved. + * + * SPDX-License-Identifier: BSD-3-Clause + */ + +#ifndef _ODP_TEST_ATOMIC_H_ +#define _ODP_TEST_ATOMIC_H_ + +#include + +/* test functions: */ +void drvatomic_test_atomic_inc_dec(void); +void drvatomic_test_atomic_add_sub(void); +void drvatomic_test_atomic_fetch_inc_dec(void); +void drvatomic_test_atomic_fetch_add_sub(void); +void drvatomic_test_atomic_max_min(void); +void drvatomic_test_atomic_cas_inc_dec(void); +void drvatomic_test_atomic_xchg(void); +void drvatomic_test_atomic_non_relaxed(void); +void drvatomic_test_atomic_op_lock_free(void); + +/* test arrays: */ +extern odp_testinfo_t drvatomic_suite_atomic[]; + +/* test registry: */ +extern odp_suiteinfo_t drvatomic_suites[]; + +/* executable init/term functions: */ +int drvatomic_init(odp_instance_t *inst); + +/* main test program: */ +int drvatomic_main(int argc, char *argv[]); + +#endif diff --git a/test/common_plat/validation/drv/atomic/atomic_main.c b/test/common_plat/validation/drv/drvatomic/drvatomic_main.c similarity index 72% rename from test/common_plat/validation/drv/atomic/atomic_main.c rename to test/common_plat/validation/drv/drvatomic/drvatomic_main.c index b290087..390a3e2 100644 --- a/test/common_plat/validation/drv/atomic/atomic_main.c +++ b/test/common_plat/validation/drv/drvatomic/drvatomic_main.c @@ -4,9 +4,9 @@ * SPDX-License-Identifier: BSD-3-Clause */ -#include "atomic.h" +#include "drvatomic.h" int main(int argc, char *argv[]) { - return atomic_main(argc, argv); + return drvatomic_main(argc, argv); } diff --git a/test/linux-generic/Makefile.am b/test/linux-generic/Makefile.am index 1cee6e7..df10c61 100644 --- a/test/linux-generic/Makefile.am +++ b/test/linux-generic/Makefile.am @@ -34,7 +34,7 @@ TESTS = validation/api/pktio/pktio_run.sh \ $(ALL_API_VALIDATION_DIR)/traffic_mngr/traffic_mngr_main$(EXEEXT) \ $(ALL_API_VALIDATION_DIR)/shmem/shmem_main$(EXEEXT) \ $(ALL_API_VALIDATION_DIR)/system/system_main$(EXEEXT) \ - $(ALL_DRV_VALIDATION_DIR)/atomic/atomic_main$(EXEEXT) \ + $(ALL_DRV_VALIDATION_DIR)/drvatomic/drvatomic_main$(EXEEXT) \ ring/ring_main$(EXEEXT) SUBDIRS += validation/api/pktio\