Disable Softdevice from observer crashes; "don't do that" should be documented?

RE: Disable Softdevice from observer crashes; "don't do that" should be documented?

ask6483 — Mon, 28 Aug 2023 19:32:37 GMT

Hi,

This bug will also happen when NRF_PWR_MGMT_CONFIG_USE_SCHEDULER is enabled. System power off is delayed then and nrf_sdh_ble_evts_poll() runs causing the crash. I was incredibly lucky to find this post ;)

RE: Disable Softdevice from observer crashes; "don't do that" should be documented?

Sigurd — Mon, 31 May 2021 07:49:59 GMT

Hi,

This has unfortunately not been fixed in SDK 17.0.2. nrf_sdh_soc_evts_poll and nrf_sdh_ble_evts_poll functions may cause reset if nrf_sdh_disable_request is called from observer. Proposed fix is to add an exception for NRF_ERROR_SOFTDEVICE_NOT_ENABLED in nrf_sdh_soc.c and nrf_sdh_ble.c

    if (ret_code != NRF_ERROR_NOT_FOUND && ret_code != NRF_ERROR_SOFTDEVICE_NOT_ENABLED)
    {
        APP_ERROR_HANDLER(ret_code);
    }

RE: Disable Softdevice from observer crashes; "don't do that" should be documented?

Keith Wakeham — Sat, 29 May 2021 03:24:30 GMT

Just adding a note to this old post. Butch's solution works after days of pulling my hair out on this. Keep in mind that even in SDK 17 examples like buttonless DFU is written with the nrf_sdh_disable_request() inside the event that Sigurd said causes a bug.

The reason it works in their examples is because they go to system power off BEFORE the observer runs nrf_sdh_ble_evts_poll() thus causing the crash. So basically, the examples are a bit "buggy" here.

RE: Disable Softdevice from observer crashes; "don't do that" should be documented?

Sigurd — Wed, 02 May 2018 09:34:07 GMT

Hi,

Thank you for reporting this.

What is happening is that we are pulling events from the SoftDevice in nrf_sdh_ble_evts_poll(), and are forwarding them to the observers. We do this until we no longer have any new events from the SoftDevice to process, and sd_ble_evt_get() returns NRF_ERROR_NOT_FOUND. But if we disable the SoftDevice in one of the observers, the next call to sd_ble_evt_get() will return NRF_ERROR_SOFTDEVICE_NOT_ENABLED, and we will enter the error-handler because only NRF_ERROR_NOT_FOUND and NRF_SUCCESS are considered valid return codes.