Skip to content

[noup] zephyr: Create dedicated heap for supplicant#74

Merged
carlescufi merged 1 commit intozephyrproject-rtos:mainfrom
rado17:supplicant_dedicated_heap
Jul 21, 2025
Merged

[noup] zephyr: Create dedicated heap for supplicant#74
carlescufi merged 1 commit intozephyrproject-rtos:mainfrom
rado17:supplicant_dedicated_heap

Conversation

@rado17
Copy link
Copy Markdown
Contributor

@rado17 rado17 commented Jan 2, 2025

Create dedicated heap for supplicant operations. Currently, WPA supplicant uses the k_malloc, which is shared across various subsystems, and we cannot enforce a minimum availability. Using a dedicated HEAP that can be pre-allocated solves this by always having necessary memory.

@MaochenWang1
Copy link
Copy Markdown
Collaborator

why we need this dedicated heap for supplicant?
BTW, currently WPA supplicant uses the kernel k_malloc, not the libc malloc

@rado17
Copy link
Copy Markdown
Contributor Author

rado17 commented Jan 2, 2025

@sachinthegreen

why we need this dedicated heap for supplicant? BTW, currently WPA supplicant uses the kernel k_malloc, not the libc malloc

Updated the description to correct this. Since the kernel heap is shared across various subsystems, we cannot enforce a minimum availability for supplicant. Using a pre-allocated heap solves this problem.

@rado17 rado17 force-pushed the supplicant_dedicated_heap branch from 84f3e4d to 7c3d29d Compare June 9, 2025 07:35
@rado17 rado17 requested a review from MaochenWang1 as a code owner June 9, 2025 07:35
Copy link
Copy Markdown
Collaborator

@krish2718 krish2718 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add a config option to use global or dedicated heap.

@rado17 rado17 force-pushed the supplicant_dedicated_heap branch from a8bf183 to 30ab616 Compare July 2, 2025 12:23
@rado17 rado17 requested a review from krish2718 July 2, 2025 12:55
Create dedicated heap for supplicant operations.

Signed-off-by: Ravi Dondaputi <ravi.dondaputi@nordicsemi.no>
@rado17 rado17 force-pushed the supplicant_dedicated_heap branch from 30ab616 to 9220b5a Compare July 3, 2025 10:15
@rado17 rado17 requested a review from jukkar July 3, 2025 11:40
@carlescufi carlescufi merged commit 09fe7bf into zephyrproject-rtos:main Jul 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants