[lng-odp] [PATCH v3 1/11] linux-gen: packet: roll back to copy based references

2017-09-14 Thread Github ODP bot
From: Petri Savolainen In order to re-implement segmentation for the new reference implementation roll back to the copy based reference implementation, which have been proved to work OK. This zero copy reference implementation introduced multiple issues: * segment

[lng-odp] [PATCH CLOUD-DEV v1 0/1] linux-gen: buffer: remove meta data of odp_buffer_hdr_t

2017-09-14 Thread Github ODP bot
Removes 152 bytes for linux-generic from odp_buffer_hdr_t. The size of the meta data was 48 cache lines for linux-generic. With this change the size of the meta data is 32 cache lines for linux-generic. Signed-off-by: Kevin Wang kevin.w...@arm.com github

[lng-odp] [PATCH v3 2/11] validation: packet: relax small concat test

2017-09-14 Thread Github ODP bot
From: Petri Savolainen Limit number of consecutive small concat to fraction of the number of packets in the pool. An implementation linking packets together (no copy) cannot be expected to support one byte concats up to max packet size. Signed-off-by: Petri

[lng-odp] [PATCH v3 7/11] linux-gen: packet: implement static references

2017-09-14 Thread Github ODP bot
From: Petri Savolainen Implemented static references with a reference counter. Counter is zero when not used (reference API not used). Signed-off-by: Petri Savolainen --- /** Email created from pull request 170

Re: [lng-odp] [PATCH API-NEXT v10] api: add schedule order unlock lock

2017-09-14 Thread Github ODP bot
Petri Savolainen(psavol) replied on github web page: include/odp/api/spec/schedule.h line 13 @@ -347,12 +347,15 @@ int odp_schedule_group_info(odp_schedule_group_t group, * be protected by its own ordered lock. This promotes maximum parallelism by * allowing order to maintained on a more

[lng-odp] [PATCH v5 7/7] travis: move cross-compilation into build matrix

2017-09-14 Thread Github ODP bot
From: Dmitry Eremin-Solenikov As both native and cross-compilation now use the same script it is possible to move cross-compilation setup back to build matrix. Proper cross-compiler is selected based on actual compiler (gcc/clang) and cross-target.

[lng-odp] [PATCH v5 4/7] travis: add cross-testing for 32-bit x86

2017-09-14 Thread Github ODP bot
From: Dmitry Eremin-Solenikov Signed-off-by: Dmitry Eremin-Solenikov --- /** Email created from pull request 175 (lumag:build-x86) ** https://github.com/Linaro/odp/pull/175 ** Patch:

[lng-odp] [PATCH API-NEXT v5 0/2] api: classification: add random early discard

2017-09-14 Thread Github ODP bot
Adds random early discard and back pressure feature configurable per CoS Signed-off-by: Balasubramanian Manoharan bala.manoha...@linaro.org github /** Email created from pull request 172 (bala-manoharan:random_early_discard) **

[lng-odp] [PATCH API-NEXT v5 1/2] api: classification: add random early detection and back pressure

2017-09-14 Thread Github ODP bot
From: Balasubramanian Manoharan Adds random early detection and Back pressure feature to CoS Signed-off-by: Balasubramanian Manoharan --- /** Email created from pull request 172 (bala-manoharan:random_early_discard) **

[lng-odp] [PATCH v3 10/11] linux-gen: packet: implement dynamic references

2017-09-14 Thread Github ODP bot
From: Petri Savolainen Use link headers to implement dynamic references. These are segment headers which first segment (seg[0]) does not point to itself but to a referenced segment. Link headers enable long chains of references (new dynamic references from previous

[lng-odp] [PATCH v3 9/11] linux-gen: packet: compress packet offset table

2017-09-14 Thread Github ODP bot
From: Petri Savolainen Use uint16_t (instead of size_t) for offsets and thus decrease size of the offset table into one fourth. Signed-off-by: Petri Savolainen --- /** Email created from pull request 170

Re: [lng-odp] [PATCH API-NEXT v10] api: add schedule order unlock lock

2017-09-14 Thread Github ODP bot
Petri Savolainen(psavol) replied on github web page: include/odp/api/spec/schedule.h line 43 @@ -365,7 +368,28 @@ void odp_schedule_order_lock(unsigned lock_index); * hold this lock. Must be in the range * 0..odp_queue_lock_count() - 1 */ -void

