rymanluk opened a new pull request #568: nimble/ll: Fix assert in 
ble_ll_adv_done
URL: https://github.com/apache/mynewt-nimble/pull/568
 
 
   This patch makes sure that periodic advertising uses its own scheduler
   type. It is needed to not confuse it with related extended advertising.
   
   Mentioned assert happend in the following scenario
   a) Nimble has configured and enabled connectable advertising
   b) Nimble has configured and enabled periodic advertising along with its
   extending advertising.
   c) Once Nimble connects and bonds with peer device, keys are storred
   and in that time GAP procedures are stopped i.e. advertising. In this
   moment periodic advertising and connection are ongoing.
   
   d) While advertising is OFF LL reschedules connection event. It might happen
   that periodic advertising is conflicting with this, and in this case
   connection has bigger prio. Scheduler wants to remove it but since
   periodic advertising uses BLE_LL_SCHED_TYPE_ADV, in fact it tries to
   stop already stopped advertising -> assert

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to