You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nuttx.apache.org by GitBox <gi...@apache.org> on 2022/03/29 15:42:30 UTC

[GitHub] [incubator-nuttx] CV-Bowen opened a new pull request #5906: mm_heap: heapsize align with MM_MIN_CHUNK.

CV-Bowen opened a new pull request #5906:
URL: https://github.com/apache/incubator-nuttx/pull/5906


   Make sure the (heapsize - sizeof(heap->mm_heapstart) - sizeof(heap->mm_heapend)) be aligned with MM_MIN_CHUNK to avoid memory assert question when realloc a mem.
   
   Signed-off-by: wangbowen6 <wa...@xiaomi.com>
   
   ## Summary
   
   ## Impact
   
   ## Testing
   
   


-- 
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-unsubscribe@nuttx.apache.org

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



[GitHub] [incubator-nuttx] CV-Bowen commented on pull request #5906: mm_heap: heapsize align with MM_MIN_CHUNK.

Posted by GitBox <gi...@apache.org>.
CV-Bowen commented on pull request #5906:
URL: https://github.com/apache/incubator-nuttx/pull/5906#issuecomment-1082604402


   @masayuki2009 OK,i will do this.


-- 
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-unsubscribe@nuttx.apache.org

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



[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #5906: mm_heap: heapsize align with MM_MIN_CHUNK.

Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 merged pull request #5906:
URL: https://github.com/apache/incubator-nuttx/pull/5906


   


-- 
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-unsubscribe@nuttx.apache.org

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



[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #5906: mm_heap: heapsize align with MM_MIN_CHUNK.

Posted by GitBox <gi...@apache.org>.
masayuki2009 commented on pull request #5906:
URL: https://github.com/apache/incubator-nuttx/pull/5906#issuecomment-1082602868


   @CV-Bowen 
   
   >Make sure the (heapsize - sizeof(heap->mm_heapstart) - sizeof(heap->mm_heapend)) be aligned with MM_MIN_CHUNK >to avoid memory assert question when realloc a mem.
   
   Can you add some test cases to apps/testing/mm so that we can test the assertion?


-- 
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-unsubscribe@nuttx.apache.org

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



[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #5906: mm_heap: heapsize align with MM_MIN_CHUNK.

Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 commented on pull request #5906:
URL: https://github.com/apache/incubator-nuttx/pull/5906#issuecomment-1084479157


   Ok, let's merge this patch now.


-- 
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-unsubscribe@nuttx.apache.org

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