[lng-odp] [PATCH v3 6/11] linux-gen: buffer: optimize header layout

2017-09-14 Thread Github ODP bot
From: Petri Savolainen Pack most often used buffer header fields (including seg[0]) into the first cache line. Signed-off-by: Petri Savolainen --- /** Email created from pull request 170 (psavol:master-packet-ref-rework) **

[lng-odp] [PATCH v3 3/11] validation: packet: unshared len with static reference

2017-09-14 Thread Github ODP bot
From: Petri Savolainen When static references are used all data is shared, and thus unshared length is zero. When there's no references, all data is unique to the packet (unshared len == len). Signed-off-by: Petri Savolainen --- /**

[lng-odp] [PATCH v5 3/7] travis: don't use cross-pkg-config

2017-09-14 Thread Github ODP bot
From: Dmitry Eremin-Solenikov Cross-pkg-config rewrites PKG_CONFIG_PATH to static value. Simulate it, but allow further script to expand PKG_CONFIG_PATH value. Signed-off-by: Dmitry Eremin-Solenikov --- /** Email created

[lng-odp] [PATCH v5 1/7] travis: fix powerpc test name

2017-09-14 Thread Github ODP bot
From: Dmitry Eremin-Solenikov Signed-off-by: Dmitry Eremin-Solenikov --- /** Email created from pull request 175 (lumag:build-x86) ** https://github.com/Linaro/odp/pull/175 ** Patch:

[lng-odp] [PATCH API-NEXT v7 2/2] linux-generic: pool: implementation for min and max headroom

2017-09-14 Thread Github ODP bot
From: Balasubramanian Manoharan Adds minimum and maximum headroom in pool capability Signed-off-by: Balasubramanian Manoharan --- /** Email created from pull request 152 (bala-manoharan:pool_headroom) **

[lng-odp] [PATCH CLOUD-DEV v1 1/1] linux-gen: buffer: remove meta data of odp_buffer_hdr_t

2017-09-14 Thread Github ODP bot
From: Kevin Wang Removes 152 bytes for linux-generic from odp_buffer_hdr_t. The size of the meta data was 48 cache lines for linux-generic. With this change the size of the meta data is 32 cache lines for linux-generic. Signed-off-by: Kevin Wang --- /**

[lng-odp] [PATCH v5 6/7] travis: merge native and cross-compilation scripts

2017-09-14 Thread Github ODP bot
From: Dmitry Eremin-Solenikov Merge native and cross-compilation scripts. This allows us to simplify Travis CI configuration and to enable e.g. testing of out-of-tree compilation. Signed-off-by: Dmitry Eremin-Solenikov ---

Re: [lng-odp] [PATCH v2] Packet references re-implementation

2017-09-14 Thread Github ODP bot
Petri Savolainen(psavol) replied on github web page: platform/linux-generic/odp_packet.c line 1993 @@ -2458,6 +2089,184 @@ uint64_t odp_packet_seg_to_u64(odp_packet_seg_t hdl) return _odp_pri(hdl); } +static inline void buffer_ref_inc(odp_buffer_hdr_t *buf_hdr) +{ + /* First

[lng-odp] [PATCH API-NEXT v5 2/2] linux-generic: classification: implement random early detection and back pressure

2017-09-14 Thread Github ODP bot
From: Balasubramanian Manoharan Implements Random early detection and Back pressure Signed-off-by: Balasubramanian Manoharan --- /** Email created from pull request 172 (bala-manoharan:random_early_discard) **

[lng-odp] [PATCH API-NEXT v7 1/2] api: pool: add min and max headroom in capability

2017-09-14 Thread Github ODP bot
From: Balasubramanian Manoharan Adds pool minimum and maximum headroom in pool capability Signed-off-by: Balasubramanian Manoharan --- /** Email created from pull request 152 (bala-manoharan:pool_headroom) **

[lng-odp] [PATCH v3 11/11] linux-gen: packet: optimize header layout

2017-09-14 Thread Github ODP bot
From: Petri Savolainen Pack most often used data into the first and the last cache line of buffer header, and the first cache line of packet header. Reduce offsets and head-/tailroom fields to 16 bits, since those are in maximum as large as a segment, usually much

Re: [lng-odp] [PATCH API-NEXT v10] api: add schedule order unlock lock

2017-09-14 Thread Github ODP bot
Petri Savolainen(psavol) replied on github web page: platform/linux-generic/include/odp_schedule_if.h line 4 @@ -95,6 +95,7 @@ typedef struct { odp_schedule_group_info_t *); void (*schedule_order_lock)(unsigned); void

[lng-odp] [PATCH v3 5/11] linux-gen: packet: re-implement packet segmentation

2017-09-14 Thread Github ODP bot
From: Petri Savolainen Removed limitation to fixed number of segments (CONFIG_PACKET_MAX_SEGS) per packet. Packet headers carrying segment tables are now linked together with 'next_seg' pointer. Last header containing a segment table is pointed by 'last_seg' pointer.

[lng-odp] [PATCH v3 0/11] Packet references re-implementation

2017-09-14 Thread Github ODP bot
Current zero copy reference implementation introduced multiple issues: segment pointers and length were not set correctly, but were left to NULL reference counting has a race condition which causes random assert failures also with non-reference packets there's larger than expected performance

[lng-odp] [PATCH API-NEXT v7 0/2] api: pool: add min and max headroom in pool capability

2017-09-14 Thread Github ODP bot
V7: This patch is now rebased with updates in api-next. @psavol could you review and provide feedback to the API definitions github /** Email created from pull request 152 (bala-manoharan:pool_headroom) ** https://github.com/Linaro/odp/pull/152 ** Patch:

[lng-odp] [PATCH v5 5/7] travis: enable cpp test during cross-compilation

2017-09-14 Thread Github ODP bot
From: Dmitry Eremin-Solenikov Install proper packages to enable cross-testing of linking C++ apps with ODP. Signed-off-by: Dmitry Eremin-Solenikov --- /** Email created from pull request 175 (lumag:build-x86) **

[lng-odp] [PATCH v5 0/7] Travis configuration update

2017-09-14 Thread Github ODP bot
Main goal of PR is to enable build-testing of ODP for 32-bit x86 arch. Side effects: Enable installation and link-testing when cross-compiling Enable c++ test when cross-compiling Fix c typo in powerpc test name github /** Email created from pull request

[lng-odp] [PATCH v3 4/11] validation: packet: check reference packet content

2017-09-14 Thread Github ODP bot
From: Petri Savolainen Check that reference and base packets contain the same data. Signed-off-by: Petri Savolainen --- /** Email created from pull request 170 (psavol:master-packet-ref-rework) ** https://github.com/Linaro/odp/pull/170

[lng-odp] [PATCH v3 8/11] linux-gen: packet: enhance segment prints

2017-09-14 Thread Github ODP bot
From: Petri Savolainen Print segment addresses and reference counts in addition to segment lengths. Signed-off-by: Petri Savolainen --- /** Email created from pull request 170 (psavol:master-packet-ref-rework) **

[lng-odp] [PATCH v5 2/7] travis: don't override prefix when cross-compiling

2017-09-14 Thread Github ODP bot
From: Dmitry Eremin-Solenikov Signed-off-by: Dmitry Eremin-Solenikov --- /** Email created from pull request 175 (lumag:build-x86) ** https://github.com/Linaro/odp/pull/175 ** Patch:

Re: [lng-odp] [PATCH API-NEXT v7] api: pool: add min and max headroom in pool capability

2017-09-14 Thread Github ODP bot
Bill Fischofer(Bill-Fischofer-Linaro) replied on github web page: platform/linux-generic/odp_pool.c line 16 @@ -853,7 +857,8 @@ int odp_pool_capability(odp_pool_capability_t *capa) capa->pkt.max_pools= ODP_CONFIG_POOLS; capa->pkt.max_len = CONFIG_PACKET_MAX_SEGS *

[lng-odp] [Bug 3210] packet header parsing routines should verify header checksums

2017-09-14 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=3210 --- Comment #2 from Bill Fischofer --- Need to check if PR #167 addresses this. -- You are receiving this mail because: You are on the CC list for the bug.

[lng-odp] [Bug 3201] Performance degradation due to no-copy packet reference commits

2017-09-14 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=3201 Bill Fischofer changed: What|Removed |Added Status|IN_PROGRESS |RESOLVED

[lng-odp] [Bug 2411] linux-generic/odp_schedule_sp.c contains todo items

2017-09-14 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2411 --- Comment #7 from Bill Fischofer --- Ping to Petri for update. Larger question is do we still need the odp_schedule_sp? -- You are receiving this mail because: You are on the CC list for the bug.

[lng-odp] [Bug 2254] check-odp: valgrind generates "No rule to make target"

2017-09-14 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2254 --- Comment #5 from Bill Fischofer --- Ping to Dmitry for update. -- You are receiving this mail because: You are on the CC list for the bug.

[lng-odp] [PATCH API-NEXT v11 0/12] api: add schedule order unlock lock

2017-09-14 Thread Github ODP bot
Adds odp_schedule_order_unlock_lock() api. This API combines schedule order unlock and lock into a single api for HW performance optimisation. github /** Email created from pull request 160 (bala-manoharan:api_sched_order_lock) **

Re: [lng-odp] [PATCH API-NEXT v10] api: add schedule order unlock lock

2017-09-14 Thread Github ODP bot
Balasubramanian Manoharan(bala-manoharan) replied on github web page: include/odp/api/spec/schedule.h line 43 @@ -365,7 +368,28 @@ void odp_schedule_order_lock(unsigned lock_index); * hold this lock. Must be in the range * 0..odp_queue_lock_count() - 1 */

