Maxim Uvarov has uploaded a new change for review. https://review.linaro.org/10969
Change subject: validation: timer: don't access non-existing timers ...................................................................... validation: timer: don't access non-existing timers Since e5c85d3f "validation: timer: handle early exhaustion of pool" the workers can handle if object caches retain packets, but with enough threads it can happen that a late starting thread won't be able to allocate any. This for loop should take that into account and not trying to access tt[0].ev. Change-Id: I3648cb7c2b7af91276f8aa4d06ee0f26e3236e7d Signed-off-by: Zoltan Kiss <[email protected]> Signed-off-by: Maxim Uvarov <[email protected]> --- M test/validation/timer/timer.c 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://review.linaro.org:29418/lng/odp refs/changes/69/10969/1 diff --git a/test/validation/timer/timer.c b/test/validation/timer/timer.c index 5854b32..7b76dbf 100644 --- a/test/validation/timer/timer.c +++ b/test/validation/timer/timer.c @@ -338,7 +338,7 @@ uint32_t ms; uint64_t prev_tick = odp_timer_current_tick(tp); - for (ms = 0; ms < 7 * RANGE_MS / 10; ms++) { + for (ms = 0; ms < 7 * RANGE_MS / 10 && allocated > 0; ms++) { odp_event_t ev; while ((ev = odp_queue_deq(queue)) != ODP_EVENT_INVALID) { /* Subtract one from prev_tick to allow for timeouts -- To view, visit https://review.linaro.org/10969 To unsubscribe, visit https://review.linaro.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I3648cb7c2b7af91276f8aa4d06ee0f26e3236e7d Gerrit-PatchSet: 1 Gerrit-Project: lng/odp Gerrit-Branch: master Gerrit-Owner: Maxim Uvarov <[email protected]> _______________________________________________ lng-odp mailing list [email protected] https://lists.linaro.org/mailman/listinfo/lng-odp
