[GitHub] [incubator-nuttx] mage0811 commented on a change in pull request #3086: improve the memory manager modularity and reduce the inforamtion explosion

[GitBox <gi...@apache.org>]

mage0811 commented on a change in pull request #3086:
URL: https://github.com/apache/incubator-nuttx/pull/3086#discussion_r596539192



##########
File path: mm/Kconfig
##########
@@ -3,6 +3,23 @@
 # see the file kconfig-language.txt in the NuttX tools repository.
 #
 
+choice
+	prompt "Build heap manager"
+	default MM_DEFAULT_MANAGER
+
+config MM_DEFAULT_MANAGER
+	bool "Default heap manager"
+	---help---
+		NuttX original memory manager strategy.
+
+config MM_CUSTOMIZE_MANAGER
+	bool "Customized heap manager"
+	---help---
+		Customized memory manger policy. The build will failed

Review comment:
       hi，when select "MM_CUSTOMIZE_MANAGER", then the user should develop the customize mm first.
   
   please check my patch "mm: add heap policy and anta logic to support mm debug on sim platform #3093",
   this is my customize mm policy used under sim platform for sanitizers debug tool.
   




----------------------------------------------------------------
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.

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