xiaoxiang781216 commented on pull request #4667:
URL: https://github.com/apache/incubator-nuttx/pull/4667#issuecomment-946107192


   > Disclaimer: I have no familiarity with the loading of ELF files. 
Considering [this past 
commit](https://github.com/apache/incubator-nuttx/commit/b7f958a02ebf0dde83c5022e79e7f8a55939565f),
 what has been changed to prevent the memory allocation failure on 
`elf_loadctors`/`elf_loaddtors`?
   
   In the kernel mode, the userspace heap will be initialized automatically:
   
https://github.com/apache/incubator-nuttx/blob/master/mm/umm_heap/umm_initialize.c#L101-L124
   please search umm_try_initialize, you will find that all possible entry 
pointer call umm_try_initialize before the real allocation.
   BTW, the change is also important to make the delay physical page commit 
work again.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to