pussuw commented on code in PR #7966:
URL: https://github.com/apache/nuttx/pull/7966#discussion_r1059984109


##########
include/nuttx/sched.h:
##########
@@ -525,6 +526,9 @@ struct tcb_s
 
   /* Task Group *************************************************************/
 
+#ifdef CONFIG_ARCH_ADDRENV
+  FAR struct task_group_s *mm_group;   /* Active memory mappings group      */

Review Comment:
   Another option would be to store the address environment only here but that 
gets a bit dirty IMO. It would require changing group->addrenv into a pointer 
and allocating the addrenv from kheap. 
   
   My reason for not doing this is that each group needs the address 
environment, so why try allocating it separately from heap ?



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

Reply via email to