Assertion Fail Using ble_nus_client_send with UART in NRF Connect SDK 1.8.0

RE: Assertion Fail Using ble_nus_client_send with UART in NRF Connect SDK 1.8.0

Kazi Afroza Sultana — Mon, 09 May 2022 09:31:55 GMT

Hello,

Great. if you want can verify the answer or I can close the case.

Best Regards,

Kazi Afroza Sultana

RE: Assertion Fail Using ble_nus_client_send with UART in NRF Connect SDK 1.8.0

jake11212 — Fri, 06 May 2022 21:34:02 GMT

This was it, working with the line in conn.c back in, thanks so much!

RE: Assertion Fail Using ble_nus_client_send with UART in NRF Connect SDK 1.8.0

Sigurd — Fri, 29 Apr 2022 09:09:57 GMT

Hi,

PDU stands for protocol data unit, commonly known as packets. Looks like you can't call bt_nus_client_send() from the UART ISR. Instead, just put the UART data in a FIFO, e.g. with k_fifo_put(), and have a separate thread that sends the data. E.g. as done in peripheral_uart sample.

RE: Assertion Fail Using ble_nus_client_send with UART in NRF Connect SDK 1.8.0

jake11212 — Thu, 21 Apr 2022 17:22:41 GMT

[quote userid="108933" url="~/f/nordic-q-a/86948/assertion-fail-using-ble_nus_client_send-with-uart-in-nrf-connect-sdk-1-8-0/364224#364224"]Are you sending the data from the UART Interrupt Handler[/quote]

Are you referring to data sent over UART or data send with the Nordic UART Service (for BLE)?

I am just calling both directly, could that be an issue? So you're recommending I generate an interrupt then use that interrupt handler to send the data?

Also my code for sending over UART is:

int uart_send(struct uart_data_t *uart_data)
{
    int status = uart_tx(device_uart, uart_data->data, uart_data->length, SYS_FOREVER_MS);

    if (status != 0)
    {
        printk("uart_tx ( error: %d )\n", status);

        // Add to tx queue to try and process / send later ...
        k_fifo_put(&fifo_uart_tx_data, uart_data);
    }

    return status;
}

Also for more context on the error I've started debugging with Ozone, this is the full error state that I'm still working through:

The line throwing the error was in: ..\ncs\v1.8.0\zephyr\subsys\bluetooth\host\conn.c line 1152 which is:

__ASSERT_NO_MSG(!k_is_in_isr());

It has the associated comment:

* PDU must not be allocated from ISR as we block with 'K_FOREVER'

* during the allocation

I wasn't sure what this meant, could you elaborate on what might be causing this error to throw. ISR is the interrupt service routine but I'm not sure what PDR is or why my code might have caused this assert. For testing purposes I commented out the line and the send worked properly but I want to solve the issue causing the error.

RE: Assertion Fail Using ble_nus_client_send with UART in NRF Connect SDK 1.8.0

Kazi Afroza Sultana — Thu, 21 Apr 2022 12:39:54 GMT

Hello,

Are you sending the data from the UART Interrupt Handler sdk-zephyr/conn.c at v2.7.0-ncs1 · nrfconnect/sdk-zephyr · GitHub? You can use a work thread or similar to send the data instead of calling it directly from the uart_handler.

Thanks.

Best Regards,

Kazi Afroza Sultana

RE: Assertion Fail Using ble_nus_client_send with UART in NRF Connect SDK 1.8.0

Kazi Afroza Sultana — Tue, 19 Apr 2022 12:03:28 GMT

Hello,

Your case arrived in Easter vacation so a bit late reply. I have started looking at your case and will get back to you later.

Thanks.

Best Regards,

Kazi Afroza Sultana