Changeset f9425006 in mainline for generic/src/mm/as.c

Timestamp:

2006-01-13T16:29:20Z (19 years ago)

Author:

Jakub Jermar <jakub@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

214e89e3

Parents:

20d50a1

Message:

Make various mips32 tlb manipulation functions preserve EntryHi register.

Cleanup and more precise comments for previous commit.

File:

• generic/src/mm/as.c (modified) (6 diffs)

generic/src/mm/as.c

@@ -59 +59 @@
 
 /*
- * Here we assume that PFN (Physical Frame Numbers) space
+ * Here we assume that PFN (Physical Frame Number) space
  * is smaller than the width of index_t. UNALLOCATED_PFN
  * can be then used to mark mappings wich were not
- * allocated a physical frame.
+ * yet allocated a physical frame.
  */
 #define UNALLOCATED_PFN         ((index_t) -1)
@@ -142 +142 @@
                         /*
                          * Frames will be allocated on-demand by
-                         * as_page_fault().
+                         * as_page_fault() or preloaded by
+                         * as_area_load_mapping().
                          */
                         a->mapping[i] = UNALLOCATED_PFN;
@@ -160 +161 @@
         spinlock_unlock(&as->lock);
         interrupts_restore(ipl);
-        
+
         return a;
 }
@@ -181 +182 @@
         for (i = 0; i < a->size; i++) {
                 ASSERT(a->mapping[i] == UNALLOCATED_PFN);
+                ASSERT(pfn[i] != UNALLOCATED_PFN);
                 a->mapping[i] = pfn[i];
         }
@@ -195 +197 @@
  * @param page Faulting page.
  *
- * @return 0 on page fault, 1 if address space operation
+ * @return 0 on page fault, 1 on success.
  */
 int as_page_fault(__address page)
@@ -251 +253 @@
                 memsetb(frame, FRAME_SIZE, 0);
                 area->mapping[vpn] = frame / FRAME_SIZE;
+                ASSERT(area->mapping[vpn] != UNALLOCATED_PFN);
         } else {
                 frame = area->mapping[vpn] * FRAME_SIZE;