xiaoxiang781216 commented on pull request #1112: URL: https://github.com/apache/incubator-nuttx-apps/pull/1112#issuecomment-1083974976
> @xiaoxiang781216 > > > @masayuki2009 the change ensure the heap memory align with MM_MIN_CHUNK, which is a core heap manager assumption. Why this issue isn't reported before? Because the default config(CONFIG_MM_SMALL=n), 2 * SIZEOF_MM_ALLOCNODE(2 * 8)(start + end node) just equals MM_MIN_CHUNK(16). But once CONFIG_MM_SMALL is enabled, 2 * SIZEOF_MM_ALLOCNODE become 2 * 4(8). > > So if the [apache/incubator-nuttx#5906](https://github.com/apache/incubator-nuttx/pull/5906) is not merged, the issue only happens with `CONFIG_MM_SMALL=y` configuration, right? Yes, or CONFIG_DEBUG_MM==y since both option change mm_allocnode_s layout and then may make the assumption fail. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
