Skip to content
This repository has been archived by the owner on May 18, 2022. It is now read-only.

Support for closing connections #147

Open
daboross opened this issue Aug 13, 2020 · 0 comments
Open

Support for closing connections #147

daboross opened this issue Aug 13, 2020 · 0 comments
Labels
area: link layer Code Area: BLE Link Layer (establishing and maintaining connections, encryption, etc) type: enhancement A new feature or an improvement for an existing one

Comments

@daboross
Copy link
Contributor

Right now, there isn't any way to properly close a connection, flush the associated queues, and turn off the radio.

Supporting this isn't necessarily trivial, however. Quoting from @jonas-schievink from #140 (comment) (on a simple-and-incorrect radio shutdown PR):

Hmm, yeah, we want something like this eventually, but getting it correct is a bit more involved. For example, the low-priority part of the stack needs to know that the connection is being teared down to drain the packet queue before the operation can complete (otherwise establishing a new connection will start processing and transmitting packets from the old connection, which is pretty bad).

Fixing this would mean adding the ability to properly close a connection from the device, and fixing "start_advertising" so that it uses this method to close any active connections before advertising.

See also #70.

@jonas-schievink jonas-schievink added area: link layer Code Area: BLE Link Layer (establishing and maintaining connections, encryption, etc) type: enhancement A new feature or an improvement for an existing one labels Aug 13, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area: link layer Code Area: BLE Link Layer (establishing and maintaining connections, encryption, etc) type: enhancement A new feature or an improvement for an existing one
Projects
None yet
Development

No branches or pull requests

2 participants