File 19140-init-heap-pages-max-order.patch of Package xen

Overview Repositories Revisions Requests Users Attributes Meta

File 19140-init-heap-pages-max-order.patch of Package xen (Revision 7ef33d17cdcd039c4b752173f18c59d8)

Currently displaying revision 7ef33d17cdcd039c4b752173f18c59d8 , Show latest

# HG changeset patch
# User Keir Fraser <keir.fraser@citrix.com>
# Date 1233314063 0
# Node ID 9b9078a02f65fc010ad838de1c9c4e8aa4cf1009
# Parent  e1135b40e783e965ed4eccfd8e983e9f3f4259e8
page_alloc: properly honor MAX_ORDER in init_heap_pages()

The previous check was flawed in that it checked
- for a single bit to be set instead of a range of bits to be clear
- a machine address against a mask supposed to be used on MFNs.

Signed-off-by: Jan Beulich <jbeulich@novell.com>

--- a/xen/common/page_alloc.c
+++ b/xen/common/page_alloc.c
@@ -477,7 +477,6 @@ static void free_heap_pages(
  * latter is not on a MAX_ORDER boundary, then we reserve the page by
  * not freeing it to the buddy allocator.
  */
-#define MAX_ORDER_ALIGNED (1UL << (MAX_ORDER))
 static void init_heap_pages(
     unsigned int zone, struct page_info *pg, unsigned long nr_pages)
 {
@@ -499,16 +498,15 @@ static void init_heap_pages(
             init_node_heap(nid_curr);
 
         /*
-         * free pages of the same node, or if they differ, but are on a
-         * MAX_ORDER alignement boundary (which already get reserved)
+         * Free pages of the same node, or if they differ, but are on a
+         * MAX_ORDER alignment boundary (which already get reserved).
          */
-         if ( (nid_curr == nid_prev) || (page_to_maddr(pg+i) &
-                                         MAX_ORDER_ALIGNED) )
-             free_heap_pages(zone, pg+i, 0);
-         else
-             printk("Reserving non-aligned node boundary @ mfn %lu\n",
-                    page_to_mfn(pg+i));
-
+        if ( (nid_curr == nid_prev) ||
+             !(page_to_mfn(pg+i) & ((1UL << MAX_ORDER) - 1)) )
+            free_heap_pages(zone, pg+i, 0);
+        else
+            printk("Reserving non-aligned node boundary @ mfn %#lx\n",
+                   page_to_mfn(pg+i));
         nid_prev = nid_curr;
     }
 }

Places

File 19140-init-heap-pages-max-order.patch of Package xen (Revision 7ef33d17cdcd039c4b752173f18c59d8)

Places