You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@nuttx.apache.org by "CV-Bowen (via GitHub)" <gi...@apache.org> on 2023/01/30 09:33:12 UTC

[GitHub] [nuttx] CV-Bowen opened a new pull request, #8353: Reduce the memory node overhead size

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

   ## Summary
   This PR reduces the memory node overhead size to half of the original by move the `node->preceding` to previous free node in physical like TLSF does (http://www.gii.upv.es/tlsf/main/docs).
   
   Original memory layout of Allocnode and Freenode:
   ![image](https://user-images.githubusercontent.com/39286361/215438882-1825529a-3fdf-4fc0-8cc2-58dfd0e99234.png)
   
   This PR contains two commits:
   Commit 1: Exchange `node->size` and `node->preceding` position and move the MM_ALLOC_BIT to `node->size`, this commit is a preparation for move the `node->preceding` to previous free node in physical address. 
   After Commit 1, the memory layout of allocnode and freenode become:
   ![image](https://user-images.githubusercontent.com/39286361/215438963-35176f4b-5726-47d7-8f86-f36eeb924052.png)
   
   Commit 2: Add MM_PREVFREE_BIT in `node->size` and move the `node->preceding` to previous free node in physical. 
   After Commit 2, the memory layout of allocnode and freenode become:
   ![image](https://user-images.githubusercontent.com/39286361/215439734-f366b1f9-7166-4249-9384-c5c563c50205.png)
   
   ## Impact
   Dynamic Memory Managerment
   
   ## Testing
   Vela and sim mmtest
   


-- 
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] [nuttx] CV-Bowen commented on pull request #8353: mm/mm_heap: reduce the memory node overhead size

Posted by "CV-Bowen (via GitHub)" <gi...@apache.org>.

CV-Bowen commented on PR #8353:
URL: https://github.com/apache/nuttx/pull/8353#issuecomment-1409639844

   > @CV-Bowen what is "sim mmtest" ? I couldn't find it here :-)
   
   Sorry, the mmtest is `mm` command and you can enable it by set `CONFIG_TESTING_MM=y`.


-- 
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] [nuttx] acassis commented on pull request #8353: mm/mm_heap: reduce the memory node overhead size

Posted by "acassis (via GitHub)" <gi...@apache.org>.

acassis commented on PR #8353:
URL: https://github.com/apache/nuttx/pull/8353#issuecomment-1409471945

   @CV-Bowen what is "sim mmtest" ? I couldn't find it here :-)


-- 
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] [nuttx] xiaoxiang781216 merged pull request #8353: mm/mm_heap: reduce the memory node overhead size

Posted by "xiaoxiang781216 (via GitHub)" <gi...@apache.org>.

xiaoxiang781216 merged PR #8353:
URL: https://github.com/apache/nuttx/pull/8353


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