as
And I think Bill Fischofer knew about this while adding
this
,at first glance, strange union, right Bill?
Yes, it's not the best decision from style point of view, but it's fast and in
case of
an error
is supposed to be caught by time validation tests.
--
Rega
On 16.05.16 17:18, Maxim Uvarov wrote:
On 05/16/16 17:00, Ivan Khoronzhuk wrote:
Hi Maxim,
I know it's a little late, but printf was here not for testing wait until.
It was here to expose visually time for a second. As ODP doesn't have
other accurate enough time source the printf was added
}
return ret;
}
--
Regards,
Ivan Khoronzhuk
___
lng-odp mailing list
lng-odp@lists.linaro.org
https://lists.linaro.org/mailman/listinfo/lng-odp
t;Exceed lower limit\n");
+ }
+
+ if (odp_time_cmp(diff, upper_limit) > 0) {
+ fprintf(stderr, "Exceed upper limit: "
+ "diff is %" PRIu64 ", upper_limit %" PRIu64 "\n",
+ odp_time_to
on it while configuration.
Regards,usage
Zoltan
___
lng-odp mailing list
lng-odp@lists.linaro.org
https://lists.linaro.org/mailman/listinfo/lng-odp
--
Regards,
Ivan Khoronzhuk
___
lng-odp mailing list
lng-odp
CPU. There should be enough
head room to keep loop duration over 4 seconds even with
higher CPU frequencies.
Signed-off-by: Petri Savolainen <petri.savolai...@nokia.com>
Reviewed-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org>
But, it needs to be revised when main thread w
t_until(void);
void time_test_wait_ns(void);
void time_test_local_to_u64(void);
void time_test_global_to_u64(void);
+void time_test_monotony(void);
/* test arrays: */
extern odp_testinfo_t time_suite_time[];
--
Regards,
Ivan Khoronzhuk
___
lng-odp mailing list
lng-odp@lists.linaro.org
https://lists.linaro.org/mailman/listinfo/lng-odp
Hi Bill,
What about to review this one?
On 16.02.16 16:02, Ivan Khoronzhuk wrote:
When test for single thread is finished the following test for
many threads can be started, and for some implementations can
happen that future one event can arrive to main thread, as it was
requested in previous
EXT Ivan Khoronzhuk [mailto:ivan.khoronz...@linaro.org]
Sent: Thursday, April 14, 2016 5:39 PM
To: Savolainen, Petri (Nokia - FI/Espoo) <petri.savolai...@nokia.com>;
lng-odp@lists.linaro.org
Subject: Re: [lng-odp] [PATCH] validation: time: shorten test duration
What about to do smth like below
s.linaro.org/mailman/listinfo/lng-odp
___
lng-odp mailing list
lng-odp@lists.linaro.org
https://lists.linaro.org/mailman/listinfo/lng-odp
--
Regards,
Ivan Khoronzhuk
___
lng-odp mailing list
lng-odp@li
iff = odp_cpu_cycles_diff(c2, c1);
loop_time += cycle_diff / odp_cpu_hz();
c1 = c2;
};
On 14.04.16 17:24, Ivan Khoronzhuk wrote:
On 14.04.16 17:15, Savolainen, Petri (Nokia - FI/Espoo) wrote:
Hi,
I said back then that we need one test case that runs longer than 4
CPU freq,
in another way it's possible to figure out needed number of loops in runtime...
-Original Message-
From: EXT Ivan Khoronzhuk [mailto:ivan.khoronz...@linaro.org]
Sent: Thursday, April 14, 2016 5:00 PM
To: Savolainen, Petri (Nokia - FI/Espoo) <petri.savolai...@nokia.com>
void time_test_global_wait_until(void);
void time_test_wait_ns(void);
void time_test_local_to_u64(void);
void time_test_global_to_u64(void);
+void time_test_monotony(void);
/* test arrays: */
extern odp_testinfo_t time_suite_time[];
--
Regards,
Ivan Khoronzhuk
ping
On 18.02.16 17:56, Ivan Khoronzhuk wrote:
ping
On 16.02.16 16:02, Ivan Khoronzhuk wrote:
When test for single thread is finished the following test for
many threads can be started, and for some implementations can
happen that future one event can arrive to main thread
Signed-off-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org>
---
v2..v1:
- just rebased with several not important corrections
doc/users-guide/users-guide.adoc | 50
1 file changed, 45 insertions(+), 5 deletions(-)
diff --git a/doc/users-guide
I cannot find in the documentation, but is it possible to access a packets
hardware time stamp via the ODP API?
Regards
Dom
___
lng-odp mailing list
lng-odp@lists.linaro.org
https://lists.linaro.org/mailman/listinfo/lng-odp
--
Regards,
Ivan Kho
On 29.02.16 23:39, Bill Fischofer wrote:
On Mon, Feb 29, 2016 at 11:49 AM, Ivan Khoronzhuk <ivan.khoronz...@linaro.org
<mailto:ivan.khoronz...@linaro.org>> wrote:
For some systems sensitive to time deviation, would be better
to have some time backup while testing schedu
For some systems sensitive to time deviation, would be better
to have some time backup while testing scheduler time, so increase
it to 3 jiffies.
https://bugs.linaro.org/show_bug.cgi?id=2076
Signed-off-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org>
---
test/validation/scheduler/sched
Signed-off-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org>
---
doc/users-guide/users-guide.adoc | 49
1 file changed, 44 insertions(+), 5 deletions(-)
diff --git a/doc/users-guide/users-guide.adoc b/doc/users-guide/users-guide.adoc
index ea5e6aa..1
ping
On 16.02.16 16:02, Ivan Khoronzhuk wrote:
When test for single thread is finished the following test for
many threads can be started, and for some implementations can
happen that future one event can arrive to main thread, as it was
requested in previous test. As result one event can
On 17.02.16 18:44, Mike Holmes wrote:
Does this tuning need to be documented in the implementer's guide ?
I thinks, no.
On 17 February 2016 at 10:51, Zoltan Kiss <zoltan.k...@linaro.org
<mailto:zoltan.k...@linaro.org>> wrote:
On 17/02/16 15:24, Ivan Khoronzhuk wrote:
Signed-off-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org>
---
include/odp/api/time.h | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/include/odp/api/time.h b/include/odp/api/time.h
index efc5478..85692ec 100644
--- a/include/odp/api/time.h
+++ b/include/odp/api/
systems.
#define ODP_WAIT_TOLERANCE (40 * ODP_TIME_MSEC_IN_NS)
--
Regards,
Ivan Khoronzhuk
___
lng-odp mailing list
lng-odp@lists.linaro.org
https://lists.linaro.org/mailman/listinfo/lng-odp
scheduling for main thread when
it doesn't participate in the multi-threaded test. Also move
pause/resume test closer to beginning, because it's used in tests
before.
Signed-off-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org>
---
test/validation/scheduler/scheduler.c | 12 +++-
1 file c
istinfo/lng-odp
--
Regards,
Ivan Khoronzhuk
___
lng-odp mailing list
lng-odp@lists.linaro.org
https://lists.linaro.org/mailman/listinfo/lng-odp
_
lng-odp mailing list
lng-odp@lists.linaro.org
https://lists.linaro.org/mailman/listinfo/lng-odp
--
Regards,
Ivan Khoronzhuk
___
lng-odp mailing list
lng-odp@lists.linaro.org
https://lists.linaro.org/mailman/listinfo/lng-odp
ists.linaro.org <mailto:lng-odp@lists.linaro.org>
https://lists.linaro.org/mailman/listinfo/lng-odp
___
lng-odp mailing list
lng-odp@lists.linaro.org
https://lists.linaro.org/mailman/listinfo/lng-odp
--
Regards,
Ivan Khoronzhuk
___
lng-odp mailing list
lng-odp@lists.linaro.org
https://lists.linaro.org/mailman/listinfo/lng-odp
On 26.01.16 17:46, Maxim Uvarov wrote:
Signed-off-by: Maxim Uvarov <maxim.uva...@linaro.org>
Reviewed-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org>
---
test/validation/timer/timer.c | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/test/vali
It's not convenient to see reused a thread id for threads that are
finished to fast.
Signed-off-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org>
---
test/validation/shmem/shmem.c | 4
1 file changed, 4 insertions(+)
diff --git a/test/validation/shmem/shmem.c b/test/validation/shmem/s
see v2
On 22.01.16 20:09, Ivan Khoronzhuk wrote:
It's not convenient to see reused a thread id for threads that are
finished to fast.
Signed-off-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org>
---
test/validation/shmem/shmem.c | 4
1 file changed, 4 insertions(+)
diff --git
It's not convenient to see reused a thread id for threads that are
finished to fast.
Signed-off-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org>
---
test/validation/shmem/shmem.c | 4
1 file changed, 4 insertions(+)
diff --git a/test/validation/shmem/shmem.c b/test/validation/shmem/s
for allocated bufs num, that is
prio_num * QUEUES_PER_PRIO * TEST_NUM_BUFS. It's h/w dependent, and should
be in normal frames. This patch tunes resource usage in order to allow
test passing for different h/w.
Signed-off-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org>
---
test/validation/sch
the same.
Signed-off-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org>
---
platform/linux-generic/include/odp_internal.h | 3 +++
platform/linux-generic/odp_cpumask_task.c | 9 +++--
platform/linux-generic/odp_system_info.c | 22 ++
3 files changed, 20 inse
This patch series is intended to run control thread on CPU0.
It's increases worker thread CPU independence and allow to use
local time API on main thread in correct way.
Ivan Khoronzhuk (2):
linux-generic: cpumask_task: use cpumask got at init
linux-generic: init: assign affinity for main
sent v3.
On 17.12.15 19:07, Ivan Khoronzhuk wrote:
Example should free resources in right order when terminates.
Also it should have correct error path.
Signed-off-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org>
---
Since v1:
Just rebase, no functional changes
example
https://bugs.linaro.org/show_bug.cgi?id=1992
Signed-off-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org>
---
platform/linux-generic/odp_timer.c | 14 ++
1 file changed, 10 insertions(+), 4 deletions(-)
diff --git a/platform/linux-generic/odp_timer.c
b/platform/linux-g
+ Stuart Haslam
On 22.01.16 17:07, Ivan Khoronzhuk wrote:
https://bugs.linaro.org/show_bug.cgi?id=1992
Signed-off-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org>
---
platform/linux-generic/odp_timer.c | 14 ++
1 file changed, 10 insertions(+), 4 deletions(-)
diff
Example should free resources in right order when terminates.
Also it should have correct error path.
Signed-off-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org>
---
Since v2:
use less labels in one error path
Since v1:
Just rebase, no functional changes
example/timer/odp_timer_test.
sent patch with corrected tag
On 22.01.16 12:24, Ivan Khoronzhuk wrote:
The scheduler test should test scheduler, not resource limits.
For instance, prio_num * QUEUES_PER_PRIO * 4. In case if prio_num = 16,
it's easily exits the queues maximum number allowed,
16 * 16 * 4 = 1024, plus queues
for allocated bufs num, that is
prio_num * QUEUES_PER_PRIO * TEST_NUM_BUFS. It's h/w dependent, and should
be in normal frames. This patch tunes resource usage in order to allow
test passing for different h/w.
Signed-off-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org>
---
test/validation/sch
+ list
On 22.01.16 17:46, Ivan Khoronzhuk wrote:
It discussed several times when started to look at it and Bill proposed to add
some cls environment.
Finally, it's sent. After this change PMR is created while connection to CoSes,
and it's eliminates ability to connect the same PMR to several
);
+void scheduler_test_atomic(void);
+void scheduler_test_ordered(void);
void scheduler_test_1q_1t_n(void);
void scheduler_test_1q_1t_a(void);
void scheduler_test_1q_1t_o(void);
--
2.5.0
_____
On 20.01.16 21:43, Ivan Khoronzhuk wrote:
On 20.01.16 18:58, Bill Fischofer wrote:
ping.
On Wed, Dec 30, 2015 at 5:49 PM, Bill Fischofer <bill.fischo...@linaro.org
<mailto:bill.fischo...@linaro.org>> wrote:
Expand the existing scheduler chaos test to provide timing
ping
On 13.01.16 19:38, Ivan Khoronzhuk wrote:
The odp_pktio_stop() called is supposed to return error.
Signed-off-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org>
---
test/validation/pktio/pktio.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/test/validation
-by: Bill Fischofer <bill.fischo...@linaro.org>
Reviewed-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org>
---
test/validation/scheduler/scheduler.c | 41 +++
test/validation/scheduler/scheduler.h | 3 +++
2 files changed, 40 insertions(+), 4 deleti
On 21.01.16 00:02, Bill Fischofer wrote:
On Wed, Jan 20, 2016 at 3:36 PM, Ivan Khoronzhuk <ivan.khoronz...@linaro.org
<mailto:ivan.khoronz...@linaro.org>> wrote:
Seems you've missed simplification.
Can you be more specific? You requested that CHAOS_NUM_ROUNDS be redu
ping
On 13.01.16 19:18, Ivan Khoronzhuk wrote:
Now odp_time_wait_ns() can be used.
Signed-off-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org>
---
test/validation/pktio/pktio.c | 13 +
1 file changed, 1 insertion(+), 12 deletions(-)
diff --git a/test/validation/pktio/pkt
On 14.01.16 11:24, Petri Savolainen wrote:
Use only one definition for posix extension level. _GNU_SOURCE
defines _POSIX_C_SOURCE based on glibc version, and is
the only definition needed under Linux.
Signed-off-by: Petri Savolainen <petri.savolai...@nokia.com>
Reviewed-by: Ivan Khor
On 14.01.16 11:24, Petri Savolainen wrote:
Allways use _GNU_SOURCE instead of various _POSIX_C_SOURCE
defines.
Signed-off-by: Petri Savolainen <petri.savolai...@nokia.com>
Reviewed-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org>
---
example/generator/odp_generator.c | 4 ++
On 19.01.16 13:27, Maxim Uvarov wrote:
On 01/19/2016 13:23, Ivan Khoronzhuk wrote:
On 19.01.16 11:59, Maxim Uvarov wrote:
On 01/19/2016 01:54, Ivan Khoronzhuk wrote:
Maxim,
in general, I had no look in this test, it was working for me and I was happy.
At fast glance I have some comments
On 19.01.16 20:13, Maxim Uvarov wrote:
On 01/19/2016 19:56, Ivan Khoronzhuk wrote:
On 19.01.16 17:02, Ivan Khoronzhuk wrote:
On 19.01.16 16:16, Maxim Uvarov wrote:
On 01/19/2016 16:47, Ivan Khoronzhuk wrote:
On 19.01.16 15:11, Ivan Khoronzhuk wrote:
Maxim.
On 19.01.16 14:59, Maxim
be considered as fix of after-loop delay.
segfault. More detailed fix will be later.
Suggested-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org>
Signed-off-by: Maxim Uvarov <maxim.uva...@linaro.org>
---
test/validation/timer/timer.c | 2 +-
1 file changed, 1 insertion(+), 1 delet
On 19.01.16 15:11, Ivan Khoronzhuk wrote:
Maxim.
On 19.01.16 14:59, Maxim Uvarov wrote:
Increase after loop delay to:
10/10 - 7/10 = 3/10. 3/10 * 2000 + some 50ms = 650ms (instead of 1ms)
to make sure that timer hanles expired.
This patch can be considered as temporary fix for timer test
On 19.01.16 16:16, Maxim Uvarov wrote:
On 01/19/2016 16:47, Ivan Khoronzhuk wrote:
On 19.01.16 15:11, Ivan Khoronzhuk wrote:
Maxim.
On 19.01.16 14:59, Maxim Uvarov wrote:
Increase after loop delay to:
10/10 - 7/10 = 3/10. 3/10 * 2000 + some 50ms = 650ms (instead of 1ms)
to make sure
On 19.01.16 17:02, Ivan Khoronzhuk wrote:
On 19.01.16 16:16, Maxim Uvarov wrote:
On 01/19/2016 16:47, Ivan Khoronzhuk wrote:
On 19.01.16 15:11, Ivan Khoronzhuk wrote:
Maxim.
On 19.01.16 14:59, Maxim Uvarov wrote:
Increase after loop delay to:
10/10 - 7/10 = 3/10. 3/10 * 2000 + some
https://bugs.linaro.org/show_bug.cgi?id=1906
Signed-off-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org>
---
Since v5:
- add res instead of "1" while wrap
- get cycle stamps a multiple of res instead direct valuues.
- move resolution test before diff test
Since v3:
- mo
v6 is sent.
On 18.01.16 14:58, Savolainen, Petri (Nokia - FI/Espoo) wrote:
-Original Message-
From: EXT Ivan Khoronzhuk [mailto:ivan.khoronz...@linaro.org]
Sent: Monday, January 18, 2016 2:24 PM
To: Savolainen, Petri (Nokia - FI/Espoo); lng-odp@lists.linaro.org
Subject: Re: [lng-odp
ping
On 13.01.16 19:38, Ivan Khoronzhuk wrote:
The odp_pktio_stop() called is supposed to return error.
Signed-off-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org>
---
test/validation/pktio/pktio.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/test/validation
ping
On 13.01.16 19:18, Ivan Khoronzhuk wrote:
Now odp_time_wait_ns() can be used.
Signed-off-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org>
---
test/validation/pktio/pktio.c | 13 +
1 file changed, 1 insertion(+), 12 deletions(-)
diff --git a/test/validation/pktio/pkt
ping
On 14.01.16 12:56, Ivan Khoronzhuk wrote:
In case of one loop pktio the MAC address is not set in the packets
but should be.
Signed-off-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org>
---
test/validation/pktio/pktio.c | 18 ++
1 file changed, 14 insertions
ping
On 15.01.16 17:55, Stuart Haslam wrote:
On Fri, Jan 15, 2016 at 05:53:20PM +0200, Ivan Khoronzhuk wrote:
On 15.01.16 17:51, Stuart Haslam wrote:
On Wed, Jan 13, 2016 at 06:46:57PM +0200, Ivan Khoronzhuk wrote:
If packet with size > MTU is sent the test can crash farther,
so bet
On 18.01.16 13:37, Savolainen, Petri (Nokia - FI/Espoo) wrote:
-Original Message-
From: EXT Ivan Khoronzhuk [mailto:ivan.khoronz...@linaro.org]
Sent: Friday, January 15, 2016 5:07 PM
To: lng-odp@lists.linaro.org
Cc: Savolainen, Petri (Nokia - FI/Espoo); Ivan Khoronzhuk
Subject: [lng
10
+ret=$?
+# pktio_ipc1 should do clean up and exit just
+# after pktio_ipc2 exited. If it does not happen
+# kill him in test.
+sleep 1
+kill ${IPC_PID} 2>&1 > /dev/null
+if [ $? -eq 0 ]; then
+rm -rf /dev/shm/ipc_pktio_* 2>&1 > /dev/n
mer_test_odp_timer_cancel(void);
void timer_test_odp_timer_all(void);
+void timer_test_destroy_pool_with_scheduled_timers(void);
/* test arrays: */
extern odp_testinfo_t timer_suite[];
--
Regards,
Ivan Khoronzhuk
___
lng-odp mailing list
lng-odp@lists.linaro.org
https://lists.linaro.org/mailman/listinfo/lng-odp
ers_expiration();
+
while (nstale != 0) {
odp_event_t ev = odp_queue_deq(queue);
if (ev != ODP_EVENT_INVALID) {
--
Regards,
Ivan Khoronzhuk
___
lng-odp mailing list
lng-odp@lists.linaro.org
https://lists.linaro.org/mailman/listinfo/lng-odp
On 15.01.16 13:54, 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: Friday, January 15, 2016 11:16 AM
To: lng-odp@lists.linaro.org
Subject: [lng-odp] [PATCH v2] validation
https://bugs.linaro.org/show_bug.cgi?id=1906
Signed-off-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org>
---
Since v2:
- added c <= max
- replaced etalon on tmp
- added msg about skip
- handled resolution wrap
test/validation/system/system.c | 133 +
On 15.01.16 15:34, Savolainen, Petri (Nokia - FI/Espoo) wrote:
-Original Message-
From: EXT Ivan Khoronzhuk [mailto:ivan.khoronz...@linaro.org]
Sent: Friday, January 15, 2016 2:33 PM
To: lng-odp@lists.linaro.org
Cc: Savolainen, Petri (Nokia - FI/Espoo); Ivan Khoronzhuk
Subject: [lng
https://bugs.linaro.org/show_bug.cgi?id=1906
Signed-off-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org>
---
Since v3:
- modified log "wrap is not detected"
- increased try num for diff to match 16 seconds
- decreased try num for resolution to match 1 second
- allow resolution
https://bugs.linaro.org/show_bug.cgi?id=1906
Signed-off-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org>
---
test/validation/system/system.c | 130
test/validation/system/system.h | 4 ++
2 files changed, 134 insertions(+)
diff --git a/test/vali
On 15.01.16 17:51, Stuart Haslam wrote:
On Wed, Jan 13, 2016 at 06:46:57PM +0200, Ivan Khoronzhuk wrote:
If packet with size > MTU is sent the test can crash farther,
so better to stop here.
Signed-off-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org>
Signed-off-by: Stua
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 <ivan.khoronz...@linaro.org>
---
test/validation/classification/classification.h| 6 +--
.../classifi
If pktio is in not promisc mode, a packet should contain correct
MAC address.
Signed-off-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org>
---
.../classification/odp_classification_test_pmr.c | 57 ++
1 file changed, 57 insertions(+)
diff --git a/test/vali
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 checked it can be done in separate test.
Signed-off-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org>
---
.../classification/odp_classification_test
-int odp_time_global_init(void)
+int odp_time_init_global(void)
{
int ret;
_odp_time_t time;
@@ -202,3 +202,8 @@ int odp_time_global_init(void)
return ret;
}
+
+int odp_time_term_global(void)
+{
+ return 0;
+}
diff --git a/platform/linux-generic/odp_timer.c
b/pl
On 15.01.16 18:18, Stuart Haslam wrote:
Printing the pktio info every time a pktio is opened generates a
load of noise on stdout and makes it difficult to see the status
of individual test cases.
Signed-off-by: Stuart Haslam <stuart.has...@linaro.org>
Reviewed-by: Ivan Khor
On 15.01.16 17:45, Stuart Haslam wrote:
Remove an unwanted initialisation and fix a bad indentation which
crept in during a merge.
forgot to add signed-off
Reviewed-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org>
---
platform/linux-generic/pktio/loop.c | 3 +--
1 file chan
https://bugs.linaro.org/show_bug.cgi?id=1906
Signed-off-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org>
---
test/validation/system/system.c | 118
test/validation/system/system.h | 4 ++
2 files changed, 122 insertions(+)
diff --git a/test/vali
Hi, Petri
This patch contains validation tests for API you've recently added.
Could you please review it.
On 15.01.16 11:16, Ivan Khoronzhuk wrote:
https://bugs.linaro.org/show_bug.cgi?id=1906
Signed-off-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org>
---
test/validation/system/sy
-and-tested-by: Bill Fischofer <bill.fischo...@linaro.org>
Signed-off-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org>
---
Since v5:
- align strings to be less than 80 chars
- added parentheses in macros definition
Since v4:
- convert time to ns only while printing log
Since v3:
- defin
it to master branch.
Ok, I'll base it on api-next.
The patch 3 in your series is not applying on the master branch.
Further comments on each patch series.
Regards,
Bala
Regards,
Bala
On 12 January 2016 at 23:41, Ivan Khoronzhuk <ivan.khoronz...@linaro.org> wrote:
This patch series co
In case of one loop pktio the MAC address is not set in the packets
but should be.
Signed-off-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org>
---
test/validation/pktio/pktio.c | 18 ++
1 file changed, 14 insertions(+), 4 deletions(-)
diff --git a/test/validation/pktio/pkt
January 2016 at 22:29, Ivan Khoronzhuk <ivan.khoronz...@linaro.org> wrote:
It can be used to restore headroom later. Does it possible a headroom
to be > sizeof(int32)/2?
Signed-off-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org>
---
include/odp/api/packet_io.h | 4 ++--
1
https://bugs.linaro.org/show_bug.cgi?id=1906
Signed-off-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org>
---
test/validation/system/system.c | 111
test/validation/system/system.h | 4 ++
2 files changed, 115 insertions(+)
diff --git a/test/vali
separately.
On 12 January 2016 at 23:41, Ivan Khoronzhuk <ivan.khoronz...@linaro.org> wrote:
These tests are simple classification tests and better to see results
for each of them separately.
Signed-off-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org>
---
test/validation/cl
by default CoS? Also, seems I forgot to start pktio
afterwards.
But when I started it, I see that linux-generic implementation doesn't pass the
test, strange.
What is going on? My board passes it correctly.
On 12 January 2016 at 23:41, Ivan Khoronzhuk <ivan.khoronz...@linaro.org>
that mostly common pktio_loop is used. I will correct to set MAC
addresses
for each test, but odp_classification_tests.c should be sophisticated also,
later.
On 12 January 2016 at 23:41, Ivan Khoronzhuk <ivan.khoronz...@linaro.org> wrote:
If pktion is in not promisc mode, a packet should c
It can be used to restore headroom later. Does it possible a headroom
to be > sizeof(int32)/2?
Signed-off-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org>
---
include/odp/api/packet_io.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/include/odp/api/packet_io.h
Now odp_time_wait_ns() can be used.
Signed-off-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org>
---
test/validation/pktio/pktio.c | 13 +
1 file changed, 1 insertion(+), 12 deletions(-)
diff --git a/test/validation/pktio/pktio.c b/test/validation/pktio/pktio.c
index 45c11c5..2
The odp_pktio_stop() called is supposed to return error.
Signed-off-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org>
---
test/validation/pktio/pktio.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/test/validation/pktio/pktio.c b/test/validation/pktio/pktio.c
index 2
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 correct
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 <ivan.khoronz...@linaro.org>
---
.../classifi
There is no need to create default input queue for pktio if
default CoS is assigned.
Signed-off-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org>
---
.../classification/odp_classification_test_pmr.c | 54 --
1 file changed, 54 deletions(-)
diff --git a/test/vali
These tests are simple classification tests and better to see results
for each of them separately.
Signed-off-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org>
---
test/validation/classification/classification.h| 22 ---
.../classification/odp_classification_tests.c
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 <ivan.khoronz...@linaro.org>
---
test/validation/classification/classification.h| 6 +--
.../classifi
If pktion is in not promisc mode, a packet should contain correct
MAC address.
Signed-off-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org>
---
.../classification/odp_classification_test_pmr.c | 158 ++---
.../classification/odp_classification_tests.c | 30 +++-
2
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
From: Nicolas Morey-Chaisemartin <nmo...@kalray.eu>
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 <ivan.khoronz...@linaro.org>
Signed-off-by: Ivan Khoronzhuk &
While division the resolution can have fractional part, so better to
round it up, when it's needed.
Reviewed-by: Bill Fischofer <bill.fischo...@linaro.org>
Signed-off-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org>
---
test/validation/time/time.c | 3 ++-
1 file changed, 2 inse
In places of two conversions the error can be up to
2 resolutions, so increase validation range to 2 res.
Reviewed-by: Bill Fischofer <bill.fischo...@linaro.org>
Signed-off-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org>
---
test/validation/time/time.c | 20 ++--
1
1 - 100 of 612 matches
Mail list logo