Changes in kernel/generic/src/mm/as.c [75b139f:38dc82d] in mainline

File:

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

kernel/generic/src/mm/as.c

@@ -574 +574 @@
  * @param backend_data NULL or a pointer to custom backend data.
  * @param base         Starting virtual address of the area.
- *                     If set to AS_AREA_ANY, a suitable mappable area is
- *                     found.
- * @param bound        Lowest address bound if base is set to AS_AREA_ANY.
+ *                     If set to -1, a suitable mappable area is found.
+ * @param bound        Lowest address bound if base is set to -1.
  *                     Otherwise ignored.
  *
@@ -586 +585 @@
     mem_backend_data_t *backend_data, uintptr_t *base, uintptr_t bound)
 {
-        if ((*base != (uintptr_t) AS_AREA_ANY) && !IS_ALIGNED(*base, PAGE_SIZE))
+        if ((*base != (uintptr_t) -1) && !IS_ALIGNED(*base, PAGE_SIZE))
                 return NULL;
         
@@ -602 +601 @@
         mutex_lock(&as->lock);
         
-        if (*base == (uintptr_t) AS_AREA_ANY) {
+        if (*base == (uintptr_t) -1) {
                 *base = as_get_unmapped_area(as, bound, size, guarded);
                 if (*base == (uintptr_t) -1) {
@@ -889 +888 @@
                 
                                 for (; i < node_size; i++) {
-                                        pte_t *pte = page_mapping_find(as,
-                                            ptr + P2SZ(i), false);
+                                        pte_t pte;
+                                        bool found = page_mapping_find(as,
+                                            ptr + P2SZ(i), false, &pte);
                                         
-                                        ASSERT(pte);
-                                        ASSERT(PTE_VALID(pte));
-                                        ASSERT(PTE_PRESENT(pte));
+                                        ASSERT(found);
+                                        ASSERT(PTE_VALID(&pte));
+                                        ASSERT(PTE_PRESENT(&pte));
                                         
                                         if ((area->backend) &&
@@ -900 +900 @@
                                                 area->backend->frame_free(area,
                                                     ptr + P2SZ(i),
-                                                    PTE_GET_FRAME(pte));
+                                                    PTE_GET_FRAME(&pte));
                                         }
                                         
@@ -1003 +1003 @@
                         
                         for (size = 0; size < (size_t) node->value[i]; size++) {
-                                pte_t *pte = page_mapping_find(as,
-                                     ptr + P2SZ(size), false);
+                                pte_t pte;
+                                bool found = page_mapping_find(as,
+                                     ptr + P2SZ(size), false, &pte);
                                 
-                                ASSERT(pte);
-                                ASSERT(PTE_VALID(pte));
-                                ASSERT(PTE_PRESENT(pte));
+                                ASSERT(found);
+                                ASSERT(PTE_VALID(&pte));
+                                ASSERT(PTE_PRESENT(&pte));
                                 
                                 if ((area->backend) &&
@@ -1014 +1015 @@
                                         area->backend->frame_free(area,
                                             ptr + P2SZ(size),
-                                            PTE_GET_FRAME(pte));
+                                            PTE_GET_FRAME(&pte));
                                 }
                                 
@@ -1315 +1316 @@
                         
                         for (size = 0; size < (size_t) node->value[i]; size++) {
-                                pte_t *pte = page_mapping_find(as,
-                                    ptr + P2SZ(size), false);
+                                pte_t pte;
+                                bool found = page_mapping_find(as,
+                                    ptr + P2SZ(size), false, &pte);
                                 
-                                ASSERT(pte);
-                                ASSERT(PTE_VALID(pte));
-                                ASSERT(PTE_PRESENT(pte));
+                                ASSERT(found);
+                                ASSERT(PTE_VALID(&pte));
+                                ASSERT(PTE_PRESENT(&pte));
                                 
-                                old_frame[frame_idx++] = PTE_GET_FRAME(pte);
+                                old_frame[frame_idx++] = PTE_GET_FRAME(&pte);
                                 
                                 /* Remove old mapping */
@@ -1452 +1454 @@
          * we need to make sure the mapping has not been already inserted.
          */
-        pte_t *pte;
-        if ((pte = page_mapping_find(AS, page, false))) {
-                if (PTE_PRESENT(pte)) {
-                        if (((access == PF_ACCESS_READ) && PTE_READABLE(pte)) ||
-                            (access == PF_ACCESS_WRITE && PTE_WRITABLE(pte)) ||
-                            (access == PF_ACCESS_EXEC && PTE_EXECUTABLE(pte))) {
+        pte_t pte;
+        bool found = page_mapping_find(AS, page, false, &pte);
+        if (found) {
+                if (PTE_PRESENT(&pte)) {
+                        if (((access == PF_ACCESS_READ) && PTE_READABLE(&pte)) ||
+                            (access == PF_ACCESS_WRITE && PTE_WRITABLE(&pte)) ||
+                            (access == PF_ACCESS_EXEC && PTE_EXECUTABLE(&pte))) {
                                 page_table_unlock(AS, false);
                                 mutex_unlock(&area->lock);
@@ -2183 +2186 @@
 
 sysarg_t sys_as_area_create(uintptr_t base, size_t size, unsigned int flags,
-    uintptr_t bound, int pager)
+    uintptr_t bound)
 {
         uintptr_t virt = base;
-        mem_backend_t *backend;
-        mem_backend_data_t backend_data;
-
-        if (pager == AS_AREA_UNPAGED)
-                backend = &anon_backend;
-        else {
-                backend = &user_backend;
-                backend_data.pager = pager;
-        }
         as_area_t *area = as_area_create(AS, flags, size,
-            AS_AREA_ATTR_NONE, backend, &backend_data, &virt, bound);
+            AS_AREA_ATTR_NONE, &anon_backend, NULL, &virt, bound);
         if (area == NULL)
-                return (sysarg_t) AS_MAP_FAILED;
+                return (sysarg_t) -1;
         
         return (sysarg_t) virt;