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

xtensa/esp32: ESP32 not use IMEM in user heap mode #9020

Conversation

donghengqaz
Copy link
Contributor

@donghengqaz donghengqaz commented Apr 13, 2023

Summary

When users enable Separated userspace heap(ESP32_SPIRAM_USER_HEAP), system don't enable separated heap region of SRAM (ESP32_IMM_HEAP -> XTENSA_IMEM_USE_SEPARATE_HEAP), so that users can have a full larger block of heap.

If users still enable Additional region to kernel heap(ESP32_SPIRAM_COMMON_HEAP), system enable separated heap region of SRAM (ESP32_IMM_HEAP -> XTENSA_IMEM_USE_SEPARATE_HEAP), so that kernel driver can allocate heap from SRAM.

Impact

Testing

@donghengqaz donghengqaz force-pushed the feature/esp32_not_use_imem_in_user_heap_mode branch 2 times, most recently from 01cc504 to 6456d5f Compare April 13, 2023 11:08
@gustavonihei
Copy link
Contributor

@donghengqaz Please fill the required sections in the PR description.

@pkarashchenko
Copy link
Contributor

Please rebase to fix the CI

@donghengqaz donghengqaz force-pushed the feature/esp32_not_use_imem_in_user_heap_mode branch from 6456d5f to c9a202d Compare April 17, 2023 02:17
@donghengqaz
Copy link
Contributor Author

Please rebase to fix the CI

OK, it's passed.

@donghengqaz donghengqaz force-pushed the feature/esp32_not_use_imem_in_user_heap_mode branch from c9a202d to c49bec7 Compare April 18, 2023 06:38
@donghengqaz donghengqaz force-pushed the feature/esp32_not_use_imem_in_user_heap_mode branch from c49bec7 to 231c923 Compare April 18, 2023 10:01
@gustavonihei gustavonihei merged commit 45bba6e into apache:master Apr 18, 2023
26 checks passed
@donghengqaz donghengqaz deleted the feature/esp32_not_use_imem_in_user_heap_mode branch April 19, 2023 02:00
@jerpelea jerpelea added this to To-Add in Release Notes - 12.2.0 Jun 13, 2023
@jerpelea jerpelea moved this from To-Add to In Progress in Release Notes - 12.2.0 Jun 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

3 participants