Skip to content

Bluetooth: Controller: Fix stuck forced continuation under throughput #92819

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

cvinayak
Copy link
Contributor

@cvinayak cvinayak commented Jul 8, 2025

Fix stuck forced continuation of connection event under high throughput scenario. At near supervision timeout the force flag is set, but in the next connection event if full connection interval is used for data transmission and a subsequent request to abort happens it was not honoured causing the tx-rx chain to keep continuing until supervision timeout.

Thalley
Thalley previously approved these changes Jul 8, 2025
@cvinayak cvinayak added the bug The issue is a bug, or the PR is fixing a bug label Jul 8, 2025
@cvinayak cvinayak added this to the v4.2.0 milestone Jul 8, 2025
Fix stuck forced continuation of connection event under high
throughput scenario. At near supervision timeout the force
flag is set, but in the next connection event if full
connection interval is used for data transmission and a
subsequent request to abort happens it was not honoured
causing the tx-rx chain to keep continuing until supervision
timeout.

Signed-off-by: Vinayak Kariappa Chettimada <vich@nordicsemi.no>
@cvinayak cvinayak force-pushed the github_stuck_forced_continue_fix branch from 75d73f0 to 773222e Compare July 10, 2025 14:03
Thalley
Thalley previously approved these changes Jul 10, 2025
cvinayak added 2 commits July 11, 2025 12:00
Fix missing radio status reset on early abort of central
and peripheral prepare.

Signed-off-by: Vinayak Kariappa Chettimada <vich@nordicsemi.no>
Disable connection event continuation on overlap with same
connection's next event. This implementation has bugs and
needs to be fixed in subsequent commits.

Signed-off-by: Vinayak Kariappa Chettimada <vich@nordicsemi.no>
@cvinayak cvinayak removed this from the v4.2.0 milestone Jul 15, 2025
@nashif nashif removed the bug The issue is a bug, or the PR is fixing a bug label Jul 15, 2025
Introduce prepare being deferred due to previous events
desire to continue. In such case the deferred prepare param
will have the defer flag set.

Signed-off-by: Vinayak Kariappa Chettimada <vich@nordicsemi.no>
@zephyrbot zephyrbot requested a review from ppryga July 17, 2025 09:17
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants