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

nrf52_bsim fails on some tests after merging 29810 #30507

Closed
nashif opened this issue Dec 7, 2020 · 2 comments · Fixed by #30522
Closed

nrf52_bsim fails on some tests after merging 29810 #30507

nashif opened this issue Dec 7, 2020 · 2 comments · Fixed by #30522
Assignees
Labels
bug The issue is a bug, or the PR is fixing a bug priority: medium Medium impact/importance bug

Comments

@nashif
Copy link
Member

nashif commented Dec 7, 2020

Describe the bug

west build -b nrf52_bsim tests/benchmarks/latency_measure -t run
hangs, before #29810 (by @nordic-krch ) it used to run all the way to the end.

To Reproduce
Steps to reproduce the behavior:

  1. west build -b nrf52_bsim tests/benchmarks/latency_measure -t run

Expected behavior

test completes
Impact
Blocking CI in many pending PRs

Logs and console output
hangs here:

?_??: WARNING: Neither simulation id or the device number have been set. I assume you want to run without a BabbleSim phy (-nosim)
?_??: WARNING: If this is not what you wanted, check with --help how to set them
d_00: @00:00:00.000000  *** Booting Zephyr OS build zephyr-v2.4.0-2121-gde191bf3a6fb  ***
d_00: @00:00:00.000000  START - Time Measurement
d_00: @00:00:00.000000  Timing results: Clock frequency: 0 MHz
@nashif nashif added bug The issue is a bug, or the PR is fixing a bug priority: medium Medium impact/importance bug labels Dec 7, 2020
@nordic-krch
Copy link
Contributor

@aescolar i think i would need a help from you here. It seems like RTC interrupt is triggered only once. I've logged COUNTER, CC[0] and EVTEN registers after setting new CC and it looks ok.

At 0 CC is set for 1sec, at one sec RTC interrupt occurs and it is set to 2sec timeout but that one never comes.

d_00: @00:00:00.000000  COUNTER:0 CC:32768
d_00: @00:00:00.000000  EVTEN:10000
d_00: @00:00:00.000000  *** Booting Zephyr OS build zephyr-v2.4.0-2096-gad3de0a76cec  ***
d_00: @00:00:00.000000  START - Time Measurement
d_00: @00:00:00.000000  Timing results: Clock frequency: 0 MHz
d_00: @00:00:01.000000  COUNTER:32768 CC:65536
d_00: @00:00:01.000000  EVTEN:10000

@aescolar
Copy link
Member

aescolar commented Dec 8, 2020

@aescolar i think i would need a help from you here.

@nordic-krch (Today and tomorrow I'm swamped.) The default when a PR breaks master is to revert it and figure out calmly why and fix it without blocking others. If you cannot find easily what went wrong I would not delay further reverting it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug The issue is a bug, or the PR is fixing a bug priority: medium Medium impact/importance bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants