Kevin Wang (1):
Relax the assert check with enqueue fails in the scheduler test case
test/common_plat/validation/api/scheduler/scheduler.c | 1 -
1 file changed, 1 deletion(-)
--
2.7.4
In the scalable scheduler, queue is implemented by the ring buffer.
It is possible enqueue would fail if the ring buffer is full.
So just remove the assert() check.
Signed-off-by: Kevin Wang
---
test/common_plat/validation/api/scheduler/scheduler.c | 1 -
1 file changed, 1 deletion(-)
diff --gi
Ping.
> -Original Message-
> From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of Petri
> Savolainen
> Sent: Monday, April 03, 2017 1:11 PM
> To: lng-odp@lists.linaro.org
> Subject: [lng-odp] [PATCH v3 1/2] api: queue: added queue size param
>
> Added capability informatio
See my patch series: [PATCH v3 1/2] api: queue: added queue size param
> -Original Message-
> From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of
> Honnappa Nagarahalli
> Sent: Friday, April 07, 2017 7:07 AM
> To: lng-odp@lists.linaro.org
> Subject: [lng-odp] [PATCH 1/2]
If the scheduler sync type is atomic or ordered,
need to release the context.
Signed-off-by: Kevin Wang
Reviewed-by: Ola Liljedahl
---
.../common_plat/validation/api/scheduler/scheduler.c | 20 +++-
1 file changed, 11 insertions(+), 9 deletions(-)
diff --git a/test/common_plat/
First, should this patch be a part of a series with a cover letter subject
"Avoid shm namespace collisions and allow shmblock per queue". Now
everything is just separate patches - including the cover letter. Please, use
format-patch command to create the series.
> -Original Message-
Hi, Kevin
1) single patch (bug fix) may not require cover letter (community please
suggest if I misunderstands).
2) submit message typically composed like:
linux-gen: pktio: loop:
3) then when you git format-patch you will get email subject reflect the
first line
[lng-odp]
> -Original Message-
> From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of Kevin
> Wang
> Sent: Friday, April 07, 2017 11:07 AM
> To: lng-odp@lists.linaro.org
> Cc: Kevin Wang
> Subject: [lng-odp] [PATCH] validation: scheduler: Release context before
> the end of the sch
1. There is just a single patch, so maybe cover letter is not needed. Will
correct it in the next patchset.
2. I may understand your concern. So I'll see if we can just less shm block
for multi-queue.
3. We create shm block in pool using this name,and if we use this same name
to create the queue wh
1.Release context is just to be added for scalable scheduler in
scheduler_test_groups(). I think it does no harms to other scheduler here.
2.This code is to be removed for the scalable scheduler, We use ring buffer
to implement the queue. So it is possible the enqueue operation failed if
the ring
Yes, will correct it in the next patch set.
Kevin
2017-04-07 16:16 GMT+08:00 Yi He :
> Hi, Kevin
>
> 1) single patch (bug fix) may not require cover letter (community please
> suggest if I misunderstands).
> 2) submit message typically composed like:
> linux-gen: pktio: loop:
>
>
Update ABI spec with the new IPsec event type.
Signed-off-by: Janne Peltonen
---
include/odp/arch/default/api/abi/event.h | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/include/odp/arch/default/api/abi/event.h
b/include/odp/arch/default/api/abi/event.h
index ab14157..872
Signed-off-by: Nikhil Agarwal
---
include/odp/api/spec/ipsec.h | 62 +++-
1 file changed, 26 insertions(+), 36 deletions(-)
diff --git a/include/odp/api/spec/ipsec.h b/include/odp/api/spec/ipsec.h
index b3dc0ca..d39e65d 100644
--- a/include/odp/api/spec/ip
We are sharing an Overview document of patch V1 based comp interface
https://docs.google.com/document/d/1aaEo8oTvZvm2096GJzl165L571tcsKqfcaD5yypiiCA/edit?usp=sharing
. It outlines possible applications, expected API usage and few key points.
Please give your suggestions/comment w.r.t this to he
On 07.04.2017 13:00, Nikhil Agarwal wrote:
> +typedef union odp_ipsec_protocols_t {
> + /** Cipher algorithms */
> + struct {
> + /** ODP_IPSEC_ESP */
> + uint32_t esp: 1;
> +
> + /** ODP_IPSEC_AH */
> + uint32_t ah : 1;
Smal
-Original Message-
From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of Nikhil
Agarwal
Sent: 07 April 2017 15:30
To: lng-odp@lists.linaro.org
Subject: [lng-odp] [API-NEXT] API: IPSEC: Updating ipsec APIs to support sNIC
implementation.
Signed-off-by: Nikhil Agarwal
---
Reviewed-and-tested-by: Petri Savolainen
Currently make check (api-next) fails on the parser test (tap pktio) in my
system. This patch corrects that. Please merge soon.
-Petri
> -Original Message-
> From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of
> Matias Elo
> Se
Hi Nikhil,
Let me see if I understand the limitations:
Inbound IPsec processing - pkt/status destination may be:
For lookaside async mode
- SA queue: OK for pkts, OK for status
- pktio queue: not applicable
- COS: OK (for pkts)
For inline mode: -
- SA queue: NOK for pkts, OK for s
From: Kevin Wang [mailto:kevin.w...@linaro.org]
Sent: Friday, April 07, 2017 11:45 AM
To: Savolainen, Petri (Nokia - FI/Espoo)
Cc: Kevin Wang ; lng-odp@lists.linaro.org
Subject: Re: [lng-odp] [PATCH] validation: scheduler: Release context before
the end of the scheduler test
1.Release context
On 7 April 2017 at 14:06, Savolainen, Petri (Nokia - FI/Espoo) <
petri.savolai...@nokia-bell-labs.com> wrote:
>
>
> From: Kevin Wang [mailto:kevin.w...@linaro.org]
> Sent: Friday, April 07, 2017 11:45 AM
> To: Savolainen, Petri (Nokia - FI/Espoo) <
petri.savolai...@nokia-bell-labs.com>
> Cc: Kevin
Lookup modes are per SA and thus may be used in a mix. Spi_overlap
parameter is global. So, removed comment about ignoring it in
LOOKUP_SPI mode.
Signed-off-by: Petri Savolainen
---
include/odp/api/spec/ipsec.h | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/include/odp/ap
On 7 April 2017 at 01:33, Bill Fischofer wrote:
> On Thu, Apr 6, 2017 at 1:51 PM, Maxim Uvarov
> wrote:
> > On 04/06/17 13:35, Ola Liljedahl wrote:
> >> On 5 April 2017 at 23:39, Maxim Uvarov wrote:
> >>> On 04/05/17 17:30, Ola Liljedahl wrote:
> On 5 April 2017 at 14:50, Maxim Uvarov
> w
On Fri, Apr 7, 2017 at 7:35 AM, Petri Savolainen
wrote:
> Lookup modes are per SA and thus may be used in a mix. Spi_overlap
> parameter is global. So, removed comment about ignoring it in
> LOOKUP_SPI mode.
>
> Signed-off-by: Petri Savolainen
Reviewed-by: Bill Fischofer
> ---
> include/odp/a
On Fri, Apr 7, 2017 at 4:54 AM, Janne Peltonen wrote:
>
> Update ABI spec with the new IPsec event type.
>
> Signed-off-by: Janne Peltonen
Reviewed-by: Bill Fischofer
> ---
> include/odp/arch/default/api/abi/event.h | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/in
Hi All,
We have a project to adapt an existed firewall implementation to ODP framework.
The current implementation needs to generate massive number of timer which link
to each flow element. So there is requirement to run up to a million of timers
to trigger the flow event.
An straightforward s
On 6 April 2017 at 20:51, Maxim Uvarov wrote:
> On 04/06/17 13:35, Ola Liljedahl wrote:
> > On 5 April 2017 at 23:39, Maxim Uvarov wrote:
> >> On 04/05/17 17:30, Ola Liljedahl wrote:
> >>> On 5 April 2017 at 14:50, Maxim Uvarov
> wrote:
> On 04/05/17 06:57, Honnappa Nagarahalli wrote:
> >>
I think it is a good suggestion. But I understood that in C++
the comma is not allowed until C++11 so I wonder if it is
ok to add it in ODP (in C it is ok from C99 on).
Perhaps someone can just add the comma when pushing the patch?
Janne
> -Original Message-
> From: Bill Fischofe
>From test/common_plat/validation/api/scheduler/scheduler.c
#define BUFS_PER_QUEUE_EXCL 1
The scalable scheduler has a default queue size of 4096 which seemed
reasonable to us. We can always increase the value but applications and
test prorgams should not expect infinite queue size. Perha
-Original Message-
From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of Verma,
Shally
Sent: Friday, April 07, 2017 4:45 PM
To: Nikhil Agarwal ; lng-odp@lists.linaro.org
Subject: Re: [lng-odp] [API-NEXT] API: IPSEC: Updating ipsec APIs to support
sNIC implementation.
-
CoS will be applicable in case of inline mode and packets will follow default
pktio CoS.
-Original Message-
From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of Bogdan
Pricope
Sent: Friday, April 07, 2017 5:10 PM
To: Verma, Shally
Cc: lng-odp@lists.linaro.org
Subject: R
On Fri, Apr 7, 2017 at 9:41 AM, Peltonen, Janne (Nokia - FI/Espoo)
wrote:
> I think it is a good suggestion. But I understood that in C++
> the comma is not allowed until C++11 so I wonder if it is
> ok to add it in ODP (in C it is ok from C99 on).
Given that ODP "accommodates" C++ rather than su
Hi,
> > Shally - enque in order-of-completion or in-order-submission? What does
> > this mean in-
> > context of ipsec?
> > In general, there is bit of confusion when we are using terms async in ODP
> > context here.
> > It imply that queue is used to output events. an async implementation can
Thanks for your note Mario. As an aside, please subscribe to the ODP
mailing list otherwise I have to approve each post you make manually,
which will only add delay. Please see
https://lists.linaro.org/mailman/listinfo/lng-odp for that. We also
have a weekly public call on Tuesdays at 15:00 UTC (th
On 7 April 2017 at 03:12, Savolainen, Petri (Nokia - FI/Espoo)
wrote:
> First, should this patch be a part of a series with a cover letter subject
> "Avoid shm namespace collisions and allow shmblock per queue". Now
> everything is just separate patches - including the cover letter. Plea
On 7 April 2017 at 10:33, Bill Fischofer wrote:
> Thanks for your note Mario. As an aside, please subscribe to the ODP
> mailing list otherwise I have to approve each post you make manually,
> which will only add delay. Please see
> https://lists.linaro.org/mailman/listinfo/lng-odp for that. We al
On 7 April 2017 at 07:29, Ola Liljedahl wrote:
> On 7 April 2017 at 14:06, Savolainen, Petri (Nokia - FI/Espoo) <
> petri.savolai...@nokia-bell-labs.com> wrote:
>>
>>
>> From: Kevin Wang [mailto:kevin.w...@linaro.org]
>> Sent: Friday, April 07, 2017 11:45 AM
>> To: Savolainen, Petri (Nokia - FI/Es
Branch: refs/heads/api-next
Home: https://github.com/Linaro/odp
Commit: 38485719c028918b019c6a5fc67cf05053421c83
https://github.com/Linaro/odp/commit/38485719c028918b019c6a5fc67cf05053421c83
Author: Petri Savolainen
Date: 2017-04-07 (Fri, 07 Apr 2017)
Changed paths:
M i
Merged this patch.
Maxim.
On 04/07/17 15:47, Bill Fischofer wrote:
> On Fri, Apr 7, 2017 at 7:35 AM, Petri Savolainen
> wrote:
>> Lookup modes are per SA and thus may be used in a mix. Spi_overlap
>> parameter is global. So, removed comment about ignoring it in
>> LOOKUP_SPI mode.
>>
>> Signed-o
I forget to note that it will be great to update .travis file for
testing new option.
Maxim.
On 04/04/17 21:47, Brian Brooks wrote:
> This work derives from Ola Liljedahl's prototype [1] which introduced a
> scalable scheduler design based on primarily lock-free algorithms and
> data structures d
On 7 April 2017 at 08:40, Peltonen, Janne (Nokia - FI/Espoo) <
janne.pelto...@nokia.com> wrote:
> Hi,
>
> On Thu, Apr 6, 2017 at 1:46 PM, Bill Fischofer
> wrote:
> > On Thu, Apr 6, 2017 at 1:32 PM, Ola Liljedahl
> wrote:
> > > On 6 April 2017 at 13:48, Jerin Jacob
> wrote:
>
> > >> We see ORDER
On 6 April 2017 at 13:59, Petri Savolainen wrote:
>
> User may give number of scheduling groups to test
> scheduler performance with other that the default (all
> threads) group. Both pktios and threads are allocated
Isn't all *workers* a better default scheduler group? In this and in
other examp
41 matches
Mail list logo