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

Bluetooth: controller: Fix ticker implementation to avoid catch up #23070

Closed
cvinayak opened this issue Feb 25, 2020 · 1 comment · Fixed by #23966
Closed

Bluetooth: controller: Fix ticker implementation to avoid catch up #23070

cvinayak opened this issue Feb 25, 2020 · 1 comment · Fixed by #23966
Assignees
Labels
area: Bluetooth bug The issue is a bug, or the PR is fixing a bug priority: medium Medium impact/importance bug
Milestone

Comments

@cvinayak
Copy link
Contributor

Describe the bug
Relates to #22926

Temporarily ticker compatibility mode has been enabled for nRF5 platforms.

Two issues observed:

  1. Either flash driver's ticker instance or master role ticker instance do not get scheduled. Both have an interval of 7.5 ms.
  2. Flash erase takes 80 ms during which CPU is halted, master intervals try catch up consuming CPU time that cause master events not being scheduled at hard real time deadlines.

Detailed analysis to be posted here, post v2.2 release completes.

To Reproduce
Steps to reproduce the behavior:

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

Expected behavior
A clear and concise description of what you expected to happen.

Impact
What impact does this issue have on your progress (e.g., annoyance, showstopper)

Screenshots or console output
If applicable, add a screenshot (drag-and-drop an image), or console logs
(cut-and-paste text and put a code fence (```) before and after, to help
explain the issue.

Environment (please complete the following information):

  • OS: (e.g. Linux, MacOS, Windows)
  • Toolchain (e.g Zephyr SDK, ...)
  • Commit SHA or Version used

Additional context
Add any other context about the problem here.

@cvinayak cvinayak added bug The issue is a bug, or the PR is fixing a bug priority: medium Medium impact/importance bug area: Bluetooth labels Feb 25, 2020
@cvinayak cvinayak added this to the v2.3.0 milestone Feb 25, 2020
@cvinayak
Copy link
Contributor Author

cvinayak commented Apr 7, 2020

Fixed by #23815

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Bluetooth 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.

2 participants