On Mon, Nov 24, 2014 at 05:22:28PM +0800, yan.songm...@linaro.org wrote: > Hi Jerin, > Thanks for reading this patch. I'll change the problem of the use of 'shm'. > > > ->IMO, We should avoid using variables allocated from stack to share > ->the data between different execution context. > ->It will work in linux threaded run-time environment.But it will > ->have issues in different run-time environment like bare-metal. > ->IMO if any memory needs to be shared across different > ->execution environment should use the memory allocated from odp shared mem. > > Sorry ,i can't get it . You mean that sometimes we don't use odp in linux ?
ODP API's are execution environment agnostic. We have a odp implementation that supports both linux and baremetal. > If so , i think we use this only in linux, if not there are a lot of api > will not work. > This cunit test is for shm, so if i use it to test itself is not a good idea. Other way around to look its, its sunnyday test to check the shared memory accessibly across the different odp execution context. > > > > yan.songm...@linaro.org > > From: Jerin Jacob > Date: 2014-11-24 15:21 > To: Yan Songming > CC: lng-odp@lists.linaro.org > Subject: Re: [lng-odp] [PATCH v2] cunit: add shm test > On Thu, Nov 20, 2014 at 10:54:55PM +0800, Yan Songming wrote: > > Add odp_cunit_common.c for common cunit function and add the cunit > > test for the api in odp_share_memory.h which include the new api implement > > odp_shm_free. > > > > Signed-off-by: Yan Songming <yan.songm...@linaro.org> > > --- > > v2 fix some problem which maxim and mike found. > > --- > > test/validation/Makefile.am | 4 +- > > test/validation/odp_cunit_common.c | 37 ++++++++++++ > > test/validation/odp_cunit_common.h | 41 +++++++++++++ > > test/validation/odp_shm.c | 121 > > +++++++++++++++++++++++++++++++++++++ > > 4 files changed, 202 insertions(+), 1 deletion(-) > > create mode 100644 test/validation/odp_cunit_common.c > > create mode 100644 test/validation/odp_cunit_common.h > > create mode 100644 test/validation/odp_shm.c > > > > diff --git a/test/validation/Makefile.am b/test/validation/Makefile.am > > index 2e1b991..5992f72 100644 > > --- a/test/validation/Makefile.am > > +++ b/test/validation/Makefile.am > > @@ -6,11 +6,12 @@ AM_LDFLAGS += -L$(CUNIT_PATH)/lib -static -lcunit > > if ODP_CUNIT_ENABLED > > TESTS = ${bin_PROGRAMS} > > check_PROGRAMS = ${bin_PROGRAMS} > > -bin_PROGRAMS = odp_init odp_queue odp_crypto > > +bin_PROGRAMS = odp_init odp_queue odp_crypto odp_shm > > odp_init_LDFLAGS = $(AM_LDFLAGS) > > odp_queue_LDFLAGS = $(AM_LDFLAGS) > > odp_crypto_CFLAGS = $(AM_CFLAGS) -I./crypto > > odp_crypto_LDFLAGS = $(AM_LDFLAGS) > > +odp_shm_LDFLAGS = $(AM_LDFLAGS) > > endif > > > > dist_odp_init_SOURCES = odp_init.c > > @@ -18,3 +19,4 @@ dist_odp_queue_SOURCES = odp_queue.c > > dist_odp_crypto_SOURCES = crypto/odp_crypto_test_async_inp.c \ > > crypto/odp_crypto_test_sync_inp.c \ > > odp_crypto.c > > +dist_odp_shm_SOURCES = odp_shm.c odp_cunit_common.c > > diff --git a/test/validation/odp_cunit_common.c > > b/test/validation/odp_cunit_common.c > > new file mode 100644 > > index 0000000..885b981 > > --- /dev/null > > +++ b/test/validation/odp_cunit_common.c > > @@ -0,0 +1,37 @@ > > +/* Copyright (c) 2014, Linaro Limited > > + * All rights reserved. > > + * > > + * SPDX-License-Identifier: BSD-3-Clause > > + */ > > + > > +/** > > + * @file > > + * > > + * ODP test application common > > + */ > > + > > +#include <string.h> > > +#include <odp.h> > > +#include "odp_cunit_common.h" > > +#include "odph_linux.h" > > +/* Globals */ > > +static odph_linux_pthread_t thread_tbl[MAX_WORKERS]; > > + > > +/** create test thread */ > > +int odp_cunit_thread_create(void *func_ptr(void *), pthrd_arg *arg) > > +{ > > + /* Create and init additional threads */ > > + odph_linux_pthread_create(thread_tbl, arg->numthrds, 0, func_ptr, > > + (void *)arg); > > + > > + return 0; > > +} > > + > > +/** exit from test thread */ > > +int odp_cunit_thread_exit(pthrd_arg *arg) > > +{ > > + /* Wait for other threads to exit */ > > + odph_linux_pthread_join(thread_tbl, arg->numthrds); > > + > > + return 0; > > +} > > diff --git a/test/validation/odp_cunit_common.h > > b/test/validation/odp_cunit_common.h > > new file mode 100644 > > index 0000000..f6eb332 > > --- /dev/null > > +++ b/test/validation/odp_cunit_common.h > > @@ -0,0 +1,41 @@ > > +/* Copyright (c) 2014, Linaro Limited > > + * All rights reserved. > > + * > > + * SPDX-License-Identifier: BSD-3-Clause > > + */ > > + > > +/** > > + * @file > > + * > > + * ODP test application common headers > > + */ > > + > > +#ifndef ODP_CUNICT_COMMON_H > > +#define ODP_CUNICT_COMMON_H > > + > > +#define MAX_WORKERS 32 /**< Maximum number of work threads */ > > + > > +typedef struct { > > + int foo; > > + int bar; > > +} test_shared_data_t; > > + > > +/** types of tests */ > > +typedef enum { > > + ODP_SHM_TEST = 0, > > + ODP_MAX_TEST > > +} odp_test_case_e; > > + > > +/** > > + * Thread argument > > + */ > > +typedef struct { > > + int testcase; /**< specifies which set of API's to exercise */ > > + int numthrds; /**< no of pthreads to create */ > > +} pthrd_arg; > > + > > +/** create thread fro start_routine function */ > > +extern int odp_cunit_thread_create(void *func_ptr(void *), pthrd_arg *arg); > > +extern int odp_cunit_thread_exit(pthrd_arg *); > > + > > +#endif /* ODP_COMMON_H */ > > diff --git a/test/validation/odp_shm.c b/test/validation/odp_shm.c > > new file mode 100644 > > index 0000000..fea3620 > > --- /dev/null > > +++ b/test/validation/odp_shm.c > > @@ -0,0 +1,121 @@ > > +/* Copyright (c) 2014, Linaro Limited > > + * All rights reserved. > > + * > > + * SPDX-License-Identifier: BSD-3-Clause > > + */ > > + > > +#include "odp.h" > > +#include "CUnit/Basic.h" > > +#include "odp_cunit_common.h" > > + > > +#define ALIGE_SIZE (128) > > +#define TESTNAME "cunit_test_shared_data" > > + > > +odp_shm_t shm; > > + > > +__thread test_shared_data_t *test_shared_data; > > +static void *run_shm_thread(void *arg) > > +{ > > + pthrd_arg *parg = (pthrd_arg *)arg; > > + odp_shm_info_t info; > > + int thr; > > + > > + thr = odp_thread_id(); > > + > > + printf("Thread %i starts\n", thr); > > + > > + switch (parg->testcase) { > > + case ODP_SHM_TEST: > > + shm = odp_shm_lookup(TESTNAME); > > make 'shm' as local variable to detect thread specific lookup failure. > making 'shm' as global will result in overwriting 'shm' > from each thread while other thread in use. > > > + CU_ASSERT(ODP_SHM_INVALID != shm); > > + test_shared_data = odp_shm_addr(shm); > > + CU_ASSERT(0 == odp_shm_info(shm, &info)); > > + CU_ASSERT(0 == strcmp(TESTNAME, info.name)); > > + CU_ASSERT(0 == info.flags); > > + CU_ASSERT(test_shared_data == info.addr); > > + CU_ASSERT(sizeof(test_shared_data_t) == info.size); > > + CU_ASSERT(odp_sys_page_size() == info.page_size); > > + odp_shm_print_all(); > > + break; > > + default: > > + printf("Invalid test case [%d]\n", parg->testcase); > > + } > > + fflush(stdout); > > + return parg; > > +} > > + > > +static void test_odp_shm_sunnyday(void) > > +{ > > + pthrd_arg thrdarg; > > IMO, We should avoid using variables allocated from stack to share > the data between different execution context. > It will work in linux threaded run-time environment.But it will > have issues in different run-time environment like bare-metal. > IMO if any memory needs to be shared across different > execution environment should use the memory allocated from odp shared mem. > > > + > > + shm = odp_shm_reserve(TESTNAME, > > + sizeof(test_shared_data_t), ALIGE_SIZE, 0); > > + CU_ASSERT(ODP_SHM_INVALID != shm); > > + > > + CU_ASSERT(0 == odp_shm_free(shm)); > > + CU_ASSERT(ODP_SHM_INVALID == odp_shm_lookup(TESTNAME)); > > + > > + shm = odp_shm_reserve(TESTNAME, > > + sizeof(test_shared_data_t), ALIGE_SIZE, 0); > > + CU_ASSERT(ODP_SHM_INVALID != shm); > > + > > + test_shared_data = odp_shm_addr(shm); > > + CU_ASSERT(NULL != test_shared_data); > > + memset(test_shared_data, 0, sizeof(test_shared_data_t)); > > + > > + thrdarg.testcase = ODP_SHM_TEST; > > + thrdarg.numthrds = odp_sys_core_count(); > > + > > + if (thrdarg.numthrds > MAX_WORKERS) > > + thrdarg.numthrds = MAX_WORKERS; > > + > > + odp_cunit_thread_create(run_shm_thread, &thrdarg); > > + odp_cunit_thread_exit(&thrdarg); > > +} > > + > > +static int finalize(void) > > +{ > > + odp_term_local(); > > + odp_term_global(); > > + return 0; > > +} > > + > > +static int init(void) > > +{ > > + if (0 != odp_init_global(NULL, NULL)) { > > + printf("odp_init_global fail.\n"); > > + return -1; > > + } > > + if (0 != odp_init_local()) { > > + printf("odp_init_local fail.\n"); > > + return -1; > > + } > > + return 0; > > +} > > + > > +int main(void) > > +{ > > + CU_pSuite ptr_suite; > > + > > + /* initialize the CUnit test registry */ > > + if (CUE_SUCCESS != CU_initialize_registry()) > > + return CU_get_error(); > > + > > + /* add the tests to the shm suite */ > > + ptr_suite = CU_add_suite(__FILE__, init, finalize); > > + if (!ptr_suite) { > > + CU_cleanup_registry(); > > + return CU_get_error(); > > + } > > + > > + if (NULL == CU_ADD_TEST(ptr_suite, test_odp_shm_sunnyday)) { > > + CU_cleanup_registry(); > > + return CU_get_error(); > > + } > > + > > + /* Run all tests using the CUnit Basic interface */ > > + CU_basic_set_mode(CU_BRM_VERBOSE); > > + CU_basic_run_tests(); > > + CU_cleanup_registry(); > > + return CU_get_error(); > > +} > > -- > > 1.8.3.1 > > > > > > _______________________________________________ > > 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