Signed-off-by: Maxim Uvarov maxim.uva...@linaro.org
---
platform/linux-generic/pktio/socket_mmap.c | 22 ++
1 file changed, 22 insertions(+)
diff --git a/platform/linux-generic/pktio/socket_mmap.c
b/platform/linux-generic/pktio/socket_mmap.c
index 7d42678..f9ba74e 100644
---
Signed-off-by: Maxim Uvarov maxim.uva...@linaro.org
---
platform/linux-generic/pktio/loop.c | 21 +
1 file changed, 21 insertions(+)
diff --git a/platform/linux-generic/pktio/loop.c
b/platform/linux-generic/pktio/loop.c
index 188a9ee..8c45e47 100644
---
https://bugs.linaro.org/show_bug.cgi?id=1750
Bug ID: 1750
Summary: sendmmsg pktio error handle
Product: OpenDataPlane
Version: unspecified
Hardware: Other
OS: Linux
Status: UNCONFIRMED
Severity:
odp_pktio_start() and odp_pktio_stop() API function uses to control pktio
inside worker or to start/stop pktio from control thread.
Patch set includes:
- linux-generic implementation
- validation test suite
- fixes to all apps uses odp_pktio_open(). For now I left current logic
if it will be
Signed-off-by: Maxim Uvarov maxim.uva...@linaro.org
---
.../linux-generic/include/odp_packet_io_internal.h | 6
platform/linux-generic/odp_packet_io.c | 34 ++
2 files changed, 40 insertions(+)
diff --git
Stuart, does this patch look ok ?
On 7 August 2015 at 15:37, Maxim Uvarov maxim.uva...@linaro.org wrote:
On 08/07/15 15:23, Krishna Garapati wrote:
I will look into the API replacement. If I understood your comment
correctly, I am already reusing the initial calculated num_workers and
On Thu, Aug 13, 2015 at 6:45 AM, Maxim Uvarov maxim.uva...@linaro.org
wrote:
Signed-off-by: Maxim Uvarov maxim.uva...@linaro.org
---
platform/linux-generic/pktio/socket_mmap.c | 22 ++
1 file changed, 22 insertions(+)
diff --git
Signed-off-by: Nicolas Morey-Chaisemartin nmo...@kalray.eu
---
test/validation/pktio/pktio.c | 73 ---
test/validation/pktio/pktio.h | 3 +-
2 files changed, 64 insertions(+), 12 deletions(-)
diff --git a/test/validation/pktio/pktio.c
Signed-off-by: Maxim Uvarov maxim.uva...@linaro.org
---
test/validation/pktio/pktio.c | 112 ++
1 file changed, 112 insertions(+)
diff --git a/test/validation/pktio/pktio.c b/test/validation/pktio/pktio.c
index ebe34fa..f8a8981 100644
---
Signed-off-by: Maxim Uvarov maxim.uva...@linaro.org
---
platform/linux-generic/pktio/socket.c | 29 +
1 file changed, 29 insertions(+)
diff --git a/platform/linux-generic/pktio/socket.c
b/platform/linux-generic/pktio/socket.c
index 55c3594..0c56d70 100644
---
On Thu, Aug 13, 2015 at 6:45 AM, Maxim Uvarov maxim.uva...@linaro.org
wrote:
Signed-off-by: Maxim Uvarov maxim.uva...@linaro.org
---
platform/linux-generic/pktio/loop.c | 21 +
1 file changed, 21 insertions(+)
diff --git a/platform/linux-generic/pktio/loop.c
Signed-off-by: Nicolas Morey-Chaisemartin nmo...@kalray.eu
---
example/generator/odp_generator.c | 1 -
example/ipsec/odp_ipsec.c | 1 -
example/packet/odp_pktio.c | 1 -
.../linux-generic/include/odp_packet_io_internal.h |
Fixes bug #1670
Signed-off-by: Nicolas Morey-Chaisemartin nmo...@kalray.eu
---
platform/linux-generic/include/odp_packet_socket.h | 3 -
platform/linux-generic/pktio/socket.c | 85 +++---
2 files changed, 57 insertions(+), 31 deletions(-)
diff --git
Changes since v4:
Fixed bug by rewriting packet allocation to use _odp_packet_alloc() to
allocate the largest possible packet
instead of (wrongly) computing it from segment length. Also dropped now
unused data from pkt_sock_t
Changes since v3:
Fix long line
Make pktio_suite_init static
After open pktio remains stopped. Inside worker thread (after all
configuration done it should be started.) In that patch I just start
it everywhere to keep original logic and be easy for review. Putting
it inside worker threads will require thread arguments change and
understanding logic of
https://bugs.linaro.org/show_bug.cgi?id=1542
--- Comment #2 from Bala Manoharan bala.manoha...@linaro.org ---
V2 submitted: https://patches.linaro.org/52328/
--
You are receiving this mail because:
You are on the CC list for the bug.___
lng-odp
https://bugs.linaro.org/show_bug.cgi?id=1545
--- Comment #2 from Bala Manoharan bala.manoha...@linaro.org ---
v2 submitted: https://patches.linaro.org/52328/
--
You are receiving this mail because:
You are on the CC list for the bug.___
lng-odp
https://bugs.linaro.org/show_bug.cgi?id=1544
--- Comment #2 from Bala Manoharan bala.manoha...@linaro.org ---
v2 submitted: https://patches.linaro.org/52328/
--
You are receiving this mail because:
You are on the CC list for the bug.___
lng-odp
https://bugs.linaro.org/show_bug.cgi?id=1546
--- Comment #2 from Bala Manoharan bala.manoha...@linaro.org ---
v2 submitted: https://patches.linaro.org/52328/
--
You are receiving this mail because:
You are on the CC list for the bug.___
lng-odp
On 08/13/15 15:44, Bill Fischofer wrote:
@@ -72,6 +76,9 @@ static int loopback_send(pktio_entry_t
*pktio_entry, odp_packet_t pkt_tbl[],
queue_entry_t *qentry;
unsigned i;
+ if (pktio_entry-s.state == STATE_STOP)
+ return 0;
+
Signed-off-by: Zoltan Kiss zoltan.k...@linaro.org
---
test/validation/packet/packet.c | 18 ++
1 file changed, 14 insertions(+), 4 deletions(-)
diff --git a/test/validation/packet/packet.c b/test/validation/packet/packet.c
index 99a6745..36edc12 100644
---
The Testing Results page should be easier to see results from jenkins/ci
builds (along with the previous coverity badges). Please let Mike or I know
if you find defects.
http://www.opendataplane.org/testing/
--
Sincerely,
Marshall Guillory
Technical Program Enabler, Agile Coach, Scrum master
Applications can preset certain parts of the packet user area, so when that
memory will be allocated it starts from a known state. If the platform
allocates the memory during pool creation, it's enough to run the
constructor after that. If it's allocating memory on demand, it should
call the
Signed-off-by: Zoltan Kiss zoltan.k...@linaro.org
---
platform/linux-generic/odp_pool.c | 10 ++
1 file changed, 10 insertions(+)
diff --git a/platform/linux-generic/odp_pool.c
b/platform/linux-generic/odp_pool.c
index 14221fd..2cc8441 100644
--- a/platform/linux-generic/odp_pool.c
+++
Applications can read the computed hash (if any) and set it if they
changed the packet headers or if the platform haven't calculated the hash.
Signed-off-by: Zoltan Kiss zoltan.k...@linaro.org
---
v2:
- focus on RSS hash only
- use setter/getter's
include/odp/api/packet.h | 28
s/unitalicized/uninitialized
Autocorrect strikes again?
On Thu, Aug 13, 2015 at 2:04 PM, Mike Holmes mike.hol...@linaro.org wrote:
It is possible that a code change leaves s=NULL; default char_ok to
failure
Signed-off-by: Mike Holmes mike.hol...@linaro.org
---
str_sz is not used, remove it
Signed-off-by: Mike Holmes mike.hol...@linaro.org
---
test/validation/common/mask_common.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/test/validation/common/mask_common.c
b/test/validation/common/mask_common.c
index fce7725..119c2bb 100644
On Thu, Aug 13, 2015 at 12:52 PM, Zoltan Kiss zoltan.k...@linaro.org
wrote:
Applications can read the computed hash (if any) and set it if they
changed the packet headers or if the platform haven't calculated the hash.
Signed-off-by: Zoltan Kiss zoltan.k...@linaro.org
---
v2:
- focus on
retval is not used, remove it
Signed-off-by: Mike Holmes mike.hol...@linaro.org
---
test/validation/classification/odp_classification_tests.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/test/validation/classification/odp_classification_tests.c
It is possible that a code change leaves s=NULL; default char_ok to
failure
Signed-off-by: Mike Holmes mike.hol...@linaro.org
---
test/validation/system/system.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/test/validation/system/system.c b/test/validation/system/system.c
Actually, let me rescind that reviewed-by. See comments inline.
On Thu, Aug 13, 2015 at 2:41 PM, Bill Fischofer bill.fischo...@linaro.org
wrote:
On Thu, Aug 13, 2015 at 2:04 PM, Mike Holmes mike.hol...@linaro.org
wrote:
str_sz is not used, remove it
Signed-off-by: Mike Holmes
Note: I've verified that this also passes, as expected, when the checkpatch
issue identified in Part 2/3 is resolved by removing the extra parens.
On Thu, Aug 13, 2015 at 12:28 PM, Zoltan Kiss zoltan.k...@linaro.org
wrote:
Signed-off-by: Zoltan Kiss zoltan.k...@linaro.org
On Thu, Aug 13, 2015 at 12:28 PM, Zoltan Kiss zoltan.k...@linaro.org
wrote:
Signed-off-by: Zoltan Kiss zoltan.k...@linaro.org
---
platform/linux-generic/odp_pool.c | 10 ++
1 file changed, 10 insertions(+)
diff --git a/platform/linux-generic/odp_pool.c
On Thu, Aug 13, 2015 at 2:04 PM, Mike Holmes mike.hol...@linaro.org wrote:
str_sz is not used, remove it
Signed-off-by: Mike Holmes mike.hol...@linaro.org
Reviewed-by: Bill Fischofer bill.fischo...@linaro.org
---
test/validation/common/mask_common.c | 2 +-
1 file changed, 1
Some grammatical nits:
On Thu, Aug 13, 2015 at 12:28 PM, Zoltan Kiss zoltan.k...@linaro.org
wrote:
Applications can preset certain parts of the packet user area, so when that
memory will be allocated it starts from a known state. If the platform
allocates the memory during pool creation, it's
https://bugs.linaro.org/show_bug.cgi?id=1615
--- Comment #7 from Mike Holmes mike.hol...@linaro.org ---
Patch committed
https://git.linaro.org/lng/odp.git/commit/6788a0e25b2f9c244668cf3d6f71d0f2eaefd32f
That may provide the fix, watching CI for re-occurrence.
--
You are receiving this mail
On Thu, Aug 13, 2015 at 2:04 PM, Mike Holmes mike.hol...@linaro.org wrote:
retval is not used, remove it
Signed-off-by: Mike Holmes mike.hol...@linaro.org
---
test/validation/classification/odp_classification_tests.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
ping - can we close this for 1.3 ?
On 6 August 2015 at 06:22, Maxim Uvarov maxim.uva...@linaro.org wrote:
On 08/06/15 12:55, Stuart Haslam wrote:
Test using the loop interface even when running as root.
https://bugs.linaro.org/show_bug.cgi?id=1735
Signed-off-by: Stuart Haslam
https://bugs.linaro.org/show_bug.cgi?id=1615
Mike Holmes mike.hol...@linaro.org changed:
What|Removed |Added
Assignee|christian.ziet...@linaro.or |lng-odp@lists.linaro.org
https://bugs.linaro.org/show_bug.cgi?id=1746
Mike Holmes mike.hol...@linaro.org changed:
What|Removed |Added
Priority|--- |Lowest
https://bugs.linaro.org/show_bug.cgi?id=1670
--- Comment #13 from Mike Holmes mike.hol...@linaro.org ---
Latest v5 http://patches.opendataplane.org/patch/2732/
--
You are receiving this mail because:
You are on the CC list for the bug.___
lng-odp
41 matches
Mail list logo
