Changeset c5396c1 in mainline for kernel/generic/src/mm/frame.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/generic/src/mm/frame.c (modified) (5 diffs)

kernel/generic/src/mm/frame.c

@@ -60 +60 @@
 #include <config.h>
 #include <str.h>
-
-#define BITMAP_BLOCK_SIZE  128
 
 zones_t zones;
@@ -410 +408 @@
         
         bitmap_initialize(&zones.info[z1].bitmap, zones.info[z1].count,
-            BITMAP_BLOCK_SIZE, confdata +
-            (sizeof(frame_t) * zones.info[z1].count));
+            confdata + (sizeof(frame_t) * zones.info[z1].count));
         bitmap_clear_range(&zones.info[z1].bitmap, 0, zones.info[z1].count);
         
@@ -578 +575 @@
                  */
                 
-                bitmap_initialize(&zone->bitmap, count, BITMAP_BLOCK_SIZE,
-                    confdata + (sizeof(frame_t) * count));
+                bitmap_initialize(&zone->bitmap, count, confdata +
+                    (sizeof(frame_t) * count));
                 bitmap_clear_range(&zone->bitmap, 0, count);
                 
@@ -591 +588 @@
                         frame_initialize(&zone->frames[i]);
         } else {
-                bitmap_initialize(&zone->bitmap, 0, 0, NULL);
+                bitmap_initialize(&zone->bitmap, 0, NULL);
                 zone->frames = NULL;
         }
@@ -605 +602 @@
 size_t zone_conf_size(size_t count)
 {
-        return (count * sizeof(frame_t) +
-            bitmap_size(count, BITMAP_BLOCK_SIZE));
+        return (count * sizeof(frame_t) + bitmap_size(count));
 }