There is no big difference when to assign default CoS to pktio,
but in usual case it's done before creating any chain, if it's
needed to be also checked it can be done in separate test.
Signed-off-by: Ivan Khoronzhuk
---
.../classification/odp_classification_test_pm
The classification tests should use supported l3 PMR where it's
not important. For validating concrete PMRs the separate test exists.
Signed-off-by: Ivan Khoronzhuk
---
test/validation/classification/classification.h| 6 +--
.../classification/odp_classification_basic.c
This patch series corrects classification tests to be a little bit
adoptive.
Ivan Khoronzhuk (5):
validation: cls: adopt for supported l3 PMR
validation: cls: assign default CoS before creating chain
validation: cls: test_pmr: don't create default input queue
validation: cls: use co
In places of two conversions the error can be up to
2 resolutions, so increase validation range to 2 res.
Reviewed-by: Bill Fischofer
Signed-off-by: Ivan Khoronzhuk
---
test/validation/time/time.c | 20 ++--
1 file changed, 10 insertions(+), 10 deletions(-)
diff --git a/test
While division the resolution can have fractional part, so better to
round it up, when it's needed.
Reviewed-by: Bill Fischofer
Signed-off-by: Ivan Khoronzhuk
---
test/validation/time/time.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/test/validation/time/tim
From: Nicolas Morey-Chaisemartin
Computation were done on a local variable and never stored back
to the global value so both local_res and global_res were always 0.
Reviewed-by: Ivan Khoronzhuk
Signed-off-by: Ivan Khoronzhuk
Signed-off-by: Nicolas Morey-Chaisemartin
---
test/validation/time
This series corrects limits based on resolution.
Since v2:
- remove return
Since v1:
- no functional changes
- replaceed "[lng-odp] [PATCH 1/3] validation: time: initialize resolution vars"
on "validation: time: store local and global resolution" from Nicolas
Ivan Khoronzh
sent v5
On 04.01.16 17:35, Ivan Khoronzhuk wrote:
This test example uses linux helper functions to schedule worker
threads in order to check global time monotony between the threads.
Each thread receives event from it's own queue, checks global time on
correctness, writes in the buffe
ng.
Signed-off-by: Ivan Khoronzhuk
Reviewed-and-tested-by: Bill Fischofer
---
Since v4:
- convert time to ns only while printing log
Since v3:
- defined log entry size
Since v2:
-added binary to .gitignore
Since v1:
- replaced many includes on odp.h
- corrected a little error path
- increased nu
In places of two conversions the error can be up to
2 resolutions, so increase validation range to 2 res.
Reviewed-by: Bill Fischofer
Signed-off-by: Ivan Khoronzhuk
---
test/validation/time/time.c | 20 ++--
1 file changed, 10 insertions(+), 10 deletions(-)
diff --git a/test
While division the resolution can have fractional part, so better to
round it up, when it's needed.
Reviewed-by: Bill Fischofer
Signed-off-by: Ivan Khoronzhuk
---
test/validation/time/time.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/test/validation/time/tim
From: Nicolas Morey-Chaisemartin
Computation were done on a local variable and never stored back
to the global value so both local_res and global_res were always 0.
Reviewed-by: Ivan Khoronzhuk
Signed-off-by: Ivan Khoronzhuk
Signed-off-by: Nicolas Morey-Chaisemartin
---
test/validation/time
This series corrects limits based on resolution.
Since v1:
- no functional changes
- replaceed "[lng-odp] [PATCH 1/3] validation: time: initialize resolution vars"
on "validation: time: store local and global resolution" from Nicolas
Ivan Khoronzhuk (2):
validation: time:
I've included this patch with my reviewed-by in v2 of initial series as
it contains replacement on your patch:
"[lng-odp] [PATCH 0/3 v2] correct limits validation"
On 06.01.16 19:05, Ivan Khoronzhuk wrote:
It was in SPAM (why? :-|), strange.
On 06.01.16 18:31, Nicolas Mor
: Ivan Khoronzhuk
---
test/validation/time/time.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/test/validation/time/time.c b/test/validation/time/time.c
index 8b469b0..b4e53ec 100644
--- a/test/validation/time/time.c
+++ b/test/validation/time/time.c
@@ -31,7
Oh, sorry. It's your patch. It's better.
Ok, will delete it from series and rebase on it.
Reviewed-by: Ivan Khoronzhuk
but I cannot see this patch in my mail box. strange.
On 06.01.16 18:49, Ivan Khoronzhuk wrote:
Computation were done on a local variable and never stored bac
ersions come back to the same value */
--
2.6.3.372.gcb93895
___
lng-odp mailing list
lng-odp@lists.linaro.org
https://lists.linaro.org/mailman/listinfo/lng-odp
--
Regards,
Ivan Khoronzhuk
___
lng-odp mailing
hard for implementors and it can be
tuned if smth was missed.
On 06.01.16 17:09, Nicolas Morey-Chaisemartin wrote:
Just realized that resolution is a frequency, not a period. My bad !
Nicolas
On 01/06/2016 04:03 PM, Nicolas Morey-Chaisemartin wrote:
On 12/17/2015 04:29 PM, Ivan Khoronzhuk w
ignore this. see series "correct validation limits"
On 05.01.16 16:41, Ivan Khoronzhuk wrote:
The resolution vars were not initialized at init, as result the resolution
is ideal.
Signed-off-by: Ivan Khoronzhuk
---
test/validation/time/time.c | 7 ---
1 file changed, 4 insert
In places of two conversions the error can be up to
2 resolutions, so increase validation range to 2 res.
Signed-off-by: Ivan Khoronzhuk
---
test/validation/time/time.c | 20 ++--
1 file changed, 10 insertions(+), 10 deletions(-)
diff --git a/test/validation/time/time.c b/test
While division the resolution can have fractional part, so better to
round it up, when it's needed.
Signed-off-by: Ivan Khoronzhuk
---
test/validation/time/time.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/test/validation/time/time.c b/test/validation/time/t
The resolution vars were not initialized at init, as result the resolution
is ideal. This patch corrects it, to use real resolution.
Signed-off-by: Ivan Khoronzhuk
---
test/validation/time/time.c | 7 ---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/test/validation/time
This series corrects limits based on resolution.
Ivan Khoronzhuk (3):
validation: time: initialize resolution vars
validation: time: round up resolution
validation: time: increase limit to check to 2 res
test/validation/time/time.c | 31 +--
1 file changed, 17
The resolution vars were not initialized at init, as result the resolution
is ideal.
Signed-off-by: Ivan Khoronzhuk
---
test/validation/time/time.c | 7 ---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/test/validation/time/time.c b/test/validation/time/time.c
index 8b469b0
sent v4 with no functional changes.
On 04.01.16 16:41, Ivan Khoronzhuk wrote:
Since v2:
-added binary to .gitignore
Since v1:
- replaced many includes on odp.h
- corrected a little error path
- increased num of iterations
- rebased on ODPv1.6
--
Regards,
Ivan Khoronzhuk
ng.
Signed-off-by: Ivan Khoronzhuk
Reviewed-and-tested-by: Bill Fischofer
---
Since v3:
- defined log entry size
Since v2:
-added binary to .gitignore
Since v1:
- replaced many includes on odp.h
- corrected a little error path
- increased num of iterations
- rebased on ODPv1.6
ng.
Signed-off-by: Ivan Khoronzhuk
Reviewed-and-tested-by: Bill Fischofer
---
Since v2:
-added binary to .gitignore
Since v1:
- replaced many includes on odp.h
- corrected a little error path
- increased num of iterations
- rebased on ODPv1.6
configure.ac| 1 +
exam
sent v3 with your reviewed-by.
On 03.01.16 15:11, Bill Fischofer wrote:
You need to create a .gitignore file in example/time and add the output binary
(odp_global_time_test) to it for this patch to be complete. Other than that:
On Thu, Dec 31, 2015 at 5:46 AM, Ivan Khoronzhuk
ng.
Signed-off-by: Ivan Khoronzhuk
---
Since v2:
- replaced many includes on odp.h
- corrected a little error path
- increased num of iterations
- rebased on ODPv1.6
configure.ac| 1 +
example/Makefile.am | 2 +-
example/time/Makefile.am| 10 +
It's for master but with global time API.
On 24.12.15 15:54, Maxim Uvarov wrote:
is it for master or api-next?
Maxim.
On 12/23/2015 16:03, Ivan Khoronzhuk wrote:
This test example uses linux helper functions to schedule worker
threads in order to check global time monotony betwee
Reviewed-by: Bill Fischofer
Signed-off-by: Ivan Khoronzhuk
---
doc/users-guide/users-guide.adoc | 29 -
1 file changed, 28 insertions(+), 1 deletion(-)
diff --git a/doc/users-guide/users-guide.adoc b/doc/users-guide/users-guide.adoc
index 7ec7957..8bc8521 100644
On 23.12.15 15:28, Bill Fischofer wrote:
On Wed, Dec 23, 2015 at 7:24 AM, Bill Fischofer mailto:bill.fischo...@linaro.org>> wrote:
On Wed, Dec 23, 2015 at 6:32 AM, Ivan Khoronzhuk mailto:ivan.khoronz...@linaro.org>> wrote:
Signed-off-by: Ivan Khoronzhuk mailto:
eiving.
Signed-off-by: Ivan Khoronzhuk
---
configure.ac| 1 +
example/Makefile.am | 2 +-
example/time/Makefile.am| 10 ++
example/time/time_global_test.c | 360
4 files changed, 372 insertions(+), 1 deletion(
Signed-off-by: Ivan Khoronzhuk
---
doc/users-guide/users-guide.adoc | 29 -
1 file changed, 28 insertions(+), 1 deletion(-)
diff --git a/doc/users-guide/users-guide.adoc b/doc/users-guide/users-guide.adoc
index 7ec7957..e8465d0 100644
--- a/doc/users-guide/users
On 23.12.15 05:38, Bill Fischofer wrote:
On Mon, Dec 21, 2015 at 6:53 AM, Ivan Khoronzhuk mailto:ivan.khoronz...@linaro.org>> wrote:
Signed-off-by: Ivan Khoronzhuk mailto:ivan.khoronz...@linaro.org>>
---
doc/users-guide/users-gui
Test checks scheduler time correctness using time API.
Signed-off-by: Ivan Khoronzhuk
---
Based on api-next as it uses new time API.
Since v2:
- replaced 2 on (20 * ODP_TIME_MSEC_IN_NS)
Since v1:
- improved a little
test/validation/scheduler/scheduler.c | 45
On 22.12.15 10:34, Savolainen, Petri (Nokia - FI/Espoo) wrote:
-Original Message-
From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of EXT
Ivan Khoronzhuk
Sent: Monday, December 21, 2015 1:02 PM
To: lng-odp@lists.linaro.org
Subject: Re: [lng-odp] [API-NEXT PATCH v2
Signed-off-by: Ivan Khoronzhuk
---
doc/users-guide/users-guide.adoc | 29 -
1 file changed, 28 insertions(+), 1 deletion(-)
diff --git a/doc/users-guide/users-guide.adoc b/doc/users-guide/users-guide.adoc
index 7ec7957..b11d050 100644
--- a/doc/users-guide/users
Mostly global time API is tested in the same way, so reuse
existent functions.
Signed-off-by: Ivan Khoronzhuk
---
test/validation/time/time.c | 189 +---
test/validation/time/time.h | 26 +++---
2 files changed, 159 insertions(+), 56 deletions(-)
diff
time
- typedef callbacks in validation tests
- test wait_until for local and global timestamps
Ivan Khoronzhuk (2):
api: time: add global time API
validation: time: update to tes global time API
include/odp/api/time.h| 30 ++
platform/linux-generic/odp_time.c | 34 +--
This patch adds global time API and implements it in linux-generic.
Signed-off-by: Ivan Khoronzhuk
---
include/odp/api/time.h| 30 ++
platform/linux-generic/odp_time.c | 34 +++---
2 files changed, 57 insertions(+), 7 deletions
sent v2.
On 18.12.15 17:34, Mike Holmes wrote:
On 18 December 2015 at 09:00, Ivan Khoronzhuk mailto:ivan.khoronz...@linaro.org>> wrote:
ping
On 03.12.15 14:58, Ivan Khoronzhuk wrote:
Signed-off-by: Ivan Khoronzhuk mailto:ivan.khoronz...@lina
Signed-off-by: Ivan Khoronzhuk
---
Since v1:
- corrected error message
platform/linux-generic/odp_init.c | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/platform/linux-generic/odp_init.c
b/platform/linux-generic/odp_init.c
index 5e19d86..4616e03 100644
--- a/platform
ping.
On 17.12.15 18:38, Ivan Khoronzhuk wrote:
Test checks scheduler time correctness using time API.
Signed-off-by: Ivan Khoronzhuk
---
Based on api-next as it uses new time API.
Since v1:
- improved a little
test/validation/scheduler/scheduler.c | 45
On 21.12.15 10:50, Savolainen, Petri (Nokia - FI/Espoo) wrote:
-Original Message-
From: EXT Ivan Khoronzhuk [mailto:ivan.khoronz...@linaro.org]
Sent: Friday, December 18, 2015 2:14 PM
To: lng-odp@lists.linaro.org
Cc: Savolainen, Petri (Nokia - FI/Espoo); Ivan Khoronzhuk
Subject
On 21.12.15 10:39, Savolainen, Petri (Nokia - FI/Espoo) wrote:
Otherwise OK, but I'd highlight that ability to *share* time stamps is the main
difference of global vs. local. See below.
-Original Message-
From: EXT Ivan Khoronzhuk [mailto:ivan.khoronz...@linaro.org]
Sent: F
On 18.12.15 17:34, Mike Holmes wrote:
On 18 December 2015 at 09:00, Ivan Khoronzhuk mailto:ivan.khoronz...@linaro.org>> wrote:
ping
On 03.12.15 14:58, Ivan Khoronzhuk wrote:
Signed-off-by: Ivan Khoronzhuk mailto:ivan.khoronz...@linaro.org>>
---
ping
On 03.12.15 14:58, Ivan Khoronzhuk wrote:
Signed-off-by: Ivan Khoronzhuk
---
platform/linux-generic/odp_init.c | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/platform/linux-generic/odp_init.c
b/platform/linux-generic/odp_init.c
index 5e19d86..087bfdf 100644
Signed-off-by: Ivan Khoronzhuk
---
doc/users-guide/users-guide.adoc | 15 ++-
1 file changed, 14 insertions(+), 1 deletion(-)
diff --git a/doc/users-guide/users-guide.adoc b/doc/users-guide/users-guide.adoc
index 7ec7957..d2b5480 100644
--- a/doc/users-guide/users-guide.adoc
+++ b
This patch add global time API and implements it in linux-generic.
Signed-off-by: Ivan Khoronzhuk
---
include/odp/api/time.h| 28
platform/linux-generic/odp_time.c | 34 +++---
2 files changed, 55 insertions(+), 7 deletions
Mostly global time API is tested in the same way, so reuse
existent functions.
Signed-off-by: Ivan Khoronzhuk
---
test/validation/time/time.c | 160 +---
test/validation/time/time.h | 23 ---
2 files changed, 137 insertions(+), 46 deletions(-)
diff
This series adds global time API and it's validation test.
Based on:
[lng-odp] [API-NEXT PATCH v5 0/6] api: time: add rate and delay APIs
https://lists.linaro.org/pipermail/lng-odp/2015-December/018678.html
Ivan Khoronzhuk (2):
api: time: add global time API
validation: time: update t
This patch adds validation test for odp_time_wait_ns/until().
Also it visually demonstrates 5 second count.
Signed-off-by: Ivan Khoronzhuk
---
test/validation/time/time.c | 53 +
test/validation/time/time.h | 2 ++
2 files changed, 55 insertions
Message-
From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of EXT
Ivan Khoronzhuk
Sent: Thursday, December 17, 2015 6:31 PM
To: lng-odp@lists.linaro.org
Subject: [lng-odp] [API-NEXT PATCH v4 1/6] api: time: add resolution and
wait API calls
This patch adds odp_time_local_res() and
Signed-off-by: Ivan Khoronzhuk
---
test/validation/time/time.c | 4
1 file changed, 4 insertions(+)
diff --git a/test/validation/time/time.c b/test/validation/time/time.c
index 4040b9f..cef 100644
--- a/test/validation/time/time.c
+++ b/test/validation/time/time.c
@@ -50,6 +50,10
This function is used to get resolution, test uses it to
check other APIs.
Signed-off-by: Ivan Khoronzhuk
---
test/validation/time/time.c | 46 ++---
test/validation/time/time.h | 1 +
2 files changed, 32 insertions(+), 15 deletions(-)
diff --git a/test
Signed-off-by: Ivan Khoronzhuk
---
test/validation/time/time.c | 10 ++
test/validation/time/time.h | 1 +
2 files changed, 11 insertions(+)
diff --git a/test/validation/time/time.c b/test/validation/time/time.c
index 50d9bdc..4040b9f 100644
--- a/test/validation/time/time.c
+++ b/test
v2:
- replaced odp_time_wait() on odp_time_wait_ns() and odp_time_wait_until()
- corrected validation tests to validate new wait APIs
Since v1:
- used "wait" instead of "delay"
- used "res" instead of "rate"
- used inlined functions in odp_time_wait()
- cor
Now odp_time_wait_ns() can be used instead of own implementation.
Signed-off-by: Ivan Khoronzhuk
---
test/performance/odp_pktio_perf.c | 16 +---
1 file changed, 1 insertion(+), 15 deletions(-)
diff --git a/test/performance/odp_pktio_perf.c
b/test/performance/odp_pktio_perf.c
This patch adds odp_time_local_res() and odp_time_wait_until(),
odp_time_wait_ns() APIs.
Signed-off-by: Ivan Khoronzhuk
---
include/odp/api/time.h| 25 ++
platform/linux-generic/odp_time.c | 98 ++-
2 files changed, 101 insertions(+), 22
Petri,
This patch corrects validation for scheduler time we had talk about some time
ago.
And removes:
- wait_time = odp_schedule_wait_time((uint64_t)-1LL);
- CU_ASSERT(wait_time > 0);
as was decided.
On 17.12.15 18:38, Ivan Khoronzhuk wrote:
Test checks scheduler t
wait time.
The inline I'll align.
Don’t break function prototypes into multiple lines. See under.
-Original Message-
From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of EXT
Ivan Khoronzhuk
Sent: Thursday, December 17, 2015 6:31 PM
To: lng-odp@lists.linaro.org
Su
Example should free resources in right order when terminates.
Also it should have correct error path.
Signed-off-by: Ivan Khoronzhuk
---
Since v1:
Just rebase, no functional changes
example/timer/odp_timer_test.c | 46 ++
1 file changed, 38 insertions
v2 is present
On 17.12.15 18:20, Ivan Khoronzhuk wrote:
Test checks scheduler time correctness using time API.
Signed-off-by: Ivan Khoronzhuk
---
Patch is based on api-next, as it uses new time API.
test/validation/scheduler/scheduler.c | 47 +--
1 file
Test checks scheduler time correctness using time API.
Signed-off-by: Ivan Khoronzhuk
---
Based on api-next as it uses new time API.
Since v1:
- improved a little
test/validation/scheduler/scheduler.c | 45 +--
1 file changed, 43 insertions(+), 2 deletions
v4 is present
On 17.12.15 17:29, Ivan Khoronzhuk wrote:
This series adds odp_time_local_res() and odp_time_wait_ns(),
odp_time_wait_until() calls to time API and it's validation
tests and usage.
Since v2:
- replaced odp_time_wait() on odp_time_wait_ns() and odp_time_wait_until()
- corr
This patch adds validation test for odp_time_wait_ns/until().
Also it visually demonstrates 5 second count.
Signed-off-by: Ivan Khoronzhuk
---
test/validation/time/time.c | 53 +
test/validation/time/time.h | 2 ++
2 files changed, 55 insertions
This function is used to get resolution, test uses it to
check other APIs.
Signed-off-by: Ivan Khoronzhuk
---
test/validation/time/time.c | 46 ++---
test/validation/time/time.h | 1 +
2 files changed, 32 insertions(+), 15 deletions(-)
diff --git a/test
Now odp_time_wait_ns() can be used instead of own implementation.
Signed-off-by: Ivan Khoronzhuk
---
test/performance/odp_pktio_perf.c | 16 +---
1 file changed, 1 insertion(+), 15 deletions(-)
diff --git a/test/performance/odp_pktio_perf.c
b/test/performance/odp_pktio_perf.c
Signed-off-by: Ivan Khoronzhuk
---
test/validation/time/time.c | 4
1 file changed, 4 insertions(+)
diff --git a/test/validation/time/time.c b/test/validation/time/time.c
index 4040b9f..cef 100644
--- a/test/validation/time/time.c
+++ b/test/validation/time/time.c
@@ -50,6 +50,10
Signed-off-by: Ivan Khoronzhuk
---
test/validation/time/time.c | 10 ++
test/validation/time/time.h | 1 +
2 files changed, 11 insertions(+)
diff --git a/test/validation/time/time.c b/test/validation/time/time.c
index 50d9bdc..4040b9f 100644
--- a/test/validation/time/time.c
+++ b/test
This patch adds odp_time_local_res() and odp_time_wait_until(),
odp_time_wait_ns() APIs.
Signed-off-by: Ivan Khoronzhuk
---
include/odp/api/time.h| 21
platform/linux-generic/odp_time.c | 100 ++
2 files changed, 101 insertions(+), 20
ation tests to validate new wait APIs
Since v1:
- used "wait" instead of "delay"
- used "res" instead of "rate"
- used inlined functions in odp_time_wait()
- corrected new API descriptions
Ivan Khoronzhuk (6):
api: time: add resolution and wait API calls
Test checks scheduler time correctness using time API.
Signed-off-by: Ivan Khoronzhuk
---
Patch is based on api-next, as it uses new time API.
test/validation/scheduler/scheduler.c | 47 +--
1 file changed, 45 insertions(+), 2 deletions(-)
diff --git a/test
This patch adds validation test for odp_time_wait_ns/until().
Also it visually demonstrates 5 second count.
Signed-off-by: Ivan Khoronzhuk
---
test/validation/time/time.c | 52 +
test/validation/time/time.h | 2 ++
2 files changed, 54 insertions
This function is used to get resolution, test uses it to
check other APIs.
Signed-off-by: Ivan Khoronzhuk
---
test/validation/time/time.c | 46 ++---
test/validation/time/time.h | 1 +
2 files changed, 32 insertions(+), 15 deletions(-)
diff --git a/test
Signed-off-by: Ivan Khoronzhuk
---
test/validation/time/time.c | 4
1 file changed, 4 insertions(+)
diff --git a/test/validation/time/time.c b/test/validation/time/time.c
index 4040b9f..cef 100644
--- a/test/validation/time/time.c
+++ b/test/validation/time/time.c
@@ -50,6 +50,10
Signed-off-by: Ivan Khoronzhuk
---
test/validation/time/time.c | 10 ++
test/validation/time/time.h | 1 +
2 files changed, 11 insertions(+)
diff --git a/test/validation/time/time.c b/test/validation/time/time.c
index 50d9bdc..4040b9f 100644
--- a/test/validation/time/time.c
+++ b/test
Now odp_time_wait_ns() can be used instead of own implementation.
Signed-off-by: Ivan Khoronzhuk
---
test/performance/odp_pktio_perf.c | 16 +---
1 file changed, 1 insertion(+), 15 deletions(-)
diff --git a/test/performance/odp_pktio_perf.c
b/test/performance/odp_pktio_perf.c
This patch adds odp_time_local_res() and odp_time_wait_until(),
odp_time_wait_ns() APIs.
Signed-off-by: Ivan Khoronzhuk
---
include/odp/api/time.h| 21
platform/linux-generic/odp_time.c | 100 ++
2 files changed, 101 insertions(+), 20
used "wait" instead of "delay"
- used "res" instead of "rate"
- used inlined functions in odp_time_wait()
- corrected new API descriptions
Ivan Khoronzhuk (6):
api: time: add resolution and wait API calls
performance: pktio_perf: use odp_time_wait_ns()
already and no need to calculate relative time.
It can be hidden inside, for lazy users...
-Original Message-
From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of EXT
Ivan Khoronzhuk
Sent: Tuesday, December 15, 2015 2:29 PM
To: lng-odp@lists.linaro.org
Subject: [lng-odp
.
+
+pkt = receive_packet(&retqueue, ODP_TIME_SEC_IN_NS);
here has to be time of scheduler, ns should be converted to with
odp_schedule_wait_time() first
Maybe here is reason to check if the function is used correctly in other places.
Here probably should be one more patch that corr
CLS_PMR_SET] = odp_pool_create(poolname, &pool_param);
+ pool_list[CLS_PMR_SET] = pool_create(poolname);
CU_ASSERT_FATAL(pool_list[CLS_PMR_SET] != ODP_POOL_INVALID);
- retval = odp_cls_cos_pool_set(cos_list[CLS_PMR_SET],
- pool_list[CLS_P
Petri,
Are you OK with this series?
On 15.12.15 14:29, Ivan Khoronzhuk wrote:
This series adds odp_time_local_res() and odp_time_wait() calls to
time API and it's validation tests and usage.
Based on:
[lng-odp] [API-NEXT PATCHv4] linux-generic: time: remove posix bleed through on
odp_t
p_timer_test.c with 1 reject...
Hunk #1 applied cleanly.
Hunk #2 applied cleanly.
Rejected hunk #3.
Hunk #4 applied cleanly.
Hunk #5 applied cleanly.
Hunk #6 applied cleanly.
Patch failed at 0001 example: timer: free resources while termination
On Mon, Dec 14, 2015 at 9:29 AM, Ivan Khoronzhuk ma
On 16.12.15 05:30, Mike Holmes wrote:
On 15 December 2015 at 07:29, Ivan Khoronzhuk mailto:ivan.khoronz...@linaro.org>> wrote:
This series adds odp_time_local_res() and odp_time_wait() calls to
time API and it's validation tests and usage.
When do we start requiring t
ons can be independent of _POSIX_C_SOURCE
level.
Signed-off-by: Bill Fischofer
Reviewed-by: Ivan Khoronzhuk
---
.../linux-generic/include/odp/plat/time_types.h| 13 +++---
platform/linux-generic/odp_time.c | 30 +++---
2 files changed, 24 inserti
This patch adds validation test for odp_time_wait(). Also it visually
demonstrates 5 second count.
Signed-off-by: Ivan Khoronzhuk
---
test/validation/time/time.c | 26 ++
test/validation/time/time.h | 1 +
2 files changed, 27 insertions(+)
diff --git a/test/validation
Signed-off-by: Ivan Khoronzhuk
---
test/validation/time/time.c | 10 ++
test/validation/time/time.h | 1 +
2 files changed, 11 insertions(+)
diff --git a/test/validation/time/time.c b/test/validation/time/time.c
index 50d9bdc..4040b9f 100644
--- a/test/validation/time/time.c
+++ b/test
This function is used to get resolution, test uses it to
check other APIs.
Signed-off-by: Ivan Khoronzhuk
---
test/validation/time/time.c | 46 ++---
test/validation/time/time.h | 1 +
2 files changed, 32 insertions(+), 15 deletions(-)
diff --git a/test
Signed-off-by: Ivan Khoronzhuk
---
test/validation/time/time.c | 4
1 file changed, 4 insertions(+)
diff --git a/test/validation/time/time.c b/test/validation/time/time.c
index 4040b9f..cef 100644
--- a/test/validation/time/time.c
+++ b/test/validation/time/time.c
@@ -50,6 +50,10
This patch adds odp_time_local_res() and odp_time_wait() APIs.
Signed-off-by: Ivan Khoronzhuk
---
include/odp/api/time.h| 17 ++
platform/linux-generic/odp_time.c | 71 ++-
2 files changed, 72 insertions(+), 16 deletions(-)
diff --git a
Now odp_time_wait() can be used instead of own implementation.
Signed-off-by: Ivan Khoronzhuk
---
test/performance/odp_pktio_perf.c | 17 ++---
1 file changed, 2 insertions(+), 15 deletions(-)
diff --git a/test/performance/odp_pktio_perf.c
b/test/performance/odp_pktio_perf.c
index
ce v1:
- used "wait" instead of "delay"
- used "res" instead of "rate"
- used inlined functions in odp_time_wait()
- corrected new API descriptions
Ivan Khoronzhuk (6):
api: time: add resolution and wait API calls
performance: pktio_perf: use odp_time_wait()
On 15.12.15 10:49, Savolainen, Petri (Nokia - FI/Espoo) wrote:
-Original Message-
From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of EXT
Ivan Khoronzhuk
Sent: Monday, December 14, 2015 8:58 PM
To: lng-odp@lists.linaro.org
Subject: [lng-odp] [API-NEXT PATCH 1/6] api
On 14.12.15 21:15, Mike Holmes wrote:
On 3 December 2015 at 06:27, Ivan Khoronzhuk mailto:ivan.khoronz...@linaro.org>> wrote:
No need to include all headers in hashtable and lineartable.
rwlock and shared_memory is enough. It allows to see which odp
parts are used.
To ma
This series adds odp_time_local_rate() and odp_time_delay() calls to
time API and it's validation tests and usage.
Ivan Khoronzhuk (6):
api: time: add rate and delay API calls
performance: pktio_perf: use odp_time_delay() function instead of
looping
validation: time: test time cons
Signed-off-by: Ivan Khoronzhuk
---
test/validation/time/time.c | 10 ++
test/validation/time/time.h | 1 +
2 files changed, 11 insertions(+)
diff --git a/test/validation/time/time.c b/test/validation/time/time.c
index 50d9bdc..4040b9f 100644
--- a/test/validation/time/time.c
+++ b/test
Signed-off-by: Ivan Khoronzhuk
---
test/validation/time/time.c | 4
1 file changed, 4 insertions(+)
diff --git a/test/validation/time/time.c b/test/validation/time/time.c
index 4040b9f..cef 100644
--- a/test/validation/time/time.c
+++ b/test/validation/time/time.c
@@ -50,6 +50,10
101 - 200 of 676 matches
Mail list logo