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.

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

tests/net/trickle failed on frdm_k64f and sam_e70_xplained with v1.14 branch. #24300

Closed
chen-png opened this issue Apr 13, 2020 · 4 comments
Closed
Assignees
Labels
area: Networking area: Tests Issues related to a particular existing or missing test bug The issue is a bug, or the PR is fixing a bug platform: Microchip SAM Microchip SAM Platform (formerly Atmel SAM) platform: NXP NXP priority: low Low impact/importance bug
Milestone

Comments

@chen-png
Copy link
Collaborator

Describe the bug
Running tests/net/trickle on frdm_k64f and sam_e70_xplained with v1.14-branch, it will failed occasionally

To Reproduce
Steps to reproduce the behavior:

  1. west build -b frdm_k64f -p auto tests/net/trickle
  2. west flash
  3. See error

Screenshots or console output
***** Booting Zephyr OS zephyr-v1.14.1-49-gff6829261918 *****
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

starting test - test_trickle_1_wait
[00:00:00.000,000] net_if: You have 1 IPv4 net_if addresses but 2 network interfaces
[00:00:00.000,000] net_if: Consider increasing CONFIG_Trickle 1 0x20000c40 callback called
PASS - test_trickle_1_wait

starting test - test_trickle_2_wait
NET_IF_MAX_IPV4_COUNT value.
[00:00:00.000,000] net_if: You have 1 IPv6 net_if addresses but 2 network interfaces
[00:00:00.000,000] net_if: Consider increasing CONFIG_NET_IF_MAX_IPV6_COUN T value.
Trickle 1 0x20000c40 callback called

**Assertion failed at  zephyr-branch-1.14-github/zephyr/tests/net/trickle/src/main.c:171: test_trickle_2_wait: (cb_2_called is false)**  

Trickle 2 no timeout
Trickle 2 0x20000c80 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

Environment (please complete the following information):

  • OS: Fedora28
  • Toolchain: Zephyr-sdk-0.10.3
  • Commit ID: ff68292
@chen-png chen-png added the bug The issue is a bug, or the PR is fixing a bug label Apr 13, 2020
@jukkar jukkar added area: Networking priority: low Low impact/importance bug labels Apr 13, 2020
@nashif
Copy link
Member

nashif commented Apr 14, 2020

@jukkar can you take a look at that one?

@jukkar
Copy link
Member

jukkar commented Apr 15, 2020

Already fixed in 2.0 by #18625
I will backport the fix to 1.14

jukkar pushed a commit to jukkar/zephyr that referenced this issue Apr 15, 2020
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>
@ioannisg ioannisg added this to the v1.14.3 milestone Apr 17, 2020
nashif pushed a commit that referenced this issue Apr 17, 2020
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>
@carlescufi carlescufi added area: Tests Issues related to a particular existing or missing test platform: Microchip SAM Microchip SAM Platform (formerly Atmel SAM) platform: NXP NXP labels Apr 30, 2020
@github-actions
Copy link

github-actions bot commented Jul 1, 2020

This issue has been marked as stale because it has been open (more than) 60 days with no activity. Remove the stale label or add a comment saying that you would like to have the label removed otherwise this issue will automatically be closed in 14 days. Note, that you can always re-open a closed issue at any time.

@github-actions github-actions bot added the Stale label Jul 1, 2020
@stephanosio stephanosio removed the Stale label Jul 1, 2020
@stephanosio
Copy link
Member

Already fixed by #24370

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Networking area: Tests Issues related to a particular existing or missing test bug The issue is a bug, or the PR is fixing a bug platform: Microchip SAM Microchip SAM Platform (formerly Atmel SAM) platform: NXP NXP priority: low Low impact/importance bug
Projects
None yet
Development

No branches or pull requests

6 participants