Changeset 20235a3 in mainline for kernel/generic/src/mm/frame.c

Timestamp:

2010-09-02T20:55:28Z (14 years ago)

Author:

Jakub Jermar <jakub@…>

Branches:

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

Children:

0c39b96

Parents:

0c61955 (diff), 3249673 (diff)
Note: this is a merge changeset, the changes displayed below correspond to the merge itself.
Use the (diff) links above to see all the changes relative to each parent.

Message:

Merge mainline changes.

File:

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

kernel/generic/src/mm/frame.c

@@ -121 +121 @@
  *
  */
-NO_TRACE static size_t zones_insert_zone(pfn_t base, size_t count)
+NO_TRACE static size_t zones_insert_zone(pfn_t base, size_t count,
+    zone_flags_t flags)
 {
         if (zones.count + 1 == ZONES_MAX) {
@@ -131 +132 @@
         for (i = 0; i < zones.count; i++) {
                 /* Check for overlap */
-                if (overlaps(base, count,
-                    zones.info[i].base, zones.info[i].count)) {
-                        printf("Zone (%p, %p) overlaps with zone (%p, %p)!\n",
-                            PFN2ADDR(base), PFN2ADDR(base + count),
-                            PFN2ADDR(zones.info[i].base),
-                            PFN2ADDR(zones.info[i].base + zones.info[i].count));
+                if (overlaps(zones.info[i].base, zones.info[i].count,
+                    base, count)) {
+                        
+                        /*
+                         * If the overlaping zones are of the same type
+                         * and the new zone is completely within the previous
+                         * one, then quietly ignore the new zone.
+                         *
+                         */
+                        
+                        if ((zones.info[i].flags != flags) ||
+                            (!iswithin(zones.info[i].base, zones.info[i].count,
+                            base, count))) {
+                                printf("Zone (%p, %p) overlaps with previous zone (%p, %p)!\n",
+                                    PFN2ADDR(base), PFN2ADDR(count),
+                                    PFN2ADDR(zones.info[i].base),
+                                    PFN2ADDR(zones.info[i].count));
+                        }
+                        
                         return (size_t) -1;
                 }
@@ -147 +161 @@
         for (j = zones.count; j > i; j--) {
                 zones.info[j] = zones.info[j - 1];
-                zones.info[j].buddy_system->data =
-                    (void *) &zones.info[j - 1];
+                if (zones.info[j].buddy_system != NULL)
+                        zones.info[j].buddy_system->data =
+                            (void *) &zones.info[j];
         }
         
@@ -748 +763 @@
         for (i = z2 + 1; i < zones.count; i++) {
                 zones.info[i - 1] = zones.info[i];
-                zones.info[i - 1].buddy_system->data =
-                    (void *) &zones.info[i - 1];
+                if (zones.info[i - 1].buddy_system != NULL)
+                        zones.info[i - 1].buddy_system->data =
+                            (void *) &zones.info[i - 1];
         }
         
@@ -898 +914 @@
                 }
                 
-                size_t znum = zones_insert_zone(start, count);
+                size_t znum = zones_insert_zone(start, count, flags);
                 if (znum == (size_t) -1) {
                         irq_spinlock_unlock(&zones.lock, true);
@@ -921 +937 @@
         
         /* Non-available zone */
-        size_t znum = zones_insert_zone(start, count);
+        size_t znum = zones_insert_zone(start, count, flags);
         if (znum == (size_t) -1) {
                 irq_spinlock_unlock(&zones.lock, true);