Donny9 commented on PR #6685: URL: https://github.com/apache/incubator-nuttx/pull/6685#issuecomment-1192479844
> @Donny9 I haven't have the chance to actually test this, but I have a proposition: > > `CONFIG_MM_BACKTRACE` enables two things now. It controls both whether we have information about the owner of a memory block (and thus enables the the relevant information in `ps`), and also produces the dump. > > I believe that these are different functions, and should be controlled independently. > > More specifically, the ownership information should _not_ depend on `CONFIG_DEBUG_MM`. This is useful information that can exist in `proc`, regardless of any debugging taking place. A task may need to know how much heap it consumes. I propose to make this a new Kconfig option. > > On the other hand, the backtrace _is_ a debugging feature, that may be enabled/disabled dependently, when actually needed. I believe that only this should depend on `CONFIG_DEBUG_MM`, and of course to the above new option. > > What do you think? @fjpanag yes, This pr has modified COFNIG_MM_BACKTRACE not to depends on CONFIG_DEBUGMM, is a completely independent config. -- 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
