Use odp_timer_capability() to determine the max timer resolution
Signed-off-by: Kevin Wang
---
example/generator/odp_generator.c | 8 +++-
example/timer/odp_timer_simple.c | 9 -
example/timer/odp_timer_test.c| 8 ++--
test/common_plat/
Signed-off-by: Kevin Wang
---
doc/users-guide/users-guide-timer.adoc | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/doc/users-guide/users-guide-timer.adoc
b/doc/users-guide/users-guide-timer.adoc
index 9cd30de..4a8bad7 100644
--- a/doc/users-guide/users-guide-timer.adoc
++
Implement a new internal function timer_res_init() to detect the max
timer resolution without overrun at the ODP init stage. It will check
timer resolution from 1ms to 100us, 10us...1ns until the timer is
overrun.
Signed-off-by: Kevin Wang
---
platform/linux-generic/odp_timer.c | 98
Currently, user needs to decide the timer resolution before creating
a timer pool. But sometimes it will cause timer overrun as the system
can't support such high resolution.
So a new API is required to expose the timer capability to the user.
Signed-off-by: Kevin Wang
---
include/odp/api/spec/t
Here is the link to Google Doc version:
https://docs.google.com/a/linaro.org/document/d/1CXqN3pZay7Ni1Z7xhxW8_tmcIaKyuU8v-l5mshZ2RGk/edit?usp=sharing
Thank you,
Honnappa
On 12 July 2017 at 15:49, Honnappa Nagarahalli
wrote:
> I tried briefly. The document converts easily. Main problem is the
> p
I tried briefly. The document converts easily. Main problem is the
pictures. The pictures need to be redrawn.
May be I will convert the document and will draw the pictures later tonight.
Thanks,
Honnappa
On 12 July 2017 at 15:14, Bill Fischofer wrote:
> Can you post a Google doc version of this
https://bugs.linaro.org/show_bug.cgi?id=2857
--- Comment #6 from Maxim Uvarov ---
yes that can be fixed. Other issue is docker has limitation of 64MB for
/dev/shm and some our tests allocate pool more then 64MB (like helper tests). I
opened separate bug for that.
--
You are receiving this mail
https://bugs.linaro.org/show_bug.cgi?id=3126
--- Comment #1 from Maxim Uvarov ---
Hm. I did not have isolcpus= in kernel but just with taskset it always pass for
me.
--
You are receiving this mail because:
You are on the CC list for the bug.
Not sure if the attachment made it to the ODP mailing list. Including
the text here:
Barry SpinneyJuly 12,2017
Combined Cipher and MAC operations
The three Encrypt/MAC Operation Kinds are:
ENCRYPT_THEN_MAC:
Encrypt
Thanks, Barry.
Copying some interested parties as well as the ODP mailing list so the
discussion can move there.
Regards,
Bill
On Wed, Jul 12, 2017 at 2:46 PM, Barry Spinney wrote:
> See attached ascii text file describing the THREE cases (whereas ODP and
> maybe DPDK only have 2).
> Hopefull
Can you post a Google doc version of this to permit shared commenting?
I can do that if you'd like, but you should probably be the doc owner.
On Wed, Jul 12, 2017 at 3:05 PM, Honnappa Nagarahalli
wrote:
> Hi,
>The discussion about what are the different configurations of the
> platform that O
Hi,
The discussion about what are the different configurations of the
platform that ODP-Cloud needs to support has come up now and then. I
thought it is good to have a consensus, on the configurations we have
to support for now, to help us guide in our future discussions. I
created a word docume
https://bugs.linaro.org/show_bug.cgi?id=3128
Maxim Uvarov changed:
What|Removed |Added
Assignee|christophe.mil...@linaro.or |lng-odp@lists.linaro.org
|g
https://bugs.linaro.org/show_bug.cgi?id=3128
Bug ID: 3128
Summary: shm in docker fails
Product: OpenDataPlane - linux- generic reference
Version: master
Hardware: Other
OS: Linux
Status: UNCONFIRMED
Severi
Hi Francois,
I will add this topic to tomorrow's ODP-Cloud call.
Thank you,
Honnappa
On 12 July 2017 at 04:51, Francois Ozog wrote:
> I'd like we sync up on key elements of the document at the arch call today.
>
> I'd like to restate a few things:
>
> - ODP Cloud single binary should support
On Wed, Jul 12, 2017 at 10:33 AM, shally verma
wrote:
> On Wed, Jul 12, 2017 at 8:37 PM, Bill Fischofer
> wrote:
>> ODP_UNUSED is useful scaffolding. The alternative is to make a null
>> reference to unused parameters to avoid compile-time warnings. For
>> example:
>>
>> int odp_xxx(uint32_t x, u
On Wed, Jul 12, 2017 at 11:16 AM, shally verma
wrote:
> On Wed, Jul 12, 2017 at 8:50 PM, Bill Fischofer
> wrote:
>> On Wed, Jul 12, 2017 at 7:35 AM, shally verma
>> wrote:
>>> I have a question on odp_pktin_recv(odp_pktin_queue_t, odp_packet_t [],
>>> int).
>>>
>>> Once app receives packets ass
On Wed, Jul 12, 2017 at 8:50 PM, Bill Fischofer
wrote:
> On Wed, Jul 12, 2017 at 7:35 AM, shally verma
> wrote:
>> I have a question on odp_pktin_recv(odp_pktin_queue_t, odp_packet_t [], int).
>>
>> Once app receives packets associated to the queue, what happens to
>> packets buffers passed to ap
https://bugs.linaro.org/show_bug.cgi?id=3078
Honnappa Nagarahalli changed:
What|Removed |Added
CC||honnappa.nagarahalli@linaro
On Wed, Jul 12, 2017 at 8:37 PM, Bill Fischofer
wrote:
> ODP_UNUSED is useful scaffolding. The alternative is to make a null
> reference to unused parameters to avoid compile-time warnings. For
> example:
>
> int odp_xxx(uint32_t x, uint32_t y)
> {
> (void)y; /* avoid unused parameter war
On Wed, Jul 12, 2017 at 7:35 AM, shally verma
wrote:
> I have a question on odp_pktin_recv(odp_pktin_queue_t, odp_packet_t [], int).
>
> Once app receives packets associated to the queue, what happens to
> packets buffers passed to application? Does implementation always
> assume that packets will
ODP_UNUSED is useful scaffolding. The alternative is to make a null
reference to unused parameters to avoid compile-time warnings. For
example:
int odp_xxx(uint32_t x, uint32_t y)
{
(void)y; /* avoid unused parameter warning for y */
...do something useful with x
}
ODP in genera
From: Dmitry Eremin-Solenikov
It was decided that it would be benefitable to live with both API types
at this point, as odp_crypto_compl_t was available for some time. So
undeprecate odp_crypto_compl_t and related functionality. Validation
tests also provide necessary tests for pref_mode and for
From: Dmitry Eremin-Solenikov
It is easy to update PREDEFINED definitions in one Doxyfile, while
forgetting about the others, as happened e.g. with ODP_DEPRECATE(x)
definition. Move PREDEFINED definitions into Doxyfile_common, so that it
is included by the rest of Doxyfiles.
Signed-off-by: Dmitr
From: Dmitry Eremin-Solenikov
Signed-off-by: Dmitry Eremin-Solenikov
---
/** Email created from pull request 74 (lumag:crypto-packet)
** https://github.com/Linaro/odp/pull/74
** Patch: https://github.com/Linaro/odp/pull/74.patch
** Base sha: ee5be324411a7520528a367967c28fc529d3bc2e
** Merge
Updates for crypto API as suggested by @psavol
github
/** Email created from pull request 74 (lumag:crypto-packet)
** https://github.com/Linaro/odp/pull/74
** Patch: https://github.com/Linaro/odp/pull/74.patch
** Base sha: ee5be324411a7520528a367967c28fc5
From: Dmitry Eremin-Solenikov
Signed-off-by: Dmitry Eremin-Solenikov
---
/** Email created from pull request 74 (lumag:crypto-packet)
** https://github.com/Linaro/odp/pull/74
** Patch: https://github.com/Linaro/odp/pull/74.patch
** Base sha: ee5be324411a7520528a367967c28fc529d3bc2e
** Merge
Move direction (inboud/outbound) specific SA parameters
into separate structures. Specification is more clear
between common and direction specific SA parameters.
Signed-off-by: Petri Savolainen
Signed-off-by: Bogdan Pricope
---
include/odp/api/spec/ipsec.h | 103 +--
Split error bits into error and warning bits. Move flags into
separate type and field. This way, all exceptions (rare but
important events) can be checked and maintained separately
from operation flags, which may be set for almost every packet.
Signed-off-by: Petri Savolainen
---
include/odp/api
Outer headers are read-only data for an implementation.
Signed-off-by: Petri Savolainen
Signed-off-by: Dmitry Eremin-Solenikov
---
include/odp/api/spec/ipsec.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/odp/api/spec/ipsec.h b/include/odp/api/spec/ipsec.h
index c
Application may track time based SA lifetime expiration
efficiently and in many ways: e.g. ODP timers may be used,
wall clock time may be polled, control plane may send
notifications, etc.
Signed-off-by: Petri Savolainen
---
include/odp/api/spec/ipsec.h | 26 --
1 file ch
Allow implementation to decide how many times a soft lifetime
expiration is reported. Application does not require strict
synchronization of soft lifetimes.
Signed-off-by: Petri Savolainen
---
include/odp/api/spec/ipsec.h | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/inc
AAD lengths are needed in crypto API but not in IPSEC API.
So, application should ignore AAD capability values in IPSEC
API.
Signed-off-by: Petri Savolainen
---
include/odp/api/spec/ipsec.h | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/include/odp/api/spec/ipsec.h b/incl
Added capability to specify maximum number of different queues that
can be used with IPsec.
Signed-off-by: Petri Savolainen
---
include/odp/api/spec/ipsec.h | 4
1 file changed, 4 insertions(+)
diff --git a/include/odp/api/spec/ipsec.h b/include/odp/api/spec/ipsec.h
index 9f061961..fa793e3
Added a parameter for passing salt/nonce for algorithms that
need it. Currently, only AES GCM needs it with 4 bytes of
salt. Later on, other cipher (or auth) algorithms may need
also salt/nonce data.
Signed-off-by: Petri Savolainen
---
include/odp/api/spec/ipsec.h | 9 +
1 file changed,
Application should not need to control initial sequence number
value as implementation is able to manage that internally.
Signed-off-by: Petri Savolainen
---
include/odp/api/spec/ipsec.h | 3 ---
1 file changed, 3 deletions(-)
diff --git a/include/odp/api/spec/ipsec.h b/include/odp/api/spec/ips
On 12.07.2017 15:56, Petri Savolainen wrote:
> Added a parameter for passing salt/nonce for algorithms that
> need it. Currently, only AES GCM needs it with 4 bytes of
> salt. Later on, other cipher or auth algorithms may need
> also salt/nonce data.
>
> Signed-off-by: Petri Savolainen
> ---
> i
Reviewed-by: Petri Savolainen
> -Original Message-
> From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of
> Github ODP bot
> Sent: Wednesday, July 12, 2017 3:00 PM
> To: lng-odp@lists.linaro.org
> Subject: [lng-odp] [PATCH API-NEXT v1 0/3] Crypto API updates
>
> Updates
From: Dmitry Eremin-Solenikov
Signed-off-by: Dmitry Eremin-Solenikov
---
/** Email created from pull request 73 (lumag:cross-2)
** https://github.com/Linaro/odp/pull/73
** Patch: https://github.com/Linaro/odp/pull/73.patch
** Base sha: 7fc6d27e937b57b31360b07028388c811f8300dc
** Merge commit
From: Dmitry Eremin-Solenikov
Following Maxim's idea use apt-get to download and install cross-packages.
Signed-off-by: Dmitry Eremin-Solenikov
---
/** Email created from pull request 73 (lumag:cross-2)
** https://github.com/Linaro/odp/pull/73
** Patch: https://github.com/Linaro/odp/pull/73.p
Per @muvarov's suggestion use apt-get instead of dpkg-cross to install
cross-packages
Add cross-compilation tests using clang
github
/** Email created from pull request 73 (lumag:cross-2)
** https://github.com/Linaro/odp/pull/73
** Patch: https://github.c
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_atomics_arm.h:18:51: error: value size does not match
regis
> -Original Message-
> From: Dmitry Eremin-Solenikov [mailto:dmitry.ereminsoleni...@linaro.org]
> Sent: Wednesday, July 12, 2017 11:53 AM
> To: Petri Savolainen ; lng-
> o...@lists.linaro.org
> Subject: Re: [lng-odp] [API-NEXT PATCH 1/9] api: ipsec: add salt parameter
>
> On 11.07.2017 1
Split error bits into error and warning bits. Move flags into
separate type and field. This way, all exceptions (rare but
important events) can be checked and maintained separately
from operation flags, which may be set for almost every packet.
Signed-off-by: Petri Savolainen
---
include/odp/api
Move direction (inboud/outbound) specific SA parameters
into separate structures. Specification is more clear
between common and direction specific SA parameters.
Signed-off-by: Petri Savolainen
Signed-off-by: Bogdan Pricope
---
include/odp/api/spec/ipsec.h | 103 +--
Outer headers are read-only data for an implementation.
Signed-off-by: Petri Savolainen
Signed-off-by: Dmitry Eremin-Solenikov
---
include/odp/api/spec/ipsec.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/odp/api/spec/ipsec.h b/include/odp/api/spec/ipsec.h
index 4
Application may track time based SA lifetime expiration
efficiently and in many ways: e.g. ODP timers may be used,
wall clock time may be polled, control plane may send
notifications, etc.
Signed-off-by: Petri Savolainen
---
include/odp/api/spec/ipsec.h | 26 --
1 file ch
Added a parameter for passing salt/nonce for algorithms that
need it. Currently, only AES GCM needs it with 4 bytes of
salt. Later on, other cipher or auth algorithms may need
also salt/nonce data.
Signed-off-by: Petri Savolainen
---
include/odp/api/spec/ipsec.h | 18 ++
1 file c
AAD lengths are needed in crypto API but not in IPSEC API.
So, application should ignore AAD capability values in IPSEC
API.
Signed-off-by: Petri Savolainen
---
include/odp/api/spec/ipsec.h | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/include/odp/api/spec/ipsec.h b/incl
Allow implementation to decide how many times a soft lifetime
expiration is reported. Application does not require strict
synchronization of soft lifetimes.
Signed-off-by: Petri Savolainen
---
include/odp/api/spec/ipsec.h | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/inc
Added capability to specify maximum number of different queues that
can be used with IPsec.
Signed-off-by: Petri Savolainen
---
include/odp/api/spec/ipsec.h | 4
1 file changed, 4 insertions(+)
diff --git a/include/odp/api/spec/ipsec.h b/include/odp/api/spec/ipsec.h
index 56363e22..f03d38d
Application should not need to control initial sequence number
value as implementation is able to manage that internally.
Signed-off-by: Petri Savolainen
---
include/odp/api/spec/ipsec.h | 3 ---
1 file changed, 3 deletions(-)
diff --git a/include/odp/api/spec/ipsec.h b/include/odp/api/spec/ips
Hi
If any implementation does not support specific mode of operation ,
say , it only support sync but not async then is it legal to mark
async API params as ODP_UNUSED? As in such case API may simply return
with failure code, thus references to params is not actually exercised
which results in to
I have a question on odp_pktin_recv(odp_pktin_queue_t, odp_packet_t [], int).
Once app receives packets associated to the queue, what happens to
packets buffers passed to application? Does implementation always
assume that packets will be freed by caller and re-allocates fresh
packets to queue?
I
On 11 July 2017 at 18:24, 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
> > Github ODP bot
> > Sent: Wednesday, July 05, 2017 5:00 PM
> > To: lng-odp@lists.linaro
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: ee5be32
From: Dmitry Eremin-Solenikov
Signed-off-by: Dmitry Eremin-Solenikov
---
/** Email created from pull request 74 (lumag:crypto-packet)
** https://github.com/Linaro/odp/pull/74
** Patch: https://github.com/Linaro/odp/pull/74.patch
** Base sha: ee5be324411a7520528a367967c28fc529d3bc2e
** Merge
From: Dmitry Eremin-Solenikov
Pass outer headers for outbound inline packets as const pointers.
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.
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
Signed-off-by: Dmitry Eremin-Solenikov
---
/** Email created from pull request 73 (lumag:cross-2)
** https://github.com/Linaro/odp/pull/73
** Patch: https://github.com/Linaro/odp/pull/73.patch
** Base sha: 7fc6d27e937b57b31360b07028388c811f8300dc
** Merge commit
Updates for crypto API as suggested by @psavol
github
/** Email created from pull request 74 (lumag:crypto-packet)
** https://github.com/Linaro/odp/pull/74
** Patch: https://github.com/Linaro/odp/pull/74.patch
** Base sha: ee5be324411a7520528a367967c28fc5
From: Dmitry Eremin-Solenikov
Following Maxim's idea use apt-get to download and install cross-packages.
Signed-off-by: Dmitry Eremin-Solenikov
---
/** Email created from pull request 73 (lumag:cross-2)
** https://github.com/Linaro/odp/pull/73
** Patch: https://github.com/Linaro/odp/pull/73.p
From: Dmitry Eremin-Solenikov
Signed-off-by: Dmitry Eremin-Solenikov
---
/** Email created from pull request 74 (lumag:crypto-packet)
** https://github.com/Linaro/odp/pull/74
** Patch: https://github.com/Linaro/odp/pull/74.patch
** Base sha: ee5be324411a7520528a367967c28fc529d3bc2e
** Merge
Per @muvarov's suggestion use apt-get instead of dpkg-cross to install
cross-packages
Add cross-compilation tests using clang
github
/** Email created from pull request 73 (lumag:cross-2)
** https://github.com/Linaro/odp/pull/73
** Patch: https://github.c
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_atomics_arm.h:18:51: error: value size does not match
regis
From: Dmitry Eremin-Solenikov
It is easy to update PREDEFINED definitions in one Doxyfile, while
forgetting about the others, as happened e.g. with ODP_DEPRECATE(x)
definition. Move PREDEFINED definitions into Doxyfile_common, so that it
is included by the rest of Doxyfiles.
Signed-off-by: Dmitr
https://bugs.linaro.org/show_bug.cgi?id=3126
Bug ID: 3126
Summary: IPC pktio test fails with taskset -c 1-2
Product: OpenDataPlane - linux- generic reference
Version: master
Hardware: Other
OS: Linux
Status: UNCONFI
I'd like we sync up on key elements of the document at the arch call today.
I'd like to restate a few things:
- ODP Cloud single binary should support multiple hardware environments
through DDF
- ODP and ODP applications should accommodate how HW uses memory on the
receive side, not the oppsoite
On 12.07.2017 12:31, Peltonen, Janne (Nokia - FI/Espoo) wrote:
> Dmitry Eremin-Solenikov wrote:
>> On 11.07.2017 15:31, Petri Savolainen wrote:
>>> Added a parameter for passing salt for AES GCM. Currently,
>>> only option for length is 4 bytes, but later on other algorithms
>>> may need more/less
Dmitry Eremin-Solenikov wrote:
> On 11.07.2017 15:31, Petri Savolainen wrote:
> > Added a parameter for passing salt for AES GCM. Currently,
> > only option for length is 4 bytes, but later on other algorithms
> > may need more/less salt data.
> >
> > Signed-off-by: Petri Savolainen
> > ---
> > i
From: Dmitry Eremin-Solenikov
loop interface can get any kind of packets, but should output only basic
packets. Reset packet subtype on loopback.
Signed-off-by: Dmitry Eremin-Solenikov
---
/** Email created from pull request 68 (lumag:packet-loop)
** https://github.com/Linaro/odp/pull/68
** P
loop interface can get any kind of packets, but should output only basic
packets. Reset packet subtype on loopback.
Signed-off-by: Dmitry Eremin-Solenikov dmitry.ereminsoleni...@linaro.org
github
/** Email created from pull request 68 (lumag:packet-loop)
**
On 11.07.2017 15:31, Petri Savolainen wrote:
> Added a parameter for passing salt for AES GCM. Currently,
> only option for length is 4 bytes, but later on other algorithms
> may need more/less salt data.
>
> Signed-off-by: Petri Savolainen
Except the suggestion for PATCH 1/9, the rest of the pa
Patch is rebased. Please help to review.
Thanks
Kevin
2017-07-11 18:00 GMT+08:00 Yi He :
> Yes, thanks, Maxim, and Petri please wait for Kevin to rebase and work out
> a new patch series.
>
> Best Regards, Yi
>
> On 11 July 2017 at 17:53, Maxim Uvarov wrote:
>
> > Yi, I need Perti review to mer
Signed-off-by: Kevin Wang
---
doc/users-guide/users-guide-timer.adoc | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/doc/users-guide/users-guide-timer.adoc
b/doc/users-guide/users-guide-timer.adoc
index 9cd30de..4a8bad7 100644
--- a/doc/users-guide/users-guide-timer.adoc
++
Use odp_timer_capability() to determine the max timer resolution
Signed-off-by: Kevin Wang
---
example/generator/odp_generator.c | 8 +++-
example/timer/odp_timer_simple.c | 9 -
example/timer/odp_timer_test.c| 8 ++--
test/common_plat/
Implement a new internal function timer_res_init() to detect the max
timer resolution without overrun at the ODP init stage. It will check
timer resolution from 1ms to 100us, 10us...1ns until the timer is
overrun.
Signed-off-by: Kevin Wang
---
platform/linux-generic/odp_timer.c | 98
Currently, user needs to decide the timer resolution before creating
a timer pool. But sometimes it will cause timer overrun as the system
can't support such high resolution.
So a new API is required to expose the timer capability to the user.
Signed-off-by: Kevin Wang
---
include/odp/api/spec/t
On 11.07.2017 15:31, Petri Savolainen wrote:
> Added a parameter for passing salt for AES GCM. Currently,
> only option for length is 4 bytes, but later on other algorithms
> may need more/less salt data.
>
> Signed-off-by: Petri Savolainen
> ---
> include/odp/api/spec/ipsec.h | 10 ++
>
Fine with me.
BR,
Bogdan
On 11 July 2017 at 15:31, Petri Savolainen wrote:
> Move direction (inboud/outbound) specific SA parameters
> into separate structures. Specification is more clear
> between common and direction specific SA parameters.
>
> Signed-off-by: Petri Savolainen
> Signed-off-by
80 matches
Mail list logo