WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-ia64-devel

[Xen-ia64-devel] [PATCH 2/7] remove p2m_xenheap boot parameter

To: xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
Subject: [Xen-ia64-devel] [PATCH 2/7] remove p2m_xenheap boot parameter
From: Isaku Yamahata <yamahata@xxxxxxxxxxxxx>
Date: Thu, 29 Jan 2009 11:21:30 +0900
Cc: Isaku Yamahata <yamahata@xxxxxxxxxxxxx>
Delivery-date: Wed, 28 Jan 2009 18:17:13 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
List-help: <mailto:xen-ia64-devel-request@lists.xensource.com?subject=help>
List-id: Discussion of the ia64 port of Xen <xen-ia64-devel.lists.xensource.com>
List-post: <mailto:xen-ia64-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-ia64-devel>, <mailto:xen-ia64-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-ia64-devel>, <mailto:xen-ia64-devel-request@lists.xensource.com?subject=unsubscribe>
Sender: xen-ia64-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mutt/1.5.18 (2008-05-17)
[IA64] remove opt_p2m_xenheap

xenheap isn't pinned down now so that allocating p2m pages from
xenheap doesn't make sense anymore.
This is preparation for removing xenheap.

Signed-off-by: Isaku Yamahata <yamahata@xxxxxxxxxxxxx>

diff --git a/xen/arch/ia64/xen/mm.c b/xen/arch/ia64/xen/mm.c
--- a/xen/arch/ia64/xen/mm.c
+++ b/xen/arch/ia64/xen/mm.c
@@ -3060,31 +3060,19 @@ static int alloc_page_type(struct page_i
        return 1;
 }
 
-static int opt_p2m_xenheap;
-boolean_param("p2m_xenheap", opt_p2m_xenheap);
-
 void *pgtable_quicklist_alloc(void)
 {
+    struct page_info *page;
     void *p;
 
     BUG_ON(dom_p2m == NULL);
-    if (!opt_p2m_xenheap) {
-        struct page_info *page = alloc_domheap_page(dom_p2m, 0);
-        if (page == NULL)
-            return NULL;
-        p = page_to_virt(page);
-        clear_page(p);
-        return p;
-    }
-    p = alloc_xenheap_pages(0);
-    if (p) {
-        clear_page(p);
-        /*
-         * This page should be read only.  At this moment, the third
-         * argument doesn't make sense.  It should be 1 when supported.
-         */
-        share_xen_page_with_guest(virt_to_page(p), dom_p2m, 0);
-    }
+
+    page = alloc_domheap_page(dom_p2m, 0);
+    if (page == NULL)
+        return NULL;
+
+    p = page_to_virt(page);
+    clear_page(p);
     return p;
 }
 
@@ -3096,8 +3084,6 @@ void pgtable_quicklist_free(void *pgtabl
     BUG_ON(page->count_info != (1 | PGC_allocated));
 
     put_page(page);
-    if (opt_p2m_xenheap)
-        free_xenheap_page(pgtable_entry);
 }
 
 void put_page_type(struct page_info *page)

Attachment: remove-opt_p2m_xenheap.patch
Description: Text Data

_______________________________________________
Xen-ia64-devel mailing list
Xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-ia64-devel
<Prev in Thread] Current Thread [Next in Thread>
  • [Xen-ia64-devel] [PATCH 2/7] remove p2m_xenheap boot parameter, Isaku Yamahata <=