Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

tests/net/trickle failed on multiple plartforms #18598

Closed
vanti opened this issue Aug 22, 2019 · 0 comments · Fixed by #18625
Closed

tests/net/trickle failed on multiple plartforms #18598

vanti opened this issue Aug 22, 2019 · 0 comments · Fixed by #18625
Assignees
@vanti
Labels
bug The issue is a bug, or the PR is fixing a bug priority: medium Medium impact/importance bug

Comments

@vanti
Copy link
Collaborator

vanti commented Aug 22, 2019 •
edited
Loading

Describe the bug
Running the test leads to a failure:

***** Booting Zephyr OS build v2.0.0-rc1-176-gc204bf3c6575 *****
Running test suite test_tickle
===================================================================
starting test - test_init
PASS - test_init
===================================================================
starting test - test_trickle_create
PASS - test_trickle_create
===================================================================
starting test - test_trickle_start
PASS - test_trickle_start
===================================================================
starting test - test_trickle_1_status
PASS - test_trickle_1_status
===================================================================
starting test - test_trickle_2_status
PASS - test_trickle_2_status
=====Trickle 1 0x20000008 callback called
==============================================================
starting test - test_trickle_1_wait
PASS - test_trickle_1_wait
===================================================================
starting test - test_trickle_2_wait
Trickle 1 0x20000008 callback called

    Assertion failed at ../src/main.c:171: test_trickle_2_wait: (cb_2_called is false)
Trickle 2 no timeout
Trickle 2 0x20000048 callback called
FAIL - test_trickle_2_wait
===================================================================
starting test - test_trickle_1_update
PASS - test_trickle_1_update
===================================================================
starting test - test_trickle_2_inc
PASS - test_trickle_2_inc
===================================================================
starting test - test_trickle_1_status
PASS - test_trickle_1_status
===================================================================
starting test - test_trickle_1_wait_long
SKIP - test_trickle_1_wait_long
===================================================================
starting test - test_trickle_stop
PASS - test_trickle_stop
===================================================================
starting test - test_trickle_1_stopped
PASS - test_trickle_1_stopped
===================================================================
Test suite test_tickle failed.
===================================================================
PROJECT EXECUTION FAILED

To Reproduce
Steps to reproduce the behavior:
0. cd tests/net/trickle

  1. mkdir build; cd build
  2. cmake .. -DBOARD=cc3220sf_launchxl
  3. make flash
  4. See error

Expected behavior
I expect the test to pass.

Environment (please complete the following information):

  • OS: Ubuntu 18.04
  • Toolchain: Zephyr SDK 0.10.3
  • Commit SHA tested: 1751648
@vanti vanti added the bug The issue is a bug, or the PR is fixing a bug label Aug 22, 2019
@jenmwms jenmwms added this to the v2.1.0 milestone Aug 23, 2019
@jenmwms jenmwms added the priority: medium Medium impact/importance bug label Aug 23, 2019
@vanti vanti mentioned this issue Aug 23, 2019
Merged
@nashif nashif removed this from the v2.1.0 milestone Aug 23, 2019
@nashif nashif mentioned this issue Aug 23, 2019
Closed
vanti added a commit to vanti/zephyr that referenced this issue Aug 23, 2019
@vanti
tests: net: trickle: use separate semaphore for second trickle timer
aca2a90 
A second semaphore is used for the second trickle timer, so that if the
first timer expires twice before the second one, the test would still
wait before proceeding to check on cb_2_called.

Fixes zephyrproject-rtos#18598.

Signed-off-by: Vincent Wan <vincent.wan@linaro.org>
@nashif nashif changed the title tests/net/trickle failed on cc3220sf_launchxl tests/net/trickle failed on multiple plartforms Aug 23, 2019
ioannisg pushed a commit that referenced this issue Aug 26, 2019
@vanti @ioannisg
tests: net: trickle: use separate semaphore for second trickle timer
fdf0af3 
A second semaphore is used for the second trickle timer, so that if the
first timer expires twice before the second one, the test would still
wait before proceeding to check on cb_2_called.

Fixes #18598.

Signed-off-by: Vincent Wan <vincent.wan@linaro.org>
LeiW000 pushed a commit to LeiW000/zephyr that referenced this issue Sep 2, 2019
@vanti
tests: net: trickle: use separate semaphore for second trickle timer
4d8b973 
A second semaphore is used for the second trickle timer, so that if the
first timer expires twice before the second one, the test would still
wait before proceeding to check on cb_2_called.

Fixes zephyrproject-rtos#18598.

Signed-off-by: Vincent Wan <vincent.wan@linaro.org>
jukkar pushed a commit to jukkar/zephyr that referenced this issue Apr 15, 2020
@vanti @jukkar
tests: net: trickle: use separate semaphore for second trickle timer
8d3dacb 
A second semaphore is used for the second trickle timer, so that if the
first timer expires twice before the second one, the test would still
wait before proceeding to check on cb_2_called.

Fixes zephyrproject-rtos#18598.

Backported to v1.14 in order to fix PR zephyrproject-rtos#24300
Fixes zephyrproject-rtos#24300

Signed-off-by: Vincent Wan <vincent.wan@linaro.org>
@jukkar jukkar mentioned this issue Apr 15, 2020
Merged
nashif pushed a commit that referenced this issue Apr 17, 2020
@vanti @nashif
tests: net: trickle: use separate semaphore for second trickle timer
4730a9b 
A second semaphore is used for the second trickle timer, so that if the
first timer expires twice before the second one, the test would still
wait before proceeding to check on cb_2_called.

Fixes #18598.

Backported to v1.14 in order to fix PR #24300
Fixes #24300

Signed-off-by: Vincent Wan <vincent.wan@linaro.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@vanti vanti

Labels
bug The issue is a bug, or the PR is fixing a bug priority: medium Medium impact/importance bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

tests: net: trickle: use separate semaphore for second trickle timer vanti/zephyr
3 participants
@nashif @vanti @jenmwms