Changes in kernel/arch/ia64/src/mm/tlb.c [1dbc43f:560b81c] in mainline

File:

• kernel/arch/ia64/src/mm/tlb.c (modified) (15 diffs)

kernel/arch/ia64/src/mm/tlb.c

@@ -477 +477 @@
 /** Instruction TLB fault handler for faults with VHPT turned off.
  *
- * @param vector Interruption vector.
+ * @param n Interruption vector.
  * @param istate Structure with saved interruption state.
  *
  */
-void alternate_instruction_tlb_fault(uint64_t vector, istate_t *istate)
+void alternate_instruction_tlb_fault(unsigned int n, istate_t *istate)
 {
         uintptr_t va;
-        pte_t *t;
+        pte_t t;
         
         va = istate->cr_ifa; /* faulting address */
@@ -490 +490 @@
         ASSERT(!is_kernel_fault(va));
 
-        t = page_mapping_find(AS, va, true);
-        if (t) {
+        bool found = page_mapping_find(AS, va, true, &t);
+        if (found) {
+                ASSERT(t.p);
+
                 /*
                  * The mapping was found in software page hash table.
                  * Insert it into data translation cache.
                  */
-                itc_pte_copy(t);
+                itc_pte_copy(&t);
         } else {
                 /*
@@ -566 +568 @@
 /** Data TLB fault handler for faults with VHPT turned off.
  *
- * @param vector Interruption vector.
+ * @param n Interruption vector.
  * @param istate Structure with saved interruption state.
  *
  */
-void alternate_data_tlb_fault(uint64_t vector, istate_t *istate)
+void alternate_data_tlb_fault(unsigned int n, istate_t *istate)
 {
         if (istate->cr_isr.sp) {
@@ -600 +602 @@
         
         
-        pte_t *entry = page_mapping_find(as, va, true);
-        if (entry) {
+        pte_t t;
+        bool found = page_mapping_find(as, va, true, &t);
+        if (found) {
+                ASSERT(t.p);
+
                 /*
                  * The mapping was found in the software page hash table.
                  * Insert it into data translation cache.
                  */
-                dtc_pte_copy(entry);
+                dtc_pte_copy(&t);
         } else {
                 if (try_memmap_io_insertion(va, istate))
@@ -623 +628 @@
  * This fault should not occur.
  *
- * @param vector Interruption vector.
+ * @param n Interruption vector.
  * @param istate Structure with saved interruption state.
  *
  */
-void data_nested_tlb_fault(uint64_t vector, istate_t *istate)
+void data_nested_tlb_fault(unsigned int n, istate_t *istate)
 {
         ASSERT(false);
@@ -634 +639 @@
 /** Data Dirty bit fault handler.
  *
- * @param vector Interruption vector.
+ * @param n Interruption vector.
  * @param istate Structure with saved interruption state.
  *
  */
-void data_dirty_bit_fault(uint64_t vector, istate_t *istate)
+void data_dirty_bit_fault(unsigned int n, istate_t *istate)
 {
         uintptr_t va;
-        pte_t *t;
+        pte_t t;
         as_t *as = AS;
         
@@ -649 +654 @@
                 as = AS_KERNEL;
 
-        t = page_mapping_find(as, va, true);
-        ASSERT((t) && (t->p));
-        if ((t) && (t->p) && (t->w)) {
+        bool found = page_mapping_find(as, va, true, &t);
+
+        ASSERT(found);
+        ASSERT(t.p);
+
+        if (found && t.p && t.w) {
                 /*
                  * Update the Dirty bit in page tables and reinsert
                  * the mapping into DTC.
                  */
-                t->d = true;
-                dtc_pte_copy(t);
+                t.d = true;
+                dtc_pte_copy(&t);
+                page_mapping_update(as, va, true, &t);
         } else {
                 as_page_fault(va, PF_ACCESS_WRITE, istate);
@@ -665 +674 @@
 /** Instruction access bit fault handler.
  *
- * @param vector Interruption vector.
+ * @param n Interruption vector.
  * @param istate Structure with saved interruption state.
  *
  */
-void instruction_access_bit_fault(uint64_t vector, istate_t *istate)
+void instruction_access_bit_fault(unsigned int n, istate_t *istate)
 {
         uintptr_t va;
-        pte_t *t;
+        pte_t t;
         
         va = istate->cr_ifa;  /* faulting address */
@@ -678 +687 @@
         ASSERT(!is_kernel_fault(va));
         
-        t = page_mapping_find(AS, va, true);
-        ASSERT((t) && (t->p));
-        if ((t) && (t->p) && (t->x)) {
+        bool found = page_mapping_find(AS, va, true, &t);
+
+        ASSERT(found);
+        ASSERT(t.p);
+
+        if (found && t.p && t.x) {
                 /*
                  * Update the Accessed bit in page tables and reinsert
                  * the mapping into ITC.
                  */
-                t->a = true;
-                itc_pte_copy(t);
+                t.a = true;
+                itc_pte_copy(&t);
+                page_mapping_update(AS, va, true, &t);
         } else {
                 as_page_fault(va, PF_ACCESS_EXEC, istate);
@@ -694 +707 @@
 /** Data access bit fault handler.
  *
- * @param vector Interruption vector.
+ * @param n Interruption vector.
  * @param istate Structure with saved interruption state.
  *
  */
-void data_access_bit_fault(uint64_t vector, istate_t *istate)
+void data_access_bit_fault(unsigned int n, istate_t *istate)
 {
         uintptr_t va;
-        pte_t *t;
+        pte_t t;
         as_t *as = AS;
         
@@ -709 +722 @@
                 as = AS_KERNEL;
 
-        t = page_mapping_find(as, va, true);
-        ASSERT((t) && (t->p));
-        if ((t) && (t->p)) {
+        bool found = page_mapping_find(as, va, true, &t);
+
+        ASSERT(found);
+        ASSERT(t.p);
+
+        if (found && t.p) {
                 /*
                  * Update the Accessed bit in page tables and reinsert
                  * the mapping into DTC.
                  */
-                t->a = true;
-                dtc_pte_copy(t);
+                t.a = true;
+                dtc_pte_copy(&t);
+                page_mapping_update(as, va, true, &t);
         } else {
                 if (as_page_fault(va, PF_ACCESS_READ, istate) == AS_PF_FAULT) {
@@ -729 +746 @@
 /** Data access rights fault handler.
  *
- * @param vector Interruption vector.
+ * @param n Interruption vector.
  * @param istate Structure with saved interruption state.
  *
  */
-void data_access_rights_fault(uint64_t vector, istate_t *istate)
+void data_access_rights_fault(unsigned int n, istate_t *istate)
 {
         uintptr_t va;
-        pte_t *t;
+        pte_t t;
         
         va = istate->cr_ifa;  /* faulting address */
@@ -745 +762 @@
          * Assume a write to a read-only page.
          */
-        t = page_mapping_find(AS, va, true);
-        ASSERT((t) && (t->p));
-        ASSERT(!t->w);
+        bool found = page_mapping_find(AS, va, true, &t);
+
+        ASSERT(found);
+        ASSERT(t.p);
+        ASSERT(!t.w);
+
         as_page_fault(va, PF_ACCESS_WRITE, istate);
 }
@@ -753 +773 @@
 /** Page not present fault handler.
  *
- * @param vector Interruption vector.
+ * @param n Interruption vector.
  * @param istate Structure with saved interruption state.
  *
  */
-void page_not_present(uint64_t vector, istate_t *istate)
+void page_not_present(unsigned int n, istate_t *istate)
 {
         uintptr_t va;
-        pte_t *t;
+        pte_t t;
         
         va = istate->cr_ifa;  /* faulting address */
@@ -766 +786 @@
         ASSERT(!is_kernel_fault(va));
 
-        t = page_mapping_find(AS, va, true);
-        ASSERT(t);
-        
-        if (t->p) {
+        bool found = page_mapping_find(AS, va, true, &t);
+
+        ASSERT(found);
+        
+        if (t.p) {
                 /*
                  * If the Present bit is set in page hash table, just copy it
                  * and update ITC/DTC.
                  */
-                if (t->x)
-                        itc_pte_copy(t);
+                if (t.x)
+                        itc_pte_copy(&t);
                 else
-                        dtc_pte_copy(t);
+                        dtc_pte_copy(&t);
         } else {
                 as_page_fault(va, PF_ACCESS_READ, istate);