August 2017 - git-lng-odp - lists.linaro.org

[COMMIT] branch cloud-dev updated. v1.15.0.0-586-g863d61cc

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, cloud-dev has been updated via 863d61cc3ed82f9f0725c792910e8a05cc5d5a82 (commit) via e420668cd3886f003c8bd6022e210bf08a0ee3b5 (commit) via bb3400df72401d88aa8e79fb0244a2b90f3bd138 (commit) via 2bbf95311cb8136b7e769deda6254a9e9b26250b (commit) via 4ce95649367da9edd140ba9c25daaaf292edf883 (commit) via 03aa7c3572865dc03df1cf8c33b969ca4b17c7e8 (commit) via dca4f5bd8218f30a593f6fb37c060de41d1ca9b1 (commit) via 5a6d8798bc436fa3a7e90fcf105362eabb21db0c (commit) via cd15a2125781f4eefdab1f765999a59504fb2939 (commit) via 757439849936bb76885f27677fc8997a67fa2a3c (commit) via 92ad20c03bf4737558265a1ff942666d7c4ec3d8 (commit) via fd2ca1209dc19c8e8199ee556ad2c9a786eabf78 (commit) via 2394317ab247fa14b4e239aec512afee7eac4524 (commit) via 09ce9406e09eacadf8f739ae8499323ad1401d36 (commit) via a165ff4093f666e5481ae5113f70b41e3354520d (commit) via 41abc837a5c25d8df3429398bcb51c72b83999e5 (commit) via e76b25aa5cafbf96133fcbd83b856090b0e95f33 (commit) via ab62d19d4811e2699ac0614bf4ba1d481bb3b2b1 (commit) via 641717bac482831a8f9702af7f941cfc349ada91 (commit) via c5436c447df98e0ec78de3620c7fe48141fef760 (commit) via 2fff8b0247a0da108c45c9febc1029bc8e12d347 (commit) via 8390f890d4bd2babb63a24f7b15d2f4763e44050 (commit) via e02f89b3a981d5f87332cd9a3dfd6b532f92c3bc (commit) via 8eeb3366d62c34665bcef2392b22b73489d70f88 (commit) via 705062dc19a4ce5e6ae2224c21bc9e32dae64ebc (commit) via b4cd6c50f9f0c2e1fa975b768253d4f4b35fad07 (commit) via e65248393a0d7263a26e4e8e4f64ac12c7142c54 (commit) via 4537d096721a753086592ed5a989beab5647bcfa (commit) via 93cbd8caf4ab5ccf22c092d42df4028969ed8a7b (commit) via 2050915d34995f4be695a88dd99e9275f346733b (commit) via ded0918391865956e3b00df3c43e4311a7285d4b (commit) via a2040db93b114fb9e1da9aeca1d0a1ea91699448 (commit) via 999042e51837a7ed8c7fb616f75707349515fc3d (commit) via aaceddaf56175ee207fcbf9c241a7839bdea091b (commit) via a1786671550c5f83a6c9e7cecd9da4cead7b4cc9 (commit) via cd0cd7af17c1c042288141b7968ccf5385398493 (commit) via c177532c1033b0dfa918774670bb61ad13f56730 (commit) via f319f7fe8524a2606b32979c365855b83fb4571e (commit) via 2e474c1a2af0bc4ef28706deee59000c5c6f34f4 (commit) via acb029364daa3299ea416f1fd2e8feb60e99d0fd (commit) via d4fe8e46c05d9b590c762502489386421ee025b1 (commit) via 8745330b32579ab66354f9d2043154f8ef326290 (commit) via e0738b61c3257145c50d4751bcf4e20b5fc5f0ec (commit) via a4afaf9198ac0621b4c8e5ade82cb4f98aeb61cd (commit) via 30ed14c05ac17406c2f7890b34c4ac580b5e7a45 (commit) via bb0ca193f8c28036fdf8b3d6c8aa5d4f11980e6c (commit) via 10ef82e6d4db8c02dd5caa2e97172688e829b90c (commit) via c16f1363303cd5fc11324acbc4dfebe0a9680a41 (commit) via fbdc9ca1e38e6c97014e68bf7b83e59096a170b7 (commit) via 2e08ac6e40551704c43233b40a9227a19fd4623b (commit) via 5386c44d204f0b42f2773cf87126ff8981bf3270 (commit) via 56dbc8702500f7d7e518fcea4f3b1ff853722004 (commit) via 70714a8c35381330647a3578d3b853af4ea4a045 (commit) via 79336f5eaee25168a5e4ad7fc0aee07395d4ca1f (commit) via ed136f929f0edf2edd4bb8cf2243b678b5f5971e (commit) via c84d3a864bf6471b2a1d433ad378c3b609460e5d (commit) via b6308d6f7cf7bf42b5ff6807ae659db83a66ce78 (commit) via 422ed4f2be27223b53c39a3d3f7b973781a6a420 (commit) via d3cae7aa012174bac6eecdca4ee3189c62400c48 (commit) via c444e19382a27c5d15e537f55c90a03963b86901 (commit) via 985db92075ffcdc48eb89954a90429e4cb9b4501 (commit) via daa04baf2cc2fc562dbf4ee8eaeed00f9594e08a (commit) via f4f7679d16917d9a7c1b2220e351fd27733ee96b (commit) via fa9550ca8f4f0a77baf6d156b441fa2d703bbdf8 (commit) via 063d713d0c167c56b4497be566b2310a5a2daf42 (commit) via 3eeb249a4774f12d2d16ca349a591654eae620ae (commit) via 4d2cf5b4a536c243089f41ade86d5dc60844f3d3 (commit) via 3a79862d2a8f9ea8eaa53d408f0763e44e522da7 (commit) via db7cc41aeb559dd296f3a6d8570aa10326a31d5e (commit) via 95ba4b394009d92c29c2e22f0776e90bb4c6edec (commit) via 9bb623dfe97590991dfe3ab04e1dcfb9ff745169 (commit) via 8b1dd78079de09d5f51c61b5b2ede2766ac74685 (commit) via 920857807c4d7831ada68ebed4ffa542564ee3a5 (commit) via 43bfbbf66db3d8fd6a22762d7ac346461857f653 (commit) via a959a61e3d00965af8b93536e3fbc3bc09cad1e9 (commit) via 9e4826c45af05eb339d129140ac88bc9d4e5a2fd (commit) via fc539b10fff8b92c234f422905eec2266ec13b26 (commit) via 04e5b3dd319d1affb61e0f6860e255082a6c15df (commit) via c65cfd73c6103d02f33a62099391a98cf60d098f (commit) via ae42f98c1a1c309c0718a3b5fc8589660de0f867 (commit) via 7e71ca2883bbdeb4765e476fcfe3334e4fbfbd88 (commit) via 333c585111723d2cb7dae0cf1517cc1f3453df4d (commit) via bc36acc21368ff3f27d5b3836e8524747e0ab4c9 (commit) via f49ce736d461c3e8c2534ed216e8a70e8bee954e (commit) via d4b85d07214fb1f566b8ac346375cfe097c0bb51 (commit) via 127c73198b07dfe413e8ce401bb26f3e5ff71e91 (commit) via 2cd78610004d755dbe7a8e9c29086791793aea19 (commit) via e58e1fc97f392f63a267c2252656367b12417152 (commit) via a3ff4f6726582e41dd1398a6939a1877b9f25edc (commit) via 4e3598f34f16aa9e50ac1aaae2b4074cd806e419 (commit) via 697c4a58d57c286e679dd570722d8875ccf05a27 (commit) via 6879ecd2d136508e59cf8d9b8f15c6475ba74c99 (commit) via 698d82ce41a3f6ca08c6f218784d3eea6ac81e42 (commit) via cd097450cbf9d7bb6cd24654f8d43ceade5b114a (commit) via 490f4bf22129638899ce71c99a8847e8ba849692 (commit) via 77f1b6743f97c6265e18cd47d0f597b44f71be2d (commit) via dcea50f7ad1f5473d74797e58eac5ba4abb939d1 (commit) via 7fc6d27e937b57b31360b07028388c811f8300dc (commit) via 104993f09d06c9ffc5ff1b2b57e44bfddefb3062 (commit) via c6dc5055a2acdf4f22e37823a86bb76b86f40b63 (commit) via 66898d97524683ae6f39b0624ce5a67f1cd8f603 (commit) via 51dc8377db6b67090224cab32a6663d7168c4e05 (commit) via 780073842dd6d2d52a52e9d59def2f13ae7d8427 (commit) via 89b7caa359acc5f213e233a3eb1ee1e9c661c246 (commit) via ce8c72bb9e8b1e162765b26b4249b7ecf1b32355 (commit) via a664b1d5e6e75a2b2110025c5bbf7ae727142bf4 (commit) via ee5be324411a7520528a367967c28fc529d3bc2e (commit) via 533a2fe155aa16268bfe4091a3a056a78d5dbfc6 (commit) via b4edd15ea49b38a0c1ed2bd021ee36aa296a79b0 (commit) via 99b74cbb3c77902322933192ce68f8c1abe21e84 (commit) via d0e3acc0cfdab0d4c66b49ea70207d8938569b80 (commit) via c6e3edd5dc52764e4f1772341dc568a549d7eaa9 (commit) via 26644862f90f4dc2f6ed99534d16f8b4d136de82 (commit) via e777674f2d484e98e3329c3e22022fb4c6d86069 (commit) via d9473fe65b531bfbe54d83d4e429d72e553c17dc (commit) via 98b09c8289e0888ce09f68f454a066a415597ea2 (commit) via f72ae9c821eed99323a98a012659acd655095bb3 (commit) via 08863bb877feb8643d1583e29912cc08f2ca8105 (commit) via d55156b2eacf07e88cbff9f51d60238536dfb7ad (commit) via 1239cc0c12ad2be11d329eebbe4df7fad05b03d7 (commit) via e3fce2fca6067b03f7fd18c9a438d2058521fe11 (commit) via 29ad363b5ab3c5e6569f20ddf63e64cc53eeb019 (commit) via 8ef745b7b78b49fc58d14e7b79b450c0bb560558 (commit) via d9c85310cb82f83010c839b6a1bbb5a1fa672ceb (commit) via 5a38050f431d904b7354cd50611304da0a601727 (commit) via 200782c3da2104a020f278691b3b23b8d55893e0 (commit) via 0275cc02d5cdb54b955cfce1988f5ee158ad0423 (commit) via d30dc0774647b3f635afea67e56147b8e6e3f24e (commit) via bf5d48ee13aa4fb737ee430038d5f2b865f45d47 (commit) via 1ba2bcd7b9ebfa1a5d8dda257c7b309d3553fa03 (commit) via 8869e18f490d0ffc35e0359a92ef96eacbf5b0b7 (commit) via 91c4f94f1ed01ed22b7cd489016016b0599f4065 (commit) via 1cb0a8863eb8309c3e532a168fed34d6e3257218 (commit) via faf97a341df8e92fd692029fe2c6c353f043950b (commit) via 29641a1e0d0868c09f35f9f32e6c8491efb62b2d (commit) via cfaee18006aab817656ffaf09e644b570ff32c8e (commit) via b02fe5f06a81a190d4c4fe65af74b800cf073436 (commit) via c7718962c6633c80eb71a0400d89c31f11f88045 (commit) via 79ce62cf3266b0d1b05acc68be382cd6271b7371 (commit) via 39e4ebd39f2d171bbfef5bfee6596033b83c56a0 (commit) via f9e7f458cf7151c75a9145272c62306b96f9ff27 (commit) via 5a2ed3eefb1a263139282c376741adb2c355bd5a (commit) via b668182d6ea0cb942c2cf43771c618c9457bf146 (commit) via eb564bfb6f813ec3f2fe8a8d4ce0da25220b4215 (commit) via 8fea9ff7d8fde984981e681f49133dbda8429b7f (commit) via 2541e8688750f2a045aff3dbf544868be3ebf787 (commit) via b2b516c03ae6eb57e0055c98b33e946e9529f215 (commit) via d5d52b65d262698694a5fab0b3d9bd12fa4a6460 (commit) via 6e594990575df83dcbb70f5f86ad52f7c5a0f96d (commit) via c50d47bd420c675552dd0b82638a073b67b2ae4f (commit) via 933fd66f5b7230a00ee0ef7acde5cc1c9af63b45 (commit) via 8ae6373c8844408ad080b58fb3083b30e8dc7ea6 (commit) via 15c97427d01c81dc9f4d0aafe9b0a99cdb2d1fc6 (commit) via 96b36df87f23bf32ab218321f1fc35703f092eb4 (commit) via 61e4b959b31854f83695cbbdd2e2fa0389fa4da6 (commit) via 6096ae389fe221d366d1a7ba8a7169c836a820de (commit) via df2cf17f3cdd3313c9982aacc7adcddfd20ea29a (commit) via fbd17f06236c14c05fb2d577b9027f96cec94a87 (commit) via e33884ef62a274c187c30f4bebb34d8a48190cfa (commit) via 0893a0d99b3b718369d77eacd88db3b7aedc8332 (commit) via 05a9cff266565a8c7d24eb29d6128f2ad70bbccb (commit) via 00a21c6dce65a30c8250db59a42a43c658e8ca1b (commit) via 08463cdedd4ab64ea94dbf90c7417bf403bf7aaa (commit) via 500cb603d5e1cb4926986242ad51d995b494c2e8 (commit) via dd394c9d9a88161e157317fa8a6eba92cb6f2e01 (commit) via 60ad22e8bd7624432e9504d32309c691fedcf27f (commit) via 4a38c7c61a370007bea53f9bb36279ce3ba42233 (commit) via e91a24138eec9054fc083174e3d071269dc1e4b6 (commit) via 3687001250f4b50af8c2f9d9aa61d3ee99976a1e (commit) via ceeab69f3af67701adb524c7b9757d19cefb1110 (commit) via 7a5813042d58598e1c66243d8cfed548302edfc4 (commit) via 759a54bf66317fee539f2b2649c283be937d832a (commit) via 9fd48a9215a7831ca951839b7187bd1eb3f7bb06 (commit) via bf554b2d9472ed8ed04580904d0aa906cdbe1e83 (commit) via ac89f9c568830498b186dff33e908053dbdc88c7 (commit) via 5b08cc290a4a84ed0ffbc4a591427f1891b9ace3 (commit) via 5551c0d76bd721110a0fa4ba1423fe1edb0fd530 (commit) via 9b78da8e2373ab0530cfbbd72b2229059b57ddf1 (commit) via 689145bcef5ada4304f1d97d6b72e10bf8308ac0 (commit) via c6a309f00f882fabe70fa535ebe8b765f6e9bd11 (commit) via a4add29291572219143dc667db4e8a7832a5692b (commit) via 497c339f94e35d5fadeff539cebfbf44d698dc7e (commit) via fdd3df3547c78d81d1244eac3fa65512e9023237 (commit) via e5b211403233dda67507f18fd91f9826022fd287 (commit) via d232db102fc77e6cb2fd9d62ba387c22f68c042a (commit) from 370f93139e16633390cb28acda6296da8fee006d (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 863d61cc3ed82f9f0725c792910e8a05cc5d5a82 Merge: 370f9313 e420668c Author: Yi He <yi.he(a)linaro.org> Date: Wed Aug 9 09:28:17 2017 +0800 Merge branch 'api-next' into cloud-dev Signed-off-by: Yi He <yi.he(a)linaro.org> diff --cc platform/linux-generic/Makefile.am index 4a7b77b8,cfccaa98..1d703cea --- a/platform/linux-generic/Makefile.am +++ b/platform/linux-generic/Makefile.am @@@ -6,9 -6,9 +6,10 @@@ include $(top_srcdir)/platform/@with_pl AM_CFLAGS += -I$(srcdir)/include AM_CFLAGS += -I$(top_srcdir)/include +AM_CFLAGS += -I$(top_srcdir)/frameworks/modular AM_CFLAGS += -I$(top_srcdir)/include/odp/arch/@ARCH_ABI@ AM_CFLAGS += -I$(top_builddir)/include + AM_CFLAGS += -I$(top_srcdir)/arch/@ARCH_DIR@ AM_CFLAGS += -Iinclude AM_CFLAGS += -DSYSCONFDIR=\"@sysconfdir@\" AM_CFLAGS += -D_ODP_PKTIO_IPC @@@ -293,14 -321,16 +324,23 @@@ if HAVE_PCA __LIB__libodp_linux_la_SOURCES += pktio/pcap.c endif +# Build modular framework into odp-linux library +modularframeworkdir = $(top_srcdir)/frameworks/modular +noinst_HEADERS += $(modularframeworkdir)/list.h \ + $(modularframeworkdir)/odp_module.h + +__LIB__libodp_linux_la_SOURCES += ../../frameworks/modular/odp_module.c + __LIB__libodp_linux_la_LIBADD = $(ATOMIC_LIBS) + __LIB__libodp_linux_la_LIBADD += $(OPENSSL_LIBS) + __LIB__libodp_linux_la_LIBADD += $(DPDK_LIBS) $(DPDK_PMDS) + __LIB__libodp_linux_la_LIBADD += $(PTHREAD_LIBS) + __LIB__libodp_linux_la_LIBADD += $(TIMER_LIBS) + __LIB__libodp_linux_la_LIBADD += $(LIBCONFIG_LIBS) + + if HAVE_PCAP + __LIB__libodp_linux_la_LIBADD += $(PCAP_LIBS) + endif # Create symlink for ABI header files. Application does not need to use the arch # specific include path for installed files. ----------------------------------------------------------------------- Summary of changes: .travis.yml | 178 +- DEPENDENCIES | 5 +- Makefile.am | 10 +- configure.ac | 11 +- doc/Doxyfile_common | 12 + doc/Makefile.am | 8 +- doc/Makefile.inc | 6 +- doc/application-api-guide/Doxyfile | 2 - doc/driver-api-guide/Doxyfile | 8 - doc/helper-guide/Doxyfile | 8 - doc/implementers-guide/Makefile.am | 5 +- doc/platform-api-guide/Doxyfile | 9 - doc/process-guide/Makefile.am | 45 +- doc/users-guide/Makefile.am | 89 +- doc/users-guide/users-guide-pktio.adoc | 14 +- doc/users-guide/users-guide-timer.adoc | 3 +- doc/users-guide/users-guide-tm.adoc | 4 +- doc/users-guide/users-guide.adoc | 32 +- example/Makefile.inc | 2 +- example/generator/odp_generator.c | 9 +- example/ipfragreass/odp_ipfragreass_atomics_arm.h | 19 +- example/ipsec/Makefile.am | 2 + example/ipsec/odp_ipsec.c | 124 +- example/ipsec/odp_ipsec_cache.c | 6 +- example/ipsec/odp_ipsec_cache.h | 1 + example/ipsec/odp_ipsec_misc.h | 2 +- example/timer/odp_timer_simple.c | 10 +- example/timer/odp_timer_test.c | 9 +- helper/cuckootable.c | 1 + helper/iplookuptable.c | 1 + include/odp/api/spec/classification.h | 91 +- include/odp/api/spec/crypto.h | 170 +- include/odp/api/spec/ipsec.h | 282 +-- include/odp/api/spec/packet_io.h | 8 +- include/odp/api/spec/system_info.h | 19 + include/odp/api/spec/timer.h | 27 + include/odp/arch/default/api/abi/buffer.h | 7 +- include/odp/arch/default/api/abi/event.h | 5 +- m4/ax_check_compile_flag.m4 | 72 + m4/ax_valgrind_check.m4 | 145 +- pkgconfig/libodp-linux.pc.in | 4 +- platform/Makefile.inc | 3 +- platform/linux-generic/Makefile.am | 42 +- platform/linux-generic/_fdserver.c | 33 +- platform/linux-generic/_ishm.c | 131 +- platform/linux-generic/arch/arm/odp_atomic.h | 212 +++ platform/linux-generic/arch/arm/odp_cpu.h | 73 + platform/linux-generic/arch/arm/odp_cpu_idling.h | 51 + platform/linux-generic/arch/arm/odp_llsc.h | 253 +++ platform/linux-generic/arch/default/odp_cpu.h | 43 + platform/linux-generic/arch/mips64/odp_cpu.h | 43 + platform/linux-generic/arch/powerpc/odp_cpu.h | 43 + platform/linux-generic/arch/x86/odp_cpu.h | 43 + platform/linux-generic/include/_ishm_internal.h | 1 + .../include/odp/api/plat/buffer_types.h | 6 +- .../include/odp/api/plat/event_types.h | 5 +- .../include/odp/api/plat/packet_flag_inlines.h | 3 +- .../include/odp/api/plat/packet_inlines.h | 74 +- .../include/odp/api/plat/packet_inlines_api.h | 16 - .../include/odp/api/plat/packet_types.h | 12 + .../include/odp/api/plat/schedule_types.h | 4 +- .../include/odp/api/plat/strong_types.h | 4 +- .../include/odp/api/plat/timer_types.h | 2 +- platform/linux-generic/include/odp_bitset.h | 212 +++ .../linux-generic/include/odp_buffer_inlines.h | 6 +- .../linux-generic/include/odp_buffer_internal.h | 32 +- .../include/odp_classification_datamodel.h | 64 +- .../include/odp_classification_inlines.h | 2 +- .../include/odp_classification_internal.h | 92 +- .../linux-generic/include/odp_config_internal.h | 23 +- .../linux-generic/include/odp_crypto_internal.h | 11 +- platform/linux-generic/include/odp_internal.h | 6 +- platform/linux-generic/include/odp_llqueue.h | 311 +++ platform/linux-generic/include/odp_packet_dpdk.h | 2 - .../linux-generic/include/odp_packet_internal.h | 169 +- .../linux-generic/include/odp_packet_io_internal.h | 18 +- .../include/odp_packet_io_ipc_internal.h | 8 - .../linux-generic/include/odp_packet_io_queue.h | 48 - platform/linux-generic/include/odp_pool_internal.h | 42 +- platform/linux-generic/include/odp_queue_if.h | 20 +- .../linux-generic/include/odp_queue_internal.h | 12 - .../include/odp_queue_scalable_internal.h | 104 + platform/linux-generic/include/odp_schedule_if.h | 16 +- .../linux-generic/include/odp_schedule_scalable.h | 139 ++ .../include/odp_schedule_scalable_config.h | 52 + .../include/odp_schedule_scalable_ordered.h | 132 ++ platform/linux-generic/include/protocols/thash.h | 105 ++ platform/linux-generic/m4/configure.m4 | 18 +- platform/linux-generic/m4/odp_dpdk.m4 | 41 +- platform/linux-generic/m4/odp_netmap.m4 | 7 +- platform/linux-generic/m4/odp_openssl.m4 | 17 +- platform/linux-generic/m4/odp_pcap.m4 | 10 +- platform/linux-generic/m4/odp_pthread.m4 | 5 - platform/linux-generic/m4/odp_schedule.m4 | 55 +- platform/linux-generic/m4/odp_timer.m4 | 8 + platform/linux-generic/odp_buffer.c | 4 +- platform/linux-generic/odp_classification.c | 312 ++- platform/linux-generic/odp_crypto.c | 418 +++-- platform/linux-generic/odp_event.c | 15 + platform/linux-generic/odp_init.c | 79 +- platform/linux-generic/odp_packet.c | 767 +++++--- platform/linux-generic/odp_packet_io.c | 171 +- platform/linux-generic/odp_pkt_queue.c | 39 +- platform/linux-generic/odp_pool.c | 158 +- platform/linux-generic/odp_queue.c | 195 +- platform/linux-generic/odp_queue_if.c | 8 + platform/linux-generic/odp_queue_scalable.c | 997 ++++++++++ platform/linux-generic/odp_schedule.c | 161 +- platform/linux-generic/odp_schedule_if.c | 6 + platform/linux-generic/odp_schedule_iquery.c | 161 +- platform/linux-generic/odp_schedule_scalable.c | 1980 ++++++++++++++++++++ .../linux-generic/odp_schedule_scalable_ordered.c | 345 ++++ platform/linux-generic/odp_schedule_sp.c | 18 +- platform/linux-generic/odp_system_info.c | 36 + platform/linux-generic/odp_time.c | 4 +- platform/linux-generic/odp_timer.c | 101 +- platform/linux-generic/odp_traffic_mngr.c | 12 +- platform/linux-generic/pktio/dpdk.c | 692 +++++-- platform/linux-generic/pktio/ipc.c | 97 +- platform/linux-generic/pktio/socket_mmap.c | 6 +- test/Makefile.inc | 3 +- test/common_plat/common/Makefile.am | 1 + test/common_plat/m4/validation.m4 | 19 +- test/common_plat/performance/odp_crypto.c | 156 +- test/common_plat/performance/odp_l2fwd.c | 12 +- test/common_plat/performance/odp_pktio_ordered.c | 4 + test/common_plat/validation/api/Makefile.inc | 3 + test/common_plat/validation/api/buffer/buffer.c | 50 +- .../validation/api/classification/classification.h | 1 + .../api/classification/odp_classification_basic.c | 4 +- .../api/classification/odp_classification_common.c | 5 +- .../classification/odp_classification_test_pmr.c | 131 +- .../api/classification/odp_classification_tests.c | 2 + .../classification/odp_classification_testsuites.h | 3 +- test/common_plat/validation/api/crypto/crypto.c | 12 +- .../validation/api/crypto/odp_crypto_test_inp.c | 341 +++- .../validation/api/crypto/odp_crypto_test_inp.h | 8 +- test/common_plat/validation/api/packet/packet.c | 21 +- test/common_plat/validation/api/queue/queue.c | 8 +- test/common_plat/validation/api/system/system.c | 21 + test/common_plat/validation/api/system/system.h | 1 + test/common_plat/validation/api/time/Makefile.am | 1 + test/common_plat/validation/api/timer/timer.c | 62 +- .../validation/api/traffic_mngr/Makefile.am | 3 +- test/linux-generic/Makefile.inc | 4 +- test/linux-generic/pktio_ipc/ipc_common.h | 4 + test/linux-generic/pktio_ipc/pktio_ipc1.c | 38 +- test/linux-generic/pktio_ipc/pktio_ipc2.c | 24 +- test/linux-generic/pktio_ipc/pktio_ipc_run.sh | 39 +- .../validation/api/shmem/shmem_common.h | 5 +- .../validation/api/shmem/shmem_linux.c | 40 +- .../validation/api/shmem/shmem_odp1.c | 10 +- .../validation/api/shmem/shmem_odp2.c | 5 +- 153 files changed, 9886 insertions(+), 2286 deletions(-) mode change 100755 => 100644 doc/users-guide/users-guide.adoc create mode 100644 m4/ax_check_compile_flag.m4 create mode 100644 platform/linux-generic/arch/arm/odp_atomic.h create mode 100644 platform/linux-generic/arch/arm/odp_cpu.h create mode 100644 platform/linux-generic/arch/arm/odp_cpu_idling.h create mode 100644 platform/linux-generic/arch/arm/odp_llsc.h create mode 100644 platform/linux-generic/arch/default/odp_cpu.h create mode 100644 platform/linux-generic/arch/mips64/odp_cpu.h create mode 100644 platform/linux-generic/arch/powerpc/odp_cpu.h create mode 100644 platform/linux-generic/arch/x86/odp_cpu.h create mode 100644 platform/linux-generic/include/odp_bitset.h create mode 100644 platform/linux-generic/include/odp_llqueue.h delete mode 100644 platform/linux-generic/include/odp_packet_io_queue.h create mode 100644 platform/linux-generic/include/odp_queue_scalable_internal.h create mode 100644 platform/linux-generic/include/odp_schedule_scalable.h create mode 100644 platform/linux-generic/include/odp_schedule_scalable_config.h create mode 100644 platform/linux-generic/include/odp_schedule_scalable_ordered.h create mode 100644 platform/linux-generic/include/protocols/thash.h create mode 100644 platform/linux-generic/m4/odp_timer.m4 create mode 100644 platform/linux-generic/odp_queue_scalable.c create mode 100644 platform/linux-generic/odp_schedule_scalable.c create mode 100644 platform/linux-generic/odp_schedule_scalable_ordered.c hooks/post-receive --

6 years, 8 months

1
0
0 0

[COMMIT] branch odp-dpdk-import created. v1.15.0.0-401-g370f9313

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, odp-dpdk-import has been created at 370f93139e16633390cb28acda6296da8fee006d (commit) - Log ----------------------------------------------------------------- ----------------------------------------------------------------------- hooks/post-receive --

6 years, 8 months

1
0
0 0

[COMMIT] branch api-next updated. v1.15.0.0-556-ge420668c

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 via e420668cd3886f003c8bd6022e210bf08a0ee3b5 (commit) via bb3400df72401d88aa8e79fb0244a2b90f3bd138 (commit) via 2bbf95311cb8136b7e769deda6254a9e9b26250b (commit) via 4ce95649367da9edd140ba9c25daaaf292edf883 (commit) via 03aa7c3572865dc03df1cf8c33b969ca4b17c7e8 (commit) via dca4f5bd8218f30a593f6fb37c060de41d1ca9b1 (commit) via 5a6d8798bc436fa3a7e90fcf105362eabb21db0c (commit) via cd15a2125781f4eefdab1f765999a59504fb2939 (commit) via 757439849936bb76885f27677fc8997a67fa2a3c (commit) via 92ad20c03bf4737558265a1ff942666d7c4ec3d8 (commit) via fd2ca1209dc19c8e8199ee556ad2c9a786eabf78 (commit) from 2394317ab247fa14b4e239aec512afee7eac4524 (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 e420668cd3886f003c8bd6022e210bf08a0ee3b5 Merge: 2394317a bb3400df Author: Maxim Uvarov <maxim.uvarov(a)linaro.org> Date: Tue Aug 8 17:04:12 2017 +0300 Merge branch 'master' into api-next diff --cc platform/linux-generic/include/odp_classification_internal.h index aca8e8c6,369636d5..208b0f5b --- a/platform/linux-generic/include/odp_classification_internal.h +++ b/platform/linux-generic/include/odp_classification_internal.h @@@ -59,91 -49,14 +49,12 @@@ This function should be called during p **/ int pktio_classifier_init(pktio_entry_t *pktio); - /** - @internal - match_pmr_cos - - Match a PMR chain with a Packet and return matching CoS - This function gets called recursively to check the chained PMR Term value - with the packet. - - **/ - cos_t *match_pmr_cos(cos_t *cos, const uint8_t *pkt_addr, pmr_t *pmr, - odp_packet_hdr_t *hdr); --/** --@internal - CoS associated with L3 QoS value - - This function returns the CoS associated with L3 QoS value - **/ - cos_t *match_qos_l3_cos(pmr_l3_cos_t *l3_cos, const uint8_t *pkt_addr, - odp_packet_hdr_t *hdr); - -Flow Signature Calculation +void _odp_cls_update_hash_proto(cos_t *cos, + odp_pktin_hash_proto_t hash_proto); - /** - @internal - CoS associated with L2 QoS value - - This function returns the CoS associated with L2 QoS value - **/ - cos_t *match_qos_l2_cos(pmr_l2_cos_t *l2_cos, const uint8_t *pkt_addr, - odp_packet_hdr_t *hdr); - /** - @internal - Flow Signature Calculation - --This function calculates the Flow Signature for a packet based on --CoS and updates in Packet Meta Data --**/ --int update_flow_signature(uint8_t *pkt_addr, cos_t *cos); - - /** - @internal - Allocate a odp_pmr_t Handle - */ - odp_pmr_t alloc_pmr(pmr_t **pmr); - - /** - @internal - Pointer to pmr_t Handle - This function checks for validity of odp_pmr_t Handle - */ - pmr_t *get_pmr_entry(odp_pmr_t pmr_id); - - /** - @internal - Pointer to pmr_t Handle - */ - pmr_t *get_pmr_entry_internal(odp_pmr_t pmr_id); - - /** - @internal - Pointer to odp_cos_t Handle - */ - cos_t *get_cos_entry(odp_cos_t cos_id); - - /** - @internal - Pointer to odp_cos_t Handle - This function checks for validity of odp_cos_t Handle - */ - cos_t *get_cos_entry_internal(odp_cos_t cos_id); - - /** - @internal - Verify PMR with a Packet - - This function goes through each PMR_TERM value in pmr_t structure and - calls verification function for each term.Returns 1 if PMR matches or 0 - Otherwise. - **/ - int verify_pmr(pmr_t *pmr, const uint8_t *pkt_addr, odp_packet_hdr_t *pkt_hdr); - +uint32_t packet_rss_hash(odp_packet_hdr_t *pkt_hdr, + odp_cls_hash_proto_t hash_proto, + const uint8_t *base); #ifdef __cplusplus } diff --cc platform/linux-generic/odp_classification.c index 5a76aa5e,5e9173dc..f9a82a2e --- a/platform/linux-generic/odp_classification.c +++ b/platform/linux-generic/odp_classification.c @@@ -31,18 -30,8 +31,19 @@@ static cos_tbl_t *cos_tbl; static pmr_tbl_t *pmr_tbl; +static _cls_queue_grp_tbl_t *queue_grp_tbl; + +const rss_key default_rss = { + .u8 = { + 0x6d, 0x5a, 0x56, 0xda, 0x25, 0x5b, 0x0e, 0xc2, + 0x41, 0x67, 0x25, 0x3d, 0x43, 0xa3, 0x8f, 0xb0, + 0xd0, 0xca, 0x2b, 0xcb, 0xae, 0x7b, 0x30, 0xb4, + 0x77, 0xcb, 0x2d, 0xa3, 0x80, 0x30, 0xf2, 0x0c, + 0x6a, 0x42, 0xb7, 0x3b, 0xbe, 0xac, 0x01, 0xfa, + } +}; + static cos_t *get_cos_entry_internal(odp_cos_t cos_id) { return &cos_tbl->cos_entry[_odp_typeval(cos_id)]; @@@ -293,9 -226,10 +299,10 @@@ odp_pmr_t alloc_pmr(pmr_t **pmr return ODP_PMR_INVAL; } + static cos_t *get_cos_entry(odp_cos_t cos_id) { - if (_odp_typeval(cos_id) >= ODP_COS_MAX_ENTRY || + if (_odp_typeval(cos_id) >= CLS_COS_MAX_ENTRY || cos_id == ODP_COS_INVALID) return NULL; if (cos_tbl->cos_entry[_odp_typeval(cos_id)].s.valid == 0) @@@ -303,9 -237,10 +310,10 @@@ return &cos_tbl->cos_entry[_odp_typeval(cos_id)]; } + static pmr_t *get_pmr_entry(odp_pmr_t pmr_id) { - if (_odp_typeval(pmr_id) >= ODP_PMR_MAX_ENTRY || + if (_odp_typeval(pmr_id) >= CLS_PMR_MAX_ENTRY || pmr_id == ODP_PMR_INVAL) return NULL; if (pmr_tbl->pmr[_odp_typeval(pmr_id)].s.valid == 0) @@@ -966,74 -871,7 +989,75 @@@ int cls_classify_packet(pktio_entry_t * return 0; } +uint32_t packet_rss_hash(odp_packet_hdr_t *pkt_hdr, + odp_cls_hash_proto_t hash_proto, + const uint8_t *base) +{ + thash_tuple_t tuple; + const _odp_ipv4hdr_t *ipv4; + const _odp_udphdr_t *udp; + const _odp_tcphdr_t *tcp; + const _odp_ipv6hdr_t *ipv6; + uint32_t hash; + uint32_t tuple_len; + + tuple_len = 0; + hash = 0; + if (pkt_hdr->p.input_flags.ipv4) { + if (hash_proto.ipv4) { + /* add ipv4 */ + ipv4 = (const _odp_ipv4hdr_t *)base + + pkt_hdr->p.l3_offset; + tuple.v4.src_addr = ipv4->src_addr; + tuple.v4.dst_addr = ipv4->dst_addr; + tuple_len += 2; + } + + if (pkt_hdr->p.input_flags.tcp && hash_proto.tcp) { + /* add tcp */ + tcp = (const _odp_tcphdr_t *)base + + pkt_hdr->p.l4_offset; + tuple.v4.sport = tcp->src_port; + tuple.v4.dport = tcp->dst_port; + tuple_len += 1; + } else if (pkt_hdr->p.input_flags.udp && hash_proto.udp) { + /* add udp */ + udp = (const _odp_udphdr_t *)base + + pkt_hdr->p.l4_offset; + tuple.v4.sport = udp->src_port; + tuple.v4.dport = udp->dst_port; + tuple_len += 1; + } + } else if (pkt_hdr->p.input_flags.ipv6) { + if (hash_proto.ipv6) { + /* add ipv6 */ + ipv6 = (const _odp_ipv6hdr_t *)base + + pkt_hdr->p.l3_offset; + thash_load_ipv6_addr(ipv6, &tuple); + tuple_len += 8; + } + if (pkt_hdr->p.input_flags.tcp && hash_proto.tcp) { + tcp = (const _odp_tcphdr_t *)base + + pkt_hdr->p.l4_offset; + tuple.v4.sport = tcp->src_port; + tuple.v4.dport = tcp->dst_port; + tuple_len += 1; + } else if (pkt_hdr->p.input_flags.udp && hash_proto.udp) { + /* add udp */ + udp = (const _odp_udphdr_t *)base + + pkt_hdr->p.l4_offset; + tuple.v4.sport = udp->src_port; + tuple.v4.dport = udp->dst_port; + tuple_len += 1; + } + } + if (tuple_len) + hash = thash_softrss((uint32_t *)&tuple, + tuple_len, default_rss); + return hash; +} + + static cos_t *match_qos_l3_cos(pmr_l3_cos_t *l3_cos, const uint8_t *pkt_addr, odp_packet_hdr_t *hdr) { ----------------------------------------------------------------------- Summary of changes: platform/linux-generic/_fdserver.c | 2 +- .../linux-generic/include/odp_buffer_internal.h | 4 - .../include/odp_classification_internal.h | 89 ---------------------- .../linux-generic/include/odp_packet_io_internal.h | 13 ---- .../linux-generic/include/odp_queue_internal.h | 5 -- platform/linux-generic/include/odp_schedule_if.h | 1 - platform/linux-generic/odp_classification.c | 31 ++++++++ platform/linux-generic/odp_packet_io.c | 28 +++---- platform/linux-generic/odp_pool.c | 13 ---- platform/linux-generic/odp_queue.c | 14 +--- platform/linux-generic/odp_schedule.c | 10 +-- platform/linux-generic/odp_schedule_iquery.c | 10 +-- 12 files changed, 56 insertions(+), 164 deletions(-) hooks/post-receive --

6 years, 8 months

1
0
0 0

[COMMIT] branch master updated. v1.15.0.0-113-gbb3400df

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 bb3400df72401d88aa8e79fb0244a2b90f3bd138 (commit) via 2bbf95311cb8136b7e769deda6254a9e9b26250b (commit) via 4ce95649367da9edd140ba9c25daaaf292edf883 (commit) via 03aa7c3572865dc03df1cf8c33b969ca4b17c7e8 (commit) via dca4f5bd8218f30a593f6fb37c060de41d1ca9b1 (commit) via 5a6d8798bc436fa3a7e90fcf105362eabb21db0c (commit) via cd15a2125781f4eefdab1f765999a59504fb2939 (commit) via 757439849936bb76885f27677fc8997a67fa2a3c (commit) via 92ad20c03bf4737558265a1ff942666d7c4ec3d8 (commit) via fd2ca1209dc19c8e8199ee556ad2c9a786eabf78 (commit) from e76b25aa5cafbf96133fcbd83b856090b0e95f33 (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 bb3400df72401d88aa8e79fb0244a2b90f3bd138 Author: Dmitry Eremin-Solenikov <dmitry.ereminsolenikov(a)linaro.org> Date: Fri Aug 4 11:37:49 2017 +0300 linux-gen: sched iquery: mark thread_local as static 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_schedule_iquery.c b/platform/linux-generic/odp_schedule_iquery.c index ee2be031..c38c7c01 100644 --- a/platform/linux-generic/odp_schedule_iquery.c +++ b/platform/linux-generic/odp_schedule_iquery.c @@ -226,7 +226,7 @@ struct sched_thread_local { static sched_global_t *sched; /* Thread local scheduler context */ -__thread sched_thread_local_t thread_local; +static __thread sched_thread_local_t thread_local; static int schedule_init_global(void) { commit 2bbf95311cb8136b7e769deda6254a9e9b26250b Author: Dmitry Eremin-Solenikov <dmitry.ereminsolenikov(a)linaro.org> Date: Fri Aug 4 11:37:49 2017 +0300 linux-gen: sched: mark sched_local as static 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_schedule.c b/platform/linux-generic/odp_schedule.c index 4c99f602..79b21ac7 100644 --- a/platform/linux-generic/odp_schedule.c +++ b/platform/linux-generic/odp_schedule.c @@ -264,7 +264,7 @@ typedef struct { static sched_global_t *sched; /* Thread local scheduler context */ -__thread sched_local_t sched_local; +static __thread sched_local_t sched_local; /* Function prototypes */ static inline void schedule_release_context(void); commit 4ce95649367da9edd140ba9c25daaaf292edf883 Author: Dmitry Eremin-Solenikov <dmitry.ereminsolenikov(a)linaro.org> Date: Fri Aug 4 11:29:34 2017 +0300 linux-gen: queue: drop two unused functions 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/include/odp_queue_internal.h b/platform/linux-generic/include/odp_queue_internal.h index c5ff1e7d..dd846d59 100644 --- a/platform/linux-generic/include/odp_queue_internal.h +++ b/platform/linux-generic/include/odp_queue_internal.h @@ -61,9 +61,6 @@ union queue_entry_u { uint8_t pad[ROUNDUP_CACHE_LINE(sizeof(struct queue_entry_s))]; }; -void queue_lock(queue_entry_t *queue); -void queue_unlock(queue_entry_t *queue); - static inline uint32_t queue_to_id(odp_queue_t handle) { return _odp_typeval(handle) - 1; diff --git a/platform/linux-generic/odp_queue.c b/platform/linux-generic/odp_queue.c index e2f8c067..7ad346ad 100644 --- a/platform/linux-generic/odp_queue.c +++ b/platform/linux-generic/odp_queue.c @@ -593,16 +593,6 @@ static int queue_init(queue_entry_t *queue, const char *name, return 0; } -void queue_lock(queue_entry_t *queue) -{ - LOCK(&queue->s.lock); -} - -void queue_unlock(queue_entry_t *queue) -{ - UNLOCK(&queue->s.lock); -} - static void queue_param_init(odp_queue_param_t *params) { memset(params, 0, sizeof(odp_queue_param_t)); commit 03aa7c3572865dc03df1cf8c33b969ca4b17c7e8 Author: Dmitry Eremin-Solenikov <dmitry.ereminsolenikov(a)linaro.org> Date: Fri Aug 4 11:26:28 2017 +0300 linux-gen: pktio: mark pktio_tbl as static 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_packet_io.c b/platform/linux-generic/odp_packet_io.c index 7958786d..9419adf6 100644 --- a/platform/linux-generic/odp_packet_io.c +++ b/platform/linux-generic/odp_packet_io.c @@ -36,7 +36,7 @@ * Must be power of two. */ #define SLEEP_CHECK 32 -pktio_table_t *pktio_tbl; +static pktio_table_t *pktio_tbl; /* pktio pointer entries ( for inlines) */ void *pktio_entry_ptr[ODP_CONFIG_PKTIO_ENTRIES]; commit dca4f5bd8218f30a593f6fb37c060de41d1ca9b1 Author: Dmitry Eremin-Solenikov <dmitry.ereminsolenikov(a)linaro.org> Date: Fri Aug 4 11:25:41 2017 +0300 linux-gen: pktio: drop/staticize leftovers from single queue shim 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/include/odp_packet_io_internal.h b/platform/linux-generic/include/odp_packet_io_internal.h index 20a86557..1a4e345f 100644 --- a/platform/linux-generic/include/odp_packet_io_internal.h +++ b/platform/linux-generic/include/odp_packet_io_internal.h @@ -252,19 +252,6 @@ static inline void pktio_cls_enabled_set(pktio_entry_t *entry, int ena) entry->s.cls_enabled = ena; } -/* - * Dummy single queue implementations of multi-queue API - */ -int single_capability(odp_pktio_capability_t *capa); -int single_input_queues_config(pktio_entry_t *entry, - const odp_pktin_queue_param_t *param); -int single_output_queues_config(pktio_entry_t *entry, - const odp_pktout_queue_param_t *param); -int single_recv_queue(pktio_entry_t *entry, int index, odp_packet_t packets[], - int num); -int single_send_queue(pktio_entry_t *entry, int index, - const odp_packet_t packets[], int num); - extern const pktio_if_ops_t netmap_pktio_ops; extern const pktio_if_ops_t dpdk_pktio_ops; extern const pktio_if_ops_t sock_mmsg_pktio_ops; diff --git a/platform/linux-generic/odp_packet_io.c b/platform/linux-generic/odp_packet_io.c index f78f70ae..7958786d 100644 --- a/platform/linux-generic/odp_packet_io.c +++ b/platform/linux-generic/odp_packet_io.c @@ -1070,6 +1070,17 @@ int odp_pktio_term_global(void) return ret; } +static +int single_capability(odp_pktio_capability_t *capa) +{ + memset(capa, 0, sizeof(odp_pktio_capability_t)); + capa->max_input_queues = 1; + capa->max_output_queues = 1; + capa->set_op.op.promisc_mode = 1; + + return 0; +} + int odp_pktio_capability(odp_pktio_t pktio, odp_pktio_capability_t *capa) { pktio_entry_t *entry; @@ -1675,13 +1686,3 @@ int odp_pktout_send(odp_pktout_queue_t queue, const odp_packet_t packets[], return entry->s.ops->send(entry, queue.index, packets, num); } - -int single_capability(odp_pktio_capability_t *capa) -{ - memset(capa, 0, sizeof(odp_pktio_capability_t)); - capa->max_input_queues = 1; - capa->max_output_queues = 1; - capa->set_op.op.promisc_mode = 1; - - return 0; -} commit 5a6d8798bc436fa3a7e90fcf105362eabb21db0c Author: Dmitry Eremin-Solenikov <dmitry.ereminsolenikov(a)linaro.org> Date: Fri Aug 4 11:25:05 2017 +0300 linux-gen: scheduler: drop unused callback 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/include/odp_schedule_if.h b/platform/linux-generic/include/odp_schedule_if.h index 4cd8c3e5..e842fb46 100644 --- a/platform/linux-generic/include/odp_schedule_if.h +++ b/platform/linux-generic/include/odp_schedule_if.h @@ -67,7 +67,6 @@ extern const schedule_fn_t *sched_fn; /* Interface for the scheduler */ int sched_cb_pktin_poll(int pktio_index, int num_queue, int index[]); void sched_cb_pktio_stop_finalize(int pktio_index); -int sched_cb_num_pktio(void); odp_queue_t sched_cb_queue_handle(uint32_t queue_index); void sched_cb_queue_destroy_finalize(uint32_t queue_index); int sched_cb_queue_deq_multi(uint32_t queue_index, odp_event_t ev[], int num); diff --git a/platform/linux-generic/odp_packet_io.c b/platform/linux-generic/odp_packet_io.c index b4523389..f78f70ae 100644 --- a/platform/linux-generic/odp_packet_io.c +++ b/platform/linux-generic/odp_packet_io.c @@ -727,11 +727,6 @@ void sched_cb_pktio_stop_finalize(int pktio_index) unlock_entry(entry); } -int sched_cb_num_pktio(void) -{ - return ODP_CONFIG_PKTIO_ENTRIES; -} - uint32_t odp_pktio_mtu(odp_pktio_t hdl) { pktio_entry_t *entry; commit cd15a2125781f4eefdab1f765999a59504fb2939 Author: Dmitry Eremin-Solenikov <dmitry.ereminsolenikov(a)linaro.org> Date: Fri Aug 4 11:23:39 2017 +0300 linux-gen: classification: mark several functions as static 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/include/odp_classification_internal.h b/platform/linux-generic/include/odp_classification_internal.h index 78eaac90..369636d5 100644 --- a/platform/linux-generic/include/odp_classification_internal.h +++ b/platform/linux-generic/include/odp_classification_internal.h @@ -30,16 +30,6 @@ extern "C" { /** @internal -match_qos_cos - -Select a CoS for the given Packet based on QoS values -This function returns the COS object matching the L2 and L3 QoS -based on the l3_preference value of the pktio -**/ -cos_t *match_qos_cos(pktio_entry_t *entry, const uint8_t *pkt_addr, - odp_packet_hdr_t *hdr); -/** -@internal Packet Classifier @@ -61,34 +51,6 @@ int pktio_classifier_init(pktio_entry_t *pktio); /** @internal -match_pmr_cos - -Match a PMR chain with a Packet and return matching CoS -This function gets called recursively to check the chained PMR Term value -with the packet. - -**/ -cos_t *match_pmr_cos(cos_t *cos, const uint8_t *pkt_addr, pmr_t *pmr, - odp_packet_hdr_t *hdr); -/** -@internal -CoS associated with L3 QoS value - -This function returns the CoS associated with L3 QoS value -**/ -cos_t *match_qos_l3_cos(pmr_l3_cos_t *l3_cos, const uint8_t *pkt_addr, - odp_packet_hdr_t *hdr); - -/** -@internal -CoS associated with L2 QoS value - -This function returns the CoS associated with L2 QoS value -**/ -cos_t *match_qos_l2_cos(pmr_l2_cos_t *l2_cos, const uint8_t *pkt_addr, - odp_packet_hdr_t *hdr); -/** -@internal Flow Signature Calculation This function calculates the Flow Signature for a packet based on @@ -96,48 +58,6 @@ CoS and updates in Packet Meta Data **/ int update_flow_signature(uint8_t *pkt_addr, cos_t *cos); -/** -@internal -Allocate a odp_pmr_t Handle -*/ -odp_pmr_t alloc_pmr(pmr_t **pmr); - -/** -@internal -Pointer to pmr_t Handle -This function checks for validity of odp_pmr_t Handle -*/ -pmr_t *get_pmr_entry(odp_pmr_t pmr_id); - -/** -@internal -Pointer to pmr_t Handle -*/ -pmr_t *get_pmr_entry_internal(odp_pmr_t pmr_id); - -/** -@internal -Pointer to odp_cos_t Handle -*/ -cos_t *get_cos_entry(odp_cos_t cos_id); - -/** -@internal -Pointer to odp_cos_t Handle -This function checks for validity of odp_cos_t Handle -*/ -cos_t *get_cos_entry_internal(odp_cos_t cos_id); - -/** -@internal -Verify PMR with a Packet - -This function goes through each PMR_TERM value in pmr_t structure and -calls verification function for each term.Returns 1 if PMR matches or 0 -Otherwise. -**/ -int verify_pmr(pmr_t *pmr, const uint8_t *pkt_addr, odp_packet_hdr_t *pkt_hdr); - #ifdef __cplusplus } #endif diff --git a/platform/linux-generic/odp_classification.c b/platform/linux-generic/odp_classification.c index 08b09078..5e9173dc 100644 --- a/platform/linux-generic/odp_classification.c +++ b/platform/linux-generic/odp_classification.c @@ -31,11 +31,13 @@ static cos_tbl_t *cos_tbl; static pmr_tbl_t *pmr_tbl; +static cos_t *get_cos_entry_internal(odp_cos_t cos_id) { return &cos_tbl->cos_entry[_odp_typeval(cos_id)]; } +static pmr_t *get_pmr_entry_internal(odp_pmr_t pmr_id) { return &pmr_tbl->pmr[_odp_typeval(pmr_id)]; @@ -200,6 +202,10 @@ odp_cos_t odp_cls_cos_create(const char *name, odp_cls_cos_param_t *param) return ODP_COS_INVALID; } +/* + * Allocate an odp_pmr_t Handle + */ +static odp_pmr_t alloc_pmr(pmr_t **pmr) { int i; @@ -220,6 +226,7 @@ odp_pmr_t alloc_pmr(pmr_t **pmr) return ODP_PMR_INVAL; } +static cos_t *get_cos_entry(odp_cos_t cos_id) { if (_odp_typeval(cos_id) >= ODP_COS_MAX_ENTRY || @@ -230,6 +237,7 @@ cos_t *get_cos_entry(odp_cos_t cos_id) return &cos_tbl->cos_entry[_odp_typeval(cos_id)]; } +static pmr_t *get_pmr_entry(odp_pmr_t pmr_id) { if (_odp_typeval(pmr_id) >= ODP_PMR_MAX_ENTRY || @@ -599,6 +607,11 @@ odp_pool_t odp_cls_cos_pool(odp_cos_t cos_id) return cos->s.pool; } +/* + * This function goes through each PMR_TERM value in pmr_t structure and calls + * verification function for each term.Returns 1 if PMR matches or 0 otherwise. + */ +static int verify_pmr(pmr_t *pmr, const uint8_t *pkt_addr, odp_packet_hdr_t *pkt_hdr) { int pmr_failure = 0; @@ -717,6 +730,12 @@ int verify_pmr(pmr_t *pmr, const uint8_t *pkt_addr, odp_packet_hdr_t *pkt_hdr) return true; } +/* + * Match a PMR chain with a Packet and return matching CoS + * This function gets called recursively to check the chained PMR Term value + * with the packet. + */ +static cos_t *match_pmr_cos(cos_t *cos, const uint8_t *pkt_addr, pmr_t *pmr, odp_packet_hdr_t *hdr) { @@ -763,6 +782,10 @@ int pktio_classifier_init(pktio_entry_t *entry) return 0; } +static +cos_t *match_qos_cos(pktio_entry_t *entry, const uint8_t *pkt_addr, + odp_packet_hdr_t *hdr); + /** Select a CoS for the given Packet based on pktio @@ -848,6 +871,7 @@ int cls_classify_packet(pktio_entry_t *entry, const uint8_t *base, return 0; } +static cos_t *match_qos_l3_cos(pmr_l3_cos_t *l3_cos, const uint8_t *pkt_addr, odp_packet_hdr_t *hdr) { @@ -869,6 +893,7 @@ cos_t *match_qos_l3_cos(pmr_l3_cos_t *l3_cos, const uint8_t *pkt_addr, return cos; } +static cos_t *match_qos_l2_cos(pmr_l2_cos_t *l2_cos, const uint8_t *pkt_addr, odp_packet_hdr_t *hdr) { @@ -888,6 +913,12 @@ cos_t *match_qos_l2_cos(pmr_l2_cos_t *l2_cos, const uint8_t *pkt_addr, return cos; } +/* + * Select a CoS for the given Packet based on QoS values + * This function returns the COS object matching the L2 and L3 QoS + * based on the l3_preference value of the pktio +*/ +static cos_t *match_qos_cos(pktio_entry_t *entry, const uint8_t *pkt_addr, odp_packet_hdr_t *hdr) { commit 757439849936bb76885f27677fc8997a67fa2a3c Author: Dmitry Eremin-Solenikov <dmitry.ereminsolenikov(a)linaro.org> Date: Fri Aug 4 11:23:13 2017 +0300 linux-gen: fdserver: mark client_lock as static 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/_fdserver.c b/platform/linux-generic/_fdserver.c index 4f74c614..d1b16f62 100644 --- a/platform/linux-generic/_fdserver.c +++ b/platform/linux-generic/_fdserver.c @@ -69,7 +69,7 @@ #endif /* when accessing the client functions, clients should be mutexed: */ -odp_spinlock_t *client_lock; +static odp_spinlock_t *client_lock; /* define the tables of file descriptors handled by this server: */ #define FDSERVER_MAX_ENTRIES 256 commit 92ad20c03bf4737558265a1ff942666d7c4ec3d8 Author: Dmitry Eremin-Solenikov <dmitry.ereminsolenikov(a)linaro.org> Date: Fri Aug 4 11:22:44 2017 +0300 linux-gen: pool: drop two unused functions 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/include/odp_buffer_internal.h b/platform/linux-generic/include/odp_buffer_internal.h index 20127004..a580c82d 100644 --- a/platform/linux-generic/include/odp_buffer_internal.h +++ b/platform/linux-generic/include/odp_buffer_internal.h @@ -107,10 +107,6 @@ ODP_STATIC_ASSERT(CONFIG_PACKET_MAX_SEGS < 256, ODP_STATIC_ASSERT(BUFFER_BURST_SIZE < 256, "BUFFER_BURST_SIZE_TOO_LARGE"); -/* Forward declarations */ -int seg_alloc_tail(odp_buffer_hdr_t *buf_hdr, int segcount); -void seg_free_tail(odp_buffer_hdr_t *buf_hdr, int segcount); - #ifdef __cplusplus } #endif diff --git a/platform/linux-generic/odp_pool.c b/platform/linux-generic/odp_pool.c index f2410b6e..c739adbc 100644 --- a/platform/linux-generic/odp_pool.c +++ b/platform/linux-generic/odp_pool.c @@ -890,19 +890,6 @@ uint64_t odp_pool_to_u64(odp_pool_t hdl) return _odp_pri(hdl); } -int seg_alloc_tail(odp_buffer_hdr_t *buf_hdr, int segcount) -{ - (void)buf_hdr; - (void)segcount; - return 0; -} - -void seg_free_tail(odp_buffer_hdr_t *buf_hdr, int segcount) -{ - (void)buf_hdr; - (void)segcount; -} - int odp_buffer_is_valid(odp_buffer_t buf) { pool_t *pool; commit fd2ca1209dc19c8e8199ee556ad2c9a786eabf78 Author: Dmitry Eremin-Solenikov <dmitry.ereminsolenikov(a)linaro.org> Date: Fri Aug 4 11:21:31 2017 +0300 linux-gen: hide get_qentry Replace two last index->qentry lookups with pointers to quentry, thus making it possible to make get_qentry static in odp_queue.c 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/include/odp_queue_internal.h b/platform/linux-generic/include/odp_queue_internal.h index 032dde88..c5ff1e7d 100644 --- a/platform/linux-generic/include/odp_queue_internal.h +++ b/platform/linux-generic/include/odp_queue_internal.h @@ -61,8 +61,6 @@ union queue_entry_u { uint8_t pad[ROUNDUP_CACHE_LINE(sizeof(struct queue_entry_s))]; }; -queue_entry_t *get_qentry(uint32_t queue_id); - void queue_lock(queue_entry_t *queue); void queue_unlock(queue_entry_t *queue); diff --git a/platform/linux-generic/odp_queue.c b/platform/linux-generic/odp_queue.c index 904c9ef2..e2f8c067 100644 --- a/platform/linux-generic/odp_queue.c +++ b/platform/linux-generic/odp_queue.c @@ -43,6 +43,9 @@ typedef struct queue_table_t { static queue_table_t *queue_tbl; +static +queue_entry_t *get_qentry(uint32_t queue_id); + static inline queue_entry_t *handle_to_qentry(odp_queue_t handle) { uint32_t queue_id; @@ -56,6 +59,7 @@ static inline odp_queue_t queue_from_id(uint32_t queue_id) return _odp_cast_scalar(odp_queue_t, queue_id + 1); } +static queue_entry_t *get_qentry(uint32_t queue_id) { return &queue_tbl->queue[queue_id]; diff --git a/platform/linux-generic/odp_schedule.c b/platform/linux-generic/odp_schedule.c index bbcc85a5..4c99f602 100644 --- a/platform/linux-generic/odp_schedule.c +++ b/platform/linux-generic/odp_schedule.c @@ -130,7 +130,7 @@ ODP_STATIC_ASSERT((8 * sizeof(pri_mask_t)) >= QUEUES_PER_PRIO, /* Storage for stashed enqueue operation arguments */ typedef struct { odp_buffer_hdr_t *buf_hdr[QUEUE_MULTI_MAX]; - uint32_t queue_index; + queue_entry_t *queue_entry; int num; } ordered_stash_t; @@ -682,12 +682,10 @@ static inline void ordered_stash_release(void) for (i = 0; i < sched_local.ordered.stash_num; i++) { queue_entry_t *queue_entry; - uint32_t queue_index; odp_buffer_hdr_t **buf_hdr; int num; - queue_index = sched_local.ordered.stash[i].queue_index; - queue_entry = get_qentry(queue_index); + queue_entry = sched_local.ordered.stash[i].queue_entry; buf_hdr = sched_local.ordered.stash[i].buf_hdr; num = sched_local.ordered.stash[i].num; @@ -788,7 +786,7 @@ static int schedule_ord_enq_multi(queue_t q_int, void *buf_hdr[], return 0; } - sched_local.ordered.stash[stash_num].queue_index = dst_queue->s.index; + sched_local.ordered.stash[stash_num].queue_entry = dst_queue; sched_local.ordered.stash[stash_num].num = num; for (i = 0; i < num; i++) sched_local.ordered.stash[stash_num].buf_hdr[i] = buf_hdr[i]; diff --git a/platform/linux-generic/odp_schedule_iquery.c b/platform/linux-generic/odp_schedule_iquery.c index d374dedd..ee2be031 100644 --- a/platform/linux-generic/odp_schedule_iquery.c +++ b/platform/linux-generic/odp_schedule_iquery.c @@ -170,7 +170,7 @@ typedef struct { /* Storage for stashed enqueue operation arguments */ typedef struct { odp_buffer_hdr_t *buf_hdr[QUEUE_MULTI_MAX]; - uint32_t queue_index; + queue_entry_t *queue_entry; int num; } ordered_stash_t; @@ -1127,12 +1127,10 @@ static inline void ordered_stash_release(void) for (i = 0; i < thread_local.ordered.stash_num; i++) { queue_entry_t *queue_entry; - uint32_t queue_index; odp_buffer_hdr_t **buf_hdr; int num; - queue_index = thread_local.ordered.stash[i].queue_index; - queue_entry = get_qentry(queue_index); + queue_entry = thread_local.ordered.stash[i].queue_entry; buf_hdr = thread_local.ordered.stash[i].buf_hdr; num = thread_local.ordered.stash[i].num; @@ -1219,7 +1217,7 @@ static int schedule_ord_enq_multi(queue_t q_int, void *buf_hdr[], return 0; } - thread_local.ordered.stash[stash_num].queue_index = dst_queue->s.index; + thread_local.ordered.stash[stash_num].queue_entry = dst_queue; thread_local.ordered.stash[stash_num].num = num; for (i = 0; i < num; i++) thread_local.ordered.stash[stash_num].buf_hdr[i] = buf_hdr[i]; ----------------------------------------------------------------------- Summary of changes: platform/linux-generic/_fdserver.c | 2 +- .../linux-generic/include/odp_buffer_internal.h | 4 -- .../include/odp_classification_internal.h | 80 ---------------------- .../linux-generic/include/odp_packet_io_internal.h | 13 ---- .../linux-generic/include/odp_queue_internal.h | 5 -- platform/linux-generic/include/odp_schedule_if.h | 1 - platform/linux-generic/odp_classification.c | 31 +++++++++ platform/linux-generic/odp_packet_io.c | 28 ++++---- platform/linux-generic/odp_pool.c | 13 ---- platform/linux-generic/odp_queue.c | 14 ++-- platform/linux-generic/odp_schedule.c | 10 ++- platform/linux-generic/odp_schedule_iquery.c | 10 ++- 12 files changed, 56 insertions(+), 155 deletions(-) hooks/post-receive --

6 years, 8 months

1
0
0 0

[COMMIT] branch api-next updated. v1.15.0.0-545-g2394317a

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 via 2394317ab247fa14b4e239aec512afee7eac4524 (commit) from 09ce9406e09eacadf8f739ae8499323ad1401d36 (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 2394317ab247fa14b4e239aec512afee7eac4524 Author: Maxim Uvarov <maxim.uvarov(a)linaro.org> Date: Mon Aug 7 16:07:58 2017 +0300 Revert "api: schedule: add thread removal correction" This reverts commit 09ce9406 api: schedule: add thread removal correction Needed some more discussion before merging this. Signed-off-by: Maxim Uvarov <maxim.uvarov(a)linaro.org> diff --git a/include/odp/api/spec/schedule.h b/include/odp/api/spec/schedule.h index f1671462..8244746d 100644 --- a/include/odp/api/spec/schedule.h +++ b/include/odp/api/spec/schedule.h @@ -234,9 +234,8 @@ odp_schedule_group_t odp_schedule_group_create(const char *name, * Schedule group destroy * * Destroys a schedule group. All queues belonging to the schedule group must - * be destroyed before destroying the group. All threads that have joined - * the group must leave before destroying the group. Other operations on this - * group must not be invoked in parallel. + * be destroyed before destroying the group. Other operations on this group + * must not be invoked in parallel. * * @param group Schedule group handle * ----------------------------------------------------------------------- Summary of changes: include/odp/api/spec/schedule.h | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) hooks/post-receive --

6 years, 9 months

1
0
0 0

[COMMIT] branch api-next updated. v1.15.0.0-544-g09ce9406

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 via 09ce9406e09eacadf8f739ae8499323ad1401d36 (commit) from a165ff4093f666e5481ae5113f70b41e3354520d (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 09ce9406e09eacadf8f739ae8499323ad1401d36 Author: Honnappa Nagarahalli <honnappa.nagarahalli(a)arm.com> Date: Wed Jun 14 16:48:24 2017 -0500 api: schedule: add thread removal correction Clarify thread removal status from scheduler group before calling odp_schedule_group_destroy. Signed-off-by: Honnappa Nagarahalli <honnappa.nagarahalli(a)arm.com> Reviewed-by: Brian Brooks <brian.brooks(a)arm.com> Reviewed-by: Ola Liljedahl <ola.liljedahl(a)arm.com> Reviewed-by: Bill Fischofer <bill.fischofer(a)linaro.org> Signed-off-by: Maxim Uvarov <maxim.uvarov(a)linaro.org> diff --git a/include/odp/api/spec/schedule.h b/include/odp/api/spec/schedule.h index 8244746d..f1671462 100644 --- a/include/odp/api/spec/schedule.h +++ b/include/odp/api/spec/schedule.h @@ -234,8 +234,9 @@ odp_schedule_group_t odp_schedule_group_create(const char *name, * Schedule group destroy * * Destroys a schedule group. All queues belonging to the schedule group must - * be destroyed before destroying the group. Other operations on this group - * must not be invoked in parallel. + * be destroyed before destroying the group. All threads that have joined + * the group must leave before destroying the group. Other operations on this + * group must not be invoked in parallel. * * @param group Schedule group handle * ----------------------------------------------------------------------- Summary of changes: include/odp/api/spec/schedule.h | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) hooks/post-receive --

6 years, 9 months

1
0
0 0

[COMMIT] branch api-next updated. v1.15.0.0-543-ga165ff40

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 via a165ff4093f666e5481ae5113f70b41e3354520d (commit) via 41abc837a5c25d8df3429398bcb51c72b83999e5 (commit) via e76b25aa5cafbf96133fcbd83b856090b0e95f33 (commit) via ab62d19d4811e2699ac0614bf4ba1d481bb3b2b1 (commit) via 641717bac482831a8f9702af7f941cfc349ada91 (commit) via c5436c447df98e0ec78de3620c7fe48141fef760 (commit) via 2fff8b0247a0da108c45c9febc1029bc8e12d347 (commit) from 8390f890d4bd2babb63a24f7b15d2f4763e44050 (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 a165ff4093f666e5481ae5113f70b41e3354520d Author: Dmitry Eremin-Solenikov <dmitry.ereminsolenikov(a)linaro.org> Date: Mon Jul 24 16:57:30 2017 +0300 api: crypto: clarify compl_queue queue in crypto_session_param Properly mention odp_crypto_operation() in the compl_queue param description. 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/include/odp/api/spec/crypto.h b/include/odp/api/spec/crypto.h index 6d4807c4..3b8e9ddb 100644 --- a/include/odp/api/spec/crypto.h +++ b/include/odp/api/spec/crypto.h @@ -312,8 +312,9 @@ typedef struct odp_crypto_session_param_t { /** Async mode completion event queue * - * The completion queue is used to return odp_crypto_op_enq() - * results to the application. + * The completion queue is used to return completions from + * odp_crypto_operation() or odp_crypto_op_enq() results to the + * application. */ odp_queue_t compl_queue; commit 41abc837a5c25d8df3429398bcb51c72b83999e5 Merge: 8390f890 e76b25aa Author: Maxim Uvarov <maxim.uvarov(a)linaro.org> Date: Fri Aug 4 19:56:42 2017 +0300 Merge branch 'master' into api-next Signed-off-by: Maxim Uvarov <maxim.uvarov(a)linaro.org> ----------------------------------------------------------------------- Summary of changes: include/odp/api/spec/crypto.h | 5 +- platform/linux-generic/odp_classification.c | 3 - platform/linux-generic/odp_packet_io.c | 2 + platform/linux-generic/odp_pkt_queue.c | 39 +++++- .../validation/api/classification/classification.h | 1 + .../api/classification/odp_classification_basic.c | 4 +- .../api/classification/odp_classification_common.c | 5 +- .../classification/odp_classification_test_pmr.c | 131 ++++++++++++++++++--- .../api/classification/odp_classification_tests.c | 2 + .../classification/odp_classification_testsuites.h | 3 +- 10 files changed, 164 insertions(+), 31 deletions(-) hooks/post-receive --

6 years, 9 months

1
0
0 0

[COMMIT] branch master updated. v1.15.0.0-103-ge76b25aa

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 e76b25aa5cafbf96133fcbd83b856090b0e95f33 (commit) via ab62d19d4811e2699ac0614bf4ba1d481bb3b2b1 (commit) from 641717bac482831a8f9702af7f941cfc349ada91 (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 e76b25aa5cafbf96133fcbd83b856090b0e95f33 Author: Ilias Apalodimas <ilias.apalodimas(a)linaro.org> Date: Thu Aug 3 21:16:34 2017 +0300 linux-gen: queue: added additional missing free Added 2 additional missing frees in _odp_queue_pool_create() as requested on review. Signed-off-by: Ilias Apalodimas <ilias.apalodimas(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_pkt_queue.c b/platform/linux-generic/odp_pkt_queue.c index 06420a96..f1086096 100644 --- a/platform/linux-generic/odp_pkt_queue.c +++ b/platform/linux-generic/odp_pkt_queue.c @@ -241,8 +241,11 @@ _odp_int_queue_pool_t _odp_queue_pool_create(uint32_t max_num_queues, */ initial_free_list_size = MIN(64 * 1024, max_queued_pkts / 4); rc = pkt_queue_free_list_add(pool, initial_free_list_size); - if (rc < 0) + if (rc < 0) { + free(pool->queue_num_tbl); + free(pool); return _ODP_INT_QUEUE_POOL_INVALID; + } /* Discard the first queue blk with idx 0 */ queue_blk_alloc(pool, &first_queue_blk_idx); commit ab62d19d4811e2699ac0614bf4ba1d481bb3b2b1 Author: Ilias Apalodimas <apalos(a)gmail.com> Date: Wed Aug 2 11:15:14 2017 +0300 linux-gen: queue: checks on memory allocations Fixes: https://bugs.linaro.org/show_bug.cgi?id=3182 by adding proper checks on allocated memory. Signed-off-by: Ilias Apalodimas <ilias.apalodimas(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_pkt_queue.c b/platform/linux-generic/odp_pkt_queue.c index 7c6cd87e..06420a96 100644 --- a/platform/linux-generic/odp_pkt_queue.c +++ b/platform/linux-generic/odp_pkt_queue.c @@ -85,17 +85,27 @@ static queue_blk_t *blk_idx_to_queue_blk(queue_pool_t *queue_pool, return &queue_region_desc->queue_blks->blks[blk_tbl_idx]; } +static void free_alloced_queue_blks(uint32_t end, queue_blks_t *blk_array[]) +{ + uint32_t i; + + for (i = 0; i < end; i++) + free(blk_array[i]); +} + static int pkt_queue_free_list_add(queue_pool_t *pool, uint32_t num_queue_blks) { queue_region_desc_t *region_desc; queue_blks_t *queue_blks; + queue_blks_t *alloced_queue_blks[num_queue_blks]; queue_blk_t *queue_blk; uint32_t which_region, blks_added, num_blks, start_idx; - uint32_t malloc_len, blks_to_add, cnt, i; + uint32_t malloc_len, blks_to_add, cnt, i, alloc_cnt; which_region = pool->current_region; blks_added = 0; + alloc_cnt = 0; while ((blks_added < num_queue_blks) && (pool->all_regions_used == 0)) { region_desc = &pool->queue_region_descs[which_region]; start_idx = region_desc->next_blk_idx; @@ -104,6 +114,13 @@ static int pkt_queue_free_list_add(queue_pool_t *pool, if (!queue_blks) { malloc_len = num_blks * sizeof(queue_blk_t); queue_blks = malloc(malloc_len); + if (!queue_blks) { + free_alloced_queue_blks(alloc_cnt, + alloced_queue_blks); + return -1; + } + alloced_queue_blks[alloc_cnt] = queue_blks; + alloc_cnt++; for (i = 0; i < num_blks; i++) init_queue_blk(&queue_blks->blks[i]); @@ -194,8 +211,19 @@ _odp_int_queue_pool_t _odp_queue_pool_create(uint32_t max_num_queues, int rc; pool = malloc(sizeof(queue_pool_t)); + if (!pool) + return _ODP_INT_QUEUE_POOL_INVALID; + memset(pool, 0, sizeof(queue_pool_t)); + malloc_len = max_num_queues * sizeof(uint32_t); + pool->queue_num_tbl = malloc(malloc_len); + if (!pool->queue_num_tbl) { + free(pool); + return _ODP_INT_QUEUE_POOL_INVALID; + } + memset(pool->queue_num_tbl, 0, malloc_len); + /* Initialize the queue_blk_tbl_sizes array based upon the * max_queued_pkts. */ @@ -223,10 +251,6 @@ _odp_int_queue_pool_t _odp_queue_pool_create(uint32_t max_num_queues, pool->max_queued_pkts = max_queued_pkts; pool->next_queue_num = 1; - malloc_len = max_num_queues * sizeof(uint32_t); - pool->queue_num_tbl = malloc(malloc_len); - memset(pool->queue_num_tbl, 0, malloc_len); - pool->min_free_list_size = pool->free_list_size; pool->peak_free_list_size = pool->free_list_size; return (_odp_int_queue_pool_t)(uintptr_t)pool; ----------------------------------------------------------------------- Summary of changes: platform/linux-generic/odp_pkt_queue.c | 39 ++++++++++++++++++++++++++++------ 1 file changed, 33 insertions(+), 6 deletions(-) hooks/post-receive --

6 years, 9 months

1
0
0 0

[COMMIT] branch master updated. v1.15.0.0-101-g641717ba

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 641717bac482831a8f9702af7f941cfc349ada91 (commit) via c5436c447df98e0ec78de3620c7fe48141fef760 (commit) via 2fff8b0247a0da108c45c9febc1029bc8e12d347 (commit) from 4537d096721a753086592ed5a989beab5647bcfa (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 641717bac482831a8f9702af7f941cfc349ada91 Author: Balasubramanian Manoharan <bala.manoharan(a)linaro.org> Date: Wed Aug 2 07:12:00 2017 -0700 validation: classification: add test case for classifier enable boolean updates test suite to enable classifier in odp_pktin_queue_param_t config adds test case for classifier enable boolean Fixes https://bugs.linaro.org/show_bug.cgi?id=3177 Signed-off-by: Balasubramanian Manoharan <bala.manoharan(a)linaro.org> Signed-off-by: Maxim Uvarov <maxim.uvarov(a)linaro.org> diff --git a/test/common_plat/validation/api/classification/classification.h b/test/common_plat/validation/api/classification/classification.h index 53c527f3..326177ad 100644 --- a/test/common_plat/validation/api/classification/classification.h +++ b/test/common_plat/validation/api/classification/classification.h @@ -102,6 +102,7 @@ void classification_test_pmr_term_vlan_id_0(void); void classification_test_pmr_term_vlan_id_x(void); void classification_test_pmr_term_eth_type_0(void); void classification_test_pmr_term_eth_type_x(void); +void classification_test_pktin_classifier_flag(void); /* test arrays: */ extern odp_testinfo_t classification_suite_basic[]; diff --git a/test/common_plat/validation/api/classification/odp_classification_basic.c b/test/common_plat/validation/api/classification/odp_classification_basic.c index 9817287e..3f4a82ed 100644 --- a/test/common_plat/validation/api/classification/odp_classification_basic.c +++ b/test/common_plat/validation/api/classification/odp_classification_basic.c @@ -87,7 +87,7 @@ void classification_test_create_pmr_match(void) pkt_pool = pool_create("pkt_pool"); CU_ASSERT_FATAL(pkt_pool != ODP_POOL_INVALID); - pktio = create_pktio(ODP_QUEUE_TYPE_SCHED, pkt_pool); + pktio = create_pktio(ODP_QUEUE_TYPE_SCHED, pkt_pool, true); CU_ASSERT_FATAL(pktio != ODP_PKTIO_INVALID); configure_default_cos(pktio, &default_cos, @@ -271,7 +271,7 @@ void classification_test_pmr_composite_create(void) pkt_pool = pool_create("pkt_pool"); CU_ASSERT_FATAL(pkt_pool != ODP_POOL_INVALID); - pktio = create_pktio(ODP_QUEUE_TYPE_SCHED, pkt_pool); + pktio = create_pktio(ODP_QUEUE_TYPE_SCHED, pkt_pool, true); CU_ASSERT_FATAL(pktio != ODP_PKTIO_INVALID); configure_default_cos(pktio, &default_cos, diff --git a/test/common_plat/validation/api/classification/odp_classification_common.c b/test/common_plat/validation/api/classification/odp_classification_common.c index eca30b87..de8a9327 100644 --- a/test/common_plat/validation/api/classification/odp_classification_common.c +++ b/test/common_plat/validation/api/classification/odp_classification_common.c @@ -24,7 +24,8 @@ static uint8_t IPV6_DST_ADDR[ODPH_IPV6ADDR_LEN] = { 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0xFF, 0xFF, 10, 0, 0, 100 }; -odp_pktio_t create_pktio(odp_queue_type_t q_type, odp_pool_t pool) +odp_pktio_t create_pktio(odp_queue_type_t q_type, odp_pool_t pool, + odp_bool_t cls_enable) { odp_pktio_t pktio; odp_pktio_param_t pktio_param; @@ -50,6 +51,8 @@ odp_pktio_t create_pktio(odp_queue_type_t q_type, odp_pool_t pool) odp_pktin_queue_param_init(&pktin_param); pktin_param.queue_param.sched.sync = ODP_SCHED_SYNC_ATOMIC; + pktin_param.classifier_enable = cls_enable; + pktin_param.hash_enable = false; if (odp_pktin_queue_config(pktio, &pktin_param)) { fprintf(stderr, "pktin queue config failed.\n"); diff --git a/test/common_plat/validation/api/classification/odp_classification_test_pmr.c b/test/common_plat/validation/api/classification/odp_classification_test_pmr.c index d9524205..e6b496a1 100644 --- a/test/common_plat/validation/api/classification/odp_classification_test_pmr.c +++ b/test/common_plat/validation/api/classification/odp_classification_test_pmr.c @@ -86,6 +86,104 @@ int classification_suite_pmr_term(void) return retcode; } +void classification_test_pktin_classifier_flag(void) +{ + odp_packet_t pkt; + odph_tcphdr_t *tcp; + uint32_t seqno; + uint16_t val; + uint16_t mask; + int retval; + odp_pktio_t pktio; + odp_queue_t queue; + odp_queue_t retqueue; + odp_queue_t default_queue; + odp_cos_t default_cos; + odp_pool_t default_pool; + odp_pmr_t pmr; + odp_cos_t cos; + char cosname[ODP_COS_NAME_LEN]; + odp_cls_cos_param_t cls_param; + odp_pool_t pool; + odp_pool_t pool_recv; + odp_pmr_param_t pmr_param; + odph_ethhdr_t *eth; + + val = CLS_DEFAULT_DPORT; + mask = 0xffff; + seqno = 0; + + /* classifier is disabled in pktin queue configuration */ + pktio = create_pktio(ODP_QUEUE_TYPE_SCHED, pkt_pool, false); + CU_ASSERT_FATAL(pktio != ODP_PKTIO_INVALID); + retval = start_pktio(pktio); + CU_ASSERT(retval == 0); + + configure_default_cos(pktio, &default_cos, + &default_queue, &default_pool); + + queue = queue_create("tcp_dport1", true); + CU_ASSERT(queue != ODP_QUEUE_INVALID); + + pool = pool_create("tcp_dport1"); + CU_ASSERT_FATAL(pool != ODP_POOL_INVALID); + + sprintf(cosname, "tcp_dport"); + odp_cls_cos_param_init(&cls_param); + cls_param.pool = pool; + cls_param.queue = queue; + cls_param.drop_policy = ODP_COS_DROP_POOL; + + cos = odp_cls_cos_create(cosname, &cls_param); + CU_ASSERT(cos != ODP_COS_INVALID); + + odp_cls_pmr_param_init(&pmr_param); + pmr_param.term = ODP_PMR_TCP_DPORT; + pmr_param.match.value = &val; + pmr_param.match.mask = &mask; + pmr_param.val_sz = sizeof(val); + + pmr = odp_cls_pmr_create(&pmr_param, 1, default_cos, cos); + CU_ASSERT(pmr != ODP_PMR_INVAL); + + pkt = create_packet(default_pkt_info); + CU_ASSERT_FATAL(pkt != ODP_PACKET_INVALID); + seqno = cls_pkt_get_seq(pkt); + CU_ASSERT(seqno != TEST_SEQ_INVALID); + eth = (odph_ethhdr_t *)odp_packet_l2_ptr(pkt, NULL); + odp_pktio_mac_addr(pktio, eth->src.addr, ODPH_ETHADDR_LEN); + odp_pktio_mac_addr(pktio, eth->dst.addr, ODPH_ETHADDR_LEN); + + tcp = (odph_tcphdr_t *)odp_packet_l4_ptr(pkt, NULL); + tcp->dst_port = odp_cpu_to_be_16(CLS_DEFAULT_DPORT); + + enqueue_pktio_interface(pkt, pktio); + + /* since classifier flag is disabled in pktin queue configuration + packet will not be delivered in classifier queues */ + pkt = receive_packet(&retqueue, ODP_TIME_SEC_IN_NS); + CU_ASSERT_FATAL(pkt != ODP_PACKET_INVALID); + pool_recv = odp_packet_pool(pkt); + /* since classifier is disabled packet should not be received in + pool and queue configured with classifier */ + CU_ASSERT(pool != pool_recv); + CU_ASSERT(retqueue != queue); + CU_ASSERT(seqno == cls_pkt_get_seq(pkt)); + + odp_packet_free(pkt); + + odp_packet_free(pkt); + odp_cos_destroy(cos); + odp_cos_destroy(default_cos); + odp_cls_pmr_destroy(pmr); + stop_pktio(pktio); + odp_queue_destroy(queue); + odp_queue_destroy(default_queue); + odp_pool_destroy(pool); + odp_pool_destroy(default_pool); + odp_pktio_close(pktio); +} + void classification_test_pmr_term_tcp_dport(void) { odp_packet_t pkt; @@ -113,7 +211,7 @@ void classification_test_pmr_term_tcp_dport(void) mask = 0xffff; seqno = 0; - pktio = create_pktio(ODP_QUEUE_TYPE_SCHED, pkt_pool); + pktio = create_pktio(ODP_QUEUE_TYPE_SCHED, pkt_pool, true); CU_ASSERT_FATAL(pktio != ODP_PKTIO_INVALID); retval = start_pktio(pktio); CU_ASSERT(retval == 0); @@ -227,7 +325,7 @@ void classification_test_pmr_term_tcp_sport(void) mask = 0xffff; seqno = 0; - pktio = create_pktio(ODP_QUEUE_TYPE_SCHED, pkt_pool); + pktio = create_pktio(ODP_QUEUE_TYPE_SCHED, pkt_pool, true); CU_ASSERT_FATAL(pktio != ODP_PKTIO_INVALID); retval = start_pktio(pktio); CU_ASSERT(retval == 0); @@ -340,7 +438,7 @@ void classification_test_pmr_term_udp_dport(void) mask = 0xffff; seqno = 0; - pktio = create_pktio(ODP_QUEUE_TYPE_SCHED, pkt_pool); + pktio = create_pktio(ODP_QUEUE_TYPE_SCHED, pkt_pool, true); CU_ASSERT_FATAL(pktio != ODP_PKTIO_INVALID); retval = start_pktio(pktio); CU_ASSERT(retval == 0); @@ -456,7 +554,7 @@ void classification_test_pmr_term_udp_sport(void) mask = 0xffff; seqno = 0; - pktio = create_pktio(ODP_QUEUE_TYPE_SCHED, pkt_pool); + pktio = create_pktio(ODP_QUEUE_TYPE_SCHED, pkt_pool, true); CU_ASSERT_FATAL(pktio != ODP_PKTIO_INVALID); retval = start_pktio(pktio); CU_ASSERT(retval == 0); @@ -570,7 +668,7 @@ void classification_test_pmr_term_ipproto(void) mask = 0xff; seqno = 0; - pktio = create_pktio(ODP_QUEUE_TYPE_SCHED, pkt_pool); + pktio = create_pktio(ODP_QUEUE_TYPE_SCHED, pkt_pool, true); CU_ASSERT_FATAL(pktio != ODP_PKTIO_INVALID); retval = start_pktio(pktio); CU_ASSERT(retval == 0); @@ -679,7 +777,7 @@ void classification_test_pmr_term_dmac(void) mask = 0xffffffffffff; seqno = 0; - pktio = create_pktio(ODP_QUEUE_TYPE_SCHED, pkt_pool); + pktio = create_pktio(ODP_QUEUE_TYPE_SCHED, pkt_pool, true); CU_ASSERT_FATAL(pktio != ODP_PKTIO_INVALID); retval = start_pktio(pktio); CU_ASSERT(retval == 0); @@ -786,7 +884,7 @@ void classification_test_pmr_term_packet_len(void) mask = 0xff00; seqno = 0; - pktio = create_pktio(ODP_QUEUE_TYPE_SCHED, pkt_pool); + pktio = create_pktio(ODP_QUEUE_TYPE_SCHED, pkt_pool, true); CU_ASSERT_FATAL(pktio != ODP_PKTIO_INVALID); retval = start_pktio(pktio); CU_ASSERT(retval == 0); @@ -898,7 +996,7 @@ void classification_test_pmr_term_vlan_id_0(void) mask = 0xff00; seqno = 0; - pktio = create_pktio(ODP_QUEUE_TYPE_SCHED, pkt_pool); + pktio = create_pktio(ODP_QUEUE_TYPE_SCHED, pkt_pool, true); CU_ASSERT_FATAL(pktio != ODP_PKTIO_INVALID); retval = start_pktio(pktio); CU_ASSERT(retval == 0); @@ -1011,7 +1109,7 @@ void classification_test_pmr_term_vlan_id_x(void) mask = 0xff00; seqno = 0; - pktio = create_pktio(ODP_QUEUE_TYPE_SCHED, pkt_pool); + pktio = create_pktio(ODP_QUEUE_TYPE_SCHED, pkt_pool, true); CU_ASSERT_FATAL(pktio != ODP_PKTIO_INVALID); retval = start_pktio(pktio); CU_ASSERT(retval == 0); @@ -1124,7 +1222,7 @@ void classification_test_pmr_term_eth_type_0(void) mask = 0xffff; seqno = 0; - pktio = create_pktio(ODP_QUEUE_TYPE_SCHED, pkt_pool); + pktio = create_pktio(ODP_QUEUE_TYPE_SCHED, pkt_pool, true); CU_ASSERT_FATAL(pktio != ODP_PKTIO_INVALID); retval = start_pktio(pktio); CU_ASSERT(retval == 0); @@ -1234,7 +1332,7 @@ void classification_test_pmr_term_eth_type_x(void) mask = 0xff00; seqno = 0; - pktio = create_pktio(ODP_QUEUE_TYPE_SCHED, pkt_pool); + pktio = create_pktio(ODP_QUEUE_TYPE_SCHED, pkt_pool, true); CU_ASSERT_FATAL(pktio != ODP_PKTIO_INVALID); retval = start_pktio(pktio); CU_ASSERT(retval == 0); @@ -1347,7 +1445,7 @@ static void classification_test_pmr_pool_set(void) mask = 0xff; seqno = 0; - pktio = create_pktio(ODP_QUEUE_TYPE_SCHED, pkt_pool); + pktio = create_pktio(ODP_QUEUE_TYPE_SCHED, pkt_pool, true); CU_ASSERT_FATAL(pktio != ODP_PKTIO_INVALID); retval = start_pktio(pktio); CU_ASSERT(retval == 0); @@ -1446,7 +1544,7 @@ static void classification_test_pmr_queue_set(void) mask = 0xff; seqno = 0; - pktio = create_pktio(ODP_QUEUE_TYPE_SCHED, pkt_pool); + pktio = create_pktio(ODP_QUEUE_TYPE_SCHED, pkt_pool, true); CU_ASSERT_FATAL(pktio != ODP_PKTIO_INVALID); retval = start_pktio(pktio); CU_ASSERT(retval == 0); @@ -1539,7 +1637,7 @@ static void classification_test_pmr_term_daddr(void) const char *dst_addr = "10.0.0.99/32"; odph_ethhdr_t *eth; - pktio = create_pktio(ODP_QUEUE_TYPE_SCHED, pkt_pool); + pktio = create_pktio(ODP_QUEUE_TYPE_SCHED, pkt_pool, true); retval = start_pktio(pktio); CU_ASSERT(retval == 0); @@ -1650,7 +1748,7 @@ static void classification_test_pmr_term_ipv6daddr(void) 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF }; - pktio = create_pktio(ODP_QUEUE_TYPE_SCHED, pkt_pool); + pktio = create_pktio(ODP_QUEUE_TYPE_SCHED, pkt_pool, true); retval = start_pktio(pktio); CU_ASSERT(retval == 0); @@ -1760,7 +1858,7 @@ static void classification_test_pmr_term_ipv6saddr(void) 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF }; - pktio = create_pktio(ODP_QUEUE_TYPE_SCHED, pkt_pool); + pktio = create_pktio(ODP_QUEUE_TYPE_SCHED, pkt_pool, true); retval = start_pktio(pktio); CU_ASSERT(retval == 0); @@ -1860,5 +1958,6 @@ odp_testinfo_t classification_suite_pmr[] = { ODP_TEST_INFO(classification_test_pmr_term_vlan_id_x), ODP_TEST_INFO(classification_test_pmr_term_eth_type_0), ODP_TEST_INFO(classification_test_pmr_term_eth_type_x), + ODP_TEST_INFO(classification_test_pktin_classifier_flag), ODP_TEST_INFO_NULL, }; diff --git a/test/common_plat/validation/api/classification/odp_classification_tests.c b/test/common_plat/validation/api/classification/odp_classification_tests.c index 4f430826..a0f73e39 100644 --- a/test/common_plat/validation/api/classification/odp_classification_tests.c +++ b/test/common_plat/validation/api/classification/odp_classification_tests.c @@ -52,6 +52,8 @@ int classification_suite_init(void) odp_pktin_queue_param_init(&pktin_param); pktin_param.queue_param.sched.sync = ODP_SCHED_SYNC_ATOMIC; + pktin_param.classifier_enable = true; + pktin_param.hash_enable = false; if (odp_pktin_queue_config(pktio_loop, &pktin_param)) { fprintf(stderr, "pktin queue config failed.\n"); diff --git a/test/common_plat/validation/api/classification/odp_classification_testsuites.h b/test/common_plat/validation/api/classification/odp_classification_testsuites.h index d296923a..e1624162 100644 --- a/test/common_plat/validation/api/classification/odp_classification_testsuites.h +++ b/test/common_plat/validation/api/classification/odp_classification_testsuites.h @@ -35,7 +35,8 @@ int classification_suite_pmr_init(void); odp_packet_t create_packet(cls_packet_info_t pkt_info); int cls_pkt_set_seq(odp_packet_t pkt); uint32_t cls_pkt_get_seq(odp_packet_t pkt); -odp_pktio_t create_pktio(odp_queue_type_t q_type, odp_pool_t pool); +odp_pktio_t create_pktio(odp_queue_type_t q_type, odp_pool_t pool, + odp_bool_t cls_enable); void configure_default_cos(odp_pktio_t pktio, odp_cos_t *cos, odp_queue_t *queue, odp_pool_t *pool); int parse_ipv4_string(const char *ipaddress, uint32_t *addr, uint32_t *mask); commit c5436c447df98e0ec78de3620c7fe48141fef760 Author: Balasubramanian Manoharan <bala.manoharan(a)linaro.org> Date: Wed Aug 2 07:11:18 2017 -0700 linux-generic: classification: update classifier enable boolean removes classifier enable boolean default setting Fixes https://bugs.linaro.org/show_bug.cgi?id=3177 Signed-off-by: Balasubramanian Manoharan <bala.manoharan(a)linaro.org> Signed-off-by: Maxim Uvarov <maxim.uvarov(a)linaro.org> diff --git a/platform/linux-generic/odp_classification.c b/platform/linux-generic/odp_classification.c index cc08b9f4..08b09078 100644 --- a/platform/linux-generic/odp_classification.c +++ b/platform/linux-generic/odp_classification.c @@ -328,7 +328,6 @@ int odp_pktio_default_cos_set(odp_pktio_t pktio_in, odp_cos_t default_cos) } entry->s.cls.default_cos = cos; - pktio_cls_enabled_set(entry, 1); return 0; } @@ -403,7 +402,6 @@ int odp_cos_with_l2_priority(odp_pktio_t pktio_in, l2_cos->cos[qos_table[i]] = cos; } } - pktio_cls_enabled_set(entry, 1); UNLOCK(&l2_cos->lock); return 0; } @@ -436,7 +434,6 @@ int odp_cos_with_l3_qos(odp_pktio_t pktio_in, l3_cos->cos[qos_table[i]] = cos; } } - pktio_cls_enabled_set(entry, 1); UNLOCK(&l3_cos->lock); return 0; } commit 2fff8b0247a0da108c45c9febc1029bc8e12d347 Author: Balasubramanian Manoharan <bala.manoharan(a)linaro.org> Date: Wed Aug 2 07:11:42 2017 -0700 linux-generic: pktio: update classifier enable boolean sets classifer enable based on boolean setting on odp_pktin_queue_param_t Fixes https://bugs.linaro.org/show_bug.cgi?id=3177 Signed-off-by: Balasubramanian Manoharan <bala.manoharan(a)linaro.org> Signed-off-by: Maxim Uvarov <maxim.uvarov(a)linaro.org> diff --git a/platform/linux-generic/odp_packet_io.c b/platform/linux-generic/odp_packet_io.c index 64c6e452..b4523389 100644 --- a/platform/linux-generic/odp_packet_io.c +++ b/platform/linux-generic/odp_packet_io.c @@ -1234,6 +1234,8 @@ int odp_pktin_queue_config(odp_pktio_t pktio, return -1; } + pktio_cls_enabled_set(entry, param->classifier_enable); + if (num_queues > capa.max_input_queues) { ODP_DBG("pktio %s: too many input queues\n", entry->s.name); return -1; ----------------------------------------------------------------------- Summary of changes: platform/linux-generic/odp_classification.c | 3 - platform/linux-generic/odp_packet_io.c | 2 + .../validation/api/classification/classification.h | 1 + .../api/classification/odp_classification_basic.c | 4 +- .../api/classification/odp_classification_common.c | 5 +- .../classification/odp_classification_test_pmr.c | 131 ++++++++++++++++++--- .../api/classification/odp_classification_tests.c | 2 + .../classification/odp_classification_testsuites.h | 3 +- 8 files changed, 128 insertions(+), 23 deletions(-) hooks/post-receive --

6 years, 9 months

1
0
0 0

[COMMIT] branch cloud-dev updated. v1.15.0.0-401-g370f9313

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, cloud-dev has been updated via 370f93139e16633390cb28acda6296da8fee006d (commit) via 5f738a79248183a252478856de88bafa37344445 (commit) via 3058124d5c307149272c1b37651056451939c3bf (commit) from 1ba26aa5650c05718c177842178de6d0f70b7fc1 (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 370f93139e16633390cb28acda6296da8fee006d Author: Yi He <yi.he(a)linaro.org> Date: Thu Jul 20 13:56:36 2017 +0800 odp: add modular framework Add modular programming framework to support selectable implementations for variant software subsystems. Signed-off-by: Yi He <yi.he(a)linaro.org> Reviewed-by: Bill Fischofer <bill.fischofer(a)linaro.org> diff --git a/frameworks/modular/list.h b/frameworks/modular/list.h new file mode 100644 index 00000000..cc42b714 --- /dev/null +++ b/frameworks/modular/list.h @@ -0,0 +1,271 @@ +/* Adopted and modified Rusty Russell CCAN Project + * https://github.com/rustyrussell/ccan + * + * Permission is hereby granted, free of charge, to any person obtaining a copy + * of this software and associated documentation files (the "Software"), to deal + * in the Software without restriction, including without limitation the rights + * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell + * copies of the Software, and to permit persons to whom the Software is + * furnished to do so, subject to the following conditions: + * + * The above copyright notice and this permission notice shall be included in + * all copies or substantial portions of the Software. + * + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE + * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, + * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN + * THE SOFTWARE. + * + * Copyright (c) 2017, ARM Limited. All rights reserved. + * + * Copyright (c) 2017, Linaro Limited + * All rights reserved. + * + * SPDX-License-Identifier: BSD-3-Clause + */ + +#ifndef CCAN_LIST_H +#define CCAN_LIST_H + +#include <stddef.h> +#include <stdbool.h> + +/* Always assume the availabilities of typeof or __typeof__ */ +#if defined(__STDC__) +#define typeof __typeof__ +#endif + +/** + * struct list_node - an entry in a doubly-linked list + * @next: next entry (self if empty) + * @prev: previous entry (self if empty) + * + * This is used as an entry in a linked list. + * Example: + * struct child { + * const char *name; + * // Linked list of all us children. + * struct list_node list; + * }; + */ +struct list_node { + struct list_node *next, *prev; +}; + +/** + * struct list_head - the head of a doubly-linked list + * @node: the head node + * + * This is used as the head of a linked list. + * Example: + * struct parent { + * const char *name; + * struct list_head children; + * unsigned int num_children; + * }; + */ +struct list_head { + struct list_node node; +}; + +/** + * LIST_HEAD_INIT - initializer for an empty list_head + * @name: the name of the list. + * + * Explicit initializer for an empty list. + * + * See also: + * LIST_HEAD, list_head_init() + * + * Example: + * static struct list_head my_list = LIST_HEAD_INIT(my_list); + */ +#define LIST_HEAD_INIT(name) { { &(name).node, &(name).node } } + +/** + * list_head_init - initialize a list_head + * @h: the list_head to set to the empty list + * + * Example: + * ... + * struct parent *parent = malloc(sizeof(*parent)); + * + * list_head_init(&parent->children); + * parent->num_children = 0; + */ +static inline void list_head_init(struct list_head *h) +{ + h->node.next = &h->node; + h->node.prev = &h->node; +} + +/** + * list_node_init - initialize a list_node + * @n: the list_node to link to itself. + * + * You don't need to use this normally! But it lets you list_del(@n) + * safely. + */ +static inline void list_node_init(struct list_node *n) +{ + n->next = n; + n->prev = n; +} + +/** + * list_add_after - add an entry after an existing node in a linked list + * @p: the existing list_node to add the node after + * @n: the new list_node to add to the list. + * + * The existing list_node must already be a member of the list. + * The new list_node does not need to be initialized; it will be overwritten. + * + * Example: + * struct child c1, c2, c3; + * LIST_HEAD(h); + * + * list_add_tail(&h, &c1.list); + * list_add_tail(&h, &c3.list); + * list_add_after(&c1.list, &c2.list); + */ +static inline void list_add_after(struct list_node *p, + struct list_node *n) +{ + n->next = p->next; + n->prev = p; + p->next->prev = n; + p->next = n; +} + +/** + * list_add - add an entry at the start of a linked list. + * @h: the list_head to add the node to + * @n: the list_node to add to the list. + * + * The list_node does not need to be initialized; it will be overwritten. + * Example: + * struct child *child = malloc(sizeof(*child)); + * + * child->name = "marvin"; + * list_add(&parent->children, &child->list); + * parent->num_children++; + */ +static inline void list_add(struct list_head *h, + struct list_node *n) +{ + list_add_after(&h->node, n); +} + +/** + * list_add_before - add an entry before an existing node in a linked list + * @p: the existing list_node to add the node before + * @n: the new list_node to add to the list. + * + * The existing list_node must already be a member of the list. + * The new list_node does not need to be initialized; it will be overwritten. + * + * Example: + * list_head_init(&h); + * list_add_tail(&h, &c1.list); + * list_add_tail(&h, &c3.list); + * list_add_before(&c3.list, &c2.list); + */ +static inline void list_add_before(struct list_node *p, + struct list_node *n) +{ + n->next = p; + n->prev = p->prev; + p->prev->next = n; + p->prev = n; +} + +/** + * list_add_tail - add an entry at the end of a linked list. + * @h: the list_head to add the node to + * @n: the list_node to add to the list. + * + * The list_node does not need to be initialized; it will be overwritten. + * Example: + * list_add_tail(&parent->children, &child->list); + * parent->num_children++; + */ +static inline void list_add_tail(struct list_head *h, + struct list_node *n) +{ + list_add_before(&h->node, n); +} + +/** + * list_empty - is a list empty? + * @h: the list_head + * + * If the list is empty, returns true. + * + * Example: + * assert(list_empty(&parent->children) == (parent->num_children == 0)); + */ +static inline bool list_empty(const struct list_head *h) +{ + return (h->node.next == &h->node); +} + +/** + * list_node_detached - is a node detached from any lists? + * @n: the list_node + * + * If the list node is initialized and detached, return true. + * Always use list_node_init() and list_del_init() on list nodes. + */ +static inline bool list_node_detached(const struct list_node *n) +{ + return (n->next == n); +} + +/** + * list_del - delete an entry from an (unknown) linked list. + * @n: the list_node to delete from the list. + * + * Note that this leaves @n in an undefined state; it can be added to + * another list, but not deleted again. + * + * See also: + * list_del_from(), list_del_init() + * + * Example: + * list_del(&child->list); + * parent->num_children--; + */ +static inline void list_del(struct list_node *n) +{ + n->next->prev = n->prev; + n->prev->next = n->next; + + /* Catch use-after-del. */ + n->next = NULL; + n->prev = NULL; +} + +/** + * list_del_init - delete a node, and reset it so it can be deleted again. + * @n: the list_node to be deleted. + * + * list_del(@n) or list_del_init() again after this will be safe, + * which can be useful in some cases. + * + * See also: + * list_del_from(), list_del() + * + * Example: + * list_del_init(&child->list); + * parent->num_children--; + */ +static inline void list_del_init(struct list_node *n) +{ + list_del(n); + list_node_init(n); +} + +#endif /* CCAN_LIST_H */ diff --git a/frameworks/modular/odp_module.c b/frameworks/modular/odp_module.c new file mode 100644 index 00000000..89b7cb0d --- /dev/null +++ b/frameworks/modular/odp_module.c @@ -0,0 +1,179 @@ +/* Copyright (c) 2017, ARM Limited. All rights reserved. + * + * Copyright (c) 2017, Linaro Limited + * All rights reserved. + * + * SPDX-License-Identifier: BSD-3-Clause + */ + +#include <stdio.h> +#include <errno.h> +#include "odp_module.h" + +#define MODULE_FRAMEWORK_VERSION 0x00010000UL +ODP_SUBSYSTEM_DEFINE(module, "module framework", MODULE_FRAMEWORK_VERSION); + +/* Bootstrap log facility, enable if ODP_DEBUG_PRINT flag is set. */ +#define DBG(format, ...) \ + do { \ + if (ODP_DEBUG_PRINT == 1) \ + fprintf(stderr, format, ##__VA_ARGS__); \ + } while (0) + +/* Keep it simple, allow one registration session at a time. */ +static struct { + odp_rwlock_t lock; + odp_subsystem_t *subsystem; + odp_module_base_t *module; +} registration = { + .lock = ODP_RWLOCK_UNLOCKED, + .subsystem = NULL, + .module = NULL, +}; + +static inline int registration_sanity_check( + odp_subsystem_t *subsystem, odp_module_base_t *module) +{ + if (subsystem == NULL || module == NULL) + return -ENOENT; + + if (!list_node_detached(&module->list)) { + DBG("module %s was already registered.\n", module->name); + return -EAGAIN; + } + + return 0; +} + +/* Module is linked statically or dynamically, and are loaded by + * program loader (execve) or dynamic linker/loader (ld.so) + * + * subsystem_register_module() should complete the whole registration + * session and link the module into subsystem's module array. + */ +static int linker_register_module( + odp_subsystem_t *subsystem, odp_module_base_t *module) +{ + int sanity = registration_sanity_check(subsystem, module); + + if (sanity < 0) /* sanity check errors */ + return sanity; + + /* Allow one registration session at a time */ + odp_rwlock_write_lock(&registration.lock); + + /* Block the subsystem API calls in load new + * implementation modules. */ + odp_rwlock_write_lock(&subsystem->lock); + module->handler = NULL; /* no DSO handler */ + list_add_tail(&subsystem->modules, &module->list); + odp_rwlock_write_unlock(&subsystem->lock); + + odp_rwlock_write_unlock(&registration.lock); + return 0; +} + +static int (*do_register_module)(odp_subsystem_t *, odp_module_base_t *) + = &linker_register_module; + +static int loader_register_module( + odp_subsystem_t *subsystem, odp_module_base_t *module) +{ + int sanity = registration_sanity_check(subsystem, module); + + if (sanity < 0) /* sanity check errors */ + return sanity; + + /* Registration session lock must be held by + * module_loader_start(). */ + if (odp_rwlock_write_trylock(&registration.lock) == 0) { + registration.subsystem = subsystem; + registration.module = module; + return 0; + } + + odp_rwlock_write_unlock(&registration.lock); + return -EACCES; +} + +void odp_module_loader_start(void) +{ + odp_rwlock_write_lock(&registration.lock); + + if (registration.module != NULL || + registration.subsystem != NULL) { + DBG("module loader start warn, A previous " + "registration did not complete yet.\n"); + } + + registration.module = NULL; + registration.subsystem = NULL; + do_register_module = &loader_register_module; +} + +void odp_module_loader_end(void) +{ + if (registration.module != NULL || + registration.subsystem != NULL) { + DBG("module loader end warn, A previous " + "registration did not complete yet.\n"); + } + + registration.module = NULL; + registration.subsystem = NULL; + do_register_module = &linker_register_module; + + odp_rwlock_write_unlock(&registration.lock); +} + +int odp_module_install(void *dso, bool active) +{ + /* Bottom halves of the registration, context exclusion + * is guaranteed by module_loader_start() + */ + if (odp_rwlock_write_trylock(&registration.lock) == 0) { + odp_subsystem_t *subsystem = registration.subsystem; + odp_module_base_t *module = registration.module; + + if (subsystem != NULL && module != NULL) { + odp_rwlock_write_lock(&subsystem->lock); + + module->handler = dso; + list_add_tail(&subsystem->modules, &module->list); + + /* install as active implementation */ + if (active) /* warn: replaceable */ + subsystem->active = module; + + odp_rwlock_write_unlock(&subsystem->lock); + } + + registration.subsystem = NULL; + registration.module = NULL; + return 0; + } + + odp_rwlock_write_unlock(&registration.lock); + return -EACCES; +} + +int odp_module_abandon(void) +{ + /* Bottom halves of the registration, context exclusion + * is guaranteed by module_loader_start() + */ + if (odp_rwlock_write_trylock(&registration.lock) == 0) { + registration.subsystem = NULL; + registration.module = NULL; + return 0; + } + + odp_rwlock_write_unlock(&registration.lock); + return -EACCES; +} + +int __subsystem_register_module( + odp_subsystem_t *subsystem, odp_module_base_t *module) +{ + return do_register_module(subsystem, module); +} diff --git a/frameworks/modular/odp_module.h b/frameworks/modular/odp_module.h new file mode 100644 index 00000000..1a9e178c --- /dev/null +++ b/frameworks/modular/odp_module.h @@ -0,0 +1,296 @@ +/* Copyright (c) 2017, ARM Limited. All rights reserved. + * + * Copyright (c) 2017, Linaro Limited + * All rights reserved. + * + * SPDX-License-Identifier: BSD-3-Clause + */ + +/** + * @file + * + * Modular framework solves the problem of choosing a module + * from multiple modules of a subsystem. + * + * The choice is available during compile time or during runtime + * or initialization. The runtime choice could be using shared + * libraries or dynamic loadable libraries. + * + * Multiple modules of the same subsystem can be built into + * individual static libraries(.a), shared libraries(.so) to be + * dynamically linked or loaded, and use constructor functions + * to register themselves. + * + * A subsystem can choose one active module and provide APIs to + * switch between modules. + * + * Alternatively, subsystem can load multiple modules and + * determine the APIs route in runtime. + * + * In order to gain full possible performance, the subsystem + * allows for choosing a specific module at compile time. + * This eliminates the need to choose the module using function + * pointer table. + * + * This framework tries to minimizes dependencies to the linked + * list and rwlock facilities only. + */ + +#ifndef ODP_MODULE_H_ +#define ODP_MODULE_H_ + +#include <stdbool.h> +#include <odp/api/rwlock.h> +#include "list.h" + +/* Forward declaration */ +typedef struct odp_module_base odp_module_base_t; + +/* Subsystem */ +typedef struct odp_subsystem { + odp_rwlock_t lock; + uint32_t version; + const char *name; + const char *description; + struct list_head modules; + odp_module_base_t *active; +} odp_subsystem_t; + +/* Internally construct subsystem instance name */ +#define __subsystem(name) odp_ ## name ## _subsystem + +/* Declare an ODP subsystem in header file */ +#define ODP_SUBSYSTEM_DECLARE(name) \ + extern odp_subsystem_t __subsystem(name) + +/* Define an ODP subsystem in source file */ +#define ODP_SUBSYSTEM_DEFINE(_name, _description, _version) \ + odp_subsystem_t __subsystem(_name) = \ + { \ + .lock = ODP_RWLOCK_UNLOCKED, \ + .name = # _name, \ + .version = _version, \ + .description = _description, \ + } + +/* Internally construct subsystem API name */ +#define __odp_api(subsystem, api) odp_ ## subsystem ##_## api + +/* Subsystem API prototype name */ +#define odp_api_proto(subsystem, api) __odp_api(subsystem, api ## _proto_t) + +/* Subsystem API declaration */ +#define ODP_SUBSYSTEM_API(name, _return, api, ...) \ + extern _return __odp_api(name, api)(__VA_ARGS__); \ + typedef _return (*odp_api_proto(name, api))(__VA_ARGS__) + +/* Subsystem API stubs are weak */ +#define ODP_SUBSYSTEM_API_STUB(name, api) \ + __attribute__((weak)) __odp_api(name, api) + +/* In case a subsystem module are built as static libraries(.a) + * or preload dynamic libraries(.so), the module can use this + * macro to override the APIs weak stubs. + */ +#define ODP_SUBSYSTEM_API_OVERRIDE(name, api, _alias) \ + __attribute__((alias(#_alias))) __odp_api(name, api) + +#define odp_subsystem_constructor(name) \ + do { \ + odp_rwlock_init(&__subsystem(name).lock); \ + list_head_init(&__subsystem(name).modules); \ + __subsystem(name).active = NULL; \ + } while (0) + +#define ODP_SUBSYSTEM_CONSTRUCTOR(name) \ + static void __attribute__((constructor(101))) \ + odp_ ## name ## _subsystem_constructor(void) + +#define odp_subsystem_lock(access, name) \ + odp_rwlock_ ## access ## _lock(&__subsystem(name).lock) + +#define odp_subsystem_unlock(access, name) \ + odp_rwlock_ ## access ## _unlock(&__subsystem(name).lock) + +/* Base class to all inherited subsystem module classes */ +struct odp_module_base { + struct list_node list; + const char *name; + void *handler; /* DSO */ + int (*init_local)(void); + int (*term_local)(void); + int (*init_global)(void); + int (*term_global)(void); +}; + +/* It is required to define subsystem module class with the + * base class as its 1st member and named as "base", and also + * use ODP_MODULE_CLASS(subsystem) to create the association + * between module class name and subsystem name, like: + * + * typedef ODP_MODULE_CLASS(subsystem) { + * odp_module_base_t base; + * ...new members... + * } new_module_t; // Here pick the name you like freely + * + * It also supports forward declaration like: + * + * // Forward declaration + * typedef ODP_MODULE_CLASS(subsystem) new_module_t; + * // Laterly comes the definition + * ODP_MODULE_CLASS(subsystem) { + * odp_module_base_t base; + * ...new members... + * } + * + * Then in preprocessor macros when we have the subsystem name + * we can recover the module class type information, like: + * + * #define MACRO(subsystem) + * do { + * ODP_MODULE_CLASS(subsystem) *mod = NULL; + * odp_subsystem_foreach_module(subsystem, mod) { + * mod->xxx; // access the module class + * } + * } while(0) + */ +#define ODP_MODULE_CLASS(subsystem) struct odp_ ## subsystem ## _module + +/* Below macros assume that all subsystem module classes have + * odp_module_base_t as their 1st member named "base". + * + * This greatly reduces the complexity for module list iteration + * and module pointer recovery from its list_node member by a forced + * type conversion instead of complex calls to container_of() etc. + */ +#define __force_cast(module, node) \ + ((typeof(module))((void *)(node))) + +#define __foreach_module(pos, head) \ + for (pos = __force_cast(pos, (head)->node.next); \ + pos != __force_cast(pos, head); \ + pos = __force_cast(pos, (pos)->base.list.next)) + +#define __foreach_module_safe(pos, n, head) \ + for (pos = __force_cast(pos, (head)->node.next), \ + n = __force_cast(pos, (pos)->base.list.next); \ + pos != __force_cast(pos, head); \ + pos = n, n = __force_cast(next, (next)->base.list.next)) + +#define odp_subsystem_active_module(name, mod) \ + __force_cast(mod, __subsystem(name).active) + +#define odp_subsystem_foreach_module(name, mod) \ + __foreach_module(mod, &__subsystem(name).modules) + +#define odp_subsystem_foreach_module_safe(name, mod, next) \ + __foreach_module_safe(mod, next, &__subsystem(name).modules) + +#define odp_module_constructor(mod) list_node_init(&(mod)->base.list) + +/* Module constructors should be later than subsystem constructors, + * in statically linked scenarios (both subsystems and modules are + * linked statically). thus the priority 102 compared to the above + * subsystem constructor priority 101. + */ +#define ODP_MODULE_CONSTRUCTOR(name) \ + static void __attribute__((constructor(102))) \ + odp_ ## name ## _module_constructor(void) + +/* All subsystems' initialization and termination routines are + * the same, provide template to help generate similar routines + * automatically, examples: + * + * ODP_SUBSYSTEM_FOREACH_TEMPLATE(subsystem, init_global, DBG) + * will generate a function walk through all the modules of the + * subsystem and invoke init_global method for each. + */ +#define ODP_SUBSYSTEM_FOREACH_TEMPLATE(subs, method, print) \ +static int odp_ ## subs ##_## method(bool continue_on_errors) \ +{ \ + int result = 0; \ + ODP_MODULE_CLASS(subs) * mod = NULL; \ + \ + odp_subsystem_lock(read, subs); \ + odp_subsystem_foreach_module(subs, mod) { \ + result = mod->base.method ? \ + mod->base.method() : 0; \ + if (result < 0) { \ + print("error %d to %s subsystem %s " \ + "module %s.\n", result, #method, \ + __subsystem(subs).name, \ + mod->base.name); \ + \ + if (continue_on_errors) \ + continue; \ + else \ + goto done; \ + } \ + } \ +done: \ + odp_subsystem_unlock(read, subs); \ + return result; \ +} + +/* Subsystem Modules Registration + * + * odp_subsystem_register_module() are called by all modules in their + * constructors, whereas the modules could be: + * + * 1) built as static libraries(.a) and linked statically, or + * built as shared libraries(.so) and linked dynamically. + * + * odp_subsystem_register_module() should complete the whole + * registration session and link the module into subsystem's + * module array. + * + * 2) built as shared libraries(.so) and loaded by a module loader + * in runtime with libdl APIs + * + * The whole registration session needs to be split to aim the + * module loader to properly handle dlopen() returns, and save + * the DSO handler into module's data structure. + * + * The module loader should program in this way: + * odp_module_loader_start(); + * ...... + * for each module + * handler = dlopen(module); + * // The module constructor runs before dlopen() returns + * // which in turn calls odp_subsystem_register_module() + * if (handler is valid) + * odp_module_install(handler); + * else + * odp_module_abandon(); + * ...... + * odp_module_loader_end(); + */ + +void odp_module_loader_start(void); +void odp_module_loader_end(void); + +int odp_module_install(void *, bool active); +int odp_module_abandon(void); + +#define __maybe_unused __attribute__((unused)) +static inline void __subsystem_set_active( + odp_subsystem_t *subsystem __maybe_unused, + odp_module_base_t *module __maybe_unused) +{ +#if defined(IM_ACTIVE_MODULE) + subsystem->active = module; +#endif +} + +int __subsystem_register_module( + odp_subsystem_t *, odp_module_base_t *); + +/* Macro to allow polymorphism on module classes */ +#define odp_subsystem_register_module(name, module) \ +({ \ + odp_module_base_t *base = &(module)->base; \ + __subsystem_register_module(&__subsystem(name), base); \ + __subsystem_set_active(&__subsystem(name), base); \ +}) + +#endif diff --git a/include/odp/api/spec/atomic.h b/include/odp/api/spec/atomic.h index 408829df..54a174ea 100644 --- a/include/odp/api/spec/atomic.h +++ b/include/odp/api/spec/atomic.h @@ -59,6 +59,12 @@ extern "C" { * Atomic 32-bit unsigned integer */ +/** + * @def ODP_ATOMIC_INIT + * Compile-time atomic initializer: + * odp_atomic_xx_t atomic = ODP_ATOMIC_INIT(v); + */ + /* * 32-bit operations in RELAXED memory ordering * -------------------------------------------- diff --git a/include/odp/api/spec/rwlock.h b/include/odp/api/spec/rwlock.h index ff8a3f27..8816b6e7 100644 --- a/include/odp/api/spec/rwlock.h +++ b/include/odp/api/spec/rwlock.h @@ -36,6 +36,11 @@ extern "C" { * ODP reader/writer lock */ +/** + * @def ODP_RWLOCK_UNLOCKED + * Compile-time RW lock initializer: + * odp_rwlock_t lock = ODP_RWLOCK_UNLOCKED; + */ /** * Initialize a reader/writer lock. diff --git a/platform/linux-generic/Makefile.am b/platform/linux-generic/Makefile.am index ba4a31ea..4a7b77b8 100644 --- a/platform/linux-generic/Makefile.am +++ b/platform/linux-generic/Makefile.am @@ -6,6 +6,7 @@ include $(top_srcdir)/platform/@with_platform@/Makefile.inc AM_CFLAGS += -I$(srcdir)/include AM_CFLAGS += -I$(top_srcdir)/include +AM_CFLAGS += -I$(top_srcdir)/frameworks/modular AM_CFLAGS += -I$(top_srcdir)/include/odp/arch/@ARCH_ABI@ AM_CFLAGS += -I$(top_builddir)/include AM_CFLAGS += -Iinclude @@ -292,6 +293,13 @@ if HAVE_PCAP __LIB__libodp_linux_la_SOURCES += pktio/pcap.c endif +# Build modular framework into odp-linux library +modularframeworkdir = $(top_srcdir)/frameworks/modular +noinst_HEADERS += $(modularframeworkdir)/list.h \ + $(modularframeworkdir)/odp_module.h + +__LIB__libodp_linux_la_SOURCES += ../../frameworks/modular/odp_module.c + __LIB__libodp_linux_la_LIBADD = $(ATOMIC_LIBS) # Create symlink for ABI header files. Application does not need to use the arch diff --git a/platform/linux-generic/include/odp/api/plat/atomic_types.h b/platform/linux-generic/include/odp/api/plat/atomic_types.h index a674ac99..86c9e5b0 100644 --- a/platform/linux-generic/include/odp/api/plat/atomic_types.h +++ b/platform/linux-generic/include/odp/api/plat/atomic_types.h @@ -81,6 +81,8 @@ typedef struct odp_atomic_u64_s odp_atomic_u64_t; typedef struct odp_atomic_u32_s odp_atomic_u32_t; +#define ODP_ATOMIC_INIT(a) { .v = a } + #ifdef __cplusplus } #endif diff --git a/platform/linux-generic/include/odp/api/plat/rwlock_types.h b/platform/linux-generic/include/odp/api/plat/rwlock_types.h index f7dc0449..2c1f3660 100644 --- a/platform/linux-generic/include/odp/api/plat/rwlock_types.h +++ b/platform/linux-generic/include/odp/api/plat/rwlock_types.h @@ -30,6 +30,8 @@ struct odp_rwlock_s { typedef struct odp_rwlock_s odp_rwlock_t; +#define ODP_RWLOCK_UNLOCKED { .cnt = ODP_ATOMIC_INIT(0) } + #ifdef __cplusplus } #endif commit 5f738a79248183a252478856de88bafa37344445 Author: Yi He <yi.he(a)linaro.org> Date: Wed Jul 19 15:05:14 2017 +0800 validation: pktio: disable socket mmsg pktio It was not working already, the problem are obscured in make check because with static pktio_if_ops array socket mmap pktio always take precedence over socket mmsg pktio. Signed-off-by: Yi He <yi.he(a)linaro.org> Reviewed-by: Bill Fischofer <bill.fischofer(a)linaro.org> diff --git a/test/linux-generic/validation/api/pktio/pktio_env b/test/linux-generic/validation/api/pktio/pktio_env index 345b5bd5..3a6f9e89 100644 --- a/test/linux-generic/validation/api/pktio/pktio_env +++ b/test/linux-generic/validation/api/pktio/pktio_env @@ -103,6 +103,9 @@ setup_pktio_env() return 5 fi done + + # disable socket mmsg pktio since it was not working already + export ODP_PKTIO_DISABLE_SOCKET_MMSG=1 } cleanup_pktio_env() commit 3058124d5c307149272c1b37651056451939c3bf Author: Yi He <yi.he(a)linaro.org> Date: Thu Jul 13 15:48:20 2017 +0800 example: ddf_app: add .gitignore Signed-off-by: Yi He <yi.he(a)linaro.org> Reviewed-by: Bill Fischofer <bill.fischofer(a)linaro.org> diff --git a/example/ddf_app/.gitignore b/example/ddf_app/.gitignore new file mode 100644 index 00000000..f79a1409 --- /dev/null +++ b/example/ddf_app/.gitignore @@ -0,0 +1 @@ +odp_ddf_app ----------------------------------------------------------------------- Summary of changes: example/ddf_app/.gitignore | 1 + frameworks/modular/list.h | 271 +++++++++++++++++++ frameworks/modular/odp_module.c | 179 +++++++++++++ frameworks/modular/odp_module.h | 296 +++++++++++++++++++++ include/odp/api/spec/atomic.h | 6 + include/odp/api/spec/rwlock.h | 5 + platform/linux-generic/Makefile.am | 8 + .../include/odp/api/plat/atomic_types.h | 2 + .../include/odp/api/plat/rwlock_types.h | 2 + test/linux-generic/validation/api/pktio/pktio_env | 3 + 10 files changed, 773 insertions(+) create mode 100644 example/ddf_app/.gitignore create mode 100644 frameworks/modular/list.h create mode 100644 frameworks/modular/odp_module.c create mode 100644 frameworks/modular/odp_module.h hooks/post-receive --

6 years, 9 months

1
0
0 0

2024

2023

2022

2021

2020

2019

2018

2017

2016

git-lng-odp August 2017