Skip to content

Assert invoked in exception in ctrl.c line 7653 (zephyr 1.13.00) #11841

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

Closed
alexsven opened this issue Dec 4, 2018 · 8 comments
Closed

Assert invoked in exception in ctrl.c line 7653 (zephyr 1.13.00) #11841

alexsven opened this issue Dec 4, 2018 · 8 comments
Assignees
Labels
area: Bluetooth bug The issue is a bug, or the PR is fixing a bug priority: medium Medium impact/importance bug

Comments

@alexsven
Copy link
Collaborator

alexsven commented Dec 4, 2018

Assert gets invoked when having one connection and starting another. Spoke to @cvinayak about this and it seems the new connection wants to resize the rx_pool while it's still in use by the first connection.
if (_radio.packet_rx_data_count != free_count_rx) { /** TODO another role instance has obtained * memory from rx pool. */ //LL_ASSERT(0); }

@carlescufi carlescufi added bug The issue is a bug, or the PR is fixing a bug priority: medium Medium impact/importance bug labels Dec 5, 2018
@carlescufi
Copy link
Member

@cvinayak can you comment on this issue?

@cvinayak
Copy link
Collaborator

@alexanderwachter @carlescufi this is a bug and needs fix (redesign, as I dont like the current implementation anyway) which I will work alongwith the design to be put in place for the split architecture work.

@carlescufi
Copy link
Member

carlescufi commented Dec 18, 2018

@cvinayak so this will be fixed in the new split controller only?
Can you propose a workaround for @alexanderwachter?

@cvinayak
Copy link
Collaborator

@alexanderwachter commenting the assert has unblocked your usecase right? I think subsequent code must be handling/deferring the data resize gracefully, right?

@alexanderwachter
Copy link
Member

I think you guys have a little typo with the names. @alexanderwachter should be @alexsven ;-)

@alexsven
Copy link
Collaborator Author

alexsven commented Jan 2, 2019

@cvinayak Yes, it unblocked our usecase.

@carlescufi
Copy link
Member

@cvinayak can you update on this one please?

@cvinayak
Copy link
Collaborator

@alexsven could you verify #14553 ?

cvinayak added a commit to cvinayak/zephyr that referenced this issue Mar 15, 2019
Workaround from asserting when Rx PDU buffers from pool
needing resize is acquired by another connection. By
skipping the connection events to give some headroom for
the host to process the Rx packets and eventually make the
pool available for resize.

Fixes zephyrproject-rtos#11841.

Signed-off-by: Vinayak Kariappa Chettimada <[email protected]>
carlescufi pushed a commit that referenced this issue Mar 15, 2019
Workaround from asserting when Rx PDU buffers from pool
needing resize is acquired by another connection. By
skipping the connection events to give some headroom for
the host to process the Rx packets and eventually make the
pool available for resize.

Fixes #11841.

Signed-off-by: Vinayak Kariappa Chettimada <[email protected]>
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

No branches or pull requests

5 participants