Merge branch 'master' into prog-loader

[lunaix-os.git] / lunaix-os / kernel / asm / x86 / pfault.c

diff --git a/lunaix-os/kernel/asm/x86/pfault.c b/lunaix-os/kernel/asm/x86/pfault.c
index f6c4355a1c0fabc58fc745b315bd4d74d0c604c6..610c1cb978fd0dd1031a4d5825b6cbbacc8fbb36 100644 (file)
--- a/lunaix-os/kernel/asm/x86/pfault.c
+++ b/lunaix-os/kernel/asm/x86/pfault.c
@@ -38,7 +38,7 @@ intr_routine_page_fault(const isr_param* param)
         goto segv_term;
     }
 
         goto segv_term;
     }
 

-    if (!SEL_RPL(param->cs)) {
+    if (!SEL_RPL(param->execp->cs)) {

         // 如果是内核页错误……
         if (do_kernel(&mapping)) {
             return;
         // 如果是内核页错误……
         if (do_kernel(&mapping)) {
             return;


@@ -79,6 +79,7 @@ intr_routine_page_fault(const isr_param* param)
             }
 
             *pte = *pte | pa | PG_PRESENT;
             }
 
             *pte = *pte | pa | PG_PRESENT;

+            memset(PG_ALIGN(ptr), 0, PG_SIZE);

             goto resolved;
         }
         // permission denied on anon page (e.g., write on readonly page)
             goto resolved;
         }
         // permission denied on anon page (e.g., write on readonly page)


@@ -88,8 +89,10 @@ intr_routine_page_fault(const isr_param* param)
     // if mfile is set (Non-anonymous), then it is a mem map
     if (hit_region->mfile && !PG_IS_PRESENT(*pte)) {
         struct v_file* file = hit_region->mfile;
     // if mfile is set (Non-anonymous), then it is a mem map
     if (hit_region->mfile && !PG_IS_PRESENT(*pte)) {
         struct v_file* file = hit_region->mfile;

-        u32_t offset =
-          ((ptr - hit_region->start) & (PG_SIZE - 1)) + hit_region->foff;
+
+        ptr = PG_ALIGN(ptr);
+
+        u32_t offset = (ptr - hit_region->start) + hit_region->foff;

         uintptr_t pa = pmm_alloc_page(__current->pid, 0);
 
         if (!pa) {
         uintptr_t pa = pmm_alloc_page(__current->pid, 0);
 
         if (!pa) {


@@ -99,7 +102,6 @@ intr_routine_page_fault(const isr_param* param)
         cpu_invplg(pte);
         *pte = (*pte & 0xFFF) | pa | PG_PRESENT;
 
         cpu_invplg(pte);
         *pte = (*pte & 0xFFF) | pa | PG_PRESENT;
 

-        ptr = PG_ALIGN(ptr);

         memset(ptr, 0, PG_SIZE);
 
         int errno = 0;
         memset(ptr, 0, PG_SIZE);
 
         int errno = 0;


@@ -130,8 +132,8 @@ segv_term:
     kprintf(KERROR "(pid: %d) Segmentation fault on %p (%p:%p)\n",
             __current->pid,
             ptr,
     kprintf(KERROR "(pid: %d) Segmentation fault on %p (%p:%p)\n",
             __current->pid,
             ptr,

-            param->cs,
-            param->eip);
+            param->execp->cs,
+            param->execp->eip);

     __SIGSET(__current->sig_pending, _SIGSEGV);
     schedule();
     // should not reach
     __SIGSET(__current->sig_pending, _SIGSEGV);
     schedule();
     // should not reach