Changeset b0c2075 in mainline for kernel/generic/src/ddi/ddi.c

Timestamp:

2013-09-10T17:48:57Z (11 years ago)

Author:

Martin Decky <martin@…>

Branches:

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

Children:

85147f3

Parents:

86733f3

Message:

new physical memory allocator supporting physical address constrains
the buddy allocator framework is retired and replaced by a two-level bitmap
the allocator can allocate an arbitrary number of frames, not only a power-of-two count

Caution: Change of semantics
The physical memory allocator no longer allocates naturally aligned blocks. If you require an aligned block, specify it as the constraint.

File:

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

kernel/generic/src/ddi/ddi.c

@@ -322 +322 @@
 }
 
-NO_TRACE static int dmamem_map_anonymous(size_t size, unsigned int map_flags,
-    unsigned int flags, uintptr_t *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 frames = SIZE2FRAMES(size);
-        uint8_t order;
-        
-        /* We need the 2^order >= frames */
-        if (frames == 1)
-                order = 0;
-        else
-                order = fnzb(frames - 1) + 1;
-        
-        *phys = frame_alloc_noreserve(order, 0, 0);
+        *phys = frame_alloc_noreserve(frames, 0, constraint);
         if (*phys == 0)
                 return ENOMEM;
@@ -390 +383 @@
                  */
                 
+                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)