Changes in kernel/generic/src/mm/backend_anon.c [38dc82d:59fb782] in mainline

File:

• kernel/generic/src/mm/backend_anon.c (modified) (10 diffs)

kernel/generic/src/mm/backend_anon.c

@@ -76 +76 @@
         .page_fault = anon_page_fault,
         .frame_free = anon_frame_free,
-
-        .create_shared_data = NULL,
-        .destroy_shared_data = NULL
 };
 
@@ -121 +118 @@
          */
         mutex_lock(&area->sh_info->lock);
-        list_foreach(area->used_space.leaf_list, leaf_link, btree_node_t,
-            node) {
+        list_foreach(area->used_space.leaf_list, cur) {
+                btree_node_t *node;
                 unsigned int i;
                 
+                node = list_get_instance(cur, btree_node_t, leaf_link);
                 for (i = 0; i < node->keys; i++) {
                         uintptr_t base = node->key[i];
@@ -131 +129 @@
                         
                         for (j = 0; j < count; j++) {
-                                pte_t pte;
-                                bool found;
+                                pte_t *pte;
                         
                                 page_table_lock(area->as, false);
-                                found = page_mapping_find(area->as,
-                                    base + P2SZ(j), false, &pte);
-
-                                ASSERT(found);
-                                ASSERT(PTE_VALID(&pte));
-                                ASSERT(PTE_PRESENT(&pte));
-
+                                pte = page_mapping_find(area->as,
+                                    base + P2SZ(j), false);
+                                ASSERT(pte && PTE_VALID(pte) &&
+                                    PTE_PRESENT(pte));
                                 btree_insert(&area->sh_info->pagemap,
                                     (base + P2SZ(j)) - area->base,
-                                    (void *) PTE_GET_FRAME(&pte), NULL);
+                                    (void *) PTE_GET_FRAME(pte), NULL);
                                 page_table_unlock(area->as, false);
 
-                                pfn_t pfn = ADDR2PFN(PTE_GET_FRAME(&pte));
+                                pfn_t pfn = ADDR2PFN(PTE_GET_FRAME(pte));
                                 frame_reference_add(pfn);
                         }
@@ -197 +191 @@
                 return AS_PF_FAULT;
 
-        mutex_lock(&area->sh_info->lock);
-        if (area->sh_info->shared) {
+        if (area->sh_info) {
                 btree_node_t *leaf;
                 
@@ -208 +201 @@
                  * mapping, a new frame is allocated and the mapping is created.
                  */
+                mutex_lock(&area->sh_info->lock);
                 frame = (uintptr_t) btree_search(&area->sh_info->pagemap,
                     upage - area->base, &leaf);
@@ -239 +233 @@
                 }
                 frame_reference_add(ADDR2PFN(frame));
+                mutex_unlock(&area->sh_info->lock);
         } else {
 
@@ -260 +255 @@
                          * Reserve the memory for this page now.
                          */
-                        if (!reserve_try_alloc(1)) {
-                                mutex_unlock(&area->sh_info->lock);
+                        if (!reserve_try_alloc(1))
                                 return AS_PF_SILENT;
-                        }
                 }
 
@@ -270 +263 @@
                 km_temporary_page_put(kpage);
         }
-        mutex_unlock(&area->sh_info->lock);
         
         /*
@@ -303 +295 @@
                  * the normal unreserving frame_free().
                  */
-                frame_free(frame, 1);
+                frame_free(frame);
         } else {
                 /*
@@ -310 +302 @@
                  * manipulate the reserve or it would be given back twice.
                  */
-                frame_free_noreserve(frame, 1);
+                frame_free_noreserve(frame);
         }
 }