[lng-odp] [Bug 2903] validation: add test case for PMR LD_VNI

2017-09-14 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2903 --- Comment #7 from Bill Fischofer --- Ping to Bala for update. I assume this is needed for Tiger Moth final release? -- You are receiving this mail because: You are on the CC list for the bug.

Re: [lng-odp] [PATCH API-NEXT v10] api: add schedule order unlock lock

2017-09-14 Thread Github ODP bot
Balasubramanian Manoharan(bala-manoharan) replied on github web page: include/odp/api/spec/schedule.h line 13 @@ -347,12 +347,15 @@ int odp_schedule_group_info(odp_schedule_group_t group, * be protected by its own ordered lock. This promotes maximum parallelism by * allowing order to

Re: [lng-odp] [PATCH API-NEXT v7] api: pool: add min and max headroom in pool capability

2017-09-14 Thread Github ODP bot
Petri Savolainen(psavol) replied on github web page: include/odp/api/spec/pool.h line 28 @@ -214,6 +221,17 @@ typedef struct odp_pool_param_t { defined by pool capability pkt.max_uarea_size. Specify as 0 if no user area is needed. */

[lng-odp] [Linaro/odp] a820d8: linux-gen: packet: roll back to copy based referen...

2017-09-14 Thread GitHub
Branch: refs/heads/master Home: https://github.com/Linaro/odp Commit: a820d8a84e132652b4cef295756ccf135e3bd54b https://github.com/Linaro/odp/commit/a820d8a84e132652b4cef295756ccf135e3bd54b Author: Petri Savolainen Date: 2017-09-14 (Thu, 14 Sep

Re: [lng-odp] [PATCH API-NEXT v7] api: pool: add min and max headroom in pool capability

2017-09-14 Thread Github ODP bot
Petri Savolainen(psavol) replied on github web page: include/odp/api/spec/pool.h line 28 @@ -214,6 +221,17 @@ typedef struct odp_pool_param_t { defined by pool capability pkt.max_uarea_size. Specify as 0 if no user area is needed. */

Re: [lng-odp] [PATCH API-NEXT v7] api: pool: add min and max headroom in pool capability

2017-09-14 Thread Github ODP bot
Balasubramanian Manoharan(bala-manoharan) replied on github web page: include/odp/api/spec/pool.h line 28 @@ -214,6 +221,17 @@ typedef struct odp_pool_param_t { defined by pool capability pkt.max_uarea_size. Specify as 0 if no user area is

Re: [lng-odp] [PATCH API-NEXT v10] api: add schedule order unlock lock

2017-09-14 Thread Github ODP bot
Balasubramanian Manoharan(bala-manoharan) replied on github web page: platform/linux-generic/include/odp_schedule_if.h line 4 @@ -95,6 +95,7 @@ typedef struct { odp_schedule_group_info_t *); void (*schedule_order_lock)(unsigned); void

Re: [lng-odp] [PATCH API-NEXT v7] api: pool: add min and max headroom in pool capability

2017-09-14 Thread Github ODP bot
Petri Savolainen(psavol) replied on github web page: platform/linux-generic/odp_pool.c line 8 @@ -348,6 +348,10 @@ static odp_pool_t pool_create(const char *name, odp_pool_param_t *params, break; case ODP_POOL_PACKET: + if (params->pkt.min_headroom >

Re: [lng-odp] [PATCH API-NEXT v7] api: pool: add min and max headroom in pool capability

2017-09-14 Thread Github ODP bot
Petri Savolainen(psavol) replied on github web page: platform/linux-generic/odp_pool.c line 8 @@ -348,6 +348,10 @@ static odp_pool_t pool_create(const char *name, odp_pool_param_t *params, break; case ODP_POOL_PACKET: + if (params->pkt.min_headroom >

[lng-odp] [Linaro/odp] f8c636: travis: fix powerpc test name

2017-09-14 Thread GitHub
Branch: refs/heads/master Home: https://github.com/Linaro/odp Commit: f8c6360667380f6b3fb917ecbe5b941b23dbadf1 https://github.com/Linaro/odp/commit/f8c6360667380f6b3fb917ecbe5b941b23dbadf1 Author: Dmitry Eremin-Solenikov Date: 2017-09-14

[lng-odp] [Bug 3238] doxygen warnings on helper header files

2017-09-14 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=3238 --- Comment #3 from Bill Fischofer --- Maxim and Dmitry will figure out how to get Doxygen 1.8.13 into Travis. Bill will post PR to fix these missing comments. -- You are receiving this mail because: You are on the CC list

[lng-odp] [Bug 2876] tests use two logging mechanisms LOG_DBG and printf

2017-09-14 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2876 Bill Fischofer changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED

[lng-odp] [PATCH API-NEXT v11 7/12] api: schedule: add schedule order unlock lock api

2017-09-14 Thread Github ODP bot
From: Balasubramanian Manoharan Adds odp_schedule_order_unlock_lock() api. This api combines schedule order unlock and lock into a single api for performance optimization in HW Signed-off-by: Balasubramanian Manoharan --- /** Email created

[lng-odp] [Bug 2903] validation: add test case for PMR LD_VNI

2017-09-14 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2903 --- Comment #8 from Bala Manoharan --- I am busy with the API changes. Will update this PR once API changes are finalized -- You are receiving this mail because: You are on the CC list for the bug.

[lng-odp] [Bug 3238] doxygen warnings on helper header files

2017-09-14 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=3238 Bill Fischofer changed: What|Removed |Added Status|CONFIRMED |IN_PROGRESS ---

[lng-odp] [PATCH API-NEXT v11 9/12] linux-generic: schedule: change order lock function signature

2017-09-14 Thread Github ODP bot
From: Balasubramanian Manoharan change function param signature from unsigned to uint32_t Signed-off-by: Balasubramanian Manoharan --- /** Email created from pull request 160 (bala-manoharan:api_sched_order_lock) **

[lng-odp] [PATCH v1 1/1] doc: helper: add missing doxygen to avoid warnings

2017-09-14 Thread Github ODP bot
From: Bill Fischofer The latest version of Doxygen requires every struct and field to be documented, so add missing documentation to helper files to avoid superfluous warnings. This resolves Bug https://bugs.linaro.org/show_bug.cgi?id=3238 Signed-off-by: Bill

[lng-odp] [PATCH v1 0/1] doc: helper: add missing doxygen to avoid warnings

2017-09-14 Thread Github ODP bot
The latest version of Doxygen requires every struct and field to be documented, so add missing documentation to helper files to avoid superfluous warnings. This resolves Bug https://bugs.linaro.org/show_bug.cgi?id=3238 Signed-off-by: Bill Fischofer bill.fischo...@linaro.org

[lng-odp] [PATCH API-NEXT v11 1/12] helper: move include files to helper/include

2017-09-14 Thread Github ODP bot
From: Dmitry Eremin-Solenikov Repair build with --enable-helper-linux option. https://bugs.linaro.org/show_bug.cgi?id=3216 Signed-off-by: Dmitry Eremin-Solenikov Signed-off-by: Maxim Uvarov --- /**

[lng-odp] [PATCH API-NEXT v11 2/12] helper: chksum: odph_chksum returns value in network byte order

2017-09-14 Thread Github ODP bot
From: Dmitry Eremin-Solenikov All examples and usecases assumed network byte order for odph_chksum() return value. Instead of changing this convention, rather document that odph_chksum returns value in network byte order. Signed-off-by: Dmitry Eremin-Solenikov

[lng-odp] [PATCH API-NEXT v11 4/12] helper: chksum: verify odph_ipv4_chksum_*() functions

2017-09-14 Thread Github ODP bot
From: Dmitry Eremin-Solenikov Signed-off-by: Dmitry Eremin-Solenikov --- /** Email created from pull request 160 (bala-manoharan:api_sched_order_lock) ** https://github.com/Linaro/odp/pull/160 ** Patch:

[lng-odp] [PATCH API-NEXT v11 8/12] linux-generic: queue: change order lock count data type

2017-09-14 Thread Github ODP bot
From: Balasubramanian Manoharan change order lock count data type from unsigned to uint32_t Signed-off-by: Balasubramanian Manoharan --- /** Email created from pull request 160 (bala-manoharan:api_sched_order_lock) **

[lng-odp] [PATCH API-NEXT v11 6/12] api: schedule: remove support for nested ordered lock

2017-09-14 Thread Github ODP bot
From: Balasubramanian Manoharan removes support for nested ordered lock use-case changes function param signature from unsigned to uint32_t Signed-off-by: Balasubramanian Manoharan --- /** Email created from pull request 160

[lng-odp] [PATCH API-NEXT v11 5/12] api: queue: change order lock count data type

2017-09-14 Thread Github ODP bot
From: Balasubramanian Manoharan change order lock count data type from unsigned to uint32_t Signed-off-by: Balasubramanian Manoharan --- /** Email created from pull request 160 (bala-manoharan:api_sched_order_lock) **

[lng-odp] [PATCH API-NEXT v11 3/12] helper: ip: correct ipv4 header checksum calculation

2017-09-14 Thread Github ODP bot
From: Dmitry Eremin-Solenikov Current code for IPv4 header checksum calculation assumes that packet data is aligned on 2-byte boundary, that there are no optional headers, etc. Rewrite checksumming code to properly copy & process headers. Signed-off-by: Dmitry

[lng-odp] [PATCH API-NEXT v11 10/12] linux-generic: schedule: implement schedule order unlock lock api

2017-09-14 Thread Github ODP bot
From: Balasubramanian Manoharan Implements odp_schedule_order_unlock_lock() api. This api combines schedule order unlock and lock into a single api for performance optimization in HW. Signed-off-by: Balasubramanian Manoharan --- /** Email

[lng-odp] [PATCH API-NEXT v11 12/12] test: validation: schduler: change order lock function signature

2017-09-14 Thread Github ODP bot
From: Balasubramanian Manoharan change order lock function param signature from unsigned to uint32_t Signed-off-by: Balasubramanian Manoharan --- /** Email created from pull request 160 (bala-manoharan:api_sched_order_lock) **

[lng-odp] [PATCH API-NEXT v11 11/12] test: validation: queue: change order lock data type

2017-09-14 Thread Github ODP bot
From: Balasubramanian Manoharan change oder lock count data type from unsigned to uint32_t Signed-off-by: Balasubramanian Manoharan --- /** Email created from pull request 160 (bala-manoharan:api_sched_order_lock) **

Re: [lng-odp] [PATCH API-NEXT v7] api: pool: add min and max headroom in pool capability

2017-09-14 Thread Github ODP bot
Petri Savolainen(psavol) replied on github web page: platform/linux-generic/odp_pool.c line 16 @@ -853,7 +857,8 @@ int odp_pool_capability(odp_pool_capability_t *capa) capa->pkt.max_pools= ODP_CONFIG_POOLS; capa->pkt.max_len = CONFIG_PACKET_MAX_SEGS * max_seg_len;

[lng-odp] [PATCH API-NEXT v1 0/1] api: timer: refine timer pool param documentation

2017-09-14 Thread Github ODP bot
Specify explicitly what each timer pool parameter means. This does not change the intended use of the parameters. Signed-off-by: Petri Savolainen petri.savolai...@linaro.org github /** Email created from pull request 177 (psavol:next-timer-min-tmo) **

[lng-odp] PR #139 discussion

2017-09-14 Thread Honnappa Nagarahalli
Hi, I saw that PR #139 has a commit to remove the dpdk pkt I/O from linux-generic. It is understood very clearly that we will have only one dpdk pkt I/O, which is from linux-dpdk. My only concern is the timing (I understand that we had already agreed on timing as well :)). The concern that I

[lng-odp] [PATCH API-NEXT v1 0/19] merge master branch to api-next

2017-09-14 Thread Github ODP bot
Merge master branch to api-next after PR #170. Almost plain merge with one commit on top to put function used for crypto to common place. github /** Email created from pull request 179 (muvarov:api-next) ** https://github.com/Linaro/odp/pull/179 **

[lng-odp] [PATCH API-NEXT v1 2/19] travis: don't override prefix when cross-compiling

2017-09-14 Thread Github ODP bot
From: Dmitry Eremin-Solenikov Signed-off-by: Dmitry Eremin-Solenikov Signed-off-by: Maxim Uvarov --- /** Email created from pull request 179 (muvarov:api-next) **

[lng-odp] [PATCH API-NEXT v1 5/19] travis: enable cpp test during cross-compilation

2017-09-14 Thread Github ODP bot
From: Dmitry Eremin-Solenikov Install proper packages to enable cross-testing of linking C++ apps with ODP. Signed-off-by: Dmitry Eremin-Solenikov Signed-off-by: Maxim Uvarov --- /** Email created

[lng-odp] [PATCH API-NEXT v1 6/19] travis: merge native and cross-compilation scripts

2017-09-14 Thread Github ODP bot
From: Dmitry Eremin-Solenikov Merge native and cross-compilation scripts. This allows us to simplify Travis CI configuration and to enable e.g. testing of out-of-tree compilation. Signed-off-by: Dmitry Eremin-Solenikov

[lng-odp] [PATCH API-NEXT v1 7/19] travis: move cross-compilation into build matrix

2017-09-14 Thread Github ODP bot
From: Dmitry Eremin-Solenikov As both native and cross-compilation now use the same script it is possible to move cross-compilation setup back to build matrix. Proper cross-compiler is selected based on actual compiler (gcc/clang) and cross-target.

[lng-odp] [PATCH API-NEXT v1 9/19] validation: packet: relax small concat test

2017-09-14 Thread Github ODP bot
From: Petri Savolainen Limit number of consecutive small concat to fraction of the number of packets in the pool. An implementation linking packets together (no copy) cannot be expected to support one byte concats up to max packet size. Signed-off-by: Petri

[lng-odp] [PATCH API-NEXT v1 8/19] linux-gen: packet: roll back to copy based references

2017-09-14 Thread Github ODP bot
From: Petri Savolainen In order to re-implement segmentation for the new reference implementation roll back to the copy based reference implementation, which have been proved to work OK. This zero copy reference implementation introduced multiple issues: * segment

[lng-odp] [PATCH API-NEXT v1 4/19] travis: add cross-testing for 32-bit x86

2017-09-14 Thread Github ODP bot
From: Dmitry Eremin-Solenikov Signed-off-by: Dmitry Eremin-Solenikov Signed-off-by: Maxim Uvarov --- /** Email created from pull request 179 (muvarov:api-next) **

[lng-odp] [PATCH API-NEXT v1 3/19] travis: don't use cross-pkg-config

2017-09-14 Thread Github ODP bot
From: Dmitry Eremin-Solenikov Cross-pkg-config rewrites PKG_CONFIG_PATH to static value. Simulate it, but allow further script to expand PKG_CONFIG_PATH value. Signed-off-by: Dmitry Eremin-Solenikov Signed-off-by: Maxim

[lng-odp] [PATCH API-NEXT v1 1/19] travis: fix powerpc test name

2017-09-14 Thread Github ODP bot
From: Dmitry Eremin-Solenikov Signed-off-by: Dmitry Eremin-Solenikov Signed-off-by: Maxim Uvarov --- /** Email created from pull request 179 (muvarov:api-next) **

[lng-odp] [PATCH API-NEXT v1 10/19] validation: packet: unshared len with static reference

2017-09-14 Thread Github ODP bot
From: Petri Savolainen When static references are used all data is shared, and thus unshared length is zero. When there's no references, all data is unique to the packet (unshared len == len). Signed-off-by: Petri Savolainen

[lng-odp] [PATCH API-NEXT v1 15/19] linux-gen: packet: enhance segment prints

2017-09-14 Thread Github ODP bot
From: Petri Savolainen Print segment addresses and reference counts in addition to segment lengths. Signed-off-by: Petri Savolainen Reviewed-by: Bill Fischofer Signed-off-by: Maxim Uvarov

[lng-odp] [PATCH API-NEXT v1 11/19] validation: packet: check reference packet content

2017-09-14 Thread Github ODP bot
From: Petri Savolainen Check that reference and base packets contain the same data. Signed-off-by: Petri Savolainen Reviewed-by: Bill Fischofer Signed-off-by: Maxim Uvarov --- /**

[lng-odp] [PATCH API-NEXT v1 13/19] linux-gen: buffer: optimize header layout

2017-09-14 Thread Github ODP bot
From: Petri Savolainen Pack most often used buffer header fields (including seg[0]) into the first cache line. Signed-off-by: Petri Savolainen Reviewed-by: Bill Fischofer Signed-off-by: Maxim Uvarov

Re: [lng-odp] [PATCH CLOUD-DEV v1] linux-gen: buffer: remove meta data of odp_buffer_hdr_t

2017-09-14 Thread Github ODP bot
nagarahalli replied on github web page: platform/linux-generic/include/odp_buffer_internal.h line 25 @@ -86,9 +79,17 @@ struct odp_buffer_hdr_t { /* Event subtype. Should be ODP_EVENT_NO_SUBTYPE except packets. */ int8_tevent_subtype; +#ifndef ODP_SCHEDULE_SCALABLE

[lng-odp] [PATCH API-NEXT v1 12/19] linux-gen: packet: re-implement packet segmentation

2017-09-14 Thread Github ODP bot
From: Petri Savolainen Removed limitation to fixed number of segments (CONFIG_PACKET_MAX_SEGS) per packet. Packet headers carrying segment tables are now linked together with 'next_seg' pointer. Last header containing a segment table is pointed by 'last_seg' pointer.

[lng-odp] [PATCH API-NEXT v1 14/19] linux-gen: packet: implement static references

2017-09-14 Thread Github ODP bot
From: Petri Savolainen Implemented static references with a reference counter. Counter is zero when not used (reference API not used). Signed-off-by: Petri Savolainen Reviewed-by: Bill Fischofer

[lng-odp] [PATCH API-NEXT v1 16/19] linux-gen: packet: compress packet offset table

2017-09-14 Thread Github ODP bot
From: Petri Savolainen Use uint16_t (instead of size_t) for offsets and thus decrease size of the offset table into one fourth. Signed-off-by: Petri Savolainen Reviewed-by: Bill Fischofer Signed-off-by: Maxim

[lng-odp] [PATCH API-NEXT v1 17/19] linux-gen: packet: implement dynamic references

2017-09-14 Thread Github ODP bot
From: Petri Savolainen Use link headers to implement dynamic references. These are segment headers which first segment (seg[0]) does not point to itself but to a referenced segment. Link headers enable long chains of references (new dynamic references from previous

[lng-odp] [PATCH API-NEXT v1 18/19] linux-gen: packet: optimize header layout

2017-09-14 Thread Github ODP bot
From: Petri Savolainen Pack most often used data into the first and the last cache line of buffer header, and the first cache line of packet header. Reduce offsets and head-/tailroom fields to 16 bits, since those are in maximum as large as a segment, usually much

[lng-odp] [PATCH API-NEXT v1 19/19] linux-gen: move packet_to_buffer to packet_inlines.h

2017-09-14 Thread Github ODP bot
From: Maxim Uvarov odp_crypto.c is also uses packet_to_buffer() so move this function to common place. Signed-off-by: Maxim Uvarov --- /** Email created from pull request 179 (muvarov:api-next) ** https://github.com/Linaro/odp/pull/179 **