On 14 July 2017 at 04:33, Bill Fischofer wrote:
>
>
> On Thu, Jul 13, 2017 at 4:32 PM, Dmitry Eremin-Solenikov <
> dmitry.ereminsoleni...@linaro.org> wrote:
>
>> On 13.07.2017 21:20, Bill Fischofer wrote:
>> > On Thu, Jul 13, 2017 at 12:58 PM, Bala Manoharan
>> > wrote:
>> >> On 13 July 2017 at
OK. Yes I think 10x should be conservative enough. Make it a #define so that
it's easy to update if needed.
-Petri
From: Kevin Wang [mailto:kevin.w...@linaro.org]
Sent: Thursday, July 13, 2017 6:31 PM
To: Savolainen, Petri (Nokia - FI/Espoo)
Cc: Kevin Wang ; lng-odp@lists.linaro.org
Subject:
Reviewed-by: Petri Savolainen
Maxim, could you merge this ASAP since patch 1/4 fixes broken api-next build
(when using gcc 4.8.4)
> -Original Message-
> From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of
> Github ODP bot
> Sent: Friday, July 14, 2017 1:00 AM
> To: lng
https://bugs.linaro.org/show_bug.cgi?id=3126
Dmitry Eremin-Solenikov changed:
What|Removed |Added
CC||dmitry.ereminsolenikov@lina
Signed-off-by: Balasubramanian Manoharan
---
platform/linux-generic/include/protocols/thash.h | 108 ++
1 file changed, 108 insertions(+)
create mode 100644 platform/linux-generic/include/protocols/thash.h
diff --git a/platform/linux-generic/include/protocols/thash.h
b/pla
Enable packet hashing per CoS to be able to distribute incoming packets to
multiple queues linked with a CoS.
Signed-off-by: Balasubramanian Manoharan
---
include/odp/api/spec/classification.h | 96 +++--
include/odp/api/spec/packet_io.h | 21
2 file
Signed-off-by: Balasubramanian Manoharan
---
.../include/odp_classification_datamodel.h | 20 ++-
.../include/odp_classification_internal.h |4 +
platform/linux-generic/odp_classification.c| 182
3 files changed, 176 insertions(+), 30 deletions
Branch: refs/heads/api-next
Home: https://github.com/Linaro/odp
Commit: bc36acc21368ff3f27d5b3836e8524747e0ab4c9
https://github.com/Linaro/odp/commit/bc36acc21368ff3f27d5b3836e8524747e0ab4c9
Author: Dmitry Eremin-Solenikov
Date: 2017-07-14 (Fri, 14 Jul 2017)
Changed paths:
Merged.
Maxim.
On 14 July 2017 at 11:24, Savolainen, Petri (Nokia - FI/Espoo) <
petri.savolai...@nokia.com> wrote:
> Reviewed-by: Petri Savolainen
>
> Maxim, could you merge this ASAP since patch 1/4 fixes broken api-next
> build (when using gcc 4.8.4)
>
>
> > -Original Message-
> > From
Update for Monarch_lts release. I included Bills fixes for gcc 7 and added
Travis checks.
github
/** Email created from pull request 77 (muvarov:monarch_lts)
** https://github.com/Linaro/odp/pull/77
** Patch: https://github.com/Linaro/odp/pull/77.patch
*
From: Bill Fischofer
Resolve Bug https://bugs.linaro.org/show_bug.cgi?id=2942 by adding
casts needed to avoid compilation failures when using clang 4.0.0
included in Ubuntu 17.04, which is stricter than clang 3.8.1 which
is in Ubuntu 16.10.
Signed-off-by: Bill Fischofer
Signed-off-by: Maxim Uva
From: Brian Brooks
The GCC 7 series introduces changes that expose ODP compilation
issues. These include case statement fall through warnings, and
stricter checks on potential string overflows and other semantic
analysis.
Fixes: https://bugs.linaro.org/show_bug.cgi?id=3027
Signed-off-by: Brian
From: Brian Brooks
The GCC 7 series introduces changes that expose ODP compilation
issues. These include case statement fall through warnings, and
stricter checks on potential string overflows and other semantic
analysis.
Fixes: https://bugs.linaro.org/show_bug.cgi?id=3027
Signed-off-by: Brian
From: Mike Holmes
Use of a script in the autotools breaks github CI
Define the API version in configure and reuse it in the code
Benefits:
removes dependence on scripts to build
removes scripts and the make file from odp/scripts
removes generated .scmversion file
places api versioning right next
From: Maxim Uvarov
autoconf-archive is not available on some build environment
Number of projects simple have local copy of this file.
Do the same thing.
Signed-off-by: Maxim Uvarov
Reviewed-by: Dmitry Eremin-Solenikov
---
/** Email created from pull request 77 (muvarov:monarch_lts)
** https:
From: Maxim Uvarov
Idea if check was to detect if libatomic is needed or not,
not trap configure on not case
Fixes Linaro CI compilation for:
gcc-linaro-5.3-2016.02-x86_64_arm-linux-gnueabih
Signed-off-by: Maxim Uvarov
Reviewed-by: Bill Fischofer
---
/** Email created from pull request 77 (muv
From: Brian Brooks
Use uint64_t instead of int type.
This resolves ipfragreass build breakage with clang on 32-bit systems.
Signed-off-by: Brian Brooks
Signed-off-by: Maxim Uvarov
---
/** Email created from pull request 77 (muvarov:monarch_lts)
** https://github.com/Linaro/odp/pull/77
** Pa
From: Maxim Uvarov
Make check should skip the test instead of failing it.
Test splits RX and TX cores for packet processing. Core
0 bind to control thread. So running machine should have
at least 2 worker threads which is not enough on 1 and 2
cpus machine. CUnit uses special value 77 to mark tes
From: Maxim Uvarov
Signed-off-by: Maxim Uvarov
---
/** Email created from pull request 77 (muvarov:monarch_lts)
** https://github.com/Linaro/odp/pull/77
** Patch: https://github.com/Linaro/odp/pull/77.patch
** Base sha: 0c15c40db40834f1df217191c4b6a06303ab0872
** Merge commit sha: f06a7319a0
Hi Bill,
Existing pktio API has only support of querying MAC and MTU values.
Adds new pktio APIs to set MTU and MAC address on pktio interface.
Implementation can indicate it's supported maximum MTU size and MAC
address size in its pktio capabilities. User can call set APIs to
set values within
Adds new pktio APIs to set MTU and MAC address on pktio interface.
Signed-off-by: Vamsi Attunuru
Signed-off-by: Mahipal Challa
Signed-off-by: Shally Verma
---
include/odp/api/spec/packet_io.h | 45
1 file changed, 45 insertions(+)
diff --git a/inclu
> -Original Message-
> From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of Vamsi
> Attunuru
> Sent: Friday, July 14, 2017 1:05 PM
> To: lng-odp@lists.linaro.org
> Cc: mcha...@cavium.com; pathr...@cavium.com; vattun...@cavium.com;
> sve...@cavium.com
> Subject: [lng-odp] [
V2:
Simplify mbuf_pool_create() arguments (Krishna)
Fix bug in dpdk_send() (Krishna)
Add Travis test (Maxim)
Destroy dpdk memory pool using a callback in odp_pool_destroy(). Fixes a bug
detected by Travis.
This patch set adds an experimental zero-copy mode for DPDK pktio. Zero-copy
operation is
From: Matias Elo
Follow the API definition and use total buffer length instead of single
segment length when checking 'len' argument validity.
Reset also packet segment data pointers and lengths.
Signed-off-by: Matias Elo
---
/** Email created from pull request 78 (matiaselo:dev/dpdk_ext_mempo
From: Matias Elo
Move packet_init() inline function to odp_packet_internal.h. This way e.g.
pktio devices can use it directly.
Signed-off-by: Matias Elo
---
/** Email created from pull request 78 (matiaselo:dev/dpdk_ext_mempool_v2)
** https://github.com/Linaro/odp/pull/78
** Patch: https://gi
From: Matias Elo
Frame size should include also packet headroom and tailroom.
Signed-off-by: Matias Elo
---
/** Email created from pull request 78 (matiaselo:dev/dpdk_ext_mempool_v2)
** https://github.com/Linaro/odp/pull/78
** Patch: https://github.com/Linaro/odp/pull/78.patch
** Base sha: 4
From: Matias Elo
One can remove max 'cache_num' objects from the local cache.
Signed-off-by: Matias Elo
---
/** Email created from pull request 78 (matiaselo:dev/dpdk_ext_mempool_v2)
** https://github.com/Linaro/odp/pull/78
** Patch: https://github.com/Linaro/odp/pull/78.patch
** Base sha: 4
From: Matias Elo
Signed-off-by: Matias Elo
---
/** Email created from pull request 78 (matiaselo:dev/dpdk_ext_mempool_v2)
** https://github.com/Linaro/odp/pull/78
** Patch: https://github.com/Linaro/odp/pull/78.patch
** Base sha: 490f4bf22129638899ce71c99a8847e8ba849692
** Merge commit sha:
From: Matias Elo
The packet default headroom is modified to be the same as in DPDK to enable
interoperability between ODP and DPDK packets.
Signed-off-by: Matias Elo
---
/** Email created from pull request 78 (matiaselo:dev/dpdk_ext_mempool_v2)
** https://github.com/Linaro/odp/pull/78
** Patc
From: Matias Elo
Signed-off-by: Matias Elo
---
/** Email created from pull request 78 (matiaselo:dev/dpdk_ext_mempool_v2)
** https://github.com/Linaro/odp/pull/78
** Patch: https://github.com/Linaro/odp/pull/78.patch
** Base sha: 490f4bf22129638899ce71c99a8847e8ba849692
** Merge commit sha:
From: Matias Elo
Implements experimental zero-copy mode for DPDK pktio. This can be enabled
with additional '--enable-dpdk-zero-copy' configure flag.
This feature has been put behind an extra configure flag as it doesn't
entirely adhere to the DPDK API and may behave unexpectedly with untested
D
Hi,
ODP API should somewhere define what exactly MTU means in the context of ODP.
One can guess that transmission and reception of L2 frames larger than the
configured MTU is not guaranteed to succeed, but which bytes are taken into
account? For instance, is Ethernet FCS counted towards the MTU?
On Thu, Jul 13, 2017 at 2:17 PM, Savolainen, Petri (Nokia - FI/Espoo)
wrote:
>
>
>> -Original Message-
>> From: shally verma [mailto:shallyvermacav...@gmail.com]
>> Sent: Thursday, July 13, 2017 11:43 AM
>> To: Savolainen, Petri (Nokia - FI/Espoo)
>> Cc: lng-odp-forward ; Shally Verma
>>
On Fri, Jul 14, 2017 at 6:55 AM, shally verma
wrote:
> On Thu, Jul 13, 2017 at 2:17 PM, Savolainen, Petri (Nokia - FI/Espoo)
> wrote:
> >
> >
> >> -Original Message-
> >> From: shally verma [mailto:shallyvermacav...@gmail.com]
> >> Sent: Thursday, July 13, 2017 11:43 AM
> >> To: Savolain
Add status ID for warning messages. Outbound inline soft lifetime
expiration is currently the only source of these events. API spec
keeps simple when the same warning structure is shared with IPSEC
packet and status events.
Signed-off-by: Petri Savolainen
---
include/odp/api/spec/ipsec.h | 33 ++
On Fri, Jul 14, 2017 at 6:06 PM, Bill Fischofer
wrote:
>
>
> On Fri, Jul 14, 2017 at 6:55 AM, shally verma
> wrote:
>>
>> On Thu, Jul 13, 2017 at 2:17 PM, Savolainen, Petri (Nokia - FI/Espoo)
>> wrote:
>> >
>> >
>> >> -Original Message-
>> >> From: shally verma [mailto:shallyvermacav...@
On Fri, Jul 14, 2017 at 6:05 AM, Peltonen, Janne (Nokia - FI/Espoo) <
janne.pelto...@nokia.com> wrote:
> Hi,
>
> ODP API should somewhere define what exactly MTU means in the context of
> ODP.
>
> One can guess that transmission and reception of L2 frames larger than the
> configured MTU is not gu
On Fri, Jul 14, 2017 at 8:02 AM, shally verma
wrote:
> On Fri, Jul 14, 2017 at 6:06 PM, Bill Fischofer
> wrote:
> >
> >
> > On Fri, Jul 14, 2017 at 6:55 AM, shally verma <
> shallyvermacav...@gmail.com>
> > wrote:
> >>
> >> On Thu, Jul 13, 2017 at 2:17 PM, Savolainen, Petri (Nokia - FI/Espoo)
>
This should also be reviewed by Bala and Nikhil, but it looks reasonable to
me and reflects the discussion we had earlier this week.
On Fri, Jul 14, 2017 at 7:43 AM, Petri Savolainen <
petri.savolai...@linaro.org> wrote:
> Add status ID for warning messages. Outbound inline soft lifetime
> expira
On 14.07.2017 15:43, Petri Savolainen wrote:
> Add status ID for warning messages. Outbound inline soft lifetime
> expiration is currently the only source of these events. API spec
> keeps simple when the same warning structure is shared with IPSEC
> packet and status events.
>
> Signed-off-by: Pe
On Fri, Jul 14, 2017 at 8:30 AM, Dmitry Eremin-Solenikov <
dmitry.ereminsoleni...@linaro.org> wrote:
> On 14.07.2017 15:43, Petri Savolainen wrote:
> > Add status ID for warning messages. Outbound inline soft lifetime
> > expiration is currently the only source of these events. API spec
> > keeps
Hi Bill
The reasons below dont jive with what ODP does today. If the
routines odp_pktio_mtu() and odp_pktio_mac_addr() exist today. They
assume that MTU can be configured and returned to the datapath. As an
application developer, it does make sense for me to use 2 different
paths to talk t
> -Original Message-
> From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of
> Balasubramanian Manoharan
> Sent: Friday, July 14, 2017 12:12 PM
> To: lng-odp@lists.linaro.org
> Subject: [lng-odp] [RFC/API-NEXTv3 1/3] api: classification: add support
> for packet hashing in
From: Bill Fischofer [mailto:bill.fischo...@linaro.org]
Sent: Friday, July 14, 2017 4:40 PM
To: Dmitry Eremin-Solenikov
Cc: Petri Savolainen ; lng-odp-forward
Subject: Re: [lng-odp] [API-NEXT PATCH] api: ipsec: add warning status event
On Fri, Jul 14, 2017 at 8:30 AM, Dmitry Eremin-Solenik
On Fri, Jul 14, 2017 at 8:41 AM, Narayana Prasad Athreya <
pathr...@caviumnetworks.com> wrote:
> Hi Bill
> The reasons below dont jive with what ODP does today. If the routines
> odp_pktio_mtu() and odp_pktio_mac_addr() exist today. They assume that MTU
> can be configured and returned to the
Hi,
I think we need both MTU and MAC address to be settable from ODP.
In OFP, at some point I was considering using tap pktio instead of ofp
tap interface for slow path processing - not being able to set MAC
address was A problem.
What about DHCP by MAC address? Maybe there are systems where
pred
On 14 July 2017 at 19:18, Savolainen, Petri (Nokia - FI/Espoo) <
petri.savolai...@nokia.com> wrote:
>
>
> > -Original Message-
> > From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of
> > Balasubramanian Manoharan
> > Sent: Friday, July 14, 2017 12:12 PM
> > To: lng-odp@lis
On Fri, Jul 14, 2017 at 9:11 AM, Bogdan Pricope
wrote:
> Hi,
>
> I think we need both MTU and MAC address to be settable from ODP.
>
> In OFP, at some point I was considering using tap pktio instead of ofp
> tap interface for slow path processing - not being able to set MAC
> address was A proble
Add several basic tests for IPsec API.
This PR incorporates #69 and event subtypes implementation on which it depends.
github
/** Email created from pull request 70 (lumag:ipsec-testsuite-2)
** https://github.com/Linaro/odp/pull/70
** Patch: https://githu
From: Dmitry Eremin-Solenikov
Add several basic tests for IPsec API.
Signed-off-by: Dmitry Eremin-Solenikov
---
/** Email created from pull request 70 (lumag:ipsec-testsuite-2)
** https://github.com/Linaro/odp/pull/70
** Patch: https://github.com/Linaro/odp/pull/70.patch
** Base sha: ae42f98
Hi,
I think MAC addresses and MTU are a bit different.
There are valid use cases for application wanting to set the MAC address, even
dynamically. For instance, with VRRP one wants to receive packets destined to a
VRRP MAC address in addition to the fixed MAC address of a port. The
possibility
https://bugs.linaro.org/show_bug.cgi?id=3126
--- Comment #7 from Maxim Uvarov ---
https://github.com/Linaro/odp/pull/79
--
You are receiving this mail because:
You are on the CC list for the bug.
On 07/12 16:00:04, Github ODP bot wrote:
> From: Dmitry Eremin-Solenikov
>
> Clang 3.8 is stricter than GCC wrt register allocation vs 128-bit
> variables. Sometimes it can not understand using 128-bit var in place of
> 64-bit register resulting in the following errors:
>
> /odp_ipfragreass_atom
On Fri, Jul 14, 2017 at 10:09 AM, Peltonen, Janne (Nokia - FI/Espoo) <
janne.pelto...@nokia.com> wrote:
> Hi,
>
>
>
> I think MAC addresses and MTU are a bit different.
>
>
>
> There are valid use cases for application wanting to set the MAC address,
> even dynamically. For instance, with VRRP one
On 08.07.2017 00:16, Brian Brooks wrote:
> On 07/12 16:00:04, Github ODP bot wrote:
>> From: Dmitry Eremin-Solenikov
>>
>> Clang 3.8 is stricter than GCC wrt register allocation vs 128-bit
>> variables. Sometimes it can not understand using 128-bit var in place of
>> 64-bit register resulting in t
55 matches
Mail list logo