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

File:

• kernel/generic/src/mm/backend_anon.c (modified) (9 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
 };
 
@@ -118 +121 @@
          */
         mutex_lock(&area->sh_info->lock);
-        list_foreach(area->used_space.leaf_list, cur) {
-                btree_node_t *node;
+        list_foreach(area->used_space.leaf_list, leaf_link, 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];
@@ -191 +193 @@
                 return AS_PF_FAULT;
 
-        if (area->sh_info) {
+        mutex_lock(&area->sh_info->lock);
+        if (area->sh_info->shared) {
                 btree_node_t *leaf;
                 
@@ -201 +204 @@
                  * 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);
@@ -233 +235 @@
                 }
                 frame_reference_add(ADDR2PFN(frame));
-                mutex_unlock(&area->sh_info->lock);
         } else {
 
@@ -255 +256 @@
                          * Reserve the memory for this page now.
                          */
-                        if (!reserve_try_alloc(1))
+                        if (!reserve_try_alloc(1)) {
+                                mutex_unlock(&area->sh_info->lock);
                                 return AS_PF_SILENT;
+                        }
                 }
 
@@ -263 +266 @@
                 km_temporary_page_put(kpage);
         }
+        mutex_unlock(&area->sh_info->lock);
         
         /*
@@ -295 +299 @@
                  * the normal unreserving frame_free().
                  */
-                frame_free(frame);
+                frame_free(frame, 1);
         } else {
                 /*
@@ -302 +306 @@
                  * manipulate the reserve or it would be given back twice.
                  */
-                frame_free_noreserve(frame);
+                frame_free_noreserve(frame, 1);
         }
 }