Changeset 1916d1f in mainline for kernel/generic/src/ddi/ddi.c

Timestamp:

2011-07-12T13:41:26Z (13 years ago)

Author:

Petr Koupy <petr.koupy@…>

Branches:

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

Children:

50fc490

Parents:

11809eab (diff), 6817eba (diff)
Note: this is a merge changeset, the changes displayed below correspond to the merge itself.
Use the (diff) links above to see all the changes relative to each parent.

Message:

Merge libposix changes.

File:

• kernel/generic/src/ddi/ddi.c (modified) (4 diffs)

kernel/generic/src/ddi/ddi.c

@@ -122 +122 @@
         backend_data.frames = pages;
         
-        /* Find the zone of the physical memory */
+        /*
+         * Check if the memory region is explicitly enabled
+         * for mapping by any parea structure.
+         */
+        
+        mutex_lock(&parea_lock);
+        btree_node_t *nodep;
+        parea_t *parea = (parea_t *) btree_search(&parea_btree,
+            (btree_key_t) pf, &nodep);
+        
+        if ((parea != NULL) && (parea->frames >= pages)) {
+                if ((!priv) && (!parea->unpriv)) {
+                        mutex_unlock(&parea_lock);
+                        return EPERM;
+                }
+                
+                goto map;
+        }
+        
+        parea = NULL;
+        mutex_unlock(&parea_lock);
+        
+        /*
+         * Check if the memory region is part of physical
+         * memory generally enabled for mapping.
+         */
+        
         irq_spinlock_lock(&zones.lock, true);
         size_t znum = find_zone(ADDR2PFN(pf), pages, 0);
@@ -153 +179 @@
         }
         
-        if (zone_flags_available(zones.info[znum].flags)) {
-                /*
-                 * Frames are part of physical memory, check
-                 * if the memory region is enabled for mapping.
-                 */
-                irq_spinlock_unlock(&zones.lock, true);
-                
-                mutex_lock(&parea_lock);
-                btree_node_t *nodep;
-                parea_t *parea = (parea_t *) btree_search(&parea_btree,
-                    (btree_key_t) pf, &nodep);
-                
-                if ((!parea) || (parea->frames < pages)) {
-                        mutex_unlock(&parea_lock);
-                        return ENOENT;
-                }
-                
-                if (!priv) {
-                        if (!parea->unpriv) {
-                                mutex_unlock(&parea_lock);
-                                return EPERM;
-                        }
-                }
-                
-                mutex_unlock(&parea_lock);
-                goto map;
-        }
-        
         irq_spinlock_unlock(&zones.lock, true);
         return ENOENT;
@@ -188 +186 @@
             AS_AREA_ATTR_NONE, &phys_backend, &backend_data)) {
                 /*
-                 * The address space area could not have been created.
+                 * The address space area was not created.
                  * We report it using ENOMEM.
                  */
+                
+                if (parea != NULL)
+                        mutex_unlock(&parea_lock);
+                
                 return ENOMEM;
         }
@@ -197 +199 @@
          * Mapping is created on-demand during page fault.
          */
-        return 0;
+        
+        if (parea != NULL) {
+                parea->mapped = true;
+                mutex_unlock(&parea_lock);
+        }
+        
+        return EOK;
 }