Changeset c5396c1 in mainline for kernel/arch

Timestamp:

2013-09-12T10:27:17Z (12 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

Location:

kernel/arch

Files:

• amd64/src/ddi/ddi.c (modified) (3 diffs)
• amd64/src/proc/task.c (modified) (1 diff)
• ia32/src/ddi/ddi.c (modified) (3 diffs)
• ia32/src/proc/task.c (modified) (1 diff)
• ia64/src/ddi/ddi.c (modified) (1 diff)

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);

kernel/arch/amd64/src/proc/task.c

@@ -46 +46 @@
 {
         task->arch.iomapver = 0;
-        bitmap_initialize(&task->arch.iomap, 0, 0, NULL);
+        bitmap_initialize(&task->arch.iomap, 0, NULL);
 }
 

kernel/arch/ia32/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_setlimit(&gdt_p[TSS_DES], TSS_BASIC_SIZE + size);
         gdtr_load(&cpugdtr);

kernel/arch/ia32/src/proc/task.c

@@ -46 +46 @@
 {
         task->arch.iomapver = 0;
-        bitmap_initialize(&task->arch.iomap, 0, 0, NULL);
+        bitmap_initialize(&task->arch.iomap, 0, NULL);
 }
 

kernel/arch/ia64/src/ddi/ddi.c

@@ -60 +60 @@
                         return ENOMEM;
                 
-                void *store = malloc(bitmap_size(IO_MEMMAP_PAGES, 0), 0);
+                void *store = malloc(bitmap_size(IO_MEMMAP_PAGES), 0);
                 if (store == NULL)
                         return ENOMEM;
                 
-                bitmap_initialize(task->arch.iomap, IO_MEMMAP_PAGES, 0, store);
+                bitmap_initialize(task->arch.iomap, IO_MEMMAP_PAGES, store);
                 bitmap_clear_range(task->arch.iomap, 0, IO_MEMMAP_PAGES);
         }