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: Assert when starting periodic advertising right after extended advertising when there is a BIG #72852

Open
Thalley opened this issue May 16, 2024 · 2 comments
Assignees
Labels
area: Bluetooth Controller area: Bluetooth bug The issue is a bug, or the PR is fixing a bug known issue Known issue priority: medium Medium impact/importance bug

Comments

@Thalley
Copy link
Collaborator

Thalley commented May 16, 2024

Describe the bug
If there is a BIG created and periodic advertising is started right after extended advertising, there is an assert in the controller that is triggered:

ASSERTION FAIL [id != ((uint8_t)((uint8_t)0 - 1))] @ WEST_TOPDIR/zephyr/subsys/bluetooth/controller/ll_sw/ull_adv_aux.c:3218
@ WEST_TOPDIR/zephyr/subsys/bluetooth/controller/ll_sw/ull_adv_aux.c:3218

To Reproduce
Can be reproduced with #72690

Steps to reproduce the behavior:

  1. Remove the k_sleep in start_ext_adv from tests/bsim/bluetooth/host/iso/bis/src/bis_broadcaster.c
  2. Build and run tests/bsim/bluetooth/host/iso/bis/tests_scripts/bis.sh
  3. See the above assert

Expected behavior
Expect the controller to support starting periodic advertising directly after extended advertising.

Impact
Big annoyance, but can be circumvented with a single k_sleep

Logs and console output
See above

Environment (please complete the following information):

Additional context
N/A

@Thalley Thalley added bug The issue is a bug, or the PR is fixing a bug area: Bluetooth area: Bluetooth Controller labels May 16, 2024
@nashif nashif added the priority: medium Medium impact/importance bug label May 21, 2024
@github-actions github-actions bot added the Stale label Jul 21, 2024
@Thalley Thalley removed the Stale label Jul 21, 2024
@jori-nordic
Copy link
Collaborator

Note for would-be issue fixers: Don't forget to grep the zephyr tree for mentions of this issue in order to remove workarounds.
E.g. tests/bsim/bluetooth/host/iso/bis/src/bis_broadcaster.c and maybe others.

@zephyrproject-rtos zephyrproject-rtos deleted a comment from github-actions bot Aug 6, 2024
@cvinayak cvinayak added the known issue Known issue label Aug 6, 2024
Copy link

github-actions bot commented Oct 6, 2024

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 Oct 6, 2024
@Thalley Thalley removed the Stale label Oct 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Bluetooth Controller area: Bluetooth bug The issue is a bug, or the PR is fixing a bug known issue Known issue priority: medium Medium impact/importance bug
Projects
None yet
Development

No branches or pull requests

4 participants