drivers: wifi: Create dedicated mem pool for Wi-Fi driver #15169
Conversation
github-actions
bot
added
manifest
changelog-entry-required
|
The following west manifest projects have been modified in this Pull Request:
Note: This message is automatically posted and updated by the Manifest GitHub Action. |
rado17
changed the title
|
You can find the documentation preview for this PR at this link. It will be updated about 10 minutes after the documentation build succeeds. Note: This comment is automatically posted by the Documentation Publishing GitHub Action. |
rado17
force-pushed
the
dedicated_mem_pool
branch
from
59d9bd8 to
11ebac8
Compare
Pull in changes for dedicated memory heaps. Signed-off-by: Ravi Dondaputi <[email protected]>
Create dedicated memory pools for Wi-Fi management and data operations (defaults: 20KB for management and 60KB for data). Reduce the heap memory pool size since the Wi-Fi driver will be allocating memory from a dedicated pool. Signed-off-by: Ravi Dondaputi <[email protected]>
rado17
force-pushed
the
dedicated_mem_pool
branch
from
11ebac8 to
1b83a6a
Compare
| @@ -596,6 +596,14 @@ if NETWORKING | |||
| config HEAP_MEM_POOL_SIZE | |||
| default 153000 | |||
|
|
|||
| config NRF_WIFI_CONFIG_HEAP_MEM_POOL_SIZE | |||
There was a problem hiding this comment.
| config NRF_WIFI_CONFIG_HEAP_MEM_POOL_SIZE | |
| config NRF_WIFI_CTRL_HEAP_SIZE |
| @@ -596,6 +596,14 @@ if NETWORKING | |||
| config HEAP_MEM_POOL_SIZE | |||
| default 153000 | |||
|
|
|||
| config NRF_WIFI_CONFIG_HEAP_MEM_POOL_SIZE | |||
| int "Dedicated memory pool for configuration" | |||
There was a problem hiding this comment.
| int "Dedicated memory pool for configuration" | |
| int "Dedicated memory pool for control path" |
| int "Dedicated memory pool for configuration" | ||
| default 20480 | ||
|
|
||
| config NRF_WIFI_DATA_HEAP_MEM_POOL_SIZE |
There was a problem hiding this comment.
| config NRF_WIFI_DATA_HEAP_MEM_POOL_SIZE | |
| config NRF_WIFI_DATA_HEAP_SIZE |
|
|
||
| static void zep_shim_mem_free(void *buf) | ||
| { | ||
| if (buf) { |
There was a problem hiding this comment.
no need, ISO C semantics are followed in Zephyr.
| @@ -12,7 +12,7 @@ CONFIG_NET_PKT_RX_COUNT=30 | |||
| CONFIG_NET_PKT_TX_COUNT=29 | |||
| CONFIG_NET_BUF_RX_COUNT=30 | |||
| CONFIG_NET_BUF_TX_COUNT=58 | |||
| CONFIG_HEAP_MEM_POOL_SIZE=230000 | |||
| #CONFIG_HEAP_MEM_POOL_SIZE=140000 | |||
There was a problem hiding this comment.
please remove commented code
| @@ -32,11 +32,11 @@ CONFIG_NET_PKT_TX_COUNT=8 | |||
| # tuned for performance, but this will be revisited in the future. | |||
| CONFIG_NET_BUF_RX_COUNT=16 | |||
| CONFIG_NET_BUF_TX_COUNT=16 | |||
| CONFIG_NET_BUF_DATA_SIZE=128 | |||
| CONFIG_NET_BUF_DATA_SIZE=1500 | |||
There was a problem hiding this comment.
why is this needed? Default config is only for ping
|
This pull request has been marked as stale because it has been open (more than) 60 days with no activity. Remove the stale label or add a comment saying that you would like to have the label removed otherwise this pull request will automatically be closed in 14 days. Note, that you can always re-open a closed pull request at any time. |
|
Migrated to upstream zephyrproject-rtos/zephyr#78588 |
Create dedicated memory pools for Wi-Fi management and
data operations (defaults: 20KB for management and 60KB for data).
Reduce the heap memory pool size since the Wi-Fi driver will
be allocating memory from a dedicated pool.