Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

soc: espressif: minimal runtime heap size request #81780

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

marekmatej
Copy link

Introduce config option to make sure enough space will be available for the runtime heap.

Introduce config option to make sure enough space will be available for
the runtime heap.

Signed-off-by: Marek Matej <[email protected]>
@marekmatej marekmatej added the platform: ESP32 Espressif ESP32 label Nov 22, 2024
Comment on lines +39 to +41
Ask for reserved space at the end of DRAM. Not having this minimal size could pottentionaly
lead to statically allocate all of available DRAM, not having enough for the subsystems
which depends on ESP_HEAP_RUNTIME (such as WiFi or BT).
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
Ask for reserved space at the end of DRAM. Not having this minimal size could pottentionaly
lead to statically allocate all of available DRAM, not having enough for the subsystems
which depends on ESP_HEAP_RUNTIME (such as WiFi or BT).
Ask for reserved space at the end of DRAM. Not having this minimal size could potentially
lead to statically allocating all of available DRAM, not having enough for the subsystems
which depends on ESP_HEAP_RUNTIME (such as WiFi or BT).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
platform: ESP32 Espressif ESP32
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants