Changeset 409 for trunk/kernel/mm/vmm.h

Timestamp:

Dec 20, 2017, 4:51:09 PM (7 years ago)

Author:

alain

Message:

Fix bugs in exec

File:

• trunk/kernel/mm/vmm.h (modified) (3 diffs)

trunk/kernel/mm/vmm.h

@@ -131 +131 @@
  * - It initializes the STACK and MMAP allocators.
  * - It registers the "kentry", "args", "envs" vsegs in the VSL.
+ * - It initializes the generic page table, calling the HAL specific hal_gpt_init() function.
+ * - For TSAR it map all pages for the "kentry" vseg, that must be identity mapping.
+ * Note:
  * - The "code" and "data" vsegs are registered by the elf_load_process() function.
  * - The "stack" vsegs are dynamically created by the thread_user_create() function.
- * - The "file", "anon", "remote" vsegs are dynamically created by the mmap() syscalls.
- * - It initializes the generic page table, calling the HAL specific hal_gpt_init() function.
- * - For TSAR it map all pages for the "kentry" vseg, that must be identity mapping.
+ * - The "file", "anon", "remote" vsegs are dynamically created by the mmap() syscall.
  * TODO : Any error in this function gives a kernel panic => improve error handling.
  *********************************************************************************************
@@ -206 +207 @@
 
 /*********************************************************************************************
- * This function removes all vsegs registered in in the virtual memory manager of the
- * process identified by the <process> argument. 
- * It releases the memory allocated to the local generic page table.
+ * This function scan the list of vsegs registered in the VSL of the process 
+ * identified by the <process> argument, and for each vseg:
+ * - it unmap from the GPT and releases all mapped pages in vseg.
+ * - it removes the vseg from the process VSL.
+ * - It releases the memory allocated to the vseg descriptor.
+ * Finally, it releases the memory allocated to the GPT itself.
  *********************************************************************************************
  * @ process   : pointer on process descriptor.
@@ -286 +290 @@
 
 /*********************************************************************************************
- * This function unmaps all PTEs of a given vseg, in the generic page table associated
- * to a given process descriptor, and releases the corresponding physical memory.
- * It can be used for any type of vseg.
+ * This function unmaps all mapped PTEs of a given vseg, from the generic page table 
+ * associated to a given process descriptor, and releases the physical memory allocated
+ * to all mapped GPT entries.  It can be used for any type of vseg.
  *********************************************************************************************
  * @ process  : pointer on process descriptor.