Changes in kernel/generic/src/mm/frame.c [905721b:818fffe] in mainline

File:

• kernel/generic/src/mm/frame.c (modified) (26 diffs)

kernel/generic/src/mm/frame.c

@@ -240 +240 @@
 NO_TRACE static bool zone_can_alloc(zone_t *zone, uint8_t order)
 {
-        return ((zone->flags & ZONE_AVAILABLE) &&
-            buddy_system_can_alloc(zone->buddy_system, order));
+        return (zone_flags_available(zone->flags)
+            && buddy_system_can_alloc(zone->buddy_system, order));
 }
 
@@ -265 +265 @@
                  * Check whether the zone meets the search criteria.
                  */
-                if (ZONE_FLAGS_MATCH(zones.info[i].flags, flags)) {
+                if ((zones.info[i].flags & flags) == flags) {
                         /*
                          * Check if the zone has 2^order frames area available.
@@ -460 +460 @@
 NO_TRACE static pfn_t zone_frame_alloc(zone_t *zone, uint8_t order)
 {
-        ASSERT(zone->flags & ZONE_AVAILABLE);
+        ASSERT(zone_flags_available(zone->flags));
         
         /* Allocate frames from zone buddy system */
@@ -490 +490 @@
 NO_TRACE static size_t zone_frame_free(zone_t *zone, size_t frame_idx)
 {
-        ASSERT(zone->flags & ZONE_AVAILABLE);
+        ASSERT(zone_flags_available(zone->flags));
         
         frame_t *frame = &zone->frames[frame_idx];
@@ -518 +518 @@
 NO_TRACE static void zone_mark_unavailable(zone_t *zone, size_t frame_idx)
 {
-        ASSERT(zone->flags & ZONE_AVAILABLE);
+        ASSERT(zone_flags_available(zone->flags));
         
         frame_t *frame = zone_get_frame(zone, frame_idx);
@@ -549 +549 @@
     buddy_system_t *buddy)
 {
-        ASSERT(zones.info[z1].flags & ZONE_AVAILABLE);
-        ASSERT(zones.info[z2].flags & ZONE_AVAILABLE);
+        ASSERT(zone_flags_available(zones.info[z1].flags));
+        ASSERT(zone_flags_available(zones.info[z2].flags));
         ASSERT(zones.info[z1].flags == zones.info[z2].flags);
         ASSERT(zones.info[z1].base < zones.info[z2].base);
@@ -645 +645 @@
 NO_TRACE static void return_config_frames(size_t znum, pfn_t pfn, size_t count)
 {
-        ASSERT(zones.info[znum].flags & ZONE_AVAILABLE);
+        ASSERT(zone_flags_available(zones.info[znum].flags));
         
         size_t cframes = SIZE2FRAMES(zone_conf_size(count));
@@ -681 +681 @@
     size_t count)
 {
-        ASSERT(zones.info[znum].flags & ZONE_AVAILABLE);
+        ASSERT(zone_flags_available(zones.info[znum].flags));
         ASSERT(frame_idx + count < zones.info[znum].count);
         
@@ -723 +723 @@
          * set of flags
          */
-        if ((z1 >= zones.count) || (z2 >= zones.count) || (z2 - z1 != 1) ||
-            (zones.info[z1].flags != zones.info[z2].flags)) {
+        if ((z1 >= zones.count) || (z2 >= zones.count)
+            || (z2 - z1 != 1)
+            || (!zone_flags_available(zones.info[z1].flags))
+            || (!zone_flags_available(zones.info[z2].flags))
+            || (zones.info[z1].flags != zones.info[z2].flags)) {
                 ret = false;
                 goto errout;
@@ -825 +828 @@
         zone->buddy_system = buddy;
         
-        if (flags & ZONE_AVAILABLE) {
+        if (zone_flags_available(flags)) {
                 /*
                  * Compute order for buddy system and initialize
@@ -862 +865 @@
 {
         return (count * sizeof(frame_t) + buddy_conf_size(fnzb(count)));
-}
-
-/** Allocate external configuration frames from low memory. */
-pfn_t zone_external_conf_alloc(size_t count)
-{
-        size_t size = zone_conf_size(count);
-        size_t order = ispwr2(size) ? fnzb(size) : (fnzb(size) + 1);
-
-        return ADDR2PFN((uintptr_t) frame_alloc(order - FRAME_WIDTH,
-            FRAME_LOWMEM | FRAME_ATOMIC));
 }
 
@@ -895 +888 @@
         irq_spinlock_lock(&zones.lock, true);
         
-        if (flags & ZONE_AVAILABLE) {  /* Create available zone */
+        if (zone_flags_available(flags)) {  /* Create available zone */
                 /* Theoretically we could have NULL here, practically make sure
                  * nobody tries to do that. If some platform requires, remove
@@ -901 +894 @@
                  */
                 ASSERT(confframe != ADDR2PFN((uintptr_t ) NULL));
-
-                /* Update the known end of physical memory. */
-                config.physmem_end = max(config.physmem_end, PFN2ADDR(start + count));
                 
                 /* If confframe is supposed to be inside our zone, then make sure
@@ -924 +914 @@
                                 for (i = 0; i < init.cnt; i++)
                                         if (overlaps(addr, PFN2ADDR(confcount),
-                                            init.tasks[i].paddr,
+                                            KA2PA(init.tasks[i].addr),
                                             init.tasks[i].size)) {
                                                 overlap = true;
@@ -1086 +1076 @@
 #endif
                 
-                /*
-                 * Since the mem_avail_mtx is an active mutex, we need to disable interrupts
-                 * to prevent deadlock with TLB shootdown.
-                 */
-                ipl_t ipl = interrupts_disable();
                 mutex_lock(&mem_avail_mtx);
                 
@@ -1103 +1088 @@
                 
                 mutex_unlock(&mem_avail_mtx);
-                interrupts_restore(ipl);
                 
 #ifdef CONFIG_DEBUG
@@ -1167 +1151 @@
          * Signal that some memory has been freed.
          */
-
-        
-        /*
-         * Since the mem_avail_mtx is an active mutex, we need to disable interrupts
-         * to prevent deadlock with TLB shootdown.
-         */
-        ipl_t ipl = interrupts_disable();
         mutex_lock(&mem_avail_mtx);
         if (mem_avail_req > 0)
@@ -1183 +1160 @@
         }
         mutex_unlock(&mem_avail_mtx);
-        interrupts_restore(ipl);
         
         if (!(flags & FRAME_NO_RESERVE))
@@ -1256 +1232 @@
         
         /* Tell the architecture to create some memory */
-        frame_low_arch_init();
+        frame_arch_init();
         if (config.cpu_active == 1) {
                 frame_mark_unavailable(ADDR2PFN(KA2PA(config.base)),
@@ -1265 +1241 @@
                 size_t i;
                 for (i = 0; i < init.cnt; i++) {
-                        pfn_t pfn = ADDR2PFN(init.tasks[i].paddr);
+                        pfn_t pfn = ADDR2PFN(KA2PA(init.tasks[i].addr));
                         frame_mark_unavailable(pfn,
                             SIZE2FRAMES(init.tasks[i].size));
@@ -1279 +1255 @@
                 frame_mark_unavailable(0, 1);
         }
-        frame_high_arch_init();
-}
-
-/** Adjust bounds of physical memory region according to low/high memory split.
- *
- * @param low[in]       If true, the adjustment is performed to make the region
- *                      fit in the low memory. Otherwise the adjustment is
- *                      performed to make the region fit in the high memory.
- * @param basep[inout]  Pointer to a variable which contains the region's base
- *                      address and which may receive the adjusted base address.
- * @param sizep[inout]  Pointer to a variable which contains the region's size
- *                      and which may receive the adjusted size.
- * @retun               True if the region still exists even after the
- *                      adjustment, false otherwise.
- */
-bool frame_adjust_zone_bounds(bool low, uintptr_t *basep, size_t *sizep)
-{
-        uintptr_t limit = KA2PA(config.identity_base) + config.identity_size;
-
-        if (low) {
-                if (*basep > limit)
-                        return false;
-                if (*basep + *sizep > limit)
-                        *sizep = limit - *basep;
-        } else {
-                if (*basep + *sizep <= limit)
-                        return false;
-                if (*basep <= limit) {
-                        *sizep -= limit - *basep;
-                        *basep = limit;
-                }
-        }
-        return true;
 }
 
@@ -1350 +1293 @@
                 *total += (uint64_t) FRAMES2SIZE(zones.info[i].count);
                 
-                if (zones.info[i].flags & ZONE_AVAILABLE) {
+                if (zone_flags_available(zones.info[i].flags)) {
                         *busy += (uint64_t) FRAMES2SIZE(zones.info[i].busy_count);
                         *free += (uint64_t) FRAMES2SIZE(zones.info[i].free_count);
@@ -1401 +1344 @@
                 irq_spinlock_unlock(&zones.lock, true);
                 
-                bool available = ((flags & ZONE_AVAILABLE) != 0);
+                bool available = zone_flags_available(flags);
                 
                 printf("%-4zu", i);
@@ -1413 +1356 @@
 #endif
                 
-                printf(" %12zu %c%c%c%c%c    ", count,
-                    available ? 'A' : '-',
-                    (flags & ZONE_RESERVED) ? 'R' : '-',
-                    (flags & ZONE_FIRMWARE) ? 'F' : '-',
-                    (flags & ZONE_LOWMEM) ? 'L' : '-',
-                    (flags & ZONE_HIGHMEM) ? 'H' : '-');
+                printf(" %12zu %c%c%c      ", count,
+                    available ? 'A' : ' ',
+                    (flags & ZONE_RESERVED) ? 'R' : ' ',
+                    (flags & ZONE_FIRMWARE) ? 'F' : ' ');
                 
                 if (available)
@@ -1460 +1401 @@
         irq_spinlock_unlock(&zones.lock, true);
         
-        bool available = ((flags & ZONE_AVAILABLE) != 0);
+        bool available = zone_flags_available(flags);
         
         uint64_t size;
@@ -1470 +1411 @@
         printf("Zone size:         %zu frames (%" PRIu64 " %s)\n", count,
             size, size_suffix);
-        printf("Zone flags:        %c%c%c%c%c\n",
-            available ? 'A' : '-',
-            (flags & ZONE_RESERVED) ? 'R' : '-',
-            (flags & ZONE_FIRMWARE) ? 'F' : '-',
-            (flags & ZONE_LOWMEM) ? 'L' : '-',
-            (flags & ZONE_HIGHMEM) ? 'H' : '-');
+        printf("Zone flags:        %c%c%c\n",
+            available ? 'A' : ' ',
+            (flags & ZONE_RESERVED) ? 'R' : ' ',
+            (flags & ZONE_FIRMWARE) ? 'F' : ' ');
         
         if (available) {