Changeset 0c2d9bb in mainline for kernel/generic/src/ddi/ddi.c

Timestamp:

2013-12-25T22:54:29Z (10 years ago)

Author:

Martin Decky <martin@…>

Branches:

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

Children:

b51cf2c

Parents:

f7a33de (diff), ac36aed (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 mainline changes

File:

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

kernel/generic/src/ddi/ddi.c

@@ -314 +314 @@
 
 NO_TRACE static int dmamem_map(uintptr_t virt, size_t size, unsigned int map_flags,
-    unsigned int flags, void **phys)
+    unsigned int flags, uintptr_t *phys)
 {
         ASSERT(TASK);
@@ -322 +322 @@
 }
 
-NO_TRACE static int dmamem_map_anonymous(size_t size, unsigned int map_flags,
-    unsigned int flags, void **phys, uintptr_t *virt, uintptr_t bound)
+NO_TRACE static int dmamem_map_anonymous(size_t size, uintptr_t constraint,
+    unsigned int map_flags, unsigned int flags, uintptr_t *phys,
+    uintptr_t *virt, uintptr_t bound)
 {
         ASSERT(TASK);
         
-        size_t pages = SIZE2FRAMES(size);
-        uint8_t order;
-        
-        /* We need the 2^order >= pages */
-        if (pages == 1)
-                order = 0;
-        else
-                order = fnzb(pages - 1) + 1;
-        
-        *phys = frame_alloc_noreserve(order, FRAME_DMA);
-        if (*phys == NULL)
+        size_t frames = SIZE2FRAMES(size);
+        *phys = frame_alloc(frames, FRAME_NO_RESERVE, constraint);
+        if (*phys == 0)
                 return ENOMEM;
         
         mem_backend_data_t backend_data;
-        backend_data.base = (uintptr_t) *phys;
-        backend_data.frames = pages;
+        backend_data.base = *phys;
+        backend_data.frames = frames;
         
         if (!as_area_create(TASK->as, map_flags, size,
             AS_AREA_ATTR_NONE, &phys_backend, &backend_data, virt, bound)) {
-                frame_free_noreserve((uintptr_t) *phys);
+                frame_free_noreserve(*phys, frames);
                 return ENOMEM;
         }
@@ -361 +354 @@
 NO_TRACE static int dmamem_unmap_anonymous(uintptr_t virt)
 {
-        // TODO: This is an ugly hack
-        as_t *as = TASK->as;
-
-        mutex_lock(&as->lock);
-        as_area_t *area = find_locked_area(as, virt);
-        if (!area) {
-                mutex_unlock(&as->lock);
-                return ENOENT;
-        }
-        frame_free_noreserve(area->backend_data.base);
-        area->backend_data.base = 0;
-        area->backend_data.frames = 0;
-        mutex_unlock(&area->lock);
-        mutex_unlock(&as->lock);
-
-        return as_area_destroy(as, virt);
+        // TODO: implement unlocking & unmap
+        return EOK;
 }
 
@@ -387 +366 @@
                  */
                 
-                void *phys;
+                uintptr_t phys;
                 int rc = dmamem_map((uintptr_t) virt_ptr, size, map_flags,
                     flags, &phys);
@@ -404 +383 @@
                  */
                 
-                void *phys;
+                uintptr_t constraint;
+                int rc = copy_from_uspace(&constraint, phys_ptr,
+                    sizeof(constraint));
+                if (rc != EOK)
+                        return rc;
+                
+                uintptr_t phys;
                 uintptr_t virt = (uintptr_t) -1;
-                int rc = dmamem_map_anonymous(size, map_flags, flags,
+                rc = dmamem_map_anonymous(size, constraint, map_flags, flags,
                     &phys, &virt, bound);
                 if (rc != EOK)