git-lng-odp

git-lng-odp@lists.linaro.org

1 participants
1239 discussions

[COMMIT] branch master updated. v1.19.0.0_tigermoth-16-gf69f033d

by git＠git-us.linaro.org

This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "". The branch, master has been updated via f69f033da3b6900acdd170668cae07ac6cd959f3 (commit) via 8d957f16105fa44d483478b2c05bae735680c007 (commit) via 5028177e4e795305b514d7164ff882dcf1d3622b (commit) from 24262a9aae62290dfd1a041d0bd5c6943270251d (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log ----------------------------------------------------------------- commit f69f033da3b6900acdd170668cae07ac6cd959f3 Author: Dmitry Eremin-Solenikov <dmitry.ereminsolenikov(a)linaro.org> Date: Tue Apr 24 17:39:17 2018 +0300 validation: crypto: add missing ODP_CIPHER_ALG_AES_CTR name Signed-off-by: Dmitry Eremin-Solenikov <dmitry.ereminsolenikov(a)linaro.org> Reviewed-by: Bill Fischofer <bill.fischofer(a)linaro.org> Signed-off-by: Maxim Uvarov <maxim.uvarov(a)linaro.org> diff --git a/test/validation/api/crypto/odp_crypto_test_inp.c b/test/validation/api/crypto/odp_crypto_test_inp.c index 676b9b12..83070290 100644 --- a/test/validation/api/crypto/odp_crypto_test_inp.c +++ b/test/validation/api/crypto/odp_crypto_test_inp.c @@ -79,6 +79,8 @@ static const char *cipher_alg_name(odp_cipher_alg_t cipher) return "ODP_CIPHER_ALG_3DES_CBC"; case ODP_CIPHER_ALG_AES_CBC: return "ODP_CIPHER_ALG_AES_CBC"; + case ODP_CIPHER_ALG_AES_CTR: + return "ODP_CIPHER_ALG_AES_CTR"; case ODP_CIPHER_ALG_AES_GCM: return "ODP_CIPHER_ALG_AES_GCM"; case ODP_CIPHER_ALG_AES_CCM: commit 8d957f16105fa44d483478b2c05bae735680c007 Author: Dmitry Eremin-Solenikov <dmitry.ereminsolenikov(a)linaro.org> Date: Tue Apr 24 11:56:15 2018 +0300 travis: bump DPDK version to 17.11.2 - latest LTS release Signed-off-by: Dmitry Eremin-Solenikov <dmitry.ereminsolenikov(a)linaro.org> Reviewed-by: Matias Elo <matias.elo(a)nokia.com> Signed-off-by: Maxim Uvarov <maxim.uvarov(a)linaro.org> diff --git a/.travis.yml b/.travis.yml index 70eff748..a6cce816 100644 --- a/.travis.yml +++ b/.travis.yml @@ -49,7 +49,7 @@ env: # for individual commit validation. But you you want to track tests history # you need generated new one at https://codecov.io specific for your repo. - CODECOV_TOKEN=a733c34c-5f5c-4ff1-af4b-e9f5edb1ab5e - - DPDK_VERS="17.11" + - DPDK_VERS="17.11.2" matrix: - CONF="" - CONF="--disable-abi-compat" commit 5028177e4e795305b514d7164ff882dcf1d3622b Author: Dmitry Eremin-Solenikov <dmitry.ereminsolenikov(a)linaro.org> Date: Tue Apr 24 11:53:05 2018 +0300 travis: fetch explicitly specified DPDK version Use tag to fetch DPDK, rather than fetching top of corresponding branch. This allows us to specify stable version and stop unnecessary playing with the cache. Signed-off-by: Dmitry Eremin-Solenikov <dmitry.ereminsolenikov(a)linaro.org> Reviewed-by: Matias Elo <matias.elo(a)nokia.com> Signed-off-by: Maxim Uvarov <maxim.uvarov(a)linaro.org> diff --git a/.travis.yml b/.travis.yml index 2f47ed79..70eff748 100644 --- a/.travis.yml +++ b/.travis.yml @@ -154,12 +154,7 @@ install: fi - gem install asciidoctor - # DPDK pktio. Note that cache must be purged if dpdk version changes. - - | - CACHED_DPDK_VERS=`fgrep Version dpdk/pkg/dpdk.spec | cut -d " " -f 2` - if [ "${CACHED_DPDK_VERS}" != "${DPDK_VERS}" ]; then - rm -rf dpdk - fi + # DPDK pktio. Cache will be updated automatically to ${DPDK_VERS} - | case "$CROSS_ARCH" in "arm64") @@ -193,8 +188,17 @@ install: LIBDPDKEXT=a fi DPDK_TARGET="${DPDK_TARGET}gcc" + CACHED_DPDK_VERS=`fgrep Version dpdk/pkg/dpdk.spec | cut -d " " -f 2` + if [ ! -d dpdk -o "${CACHED_DPDK_VERS}" != "${DPDK_VERS}" ]; then + rm -rf dpdk + mkdir dpdk + pushd dpdk + git init + git -c advice.detachedHead=false fetch -q --depth=1 http://dpdk.org/git/dpdk-stable v${DPDK_VERS} + git checkout -f FETCH_HEAD + popd + fi if [ ! -f "dpdk/${TARGET}/usr/local/lib/libdpdk.$LIBDPDKEXT" ]; then - git -c advice.detachedHead=false clone -q --depth=1 --single-branch --branch=${DPDK_VERS} http://dpdk.org/git/dpdk-stable dpdk pushd dpdk git log --oneline --decorate # AArch64 && ARMv7 fixup ----------------------------------------------------------------------- Summary of changes: .travis.yml | 20 ++++++++++++-------- test/validation/api/crypto/odp_crypto_test_inp.c | 2 ++ 2 files changed, 14 insertions(+), 8 deletions(-) hooks/post-receive --

8 years, 4 months

[COMMIT] branch master updated. v1.19.0.0_tigermoth-13-g24262a9a

by git＠git-us.linaro.org

This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "". The branch, master has been updated via 24262a9aae62290dfd1a041d0bd5c6943270251d (commit) via 7e355890905a9195ff1027377991b4134e285a03 (commit) from 6947621e1b3a3c73c3ee351325f505e5e191474f (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log ----------------------------------------------------------------- commit 24262a9aae62290dfd1a041d0bd5c6943270251d Author: Petri Savolainen <petri.savolainen(a)linaro.org> Date: Tue Apr 24 15:53:59 2018 +0300 test: mmap_vlan_ins: wait for background process Each 'make check' run was still leaving a mmap_vlan_ins process behind. Wait until both processes (also the background process) has finished before tearing down the interfaces. Signed-off-by: Petri Savolainen <petri.savolainen(a)linaro.org> Signed-off-by: Maxim Uvarov <maxim.uvarov(a)linaro.org> diff --git a/platform/linux-generic/test/mmap_vlan_ins/mmap_vlan_ins.sh b/platform/linux-generic/test/mmap_vlan_ins/mmap_vlan_ins.sh index ae263ee3..b0eb046b 100755 --- a/platform/linux-generic/test/mmap_vlan_ins/mmap_vlan_ins.sh +++ b/platform/linux-generic/test/mmap_vlan_ins/mmap_vlan_ins.sh @@ -69,8 +69,8 @@ plat_mmap_vlan_ins${EXEEXT} pktiop0p1 pcap:out=${PCAP_OUT} \ plat_mmap_vlan_ins${EXEEXT} pcap:in=${PCAP_IN} pktiop1p0 \ 01:02:03:04:05:06 01:08:09:0a:0b:0c -# Give both processes time to close before removing interfaces -sleep 4 +# Wait both processes to exit before removing the interfaces +wait $! rm -f ${PCAP_OUT} cleanup_pktio_env commit 7e355890905a9195ff1027377991b4134e285a03 Author: Dmitry Eremin-Solenikov <dmitry.ereminsolenikov(a)linaro.org> Date: Mon Apr 23 23:10:03 2018 +0300 linux-gen: ipsec: parse IPv6 next header in tunnel case IPsec parsing code will fail to update next header field in internal state in case of outbound tunnel processing of IPv6 packets. Fixes: https://bugs.linaro.org/show_bug.cgi?id=3764 Signed-off-by: Dmitry Eremin-Solenikov <dmitry.ereminsolenikov(a)linaro.org> Reviewed-by: Bill Fischofer <bill.fischofer(a)linaro.org> Signed-off-by: Maxim Uvarov <maxim.uvarov(a)linaro.org> diff --git a/platform/linux-generic/odp_ipsec.c b/platform/linux-generic/odp_ipsec.c index 782cbf45..7401f590 100644 --- a/platform/linux-generic/odp_ipsec.c +++ b/platform/linux-generic/odp_ipsec.c @@ -844,6 +844,7 @@ static int ipsec_out_tunnel_parse_ipv6(ipsec_state_t *state, state->out_tunnel.ip_flabel = (ipv6hdr->ver_tc_flow & _ODP_IPV6HDR_FLOW_LABEL_MASK) >> _ODP_IPV6HDR_FLOW_LABEL_SHIFT; + state->ip_next_hdr = ipv6hdr->next_hdr; return 0; } ----------------------------------------------------------------------- Summary of changes: platform/linux-generic/odp_ipsec.c | 1 + platform/linux-generic/test/mmap_vlan_ins/mmap_vlan_ins.sh | 4 ++-- 2 files changed, 3 insertions(+), 2 deletions(-) hooks/post-receive --

8 years, 4 months

[COMMIT] branch master updated. v1.19.0.0_tigermoth-11-g6947621e

by git＠git-us.linaro.org

This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "". The branch, master has been updated via 6947621e1b3a3c73c3ee351325f505e5e191474f (commit) from 29cb860583cb906bc16eddf9a4c98d6bb37333e5 (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log ----------------------------------------------------------------- commit 6947621e1b3a3c73c3ee351325f505e5e191474f Author: Josep Puigdemont <josep.puigdemont(a)linaro.org> Date: Mon Apr 23 13:48:22 2018 +0200 linux-gen: fdserver: remove unnecessary locking The locks in fdserver's operations don't serve any purpose, there is no need to acquire a lock when registering a file descriptor to the server, specially because the only place where the registering function is used is already protected by another lock, and the same goes for deregistering. Also, the fdserver handles requests sequentially. On the other hand, removing the lock from the lookup function may return a fd that is being deregistered, but this is not protecting us from misusing it because the operation to be unregistered may be queued right after the lookup request, at which point the fd is already invalid anyway. Signed-off-by: Josep Puigdemont <josep.puigdemont(a)linaro.org> Reviewed-by: Bill Fischofer <bill.fischofer(a)linaro.org> Signed-off-by: Maxim Uvarov <maxim.uvarov(a)linaro.org> diff --git a/platform/linux-generic/odp_fdserver.c b/platform/linux-generic/odp_fdserver.c index 0e9fb0e4..774fe360 100644 --- a/platform/linux-generic/odp_fdserver.c +++ b/platform/linux-generic/odp_fdserver.c @@ -39,7 +39,6 @@ */ #include <odp_posix_extensions.h> -#include <odp/api/spinlock.h> #include <odp_internal.h> #include <odp_debug_internal.h> #include <odp_fdserver_internal.h> @@ -78,9 +77,6 @@ ODP_DBG(fmt, ##__VA_ARGS__);\ } while (0) -/* when accessing the client functions, clients should be mutexed: */ -static odp_spinlock_t *client_lock; - /* define the tables of file descriptors handled by this server: */ #define FDSERVER_MAX_ENTRIES 256 typedef struct fdentry_s { @@ -288,23 +284,18 @@ int _odp_fdserver_register_fd(fd_server_context_e context, uint64_t key, int command; int fd; - odp_spinlock_lock(client_lock); - FD_ODP_DBG("FD client register: pid=%d key=%" PRIu64 ", fd=%d\n", getpid(), key, fd_to_send); s_sock = get_socket(); - if (s_sock < 0) { - odp_spinlock_unlock(client_lock); + if (s_sock < 0) return -1; - } res = send_fdserver_msg(s_sock, FD_REGISTER_REQ, context, key, fd_to_send); if (res < 0) { ODP_ERR("fd registration failure\n"); close(s_sock); - odp_spinlock_unlock(client_lock); return -1; } @@ -313,13 +304,11 @@ int _odp_fdserver_register_fd(fd_server_context_e context, uint64_t key, if ((res < 0) || (command != FD_REGISTER_ACK)) { ODP_ERR("fd registration failure\n"); close(s_sock); - odp_spinlock_unlock(client_lock); return -1; } close(s_sock); - odp_spinlock_unlock(client_lock); return 0; } @@ -334,22 +323,17 @@ int _odp_fdserver_deregister_fd(fd_server_context_e context, uint64_t key) int command; int fd; - odp_spinlock_lock(client_lock); - FD_ODP_DBG("FD client deregister: pid=%d key=%" PRIu64 "\n", getpid(), key); s_sock = get_socket(); - if (s_sock < 0) { - odp_spinlock_unlock(client_lock); + if (s_sock < 0) return -1; - } res = send_fdserver_msg(s_sock, FD_DEREGISTER_REQ, context, key, -1); if (res < 0) { ODP_ERR("fd de-registration failure\n"); close(s_sock); - odp_spinlock_unlock(client_lock); return -1; } @@ -358,13 +342,11 @@ int _odp_fdserver_deregister_fd(fd_server_context_e context, uint64_t key) if ((res < 0) || (command != FD_DEREGISTER_ACK)) { ODP_ERR("fd de-registration failure\n"); close(s_sock); - odp_spinlock_unlock(client_lock); return -1; } close(s_sock); - odp_spinlock_unlock(client_lock); return 0; } @@ -380,19 +362,14 @@ int _odp_fdserver_lookup_fd(fd_server_context_e context, uint64_t key) int command; int fd; - odp_spinlock_lock(client_lock); - s_sock = get_socket(); - if (s_sock < 0) { - odp_spinlock_unlock(client_lock); + if (s_sock < 0) return -1; - } res = send_fdserver_msg(s_sock, FD_LOOKUP_REQ, context, key, -1); if (res < 0) { ODP_ERR("fd lookup failure\n"); close(s_sock); - odp_spinlock_unlock(client_lock); return -1; } @@ -401,7 +378,6 @@ int _odp_fdserver_lookup_fd(fd_server_context_e context, uint64_t key) if ((res < 0) || (command != FD_LOOKUP_ACK)) { ODP_ERR("fd lookup failure\n"); close(s_sock); - odp_spinlock_unlock(client_lock); return -1; } @@ -409,7 +385,6 @@ int _odp_fdserver_lookup_fd(fd_server_context_e context, uint64_t key) ODP_DBG("FD client lookup: pid=%d, key=%" PRIu64 ", fd=%d\n", getpid(), key, fd); - odp_spinlock_unlock(client_lock); return fd; } @@ -421,27 +396,21 @@ static int stop_server(void) int s_sock; /* server socket */ int res; - odp_spinlock_lock(client_lock); - FD_ODP_DBG("FD sending server stop request\n"); s_sock = get_socket(); - if (s_sock < 0) { - odp_spinlock_unlock(client_lock); + if (s_sock < 0) return -1; - } res = send_fdserver_msg(s_sock, FD_SERVERSTOP_REQ, 0, 0, -1); if (res < 0) { ODP_ERR("fd stop request failure\n"); close(s_sock); - odp_spinlock_unlock(client_lock); return -1; } close(s_sock); - odp_spinlock_unlock(client_lock); return 0; } @@ -595,12 +564,6 @@ int _odp_fdserver_init_global(void) pid_t server_pid; int res; - /* create the client spinlock that any client can see: */ - client_lock = mmap(NULL, sizeof(odp_spinlock_t), PROT_READ | PROT_WRITE, - MAP_SHARED | MAP_ANONYMOUS, -1, 0); - - odp_spinlock_init(client_lock); - snprintf(sockpath, FDSERVER_SOCKPATH_MAXLEN, FDSERVER_SOCKDIR_FORMAT, odp_global_data.shm_dir, odp_global_data.uid); ----------------------------------------------------------------------- Summary of changes: platform/linux-generic/odp_fdserver.c | 45 ++++------------------------------- 1 file changed, 4 insertions(+), 41 deletions(-) hooks/post-receive --

8 years, 4 months

[COMMIT] branch api-next updated. v1.19.0.0_tigermoth-10-g29cb8605

by git＠git-us.linaro.org

This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "". The branch, api-next has been updated discards 1c11eb2f0e612eb9b7a6cbe91a3e4ac3e61c7b80 (commit) via 29cb860583cb906bc16eddf9a4c98d6bb37333e5 (commit) via acc05760d5060d1782aa597e44c72ccb9a71b63b (commit) via c440dfceff97526d2763383afd8da064faef3d0d (commit) via 93591a6134fbc5655c0cf6a25e0b4e0668357971 (commit) via fec701841dbe2e963cf3adc7c74c98b41e059b9d (commit) via 0e24607e3075a573ebe4ac408703c76daac7236c (commit) via 5d4be5077adeec54d1956c86dcb87b186c39a7cb (commit) via 3aa3f4352083d159260cebc23b87607e73aa6f2b (commit) via 0d4271e31fe685a8a82381d21d5166f0398080c1 (commit) via d324b9de12c510ccf9fa1c1b6ece327c578ee53d (commit) via e828b4d6f503ff94c40e30b1d8babf0dcbecde91 (commit) via 1e0ac11c75eee10959d1fa674a05e746476271b3 (commit) via 8caae505c35444706cff8815c41821d80e791403 (commit) via 4b6787c84e020c350a03ec47d12e1ef68ddb6300 (commit) via 4142652b8eb397963bec48610bdb9ab6599e65fb (commit) via 60777ba4309911df0780bd8de6d96524bc2ca03c (commit) via b2632ac253a3b9f5ba7a4174acf6abe295ec3550 (commit) via 79f5c103310c36dc213a552b9534f68ce8df1132 (commit) via 2c99a9dc1978c9178e711d264aaaca2bc4702352 (commit) via 88ba35773c76c871a222fe48a0dac05765c39936 (commit) via 2e96afe0b7c4e7e41fb03bcda605ab704f538ca1 (commit) via 52ea8d51d522d7013f746748bf84bfc2e9bc042c (commit) via b7ee13d765e5d8da1ef68f0c0777cb5e40fffbf7 (commit) via a2cad95118b79f204bc13914acf43a8fc1dd2682 (commit) via a5340c53957b3174c89d03ecf2c61e9ba98ac57c (commit) via 813cd2df67ae0591b0f792eff82854c5c9c642ca (commit) via 48f8503baf847315dda6609abc01f5befe6e0284 (commit) via b1100fdba29d8f514236dbcfd6f17e7941c0256a (commit) via f96a21992e13cf401a3494f2caeec9d42f56f0d9 (commit) via 1e925d0ccc7340575ddcfa07ab7bd372d751e006 (commit) via c9d31f30ba4fd7818b4ba2ffd7bbaea74761a2d9 (commit) via cabee48a54a68f333569ee1904bc04d53da7c9c6 (commit) via 183ce9babe20e161caef6b55eccfd043b334f748 (commit) via 0fa9977e21010f963892f59a75f03e3ba800bc74 (commit) via b0b07f403ffc22349ac8dd75f47238bea40891be (commit) via bd0377ae95427e80f04b0dd5f3a8dcb0b1ab89e3 (commit) via 5555878edbda765134fbe50f3181f409963037ea (commit) via 009dab38672eaf8ab6eddc551da41a84e86915a5 (commit) via 290decaf3259b036c3a402be428b04ba100f0f81 (commit) via 61d16e162788ac0923c544b4d31bb847fa3d9189 (commit) via 83fda5a447335b95bdd991187d4efb0a15a5709a (commit) via 654507c6eeed83dd84210f7e481ded34f5d4cdca (commit) via 84f8013995d4b96ac9471cc92285c76b50464415 (commit) via 92ebb26bae583ff864bc4bef6a2776329527a069 (commit) via ed4e1bfea15d8502e50b6c210f38dd3f13683d2f (commit) via ef5998dfd0ba1405fef6dd1a7f0e9adf686fca89 (commit) via a430caf661d4a0ad8d69c67dc7a20911e3169a25 (commit) via fbe07de54f3768f32bd39a0ea9aa2a79188b1186 (commit) via 57df98eca15fe64a716f9f555714ef5fb9cd25e9 (commit) via 674ea087fba016e55f03e3966fe64fc908141f8f (commit) via a9584da4284c04e45a9584a58c6dbeb404871119 (commit) via 63123149319eb0e379dc52a3f4691993d026f3bf (commit) via 9a5a18af733c07109224e328ca0ac640ff49f845 (commit) via 06321dc028b83ea78b39eb9673859a40f9da37f7 (commit) via d6de4dfcf1a3a6c861c87d47990f793c71328eda (commit) via fc0a57306aa68a782e8611a9bd08ae10aaa5274a (commit) via 2eaf47b69ea5f2095e26e0ea597d3e8702b0da06 (commit) via 9f3311f92d28ef9fdf6ef3d8ec8b90efe7a832d6 (commit) via 68b1a52611e6ac9be306e5a17d009a6fd46c455e (commit) via 59d636481fd1771107f3eb4632c2a9fc868afc11 (commit) via 0225a8a396ddc7168be096f28b0a711184a48ef9 (commit) via 72febbae5ea55a5fd051978fbfa0f669cf0e99fc (commit) via f80f28c4046b650c180bcb9b41c859c0c8e2564e (commit) via b08aa94548163ae4d4bac2724e68ca9102667f6a (commit) via 4aacfd4d72b80a22d449d865a6c4f063afa823ab (commit) via 71ca38c00c11ebb6b4dede6d09ebdd28eb45d890 (commit) via 7e0c523ad6f209f238277e4116c21ba696edb7f4 (commit) via 12d238d96318cc9d46dd5a2893894824a633805b (commit) via bcd246e3a9948a631eb8eb77886ac1b369b13dba (commit) via c57da8ddc80e0e314957c6d11db288ef11fc1805 (commit) via 535ad269bfc6581a048d3e844576063be5d19501 (commit) via d8dd54f2325cf859703721e7a17b6cbe734d857a (commit) via edc9616a1bbc0fbc2d7f84ddafeb119d689f3f70 (commit) via 31d3492b50b057df853ad51c909ac21d6e95a4ca (commit) via 74f7b6d1a003175f1cddf979212631a0d7832399 (commit) via 3c5cc8070d3bc8b7429f0410de6ef3009ff6a28f (commit) via 4665f6d2987fb26ad006cb56b02b3ef4c170f371 (commit) via d3690d4115441dc69d9571cb5261dd005a112d39 (commit) via 364887331875180c498abb8b7681a228ca21181e (commit) via eca5df13cf7987d6e3cd3cc930851671b071e4b3 (commit) via c8dc27d1fb5c890143338439f1cdbe3b2719174c (commit) via 62ef9d5c92e0f56f45767f4c73df214d4393e158 (commit) via 41fd242a0685e79cb6f1b1e1826e6734bea2f359 (commit) via 35a90a547466db90c00d6375d8d9a3573e84e99b (commit) via 9a1d48b1262d88e55922c4e1ed9f91ed2e3376d7 (commit) via 1e70adc3ae54b0c4518bdb51d94301f1dafe4ebf (commit) via 91282a4c578716f157aa8a5e0a9bf9bd25b19a17 (commit) via e009051a0166fa25e601190354e0333aa30ad58d (commit) via 36f589b5a84804ce246bcd9a3eb9352801aaf41d (commit) via 2bbd7ace9e236d27dcfb52acfef991bd5ce39354 (commit) via 196d8f518071cc8c37dde0835cb7559d6cdd55da (commit) via 42fc5ebdcd5a1e5fbb7a19a2756a310f23a0cdd5 (commit) via bb8015d15ca4b0e2033eeca09906032a88e74bff (commit) via 3e61be54a77ed8ccc2030e88b9a26372d3f76e2c (commit) via c0da87688187c81039cf81790fed3d6ed00a956e (commit) via c2a1c48f5fc17cf4dd870057d17059ad668d29ad (commit) via c67dcec1c034a0c75c093b49e2a172bbd3d50def (commit) via 80b9a1e4d6a297ac916cbaf360e7b2114ef86a7a (commit) via 7135eda6854ba60d6379ca5799b4b58959e998b5 (commit) via 017969502cd0247c9b25e80e002ac563860f5a5d (commit) via 3a4f369fb2019761af5d4d7ad09d48f1e5d44f82 (commit) via 962600ecdbe44d386f95a57ac5a91365af0b6c39 (commit) via 6ea708c0f9f36b9a90f463dd1f7e0332f82e079b (commit) via 534667daec58ce8b2e417709e788934bea6f8f09 (commit) This update added new revisions after undoing existing revisions. That is to say, the old revision is not a strict subset of the new revision. This situation occurs when you --force push a change and generate a repository containing something like this: * -- * -- B -- O -- O -- O (1c11eb2f0e612eb9b7a6cbe91a3e4ac3e61c7b80) \ N -- N -- N (29cb860583cb906bc16eddf9a4c98d6bb37333e5) When this happens we assume that you've already had alert emails for all of the O revisions, and so we here report only the revisions in the N branch from the common base, B. Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log ----------------------------------------------------------------- ----------------------------------------------------------------------- Summary of changes: .checkpatch.conf | 5 + .shippable.yml | 21 +- .travis.yml | 20 +- CHANGELOG | 154 ++ DEPENDENCIES | 6 +- Makefile.am | 8 +- config/README | 10 + config/odp-linux-generic.conf | 58 + configure.ac | 21 +- doc/users-guide/users-guide-cls.adoc | 2 +- doc/users-guide/users-guide-crypto.adoc | 2 +- doc/users-guide/users-guide-ipsec.adoc | 4 +- doc/users-guide/users-guide-packet.adoc | 35 + doc/users-guide/users-guide-pktio.adoc | 374 ++++- doc/users-guide/users-guide-tm.adoc | 8 +- doc/users-guide/users-guide.adoc | 68 +- example/Makefile.inc | 2 + example/l2fwd/README | 7 + m4/odp_dpdk.m4 | 11 +- m4/odp_libconfig.m4 | 29 + .../m4/odp_schedule.m4 => m4/odp_scheduler.m4 | 5 + platform/Makefile.inc | 5 +- platform/linux-generic/.gitignore | 1 + platform/linux-generic/Makefile.am | 76 +- .../linux-generic/include/odp_buffer_inlines.h | 5 + ...fdserver_internal.h => odp_fdserver_internal.h} | 0 platform/linux-generic/include/odp_internal.h | 5 + .../{_ishm_internal.h => odp_ishm_internal.h} | 0 ...{_ishmphy_internal.h => odp_ishmphy_internal.h} | 0 ...ishmpool_internal.h => odp_ishmpool_internal.h} | 0 .../linux-generic/include/odp_libconfig_internal.h | 34 + platform/linux-generic/include/odp_packet_dpdk.h | 10 +- .../linux-generic/include/odp_packet_internal.h | 4 + platform/linux-generic/include/odp_packet_netmap.h | 7 + .../linux-generic/include/odp_queue_internal.h | 24 + platform/linux-generic/include/odp_queue_lf.h | 1 - platform/linux-generic/include/odp_schedule_if.h | 8 +- .../include/odp_schedule_scalable_ordered.h | 2 +- platform/linux-generic/libodp-linux.pc.in | 1 + platform/linux-generic/m4/configure.m4 | 5 +- platform/linux-generic/odp_crypto.c | 256 +++- .../linux-generic/{_fdserver.c => odp_fdserver.c} | 77 +- platform/linux-generic/odp_impl.c | 30 +- platform/linux-generic/odp_init.c | 14 + platform/linux-generic/odp_ipsec.c | 214 +-- platform/linux-generic/odp_ipsec_sad.c | 9 + platform/linux-generic/{_ishm.c => odp_ishm.c} | 131 +- .../linux-generic/{_ishmphy.c => odp_ishmphy.c} | 6 +- .../linux-generic/{_ishmpool.c => odp_ishmpool.c} | 4 +- platform/linux-generic/odp_libconfig.c | 135 ++ platform/linux-generic/odp_packet.c | 239 +++ platform/linux-generic/odp_packet_io.c | 145 +- platform/linux-generic/odp_pool.c | 9 +- platform/linux-generic/odp_queue_basic.c | 213 ++- platform/linux-generic/odp_queue_lf.c | 2 +- platform/linux-generic/odp_queue_scalable.c | 4 +- platform/linux-generic/odp_random.c | 54 + platform/linux-generic/odp_schedule_basic.c | 633 ++++---- platform/linux-generic/odp_schedule_iquery.c | 23 +- platform/linux-generic/odp_schedule_scalable.c | 4 +- platform/linux-generic/odp_schedule_sp.c | 7 +- platform/linux-generic/odp_shared_memory.c | 2 +- platform/linux-generic/odp_system_info.c | 26 +- platform/linux-generic/pktio/dpdk.c | 111 +- platform/linux-generic/pktio/ipc.c | 2 +- platform/linux-generic/pktio/loop.c | 110 +- platform/linux-generic/pktio/netmap.c | 63 + .../test/mmap_vlan_ins/mmap_vlan_ins.sh | 3 + platform/linux-generic/test/ring/Makefile.am | 2 + scripts/build-pktio-dpdk | 7 +- scripts/checkpatch.pl | 1552 ++++++++++++++++---- scripts/ci-checkpatches.sh | 4 +- test/Makefile.inc | 2 + test/miscellaneous/odp_api_from_cpp.cpp | 6 +- test/performance/.gitignore | 2 + test/performance/Makefile.am | 11 + test/performance/odp_crypto.c | 271 +++- test/performance/{odp_crypto.c => odp_ipsec.c} | 1035 +++++++------ test/performance/odp_sched_latency.c | 9 +- test/performance/odp_sched_pktio.c | 800 ++++++++++ .../{odp_l2fwd_run.sh => odp_sched_pktio_run.sh} | 69 +- .../api/classification/odp_classification_tests.c | 59 +- .../classification/odp_classification_testsuites.h | 12 + test/validation/api/crypto/odp_crypto_test_inp.c | 95 +- test/validation/api/crypto/test_vectors.h | 209 +++ test/validation/api/crypto/test_vectors_len.h | 10 + test/validation/api/ipsec/ipsec.c | 17 +- test/validation/api/ipsec/ipsec_test_out.c | 194 +++ test/validation/api/ipsec/test_vectors.h | 89 +- test/validation/api/pktio/pktio.c | 669 ++++++--- test/validation/api/scheduler/scheduler.c | 4 +- test/validation/api/timer/timer.c | 336 ++++- 92 files changed, 7070 insertions(+), 1958 deletions(-) create mode 100644 config/odp-linux-generic.conf create mode 100644 m4/odp_libconfig.m4 rename platform/linux-generic/m4/odp_schedule.m4 => m4/odp_scheduler.m4 (73%) rename platform/linux-generic/include/{_fdserver_internal.h => odp_fdserver_internal.h} (100%) rename platform/linux-generic/include/{_ishm_internal.h => odp_ishm_internal.h} (100%) rename platform/linux-generic/include/{_ishmphy_internal.h => odp_ishmphy_internal.h} (100%) rename platform/linux-generic/include/{_ishmpool_internal.h => odp_ishmpool_internal.h} (100%) create mode 100644 platform/linux-generic/include/odp_libconfig_internal.h rename platform/linux-generic/{_fdserver.c => odp_fdserver.c} (91%) rename platform/linux-generic/{_ishm.c => odp_ishm.c} (98%) rename platform/linux-generic/{_ishmphy.c => odp_ishmphy.c} (98%) rename platform/linux-generic/{_ishmpool.c => odp_ishmpool.c} (99%) create mode 100644 platform/linux-generic/odp_libconfig.c create mode 100644 platform/linux-generic/odp_random.c copy test/performance/{odp_crypto.c => odp_ipsec.c} (56%) create mode 100644 test/performance/odp_sched_pktio.c copy test/performance/{odp_l2fwd_run.sh => odp_sched_pktio_run.sh} (50%) hooks/post-receive --

8 years, 4 months

[COMMIT] branch master updated. v1.19.0.0_tigermoth-10-g29cb8605

by git＠git-us.linaro.org

This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "". The branch, master has been updated via 29cb860583cb906bc16eddf9a4c98d6bb37333e5 (commit) from acc05760d5060d1782aa597e44c72ccb9a71b63b (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log ----------------------------------------------------------------- commit 29cb860583cb906bc16eddf9a4c98d6bb37333e5 Author: Petri Savolainen <petri.savolainen(a)linaro.org> Date: Mon Apr 23 13:52:31 2018 +0300 test: crypto perf: check algorithm capabilities Test only those algorithms that implementation supports. Signed-off-by: Petri Savolainen <petri.savolainen(a)linaro.org> Reviewed-by: Dmitry Eremin-Solenikov <dmitry.ereminsolenikov(a)linaro.org> Signed-off-by: Maxim Uvarov <maxim.uvarov(a)linaro.org> diff --git a/test/performance/odp_crypto.c b/test/performance/odp_crypto.c index 8841c127..8a5f2c5b 100644 --- a/test/performance/odp_crypto.c +++ b/test/performance/odp_crypto.c @@ -187,6 +187,14 @@ typedef struct { struct rusage ru_thread; /**< Rusage value for current thread */ } time_record_t; +/* Arguments for one test run */ +typedef struct test_run_arg_t { + crypto_args_t crypto_args; + crypto_alg_config_t *crypto_alg_config; + odp_crypto_capability_t crypto_capa; + +} test_run_arg_t; + static void parse_args(int argc, char *argv[], crypto_args_t *cargs); static void usage(char *progname); @@ -856,18 +864,106 @@ run_measure_one(crypto_args_t *cargs, return rc < 0 ? rc : 0; } +static int check_cipher_alg(odp_crypto_capability_t *capa, + odp_cipher_alg_t alg) +{ + switch (alg) { + case ODP_CIPHER_ALG_NULL: + if (capa->ciphers.bit.null) + return 0; + case ODP_CIPHER_ALG_DES: + if (capa->ciphers.bit.des) + return 0; + case ODP_CIPHER_ALG_3DES_CBC: + if (capa->ciphers.bit.trides_cbc) + return 0; + case ODP_CIPHER_ALG_AES_CBC: + if (capa->ciphers.bit.aes_cbc) + return 0; + case ODP_CIPHER_ALG_AES_CTR: + if (capa->ciphers.bit.aes_ctr) + return 0; + case ODP_CIPHER_ALG_AES_GCM: + if (capa->ciphers.bit.aes_gcm) + return 0; + case ODP_CIPHER_ALG_AES_CCM: + if (capa->ciphers.bit.aes_ccm) + return 0; + case ODP_CIPHER_ALG_CHACHA20_POLY1305: + if (capa->ciphers.bit.chacha20_poly1305) + return 0; + default: + return -1; + } +} + +static int check_auth_alg(odp_crypto_capability_t *capa, + odp_auth_alg_t alg) +{ + switch (alg) { + case ODP_AUTH_ALG_NULL: + if (capa->auths.bit.null) + return 0; + case ODP_AUTH_ALG_MD5_HMAC: + if (capa->auths.bit.md5_hmac) + return 0; + case ODP_AUTH_ALG_SHA1_HMAC: + if (capa->auths.bit.sha1_hmac) + return 0; + case ODP_AUTH_ALG_SHA256_HMAC: + if (capa->auths.bit.sha256_hmac) + return 0; + case ODP_AUTH_ALG_SHA384_HMAC: + if (capa->auths.bit.sha384_hmac) + return 0; + case ODP_AUTH_ALG_SHA512_HMAC: + if (capa->auths.bit.sha512_hmac) + return 0; + case ODP_AUTH_ALG_AES_GCM: + if (capa->auths.bit.aes_gcm) + return 0; + case ODP_AUTH_ALG_AES_GMAC: + if (capa->auths.bit.aes_gmac) + return 0; + case ODP_AUTH_ALG_AES_CCM: + if (capa->auths.bit.aes_ccm) + return 0; + case ODP_AUTH_ALG_CHACHA20_POLY1305: + if (capa->auths.bit.chacha20_poly1305) + return 0; + default: + return -1; + } +} + /** * Process one algorithm. Note if paload size is specicified it is * only one run. Or iterate over set of predefined payloads. */ -static int -run_measure_one_config(crypto_args_t *cargs, - crypto_alg_config_t *config) +static int run_measure_one_config(test_run_arg_t *arg) { crypto_run_result_t result; odp_crypto_session_t session; + crypto_args_t *cargs = &arg->crypto_args; + crypto_alg_config_t *config = arg->crypto_alg_config; + odp_crypto_capability_t crypto_capa = arg->crypto_capa; int rc = 0; + if (check_cipher_alg(&crypto_capa, config->session.cipher_alg)) { + printf(" Cipher algorithm not supported\n"); + rc = 1; + } + + if (check_auth_alg(&crypto_capa, config->session.auth_alg)) { + printf(" Auth algorithm not supported\n"); + rc = 1; + } + + if (rc) { + printf(" => %s skipped\n\n", config->name); + return 0; + } + if (create_session_from_config(&session, config, cargs)) return -1; @@ -898,17 +994,9 @@ run_measure_one_config(crypto_args_t *cargs, return rc; } -typedef struct thr_arg { - crypto_args_t crypto_args; - crypto_alg_config_t *crypto_alg_config; -} thr_arg_t; - static int run_thr_func(void *arg) { - thr_arg_t *thr_args = (thr_arg_t *)arg; - - run_measure_one_config(&thr_args->crypto_args, - thr_args->crypto_alg_config); + run_measure_one_config((test_run_arg_t *)arg); return 0; } @@ -919,13 +1007,14 @@ int main(int argc, char *argv[]) odp_queue_param_t qparam; odp_pool_param_t params; odp_queue_t out_queue = ODP_QUEUE_INVALID; - thr_arg_t thr_arg; + test_run_arg_t test_run_arg; odp_cpumask_t cpumask; char cpumaskstr[ODP_CPUMASK_STR_SIZE]; int num_workers = 1; odph_odpthread_t thr[num_workers]; odp_instance_t instance; - odp_pool_capability_t capa; + odp_pool_capability_t pool_capa; + odp_crypto_capability_t crypto_capa; uint32_t max_seg_len; unsigned i; @@ -943,12 +1032,17 @@ int main(int argc, char *argv[]) /* Init this thread */ odp_init_local(instance, ODP_THREAD_WORKER); - if (odp_pool_capability(&capa)) { + if (odp_crypto_capability(&crypto_capa)) { + app_err("Crypto capability request failed.\n"); + exit(EXIT_FAILURE); + } + + if (odp_pool_capability(&pool_capa)) { app_err("Pool capability request failed.\n"); exit(EXIT_FAILURE); } - max_seg_len = capa.pkt.max_seg_len; + max_seg_len = pool_capa.pkt.max_seg_len; for (i = 0; i < sizeof(payloads) / sizeof(unsigned int); i++) { if (payloads[i] > max_seg_len) @@ -991,9 +1085,6 @@ int main(int argc, char *argv[]) if (cargs.schedule) { printf("Run in async scheduled mode\n"); - - thr_arg.crypto_args = cargs; - thr_arg.crypto_alg_config = cargs.alg_config; num_workers = odp_cpumask_default_worker(&cpumask, num_workers); (void)odp_cpumask_to_str(&cpumask, cpumaskstr, @@ -1012,12 +1103,16 @@ int main(int argc, char *argv[]) memset(thr, 0, sizeof(thr)); + test_run_arg.crypto_args = cargs; + test_run_arg.crypto_alg_config = cargs.alg_config; + test_run_arg.crypto_capa = crypto_capa; + if (cargs.alg_config) { odph_odpthread_params_t thr_params; memset(&thr_params, 0, sizeof(thr_params)); thr_params.start = run_thr_func; - thr_params.arg = &thr_arg; + thr_params.arg = &test_run_arg; thr_params.thr_type = ODP_THREAD_WORKER; thr_params.instance = instance; @@ -1025,7 +1120,7 @@ int main(int argc, char *argv[]) odph_odpthreads_create(&thr[0], &cpumask, &thr_params); odph_odpthreads_join(&thr[0]); } else { - run_measure_one_config(&cargs, cargs.alg_config); + run_measure_one_config(&test_run_arg); } } else { unsigned int i; @@ -1033,7 +1128,8 @@ int main(int argc, char *argv[]) for (i = 0; i < (sizeof(algs_config) / sizeof(crypto_alg_config_t)); i++) { - run_measure_one_config(&cargs, algs_config + i); + test_run_arg.crypto_alg_config = algs_config + i; + run_measure_one_config(&test_run_arg); } } ----------------------------------------------------------------------- Summary of changes: test/performance/odp_crypto.c | 140 +++++++++++++++++++++++++++++++++++------- 1 file changed, 118 insertions(+), 22 deletions(-) hooks/post-receive --

8 years, 4 months

[COMMIT] branch master updated. v1.19.0.0_tigermoth-9-gacc05760

by git＠git-us.linaro.org

This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "". The branch, master has been updated via acc05760d5060d1782aa597e44c72ccb9a71b63b (commit) from c440dfceff97526d2763383afd8da064faef3d0d (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log ----------------------------------------------------------------- commit acc05760d5060d1782aa597e44c72ccb9a71b63b Author: Dmitry Eremin-Solenikov <dmitry.ereminsolenikov(a)linaro.org> Date: Tue Apr 3 15:48:12 2018 +0300 validation: crypto: zero return code of odp_crypto_op/odp_crypto_op_enq crypto_main test will accept odp_crypto_op/odp_crypto_op_enq returning 0 meaning 0 packets were processed. Instead it should mark test as failedin this case. Fixes: https://bugs.linaro.org/show_bug.cgi?id=3763 Signed-off-by: Dmitry Eremin-Solenikov <dmitry.ereminsolenikov(a)linaro.org> Reviewed-by: Bill Fischofer <bill.fischofer(a)linaro.org> Signed-off-by: Maxim Uvarov <maxim.uvarov(a)linaro.org> diff --git a/test/validation/api/crypto/odp_crypto_test_inp.c b/test/validation/api/crypto/odp_crypto_test_inp.c index aa053dd5..676b9b12 100644 --- a/test/validation/api/crypto/odp_crypto_test_inp.c +++ b/test/validation/api/crypto/odp_crypto_test_inp.c @@ -200,7 +200,7 @@ static int alg_packet_op(odp_packet_t pkt, op_params.hash_result_offset = plaintext_len; rc = odp_crypto_op(&pkt, &out_pkt, &op_params, 1); - if (rc < 0) { + if (rc <= 0) { CU_FAIL("Failed odp_crypto_packet_op()"); return rc; } @@ -261,7 +261,7 @@ static int alg_packet_op_enq(odp_packet_t pkt, op_params.hash_result_offset = plaintext_len; rc = odp_crypto_op_enq(&pkt, &pkt, &op_params, 1); - if (rc < 0) { + if (rc <= 0) { CU_FAIL("Failed odp_crypto_op_enq()"); return rc; } ----------------------------------------------------------------------- Summary of changes: test/validation/api/crypto/odp_crypto_test_inp.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) hooks/post-receive --

8 years, 4 months

[COMMIT] branch master updated. v1.19.0.0_tigermoth-8-gc440dfce

by git＠git-us.linaro.org

This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "". The branch, master has been updated via c440dfceff97526d2763383afd8da064faef3d0d (commit) via 93591a6134fbc5655c0cf6a25e0b4e0668357971 (commit) via fec701841dbe2e963cf3adc7c74c98b41e059b9d (commit) via 0e24607e3075a573ebe4ac408703c76daac7236c (commit) via 5d4be5077adeec54d1956c86dcb87b186c39a7cb (commit) via 3aa3f4352083d159260cebc23b87607e73aa6f2b (commit) via 0d4271e31fe685a8a82381d21d5166f0398080c1 (commit) via d324b9de12c510ccf9fa1c1b6ece327c578ee53d (commit) from e828b4d6f503ff94c40e30b1d8babf0dcbecde91 (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log ----------------------------------------------------------------- commit c440dfceff97526d2763383afd8da064faef3d0d Author: Dmitry Eremin-Solenikov <dmitry.ereminsolenikov(a)linaro.org> Date: Sun Apr 22 10:18:17 2018 +0300 linux-generic: random: split from crypto module While random and crypto might share some implementation details, in case of linux-generic and linux-DPDK it might be easier to split them to two different files. Signed-off-by: Dmitry Eremin-Solenikov <dmitry.ereminsolenikov(a)linaro.org> Reviewed-by: Bill Fischofer <bill.fischofer(a)linaro.org> Signed-off-by: Maxim Uvarov <maxim.uvarov(a)linaro.org> diff --git a/platform/linux-generic/Makefile.am b/platform/linux-generic/Makefile.am index 4f03eab1..7d1d5554 100644 --- a/platform/linux-generic/Makefile.am +++ b/platform/linux-generic/Makefile.am @@ -175,6 +175,7 @@ __LIB__libodp_linux_la_SOURCES = \ odp_queue_if.c \ odp_queue_lf.c \ odp_queue_scalable.c \ + odp_random.c \ odp_rwlock.c \ odp_rwlock_recursive.c \ odp_schedule_basic.c \ diff --git a/platform/linux-generic/odp_crypto.c b/platform/linux-generic/odp_crypto.c index 852f0212..aca822d8 100644 --- a/platform/linux-generic/odp_crypto.c +++ b/platform/linux-generic/odp_crypto.c @@ -23,7 +23,6 @@ #include <string.h> #include <stdlib.h> -#include <openssl/rand.h> #include <openssl/hmac.h> #include <openssl/cmac.h> #include <openssl/evp.h> @@ -1954,47 +1953,6 @@ int _odp_crypto_term_local(void) return 0; } -odp_random_kind_t odp_random_max_kind(void) -{ - return ODP_RANDOM_CRYPTO; -} - -int32_t odp_random_data(uint8_t *buf, uint32_t len, odp_random_kind_t kind) -{ - int rc; - - switch (kind) { - case ODP_RANDOM_BASIC: - case ODP_RANDOM_CRYPTO: - rc = RAND_bytes(buf, len); - return (1 == rc) ? (int)len /*success*/: -1 /*failure*/; - - case ODP_RANDOM_TRUE: - default: - return -1; - } -} - -int32_t odp_random_test_data(uint8_t *buf, uint32_t len, uint64_t *seed) -{ - union { - uint32_t rand_word; - uint8_t rand_byte[4]; - } u; - uint32_t i = 0, j; - uint32_t seed32 = (*seed) & 0xffffffff; - - while (i < len) { - u.rand_word = rand_r(&seed32); - - for (j = 0; j < 4 && i < len; j++, i++) - *buf++ = u.rand_byte[j]; - } - - *seed = seed32; - return len; -} - odp_crypto_compl_t odp_crypto_compl_from_event(odp_event_t ev) { /* This check not mandated by the API specification */ diff --git a/platform/linux-generic/odp_random.c b/platform/linux-generic/odp_random.c new file mode 100644 index 00000000..22894c47 --- /dev/null +++ b/platform/linux-generic/odp_random.c @@ -0,0 +1,54 @@ +/* Copyright (c) 2014-2018, Linaro Limited + * All rights reserved. + * + * SPDX-License-Identifier: BSD-3-Clause + */ + +#include "config.h" + +#include <odp_posix_extensions.h> +#include <odp_internal.h> +#include <odp/api/random.h> + +#include <openssl/rand.h> + +odp_random_kind_t odp_random_max_kind(void) +{ + return ODP_RANDOM_CRYPTO; +} + +int32_t odp_random_data(uint8_t *buf, uint32_t len, odp_random_kind_t kind) +{ + int rc; + + switch (kind) { + case ODP_RANDOM_BASIC: + case ODP_RANDOM_CRYPTO: + rc = RAND_bytes(buf, len); + return (1 == rc) ? (int)len /*success*/: -1 /*failure*/; + + case ODP_RANDOM_TRUE: + default: + return -1; + } +} + +int32_t odp_random_test_data(uint8_t *buf, uint32_t len, uint64_t *seed) +{ + union { + uint32_t rand_word; + uint8_t rand_byte[4]; + } u; + uint32_t i = 0, j; + uint32_t seed32 = (*seed) & 0xffffffff; + + while (i < len) { + u.rand_word = rand_r(&seed32); + + for (j = 0; j < 4 && i < len; j++, i++) + *buf++ = u.rand_byte[j]; + } + + *seed = seed32; + return len; +} commit 93591a6134fbc5655c0cf6a25e0b4e0668357971 Author: Petri Savolainen <petri.savolainen(a)linaro.org> Date: Fri Apr 20 14:25:30 2018 +0300 validation: timer: honor min timeout param value Timeout requests should be between min_tmo and max_tmo timer pool parameters, if too early/too late return codes are reported as test failures. Signed-off-by: Petri Savolainen <petri.savolainen(a)linaro.org> Reviewed-by: Bill Fischofer <bill.fischofer(a)linaro.org> Signed-off-by: Maxim Uvarov <maxim.uvarov(a)linaro.org> diff --git a/test/validation/api/timer/timer.c b/test/validation/api/timer/timer.c index f81a3d09..a34885c0 100644 --- a/test/validation/api/timer/timer.c +++ b/test/validation/api/timer/timer.c @@ -497,7 +497,7 @@ static int worker_entrypoint(void *arg TEST_UNUSED) uint32_t ncancel; uint32_t ntoolate; uint32_t ms; - uint64_t prev_tick; + uint64_t prev_tick, nsec; odp_event_t ev; struct timespec ts; uint32_t nstale; @@ -539,9 +539,9 @@ static int worker_entrypoint(void *arg TEST_UNUSED) /* Initial set all timers with a random expiration time */ nset = 0; for (i = 0; i < allocated; i++) { - tck = odp_timer_current_tick(tp) + 1 + - odp_timer_ns_to_tick(tp, (rand_r(&seed) % RANGE_MS) - * 1000000ULL); + nsec = MIN_TMO + (rand_r(&seed) % RANGE_MS) * 1000000ULL; + tck = odp_timer_current_tick(tp) + + odp_timer_ns_to_tick(tp, nsec); timer_rc = odp_timer_set_abs(tt[i].tim, tck, &tt[i].ev); if (timer_rc == ODP_TIMER_TOOEARLY) { LOG_ERR("Missed tick, setting timer\n"); @@ -585,7 +585,7 @@ static int worker_entrypoint(void *arg TEST_UNUSED) } else { odp_timer_set_t rc; uint64_t cur_tick; - uint64_t tck, nsec; + uint64_t tck; if (tt[i].ev != ODP_EVENT_INVALID) /* Timer inactive => set */ @@ -594,16 +594,18 @@ static int worker_entrypoint(void *arg TEST_UNUSED) /* Timer active => reset */ nreset++; - nsec = (rand_r(&seed) % RANGE_MS) * 1000000ULL; - tck = 1 + odp_timer_ns_to_tick(tp, nsec); + nsec = MIN_TMO + + (rand_r(&seed) % RANGE_MS) * 1000000ULL; + tck = odp_timer_ns_to_tick(tp, nsec); cur_tick = odp_timer_current_tick(tp); rc = odp_timer_set_rel(tt[i].tim, tck, &tt[i].ev); - if (rc == ODP_TIMER_TOOEARLY || - rc == ODP_TIMER_TOOLATE) { - CU_FAIL("Failed to set timer (tooearly/toolate)"); - } else if (rc != ODP_TIMER_SUCCESS) { + if (rc == ODP_TIMER_TOOEARLY) { + CU_FAIL("Failed to set timer: TOO EARLY"); + } else if (rc == ODP_TIMER_TOOLATE) { + CU_FAIL("Failed to set timer: TOO LATE"); + } else if (rc == ODP_TIMER_NOEVENT) { /* Set/reset failed, timer already expired */ ntoolate++; } else if (rc == ODP_TIMER_SUCCESS) { @@ -611,6 +613,8 @@ static int worker_entrypoint(void *arg TEST_UNUSED) tt[i].tick = cur_tick + tck; /* ODP timer owns the event now */ tt[i].ev = ODP_EVENT_INVALID; + } else { + CU_FAIL("Failed to set timer: bad return code"); } } ts.tv_sec = 0; @@ -743,7 +747,10 @@ static void timer_test_odp_timer_all(void) CU_ASSERT(strcmp(tpinfo.name, NAME) == 0); LOG_DBG("Timer pool handle: %" PRIu64 "\n", odp_timer_pool_to_u64(tp)); - LOG_DBG("#timers..: %u\n", NTIMERS); + LOG_DBG("Resolution: %" PRIu64 "\n", tparam.res_ns); + LOG_DBG("Min timeout: %" PRIu64 "\n", tparam.min_tmo); + LOG_DBG("Max timeout: %" PRIu64 "\n", tparam.max_tmo); + LOG_DBG("Num timers..: %u\n", tparam.num_timers); LOG_DBG("Tmo range: %u ms (%" PRIu64 " ticks)\n", RANGE_MS, odp_timer_ns_to_tick(tp, 1000000ULL * RANGE_MS)); commit fec701841dbe2e963cf3adc7c74c98b41e059b9d Author: Petri Savolainen <petri.savolainen(a)linaro.org> Date: Thu Apr 19 17:54:36 2018 +0300 validation: timer: fix timer_all test sync issue When timer is reset, odp_timer_set_rel() outputs the previous event handle (not ODP_EVENT_INVALID as in case of set). Test case 'timer_all' maintain test status in the event handle variable. Set the variable always to EVENT_INVALID so that both set and reset cases work the same way. Signed-off-by: Petri Savolainen <petri.savolainen(a)linaro.org> Reviewed-by: Bill Fischofer <bill.fischofer(a)linaro.org> Signed-off-by: Maxim Uvarov <maxim.uvarov(a)linaro.org> diff --git a/test/validation/api/timer/timer.c b/test/validation/api/timer/timer.c index 8ab58137..f81a3d09 100644 --- a/test/validation/api/timer/timer.c +++ b/test/validation/api/timer/timer.c @@ -609,6 +609,8 @@ static int worker_entrypoint(void *arg TEST_UNUSED) } else if (rc == ODP_TIMER_SUCCESS) { /* Save expected expiration tick on success */ tt[i].tick = cur_tick + tck; + /* ODP timer owns the event now */ + tt[i].ev = ODP_EVENT_INVALID; } } ts.tv_sec = 0; commit 0e24607e3075a573ebe4ac408703c76daac7236c Author: Petri Savolainen <petri.savolainen(a)linaro.org> Date: Thu Apr 19 14:40:21 2018 +0300 validation: timer: add plain and sched queue tests Added simple timeout delivery tests for plain and scheduled queues. Current "timer_all" test is complex and tests only plain queues. Signed-off-by: Petri Savolainen <petri.savolainen(a)linaro.org> Reviewed-by: Bill Fischofer <bill.fischofer(a)linaro.org> Signed-off-by: Maxim Uvarov <maxim.uvarov(a)linaro.org> diff --git a/test/validation/api/timer/timer.c b/test/validation/api/timer/timer.c index bd4f1b99..8ab58137 100644 --- a/test/validation/api/timer/timer.c +++ b/test/validation/api/timer/timer.c @@ -144,6 +144,170 @@ static void timer_test_timeout_pool_free(void) CU_ASSERT(odp_pool_destroy(pool) == 0); } +static void timer_test_queue_type(odp_queue_type_t queue_type) +{ + odp_pool_t pool; + const int num = 10; + odp_timeout_t tmo; + odp_event_t ev; + odp_queue_param_t queue_param; + odp_timer_pool_param_t tparam; + odp_timer_pool_t tp; + odp_queue_t queue; + odp_timer_t tim; + int i, ret, num_tmo; + uint64_t tick_base, tick; + uint64_t res_ns, period_ns, period_tick, test_period; + uint64_t diff_period, diff_test; + odp_pool_param_t params; + odp_timer_capability_t timer_capa; + odp_time_t t0, t1, t2; + + odp_pool_param_init(&params); + params.type = ODP_POOL_TIMEOUT; + params.tmo.num = num; + + pool = odp_pool_create("timeout_pool", &params); + CU_ASSERT_FATAL(pool != ODP_POOL_INVALID); + + if (odp_timer_capability(ODP_CLOCK_CPU, &timer_capa)) + CU_FAIL_FATAL("Timer capability failed") + + res_ns = 20 * ODP_TIME_MSEC_IN_NS; + + if (timer_capa.highest_res_ns > res_ns) + res_ns = timer_capa.highest_res_ns; + + tparam.res_ns = res_ns; + tparam.min_tmo = 5 * res_ns; + tparam.max_tmo = 10000 * tparam.min_tmo; + tparam.num_timers = num + 1; + tparam.priv = 0; + tparam.clk_src = ODP_CLOCK_CPU; + + LOG_DBG("\nTimer pool parameters:\n"); + LOG_DBG(" res_ns %" PRIu64 "\n", tparam.res_ns); + LOG_DBG(" min_tmo %" PRIu64 "\n", tparam.min_tmo); + LOG_DBG(" max_tmo %" PRIu64 "\n", tparam.max_tmo); + + tp = odp_timer_pool_create("timer_pool", &tparam); + if (tp == ODP_TIMER_POOL_INVALID) + CU_FAIL_FATAL("Timer pool create failed"); + + odp_timer_pool_start(); + + odp_queue_param_init(&queue_param); + if (queue_type == ODP_QUEUE_TYPE_SCHED) { + queue_param.type = ODP_QUEUE_TYPE_SCHED; + queue_param.sched.prio = ODP_SCHED_PRIO_DEFAULT; + queue_param.sched.sync = ODP_SCHED_SYNC_ATOMIC; + queue_param.sched.group = ODP_SCHED_GROUP_ALL; + } + + queue = odp_queue_create("timer_queue", &queue_param); + if (queue == ODP_QUEUE_INVALID) + CU_FAIL_FATAL("Queue create failed"); + + period_ns = 4 * tparam.min_tmo; + period_tick = odp_timer_ns_to_tick(tp, period_ns); + test_period = num * period_ns; + + LOG_DBG(" period_ns %" PRIu64 "\n", period_ns); + LOG_DBG(" period_tick %" PRIu64 "\n\n", period_tick); + + tick_base = odp_timer_current_tick(tp); + t0 = odp_time_local(); + t1 = t0; + t2 = t0; + + for (i = 0; i < num; i++) { + tmo = odp_timeout_alloc(pool); + CU_ASSERT_FATAL(tmo != ODP_TIMEOUT_INVALID); + ev = odp_timeout_to_event(tmo); + CU_ASSERT_FATAL(ev != ODP_EVENT_INVALID); + + tim = odp_timer_alloc(tp, queue, USER_PTR); + CU_ASSERT_FATAL(tim != ODP_TIMER_INVALID); + + tick = tick_base + ((i + 1) * period_tick); + ret = odp_timer_set_abs(tim, tick, &ev); + + LOG_DBG("abs timer tick %" PRIu64 "\n", tick); + if (ret == ODP_TIMER_TOOEARLY) + LOG_DBG("Too early %" PRIu64 "\n", tick); + else if (ret == ODP_TIMER_TOOLATE) + LOG_DBG("Too late %" PRIu64 "\n", tick); + else if (ret == ODP_TIMER_NOEVENT) + LOG_DBG("No event %" PRIu64 "\n", tick); + + CU_ASSERT(ret == ODP_TIMER_SUCCESS); + } + + num_tmo = 0; + + do { + if (queue_type == ODP_QUEUE_TYPE_SCHED) + ev = odp_schedule(NULL, ODP_SCHED_NO_WAIT); + else + ev = odp_queue_deq(queue); + + t2 = odp_time_local(); + diff_test = odp_time_diff_ns(t2, t0); + + if (ev != ODP_EVENT_INVALID) { + diff_period = odp_time_diff_ns(t2, t1); + t1 = odp_time_local(); + tmo = odp_timeout_from_event(ev); + tim = odp_timeout_timer(tmo); + tick = odp_timeout_tick(tmo); + + CU_ASSERT(diff_period > (period_ns - (2 * res_ns))); + CU_ASSERT(diff_period < (period_ns + (2 * res_ns))); + + LOG_DBG("timeout tick %" PRIu64 "\n", tick); + + odp_timeout_free(tmo); + CU_ASSERT(odp_timer_free(tim) == ODP_EVENT_INVALID); + + num_tmo++; + } + + } while (diff_test < (2 * test_period) && num_tmo < num); + + CU_ASSERT(num_tmo == num); + CU_ASSERT(diff_test > (test_period - tparam.min_tmo)); + CU_ASSERT(diff_test < (test_period + tparam.min_tmo)); + + /* Scalable scheduler needs this pause sequence. Otherwise, it gets + * stuck on terminate. */ + if (queue_type == ODP_QUEUE_TYPE_SCHED) { + odp_schedule_pause(); + while (1) { + ev = odp_schedule(NULL, ODP_SCHED_NO_WAIT); + if (ev == ODP_EVENT_INVALID) + break; + + CU_FAIL("Drop extra event\n"); + odp_event_free(ev); + } + } + + odp_timer_pool_destroy(tp); + + CU_ASSERT(odp_queue_destroy(queue) == 0); + CU_ASSERT(odp_pool_destroy(pool) == 0); +} + +static void timer_test_plain_queue(void) +{ + timer_test_queue_type(ODP_QUEUE_TYPE_PLAIN); +} + +static void timer_test_sched_queue(void) +{ + timer_test_queue_type(ODP_QUEUE_TYPE_SCHED); +} + static void timer_test_odp_timer_cancel(void) { odp_pool_t pool; @@ -642,6 +806,8 @@ static void timer_test_odp_timer_all(void) odp_testinfo_t timer_suite[] = { ODP_TEST_INFO(timer_test_timeout_pool_alloc), ODP_TEST_INFO(timer_test_timeout_pool_free), + ODP_TEST_INFO(timer_test_plain_queue), + ODP_TEST_INFO(timer_test_sched_queue), ODP_TEST_INFO(timer_test_odp_timer_cancel), ODP_TEST_INFO(timer_test_odp_timer_all), ODP_TEST_INFO_NULL, commit 5d4be5077adeec54d1956c86dcb87b186c39a7cb Author: Petri Savolainen <petri.savolainen(a)linaro.org> Date: Thu Apr 19 13:03:29 2018 +0300 validation: timer: don't assume low timer tick frequency Timer tick can be the same as CPU frequency. So, test cannot loop until two current_tick calls return the same tick value. Similarly, test cannot assume that expiration tick of a relative timeout call can be exactly calculated from a current_tick value. Signed-off-by: Petri Savolainen <petri.savolainen(a)linaro.org> Reviewed-by: Bill Fischofer <bill.fischofer(a)linaro.org> Signed-off-by: Maxim Uvarov <maxim.uvarov(a)linaro.org> diff --git a/test/validation/api/timer/timer.c b/test/validation/api/timer/timer.c index 9cbb08bb..bd4f1b99 100644 --- a/test/validation/api/timer/timer.c +++ b/test/validation/api/timer/timer.c @@ -291,19 +291,17 @@ static void handle_tmo(odp_event_t ev, bool stale, uint64_t prev_tick) CU_FAIL("Wrong status (stale) for fresh timeout"); if (!stale) { - /* Fresh timeout => local timer must have matching tick */ - if (ttp->tick != tick) { - LOG_DBG("Wrong tick: expected %" PRIu64 - " actual %" PRIu64 "\n", + /* tmo tick cannot be smaller than pre-calculated tick */ + if (tick < ttp->tick) { + LOG_DBG("Too small tick: pre-calculated %" PRIu64 + " timeout %" PRIu64 "\n", ttp->tick, tick); - CU_FAIL("odp_timeout_tick() wrong tick"); + CU_FAIL("odp_timeout_tick() too small tick"); } - if (ttp->ev != ODP_EVENT_INVALID) - CU_FAIL("Wrong state for fresh timer (event)"); - /* Check that timeout was delivered 'timely' */ if (tick > odp_timer_current_tick(tp)) CU_FAIL("Timeout delivered early"); + if (tick < prev_tick) { LOG_DBG("Too late tick: %" PRIu64 " prev_tick %" PRIu64"\n", @@ -421,23 +419,23 @@ static int worker_entrypoint(void *arg TEST_UNUSED) ncancel++; } } else { + odp_timer_set_t rc; + uint64_t cur_tick; + uint64_t tck, nsec; + if (tt[i].ev != ODP_EVENT_INVALID) /* Timer inactive => set */ nset++; else /* Timer active => reset */ nreset++; - uint64_t tck = 1 + odp_timer_ns_to_tick(tp, - (rand_r(&seed) % RANGE_MS) * 1000000ULL); - odp_timer_set_t rc; - uint64_t cur_tick; - /* Loop until we manage to read cur_tick and set a - * relative timer in the same tick */ - do { - cur_tick = odp_timer_current_tick(tp); - rc = odp_timer_set_rel(tt[i].tim, - tck, &tt[i].ev); - } while (cur_tick != odp_timer_current_tick(tp)); + + nsec = (rand_r(&seed) % RANGE_MS) * 1000000ULL; + tck = 1 + odp_timer_ns_to_tick(tp, nsec); + + cur_tick = odp_timer_current_tick(tp); + rc = odp_timer_set_rel(tt[i].tim, tck, &tt[i].ev); + if (rc == ODP_TIMER_TOOEARLY || rc == ODP_TIMER_TOOLATE) { CU_FAIL("Failed to set timer (tooearly/toolate)"); commit 3aa3f4352083d159260cebc23b87607e73aa6f2b Author: Petri Savolainen <petri.savolainen(a)linaro.org> Date: Thu Apr 19 11:08:14 2018 +0300 validation: timer: tick conversion may lose precision Tick <-> nsec conversion may include rounding and thus loss of precision. For example, when tick frequency is higher than 1GHz conversion to nsec lose precision by definition. Modified test to convert nsec -> tick -> nsec and allow resolution level diffrence between nsec values. Signed-off-by: Petri Savolainen <petri.savolainen(a)linaro.org> Reviewed-by: Bill Fischofer <bill.fischofer(a)linaro.org> Signed-off-by: Maxim Uvarov <maxim.uvarov(a)linaro.org> diff --git a/test/validation/api/timer/timer.c b/test/validation/api/timer/timer.c index 6d4d3ecd..9cbb08bb 100644 --- a/test/validation/api/timer/timer.c +++ b/test/validation/api/timer/timer.c @@ -25,6 +25,13 @@ /* Number of timers per thread */ #define NTIMERS 2000 +#define NAME "timer_pool" +#define RES (10 * ODP_TIME_MSEC_IN_NS / 3) +#define MIN_TMO (10 * ODP_TIME_MSEC_IN_NS / 3) +#define MAX_TMO (1000000 * ODP_TIME_MSEC_IN_NS) +#define USER_PTR ((void *)0xdead) +#define TICK_INVALID (~(uint64_t)0) + /* Barrier for thread synchronisation */ static odp_barrier_t test_barrier; @@ -41,6 +48,9 @@ static odp_atomic_u32_t ndelivtoolate; * caches may make this number lower than the capacity of the pool */ static odp_atomic_u32_t timers_allocated; +/* Timer resolution in nsec */ +static uint64_t resolution_ns; + /* Timer helper structure */ struct test_timer { odp_timer_t tim; /* Timer handle */ @@ -49,8 +59,6 @@ struct test_timer { uint64_t tick; /* Expiration tick or TICK_INVALID */ }; -#define TICK_INVALID (~(uint64_t)0) - static void timer_test_timeout_pool_alloc(void) { odp_pool_t pool; @@ -180,7 +188,6 @@ static void timer_test_odp_timer_cancel(void) if (queue == ODP_QUEUE_INVALID) CU_FAIL_FATAL("Queue create failed"); - #define USER_PTR ((void *)0xdead) tim = odp_timer_alloc(tp, queue, USER_PTR); if (tim == ODP_TIMER_INVALID) CU_FAIL_FATAL("Failed to allocate timer"); @@ -518,9 +525,7 @@ static void timer_test_odp_timer_all(void) odp_timer_pool_param_t tparam; odp_cpumask_t unused; odp_timer_pool_info_t tpinfo; - uint64_t tick; - uint64_t ns; - uint64_t t2; + uint64_t ns, tick, ns2; pthrd_arg thrdarg; odp_timer_capability_t timer_capa; @@ -546,15 +551,12 @@ static void timer_test_odp_timer_all(void) if (tbp == ODP_POOL_INVALID) CU_FAIL_FATAL("Timeout pool create failed"); -#define NAME "timer_pool" -#define RES (10 * ODP_TIME_MSEC_IN_NS / 3) -#define MIN_TMO (10 * ODP_TIME_MSEC_IN_NS / 3) -#define MAX_TMO (1000000 * ODP_TIME_MSEC_IN_NS) /* Create a timer pool */ if (odp_timer_capability(ODP_CLOCK_CPU, &timer_capa)) CU_FAIL("Error: get timer capacity failed.\n"); - tparam.res_ns = MAX(RES, timer_capa.highest_res_ns); + resolution_ns = MAX(RES, timer_capa.highest_res_ns); + tparam.res_ns = resolution_ns; tparam.min_tmo = MIN_TMO; tparam.max_tmo = MAX_TMO; tparam.num_timers = num_workers * NTIMERS; @@ -581,11 +583,26 @@ static void timer_test_odp_timer_all(void) LOG_DBG("Tmo range: %u ms (%" PRIu64 " ticks)\n", RANGE_MS, odp_timer_ns_to_tick(tp, 1000000ULL * RANGE_MS)); - for (tick = 0; tick < 1000000000000ULL; tick += 1000000ULL) { - ns = odp_timer_tick_to_ns(tp, tick); - t2 = odp_timer_ns_to_tick(tp, ns); - if (tick != t2) - CU_FAIL("Invalid conversion tick->ns->tick"); + tick = odp_timer_ns_to_tick(tp, 0); + CU_ASSERT(tick == 0); + ns2 = odp_timer_tick_to_ns(tp, tick); + CU_ASSERT(ns2 == 0); + + for (ns = resolution_ns; ns < MAX_TMO; ns += resolution_ns) { + tick = odp_timer_ns_to_tick(tp, ns); + ns2 = odp_timer_tick_to_ns(tp, tick); + + if (ns2 < ns - resolution_ns) { + LOG_DBG("FAIL ns:%" PRIu64 " tick:%" PRIu64 " ns2:%" + PRIu64 "\n", ns, tick, ns2); + CU_FAIL("tick conversion: nsec too small\n"); + } + + if (ns2 > ns + resolution_ns) { + LOG_DBG("FAIL ns:%" PRIu64 " tick:%" PRIu64 " ns2:%" + PRIu64 "\n", ns, tick, ns2); + CU_FAIL("tick conversion: nsec too large\n"); + } } /* Initialize barrier used by worker threads for synchronization */ commit 0d4271e31fe685a8a82381d21d5166f0398080c1 Author: Petri Savolainen <petri.savolainen(a)linaro.org> Date: Thu Apr 19 10:17:39 2018 +0300 validation: timer: check user pointer once Code clean up: check only once that user pointer is not NULL. Signed-off-by: Petri Savolainen <petri.savolainen(a)linaro.org> Reviewed-by: Bill Fischofer <bill.fischofer(a)linaro.org> Signed-off-by: Maxim Uvarov <maxim.uvarov(a)linaro.org> diff --git a/test/validation/api/timer/timer.c b/test/validation/api/timer/timer.c index 06148de8..6d4d3ecd 100644 --- a/test/validation/api/timer/timer.c +++ b/test/validation/api/timer/timer.c @@ -233,6 +233,10 @@ static void timer_test_odp_timer_cancel(void) static void handle_tmo(odp_event_t ev, bool stale, uint64_t prev_tick) { odp_event_subtype_t subtype; + odp_timeout_t tmo; + odp_timer_t tim; + uint64_t tick; + struct test_timer *ttp; CU_ASSERT_FATAL(ev != ODP_EVENT_INVALID); /* Internal error */ if (odp_event_type(ev) != ODP_EVENT_TIMEOUT) { @@ -255,33 +259,39 @@ static void handle_tmo(odp_event_t ev, bool stale, uint64_t prev_tick) CU_FAIL("Unexpected event subtype received"); return; } + /* Read the metadata from the timeout */ - odp_timeout_t tmo = odp_timeout_from_event(ev); - odp_timer_t tim = odp_timeout_timer(tmo); - uint64_t tick = odp_timeout_tick(tmo); - struct test_timer *ttp = odp_timeout_user_ptr(tmo); + tmo = odp_timeout_from_event(ev); + tim = odp_timeout_timer(tmo); + tick = odp_timeout_tick(tmo); + ttp = odp_timeout_user_ptr(tmo); if (tim == ODP_TIMER_INVALID) CU_FAIL("odp_timeout_timer() invalid timer"); - if (!ttp) + + if (ttp == NULL) { CU_FAIL("odp_timeout_user_ptr() null user ptr"); + return; + } - if (ttp && ttp->ev2 != ev) + if (ttp->ev2 != ev) CU_FAIL("odp_timeout_user_ptr() wrong user ptr"); - if (ttp && ttp->tim != tim) + + if (ttp->tim != tim) CU_FAIL("odp_timeout_timer() wrong timer"); if (!odp_timeout_fresh(tmo)) CU_FAIL("Wrong status (stale) for fresh timeout"); + if (!stale) { /* Fresh timeout => local timer must have matching tick */ - if (ttp && ttp->tick != tick) { + if (ttp->tick != tick) { LOG_DBG("Wrong tick: expected %" PRIu64 " actual %" PRIu64 "\n", ttp->tick, tick); CU_FAIL("odp_timeout_tick() wrong tick"); } - if (ttp && ttp->ev != ODP_EVENT_INVALID) + if (ttp->ev != ODP_EVENT_INVALID) CU_FAIL("Wrong state for fresh timer (event)"); /* Check that timeout was delivered 'timely' */ @@ -296,11 +306,9 @@ static void handle_tmo(odp_event_t ev, bool stale, uint64_t prev_tick) } } - if (ttp) { - /* Internal error */ - CU_ASSERT_FATAL(ttp->ev == ODP_EVENT_INVALID); - ttp->ev = ev; - } + /* Internal error */ + CU_ASSERT_FATAL(ttp->ev == ODP_EVENT_INVALID); + ttp->ev = ev; } /* Worker thread entrypoint which performs timer alloc/set/cancel/free commit d324b9de12c510ccf9fa1c1b6ece327c578ee53d Author: Petri Savolainen <petri.savolainen(a)linaro.org> Date: Thu Apr 19 09:57:42 2018 +0300 validation: timer: remove doxygen tags Doxygen documentation is not generated from validation test files. Remove partial and unnecessary doxygen tags. Signed-off-by: Petri Savolainen <petri.savolainen(a)linaro.org> Reviewed-by: Bill Fischofer <bill.fischofer(a)linaro.org> Signed-off-by: Maxim Uvarov <maxim.uvarov(a)linaro.org> diff --git a/test/validation/api/timer/timer.c b/test/validation/api/timer/timer.c index 1cd32cb1..06148de8 100644 --- a/test/validation/api/timer/timer.c +++ b/test/validation/api/timer/timer.c @@ -6,10 +6,6 @@ #include "config.h" -/** - * @file - */ - /* For rand_r and nanosleep */ #ifndef _GNU_SOURCE #define _GNU_SOURCE @@ -23,29 +19,29 @@ #define MAX(a, b) (((a) > (b)) ? (a) : (b)) -/** @private Timeout range in milliseconds (ms) */ +/* Timeout range in milliseconds (ms) */ #define RANGE_MS 2000 -/** @private Number of timers per thread */ +/* Number of timers per thread */ #define NTIMERS 2000 -/** @private Barrier for thread synchronisation */ +/* Barrier for thread synchronisation */ static odp_barrier_t test_barrier; -/** @private Timeout pool handle used by all threads */ +/* Timeout pool handle used by all threads */ static odp_pool_t tbp; -/** @private Timer pool handle used by all threads */ +/* Timer pool handle used by all threads */ static odp_timer_pool_t tp; -/** @private Count of timeouts delivered too late */ +/* Count of timeouts delivered too late */ static odp_atomic_u32_t ndelivtoolate; -/** @private Sum of all allocated timers from all threads. Thread-local +/* Sum of all allocated timers from all threads. Thread-local * caches may make this number lower than the capacity of the pool */ static odp_atomic_u32_t timers_allocated; -/* @private Timer helper structure */ +/* Timer helper structure */ struct test_timer { odp_timer_t tim; /* Timer handle */ odp_event_t ev; /* Timeout event */ @@ -233,7 +229,7 @@ static void timer_test_odp_timer_cancel(void) CU_FAIL_FATAL("Failed to destroy pool"); } -/* @private Handle a received (timeout) event */ +/* Handle a received (timeout) event */ static void handle_tmo(odp_event_t ev, bool stale, uint64_t prev_tick) { odp_event_subtype_t subtype; @@ -307,7 +303,7 @@ static void handle_tmo(odp_event_t ev, bool stale, uint64_t prev_tick) } } -/* @private Worker thread entrypoint which performs timer alloc/set/cancel/free +/* Worker thread entrypoint which performs timer alloc/set/cancel/free * tests */ static int worker_entrypoint(void *arg TEST_UNUSED) { @@ -506,7 +502,7 @@ static int worker_entrypoint(void *arg TEST_UNUSED) return CU_get_number_of_failures(); } -/* @private Timer test case entrypoint */ +/* Timer test case entrypoint */ static void timer_test_odp_timer_all(void) { int rc; ----------------------------------------------------------------------- Summary of changes: platform/linux-generic/Makefile.am | 1 + platform/linux-generic/odp_crypto.c | 42 ----- platform/linux-generic/odp_random.c | 54 ++++++ test/validation/api/timer/timer.c | 336 ++++++++++++++++++++++++++++-------- 4 files changed, 320 insertions(+), 113 deletions(-) create mode 100644 platform/linux-generic/odp_random.c hooks/post-receive --

8 years, 4 months

[COMMIT] annotated tag v1.19.0.0_tigermoth created. v1.19.0.0_tigermoth

by git＠git-us.linaro.org

8 years, 4 months

[COMMIT] branch tigermoth_lts created. v1.19.0.0_tigermoth

by git＠git-us.linaro.org

This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "". The branch, tigermoth_lts has been created at e828b4d6f503ff94c40e30b1d8babf0dcbecde91 (commit) - Log ----------------------------------------------------------------- ----------------------------------------------------------------------- hooks/post-receive --

8 years, 4 months

[COMMIT] annotated tag v1.19.0.0 created. v1.19.0.0

by git＠git-us.linaro.org

This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "". The annotated tag, v1.19.0.0 has been created at 200122ffa31e4cf5367edb528138d277a1f5da1e (tag) tagging e828b4d6f503ff94c40e30b1d8babf0dcbecde91 (commit) replaces v1.18.0.1_tigermoth_rc3 tagged by Maxim Uvarov on Thu Apr 19 17:56:01 2018 +0300 - Log ----------------------------------------------------------------- == OpenDataPlane (1.19.0.0) === Summary of Changes ODP v1.19.0.0 is the official Tiger Moth final release. It incorporates final implementation changes and bug fixes and completes the Tiger Moth ODP development cycle. ==== APIs No functional changes for this release. The Tiger Moth API was frozen in ODP v1.18.0.0. ===== API Documentation Update The specification for the `odp_packet_l4_chksum_status()` API has been clarified to reflect that in IPv4 UDP checksums are optional. As a result, a zero (nonexistent) checksum will be reported as `ODP_PACKET_CHKSUM_OK`. ==== C++ Test Improvements The {cpp} test included in the ODP validation suite now uses `cout` instead of `printf()` to ensure that {cpp} is being used to compile it. ==== Queue and Scheduler Configuration For the ODP Reference Implementation, The `config/odp-linux-generic.conf` file is extended with sections to control the default and maximum sizes for basic queues, and the priority spread used by the scheduler for scheduled queues. The configuration file is a template named `platform/odp-$platform.conf` so this can be easily inherited by other ODP implementations. ==== Runtime Default `config` File Read Order Improvements For the ODP Reference Implementation, the default values of the application-provided `config` file (if used) override the values provided by the built-in `config/odp-linux-generic.conf` file. === Implementation Improvements The `odp-linux` reference implementation is improved in a number of areas: ==== Netmap Ring Configuration for VALE PktIO netmap support now uses the ODP config file to allow rings used for VALE processing to be specified. The supplied defaults provide optimal performance in typical settings. ==== AES-XCBC-MAC and SHA384-HMAC These crypto/authentication algorithms are now implemented. ==== Packet Checksum Validation and Insertion Proper packet checksum validation and insertion, in conformance with the relevant ODP APIs, is now provided. === Dependency Changes ==== DPDK 17.11 Support The Tiger Moth LTS release is synchronized with the most recent DPDK LTS release for DPDK pktio support. ==== Removal of dependency on `xxd` package. This dependency is removed. The Reference Implementation build tools now use the standard `od` tool rather than the optional `xxd` package. === Performance Tests ==== `odp_sched_pktio` A new test has been added to test the performance of PktIO operations in scheduled mode. Scheduled PktIO is inherently more scalable and simpler from an application standpoint than direct (polled) I/O, but depending on the efficiency of the scheduler implementation can incur additional levels of overhead. This test can give insight into a given platform's scheduler efficiency. For the `odp-linux` reference implementation, this test has shown scheduled I/O to be within 10% of rates achievable via direct I/O, meaning that for many applications the simplicity and scalability of the event model is preferable. ==== `odp_ipsec` A new test has been added that measures outbound (TX) IPsec performance with a variety of cipher and authentication algorithms. === Example Changes ==== `l2fwd` Example The `README` file associated with this example has been clarified to explain that this example is a throughput test and as a result does not preserve packet order under all conditions. === Bug Fixes ==== https://bugs.linaro.org/show_bug.cgi?id=3611[Bug 3611] ODP linux-generic fails on AArch64 in non-ABI-compat mode. ==== https://bugs.linaro.org/show_bug.cgi?id=3657[Bug 3657] PktIO does not work with Mellanox Interfaces ==== https://bugs.linaro.org/show_bug.cgi?id=3685[Bug 3685] RX UDP checksum offload drops valid UDP packets with Niantic ==== https://bugs.linaro.org/show_bug.cgi?id=3686[Bug 3686] IP header checksum not inserted if L4 offset not set ==== https://bugs.linaro.org/show_bug.cgi?id=3690[Bug 3690] fdserver process interferes with signal handling ==== https://bugs.linaro.org/show_bug.cgi?id=3736[Bug 3736] return value not checked for some fdserver interface functions === Known Issues ==== https://bugs.linaro.org/show_bug.cgi?id=2988[Bug 2988] ODP exposes symbols outside of odp*/_odp* namespace Bill Fischofer (9): doc: userguide: typo corrections doc: userguide: change pool queue to plain queue doc: userguide: shm corrections doc: userguide: change sched type none to parallel doc: userguide: refresh pktio code examples doc: userguide: add packet checksum and parsing info doc: userguide: add pktio capability and config info example: l2fwd: update readme regarding ordered queue usage changelog: updates for odp v1.19.0.0 Bogdan Pricope (3): linux-gen: pktio: dpdk: accept UDPv4 packets with all-zero csum linux-gen: pktio: dpdk: fix IPv4 csum calculation when l4 offset is not set linux-gen: dpdk: fix runtime/default config read order Dmitry Eremin-Solenikov (20): performance: fix sched_latency test with huge cpu count shippable: reenable non-ABI-compat build for GCC test: misc: use C++ I/O instead of C tests: add IPsec performance test performance: ipsec: add AH measurements performance: ipsec: add more algorithms performance: crypto: add more algorithms linux-gen: crypto: add IV length checks linux-gen: ipsec: add proper support for AES-CCM build: move odp scheduler setting to common m4 file build: another DPDK-linking fix build: don't use xxd to hexdump config file build: fix autoconf error caused by double-registering config item linux-gen: packet: IPv4 checksum insertion linux-gen: packet: l4 checksum insertion support linux-gen: pktio: loop: support IPv4/TCP/UDP checksum generation validation: verify IPv4 and UDP checksum validation and generation validation: ipsec: fix packet checksums linux-gen: ipsec: implement outbound checksumming support validation: ipsec: validate outbound checksumming support Josep Puigdemont (8): fdserver: handle interruption by signal in accept linux-gen: shm: fill data used in do_map before the call linux-gen: shm: do not close provided file descriptors linux-gen: shm: be consistent with rest of the code fdserver: change session ID after fork linux-gen: fdserver: mask signals we don't need linux-gen: shm: check return value when registering fds fdserver: handle signal interruption in connect() Juha-Matti Tilli (1): linux-gen: netmap: ring configuration for VALE Matias Elo (2): linux-gen: dpdk: bump supported dpdk version to v17.11 linux-gen: dpdk: allocate huge page memory for all numa nodes Maxim Uvarov (9): shippable: simplify test execution add odp_ipsec to .gitignore linux-gen: clean up ishm file naming linux-gen: hide debug prints from fd server linux-gen: ishm: remove useless debug print travis: use -M option for generated patches linux-gen: fix clang on aarch64 build shippable: reenable clang with disable abi compat options configure.ac: update version to v1.19.0.0 Petri Savolainen (11): linux-gen: queue: configurable default size linux-gen: queue: configurable max size linux-gen: sched: configurable priority spread linux-gen: sched: decouple spread and group table sizes linux-gen: sched: increase max spread linux-gen: pktio: add error prints linux-gen: pktio: add index to pktio print test: sched_pktio: new scheduler performance test with pktio test: sched_pktio: add option to collect statistics test: sched_pktio: run in validation test suite api: packet: UDP checksum value of zero yhe (2): linux-gen:crypto:implement AES-XCBC-MAC and SHA384-HMAC validation:crypto:implement AES-XCBC-MAC and SHA384-HMAC ----------------------------------------------------------------------- hooks/post-receive --

8 years, 4 months

Jump to page:

2026

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

git-lng-odp