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/05/02 19:24:31 UTC

[GitHub] [incubator-nuttx] jlaitine commented on pull request #6025: Add vm map

jlaitine commented on PR #6025:
URL: https://github.com/apache/incubator-nuttx/pull/6025#issuecomment-1115274307

   Re-based against current master
   
   Changed vm_destroy to unmap any regions left, ensuring that any kernel side allocations made by drivers are also freed when a task group exits.
   
   I disagree on the review requirement of opening another file handle on mmap, and on the requirement of expanding this PR into re-writing rammaps or changing other fs parts which are working. I have now already explained why those things should not be done in this PR, many times.
   
   So I won't be doing those, feel free to close this PR if it is not accepted. I am open to relevant review comments still of course.
   


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