espressif / esp-hosted

Hosted Solution (Linux/MCU) with ESP32 (Wi-Fi + BT + BLE)
Other
674 stars 158 forks source link

Add WiFi packet flow control #455

Open udoudou opened 1 month ago

udoudou commented 1 month ago

I added flow control for wifi packets. This will greatly reduce the packet loss rate (tested by sending udp via iperf). But it only supports the sdio interface because I think the current spi communication implementation is not compatible with the flow control mechanism. At the same time, when I checked the spi communication, I found that there are still some problems: The number of SPI memory pools is only SPI_MEMPOOL_NUM_BLOCKS, but the number of spi_tx_queue and spi_rx_queue queues is three times that. I think in extreme cases, when the memory pool is exhausted and all the memory is in spi_rx_queue/spi_tx_queue, the next spi transmission will fail to apply for memory, triggering assert

CLAassistant commented 1 month ago

CLA assistant check
All committers have signed the CLA.

mantriyogesh commented 1 week ago

Hello @udoudou ,

Would it be possible to explain the change in some kind of write up?