xen-devel

[Top] [All Lists]

[Xen-devel] p2m: use xzalloc() for struct p2m_domain allocation

from [Jan Beulich]

[Permanent Link][Original]

To:	"xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Subject:	[Xen-devel] p2m: use xzalloc() for struct p2m_domain allocation
From:	"Jan Beulich" <JBeulich@xxxxxxxx>
Date:	Fri, 14 Oct 2011 08:26:12 +0100
Delivery-date:	Fri, 14 Oct 2011 00:26:42 -0700
Envelope-to:	www-data@xxxxxxxxxxxxxxxxxxx
List-help:	<mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id:	Xen developer discussion <xen-devel.lists.xensource.com>
List-post:	<mailto:xen-devel@lists.xensource.com>
List-subscribe:	<http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe:	<http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Sender:	xen-devel-bounces@xxxxxxxxxxxxxxxxxxx

Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>

--- a/xen/arch/x86/mm/p2m.c
+++ b/xen/arch/x86/mm/p2m.c
@@ -71,7 +71,6 @@ boolean_param("hap_2mb", opt_hap_2mb);
 /* Init the datastructures for later use by the p2m code */
 static void p2m_initialise(struct domain *d, struct p2m_domain *p2m)
 {
-    memset(p2m, 0, sizeof(*p2m));
     mm_lock_init(&p2m->lock);
     INIT_LIST_HEAD(&p2m->np2m_list);
     INIT_PAGE_LIST_HEAD(&p2m->pages);
@@ -100,7 +99,7 @@ p2m_init_nestedp2m(struct domain *d)
 
     mm_lock_init(&d->arch.nested_p2m_lock);
     for (i = 0; i < MAX_NESTEDP2M; i++) {
-        d->arch.nested_p2m[i] = p2m = xmalloc(struct p2m_domain);
+        d->arch.nested_p2m[i] = p2m = xzalloc(struct p2m_domain);
         if (p2m == NULL)
             return -ENOMEM;
         p2m_initialise(d, p2m);
@@ -115,7 +114,7 @@ int p2m_init(struct domain *d)
 {
     struct p2m_domain *p2m;
 
-    p2m_get_hostp2m(d) = p2m = xmalloc(struct p2m_domain);
+    p2m_get_hostp2m(d) = p2m = xzalloc(struct p2m_domain);
     if ( p2m == NULL )
         return -ENOMEM;
     p2m_initialise(d, p2m);




_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel

[More with this subject...]

<Prev in Thread]	Current Thread	[Next in Thread>
[Xen-devel] p2m: use xzalloc() for struct p2m_domain allocation, Jan Beulich <= Re: [Xen-devel] p2m: use xzalloc() for struct p2m_domain allocation, Keir Fraser Re: [Xen-devel] p2m: use xzalloc() for struct p2m_domain allocation, Tim Deegan

Previous by Date:	[Xen-devel] [PATCH] cpufreq: error path fixes, Jan Beulich
Next by Date:	[Xen-devel] [PATCH] rcu: move private declarations and definitions from header to implementation, Jan Beulich
Previous by Thread:	[Xen-devel] [PATCH] cpufreq: error path fixes, Jan Beulich
Next by Thread:	Re: [Xen-devel] p2m: use xzalloc() for struct p2m_domain allocation, Keir Fraser
Indexes:	[Date] [Thread] [Top] [All Lists]