I have fixed the issue. I will post the PR once the internal review is
completed.
Thanks,
Honnappa
On 16 October 2017 at 14:40, Maxim Uvarov wrote:
> (gdb) bt
> #0 0x7fbe66d26c37 in __GI_raise (sig=sig@entry=6) at
> ../nptl/sysdeps/unix/sysv/linux/raise.c:56
> #1 0x7fbe66d2a028 in __G
On Mon, Oct 16, 2017 at 7:59 AM, Github ODP bot wrote:
> From: Petri Savolainen
>
> Added packet pool parameter 'max_num', so that 'num' parameter
> can be round up by the implementation. Most implementations have
> a fixed segment size, and need this flexibility. For example,
> when 'len' is lar
On Mon, Oct 16, 2017 at 8:00 AM, Github ODP bot wrote:
> From: Petri Savolainen
>
> Remove anonymous union from pool parameter structure.
> Union makes it impossible to initialize parameters per
> pool type (use other values than all zeros). This change
> is not visible to applications (union was
On Mon, Oct 16, 2017 at 7:59 AM, Github ODP bot wrote:
> From: Petri Savolainen
>
> Additional packet length and number specification gives
> more information to implementation about intended packet
> length distribution in the pool. This enables e.g. correct
> initialization when pool implementa
https://bugs.linaro.org/show_bug.cgi?id=3343
Tom Trahan changed:
What|Removed |Added
CC||t...@shippable.com
--- Comment #1 from Tom Trahan
Bill Fischofer(Bill-Fischofer-Linaro) replied on github web page:
platform/linux-generic/odp_packet.c
line 241
@@ -2146,7 +2146,7 @@ odp_packet_t odp_packet_ref(odp_packet_t pkt, uint32_t
offset)
link_hdr->buf_hdr.seg[0].len = seg->len - seg_offset;
buffer_ref_inc(seg->hdr);
-
Bill Fischofer(Bill-Fischofer-Linaro) replied on github web page:
platform/linux-generic/odp_packet.c
line 18
@@ -179,10 +179,7 @@ static inline void *packet_seg_data(odp_packet_hdr_t
*pkt_hdr, uint32_t seg_idx)
static inline uint16_t packet_last_seg(odp_packet_hdr_t *pkt_hdr)
{
- if (C
Bill Fischofer(Bill-Fischofer-Linaro) replied on github web page:
platform/linux-generic/odp_packet.c
line 27
@@ -291,7 +288,7 @@ static inline void *packet_map(odp_packet_hdr_t *pkt_hdr,
if (odp_unlikely(offset >= pkt_hdr->frame_len))
return NULL;
- if (odp_likely(
Bill Fischofer(Bill-Fischofer-Linaro) replied on github web page:
platform/linux-generic/include/odp_packet_internal.h
line 9
@@ -197,7 +197,7 @@ static inline void packet_init(odp_packet_hdr_t *pkt_hdr,
uint32_t len)
uint32_t seg_len;
int num = pkt_hdr->buf_hdr.segcount;
-
From: Tom Trahan
---
/** Email created from pull request 231 (ttrahan:master)
** https://github.com/Linaro/odp/pull/231
** Patch: https://github.com/Linaro/odp/pull/231.patch
** Base sha: ec0c3145fcafa09ae3a79875e7e07dd4794583cc
** Merge commit sha: 738c618a4f25f383f42a190abcf2c6cd6d52ce62
*
Adds shippable.yml config file for testing ODP builds on Aarch64 hardware using
Shippable. This CI config also uses a Dockerfile to install additional
dependencies on top of the standard Shippable Aarch64 image.
github
/** Email created from pull request 2
From: Tom Trahan
---
/** Email created from pull request 231 (ttrahan:master)
** https://github.com/Linaro/odp/pull/231
** Patch: https://github.com/Linaro/odp/pull/231.patch
** Base sha: ec0c3145fcafa09ae3a79875e7e07dd4794583cc
** Merge commit sha: 8f4186ef21cc545334b1ebc56df86388b163ecc2
*
From: Tom Trahan
---
/** Email created from pull request 231 (ttrahan:master)
** https://github.com/Linaro/odp/pull/231
** Patch: https://github.com/Linaro/odp/pull/231.patch
** Base sha: ec0c3145fcafa09ae3a79875e7e07dd4794583cc
** Merge commit sha: 8f4186ef21cc545334b1ebc56df86388b163ecc2
*
From: Tom Trahan
---
/** Email created from pull request 231 (ttrahan:master)
** https://github.com/Linaro/odp/pull/231
** Patch: https://github.com/Linaro/odp/pull/231.patch
** Base sha: ec0c3145fcafa09ae3a79875e7e07dd4794583cc
** Merge commit sha: 8f4186ef21cc545334b1ebc56df86388b163ecc2
*
From: Tom Trahan
---
/** Email created from pull request 231 (ttrahan:master)
** https://github.com/Linaro/odp/pull/231
** Patch: https://github.com/Linaro/odp/pull/231.patch
** Base sha: ec0c3145fcafa09ae3a79875e7e07dd4794583cc
** Merge commit sha: 8f4186ef21cc545334b1ebc56df86388b163ecc2
*
From: Tom Trahan
---
/** Email created from pull request 231 (ttrahan:master)
** https://github.com/Linaro/odp/pull/231
** Patch: https://github.com/Linaro/odp/pull/231.patch
** Base sha: ec0c3145fcafa09ae3a79875e7e07dd4794583cc
** Merge commit sha: 8f4186ef21cc545334b1ebc56df86388b163ecc2
*
Adds shippable.yml config file for testing ODP builds on Aarch64 hardware using
Shippable. This CI config also uses a Dockerfile to install additional
dependencies on top of the standard Shippable Aarch64 image.
github
/** Email created from pull request 2
https://bugs.linaro.org/show_bug.cgi?id=3343
Bug ID: 3343
Summary: aarch64 timer test fails
Product: OpenDataPlane - linux- generic reference
Version: v1.15.0.0
Hardware: Other
OS: Linux
Status: UNCONFIRMED
From: Tom Trahan
---
/** Email created from pull request 231 (ttrahan:master)
** https://github.com/Linaro/odp/pull/231
** Patch: https://github.com/Linaro/odp/pull/231.patch
** Base sha: ae4f1d82ed704992c1d8284c23795b9e076b33b9
** Merge commit sha: 6f4b54ebaa9d145f3ad555f5bc3488afab42e2f3
*
From: Tom Trahan
---
/** Email created from pull request 231 (ttrahan:master)
** https://github.com/Linaro/odp/pull/231
** Patch: https://github.com/Linaro/odp/pull/231.patch
** Base sha: ae4f1d82ed704992c1d8284c23795b9e076b33b9
** Merge commit sha: 6f4b54ebaa9d145f3ad555f5bc3488afab42e2f3
*
From: Tom Trahan
---
/** Email created from pull request 231 (ttrahan:master)
** https://github.com/Linaro/odp/pull/231
** Patch: https://github.com/Linaro/odp/pull/231.patch
** Base sha: ae4f1d82ed704992c1d8284c23795b9e076b33b9
** Merge commit sha: 6f4b54ebaa9d145f3ad555f5bc3488afab42e2f3
*
From: Tom Trahan
---
/** Email created from pull request 231 (ttrahan:master)
** https://github.com/Linaro/odp/pull/231
** Patch: https://github.com/Linaro/odp/pull/231.patch
** Base sha: ae4f1d82ed704992c1d8284c23795b9e076b33b9
** Merge commit sha: 6f4b54ebaa9d145f3ad555f5bc3488afab42e2f3
*
From: Tom Trahan
---
/** Email created from pull request 231 (ttrahan:master)
** https://github.com/Linaro/odp/pull/231
** Patch: https://github.com/Linaro/odp/pull/231.patch
** Base sha: ae4f1d82ed704992c1d8284c23795b9e076b33b9
** Merge commit sha: 6f4b54ebaa9d145f3ad555f5bc3488afab42e2f3
*
Adds shippable.yml config file for testing ODP builds on Aarch64 hardware using
Shippable. This CI config also uses a Dockerfile to install additional
dependencies on top of the standard Shippable Aarch64 image.
github
/** Email created from pull request 2
(gdb) bt
#0 0x7fbe66d26c37 in __GI_raise (sig=sig@entry=6) at
../nptl/sysdeps/unix/sysv/linux/raise.c:56
#1 0x7fbe66d2a028 in __GI_abort () at abort.c:89
#2 0x0042e279 in odp_override_abort () at odp_weak.c:42
#3 0x00424604 in llq_enqueue (node=, llq=) at ./include/odp_l
./configure --enable-schedule-scalable
sudo make check
Test: pktio_test_sched_multi
/include/odp_llqueue.h:269:llq_dequeue_cond():node->next == SENTINEL
Aborted
full log is attached.
On 16 October 2017 at 19:53, Honnappa Nagarahalli <
honnappa.nagaraha...@linaro.org> wrote:
> Dmitry had
Branch: refs/heads/master
Home: https://github.com/Linaro/odp
Commit: 209c3a2e255de7aa82c4f08f8abb41c04bb3d86d
https://github.com/Linaro/odp/commit/209c3a2e255de7aa82c4f08f8abb41c04bb3d86d
Author: Dmitry Eremin-Solenikov
Date: 2017-10-16 (Mon, 16 Oct 2017)
Changed paths:
From: Tom Trahan
---
/** Email created from pull request 231 (ttrahan:master)
** https://github.com/Linaro/odp/pull/231
** Patch: https://github.com/Linaro/odp/pull/231.patch
** Base sha: ae4f1d82ed704992c1d8284c23795b9e076b33b9
** Merge commit sha: dfd2ae6b5630b49c12ab2d4579a5e5916777be1f
*
From: Tom Trahan
---
/** Email created from pull request 231 (ttrahan:master)
** https://github.com/Linaro/odp/pull/231
** Patch: https://github.com/Linaro/odp/pull/231.patch
** Base sha: ae4f1d82ed704992c1d8284c23795b9e076b33b9
** Merge commit sha: dfd2ae6b5630b49c12ab2d4579a5e5916777be1f
*
From: Tom Trahan
---
/** Email created from pull request 231 (ttrahan:master)
** https://github.com/Linaro/odp/pull/231
** Patch: https://github.com/Linaro/odp/pull/231.patch
** Base sha: ae4f1d82ed704992c1d8284c23795b9e076b33b9
** Merge commit sha: dfd2ae6b5630b49c12ab2d4579a5e5916777be1f
*
From: Tom Trahan
---
/** Email created from pull request 231 (ttrahan:master)
** https://github.com/Linaro/odp/pull/231
** Patch: https://github.com/Linaro/odp/pull/231.patch
** Base sha: ae4f1d82ed704992c1d8284c23795b9e076b33b9
** Merge commit sha: dfd2ae6b5630b49c12ab2d4579a5e5916777be1f
*
Adds shippable.yml config file for testing ODP builds on Aarch64 hardware using
Shippable. This CI config also uses a Dockerfile to install additional
dependencies on top of the standard Shippable Aarch64 image.
github
/** Email created from pull request 2
Dmitry had taken a look at these. He mentioned they can be ignored for now.
Thanks,
Honnappa
On 16 October 2017 at 03:52, Maxim Uvarov wrote:
> bootstrap generates a lot of errors, I can reproduce them locally also
>
> https://travis-ci.org/nagarahalli/odp/jobs/287783022
>
> On 16 October 2017 a
On 16 October 2017 at 01:21, Maxim Uvarov wrote:
> I will take a look. But please check that you added this 3 commits.
>
>
> c16af648 travis: purge dpdk cache on version change
>
This is not in the merge. The following 2 are in the merge
> 3cb45201 travis: build dpdk for general cpu
> 73bc4619
From: Dmitry Eremin-Solenikov
Report maximum possible antireplay window size to application. Othersise
it can try requesting arbitrary window sizes not supported by
hardware/implementation.
Signed-off-by: Dmitry Eremin-Solenikov
---
/** Email created from pull request 230 (lumag:ipsec-rws-cap)
Signed-off-by: Dmitry Eremin-Solenikov dmitry.ereminsoleni...@linaro.org
github
/** Email created from pull request 230 (lumag:ipsec-rws-cap)
** https://github.com/Linaro/odp/pull/230
** Patch: https://github.com/Linaro/odp/pull/230.patch
** Base sha: afe
From: Matias Elo
Adjust 'segmented_packet_len' so that the test packet pool is able to
allocate at least PACKET_POOL_NUM_SEG possibly segmented test packets.
Several validation tests allocate new packets, so it is not enough to be
able allocate a single segmented packet.
Signed-off-by: Matias El
From: Matias Elo
Use separate defines for the maximum number of segments per packet
(CONFIG_PACKET_MAX_SEGS) and the number of segments stored in a packet
header (CONFIG_PACKET_SEGS_PER_HDR).
A separate define is also added for the maximum packet length.
Signed-off-by: Matias Elo
---
/** Email
From: Matias Elo
Use stored packet pool segment length (pool_t.seg_len) instead of a define.
This enables creating packet pools with different segment sizes.
Signed-off-by: Matias Elo
---
/** Email created from pull request 233 (matiaselo:dev/pool_dyn_seg_len)
** https://github.com/Linaro/odp/
From: Matias Elo
Struct pool_t members 'data_size' and 'max_seg_len' were both used to store
the same value. Replace them with a single 'seg_len' member.
Signed-off-by: Matias Elo
---
/** Email created from pull request 233 (matiaselo:dev/pool_dyn_seg_len)
** https://github.com/Linaro/odp/pull
From: Matias Elo
Fix (invalid) gcc error when CONFIG_PACKET_SEGS_PER_HDR is set to one.
Signed-off-by: Matias Elo
---
/** Email created from pull request 233 (matiaselo:dev/pool_dyn_seg_len)
** https://github.com/Linaro/odp/pull/233
** Patch: https://github.com/Linaro/odp/pull/233.patch
** B
From: Petri Savolainen
Implemented max_num info for packet pools.
Signed-off-by: Petri Savolainen
---
/** Email created from pull request 234 (psavol:next-pool-param)
** https://github.com/Linaro/odp/pull/234
** Patch: https://github.com/Linaro/odp/pull/234.patch
** Base sha: afeda4d14bb6f44
This patch set modifies packet pool implementation to select segment size at
runtime based on the pool parameters. After this patch set an application is
able to create multiple packet pools with varying segment sizes.
github
/** Email created from pull re
From: Petri Savolainen
Added odp_pool_info() test for packet pools.
Signed-off-by: Petri Savolainen
---
/** Email created from pull request 234 (psavol:next-pool-param)
** https://github.com/Linaro/odp/pull/234
** Patch: https://github.com/Linaro/odp/pull/234.patch
** Base sha: afeda4d14bb6f
From: Petri Savolainen
Test packet pool subparameters when those are supported.
Signed-off-by: Petri Savolainen
---
/** Email created from pull request 234 (psavol:next-pool-param)
** https://github.com/Linaro/odp/pull/234
** Patch: https://github.com/Linaro/odp/pull/234.patch
** Base sha: a
From: Petri Savolainen
Remove anonymous union from pool parameter structure.
Union makes it impossible to initialize parameters per
pool type (use other values than all zeros). This change
is not visible to applications (union was anonymous).
Signed-off-by: Petri Savolainen
---
/** Email create
From: Petri Savolainen
Use odp_pool_param_init() to initialize pool parameters.
Signed-off-by: Petri Savolainen
---
/** Email created from pull request 234 (psavol:next-pool-param)
** https://github.com/Linaro/odp/pull/234
** Patch: https://github.com/Linaro/odp/pull/234.patch
** Base sha: a
From: Petri Savolainen
Added packet pool parameter 'max_num', so that 'num' parameter
can be round up by the implementation. Most implementations have
a fixed segment size, and need this flexibility. For example,
when 'len' is larger than the pool segment size, there may be
e.g. 2 x 'num' segment
From: Petri Savolainen
Packet pool parameters does not require application to specify
the maximum number of packet. Application is more portable, if
it does not restrict max_num, but instead uses this info field
after pool creation.
Signed-off-by: Petri Savolainen
---
/** Email created from pul
From: Petri Savolainen
Additional packet length and number specification gives
more information to implementation about intended packet
length distribution in the pool. This enables e.g. correct
initialization when pool implementation is based on multiple
fixed packet / segment sizes (subpools).
Added packet pool parameters for more fine grained pool
configuration. The basic usage of the parameters is not changed,
except that implementation may now round up 'num' by default.
Application can limit the round up with new 'max_num' parameter.
If application does not limit max_num (but is still
From: Bogdan Pricope
Implement link status tap pktio operation.
Signed-off-by: Bogdan Pricope
---
/** Email created from pull request 232 (bogdanPricope:tap_set_mac_pr)
** https://github.com/Linaro/odp/pull/232
** Patch: https://github.com/Linaro/odp/pull/232.patch
** Base sha: afeda4d14bb6f
Implement start/stop/link status/MAC address set operations of TAP interface..
github
/** Email created from pull request 232 (bogdanPricope:tap_set_mac_pr)
** https://github.com/Linaro/odp/pull/232
** Patch: https://github.com/Linaro/odp/pull/232.patch
*
From: Bogdan Pricope
Implement MAC address set function and set corresponding
capability flag.
Signed-off-by: Bogdan Pricope
---
/** Email created from pull request 232 (bogdanPricope:tap_set_mac_pr)
** https://github.com/Linaro/odp/pull/232
** Patch: https://github.com/Linaro/odp/pull/232.pa
From: Bogdan Pricope
Implement start/stop tap pktio operations.
Signed-off-by: Bogdan Pricope
---
/** Email created from pull request 232 (bogdanPricope:tap_set_mac_pr)
** https://github.com/Linaro/odp/pull/232
** Patch: https://github.com/Linaro/odp/pull/232.patch
** Base sha: afeda4d14bb6f
https://bugs.linaro.org/show_bug.cgi?id=3337
Dmitry Eremin-Solenikov changed:
What|Removed |Added
CC||dmitry.ereminsolenikov@lina
muvarov replied on github web page:
shippable.yml
line 19
@@ -0,0 +1,81 @@
+
+language: c
+
+compiler:
+ - gcc
+ - clang
+
+env:
+ matrix:
+- CONF=""
+# - CONF="--disable-abi-compat"
+# - CONF="--enable-schedule-sp"
+# - CONF="--enable-schedule-iquery"
+# - CONF="--enable-dp
muvarov replied on github web page:
Dockerfile
line 8
@@ -0,0 +1,23 @@
+FROM drydockaarch64/u16:v5.10.1
+
+RUN if $(sudo update-alternatives --list gcc); \
+ then sudo update-alternatives --remove-all gcc; \
+fi
+
+RUN sudo apt-get update && sudo apt-get install -yy \
+ gcc-4.8 \
Comme
https://bugs.linaro.org/show_bug.cgi?id=3337
Bug ID: 3337
Summary: ./test/linux-generic/run-test is not used
Product: OpenDataPlane - linux- generic reference
Version: v1.15.0.0
Hardware: Other
OS: Linux
Status: UNC
bootstrap generates a lot of errors, I can reproduce them locally also
https://travis-ci.org/nagarahalli/odp/jobs/287783022
On 16 October 2017 at 09:21, Maxim Uvarov wrote:
> I will take a look. But please check that you added this 3 commits.
>
>
> c16af648 travis: purge dpdk cache on version c
Git log entry is missing. Update with some rationale why this change is needed.
-Petri
> -Original Message-
> From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of
> Github ODP bot
> Sent: Sunday, October 15, 2017 9:00 AM
> To: lng-odp@lists.linaro.org
> Subject: [lng-odp]
61 matches
Mail list logo