Changeset c5396c1 in mainline for kernel/arch/amd64/src/ddi/ddi.c

Timestamp:

2013-09-12T10:27:17Z (11 years ago)

Author:

Martin Decky <martin@…>

Branches:

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

Children:

6e75f2d

Parents:

4a5f2b0

Message:

abandon the 2nd level bitmap
according to observations it is very rare to have allocation requests for more than 8 frames, therefore the conservative update of the 2nd level bitmap can hardly provide any benefits
the natural blocking of the bitmap (by bytes) should provide a reasonable performance

File:

• kernel/arch/amd64/src/ddi/ddi.c (modified) (3 diffs)

kernel/arch/amd64/src/ddi/ddi.c

@@ -68 +68 @@
                  */
                 
-                void *store = malloc(bitmap_size(elements, 0), FRAME_ATOMIC);
+                void *store = malloc(bitmap_size(elements), FRAME_ATOMIC);
                 if (!store)
                         return ENOMEM;
                 
                 bitmap_t oldiomap;
-                bitmap_initialize(&oldiomap, task->arch.iomap.elements, 0,
+                bitmap_initialize(&oldiomap, task->arch.iomap.elements,
                     task->arch.iomap.bits);
                 
-                bitmap_initialize(&task->arch.iomap, elements, 0, store);
+                bitmap_initialize(&task->arch.iomap, elements, store);
                 
                 /*
@@ -129 +129 @@
                 
                 bitmap_t iomap;
-                bitmap_initialize(&iomap, TSS_IOMAP_SIZE * 8, 0,
+                bitmap_initialize(&iomap, TSS_IOMAP_SIZE * 8,
                     CPU->arch.tss->iomap);
                 bitmap_copy(&iomap, &TASK->arch.iomap, elements);
@@ -157 +157 @@
         
         descriptor_t *gdt_p = (descriptor_t *) cpugdtr.base;
-        size_t size = bitmap_size(elements, 0);
+        size_t size = bitmap_size(elements);
         gdt_tss_setlimit(&gdt_p[TSS_DES], TSS_BASIC_SIZE + size);
         gdtr_load(&cpugdtr);