On Aug 28, 2006, at 11:47 AM, Hollis Blanchard wrote:

On Sun, 2006-08-27 at 20:20 +0000, Xen patchbot-xenppc-unstable wrote:

+    /* make sure we are at least as big as the RMA */
+    if (dom0_nrpages < rma_nrpages)
+        dom0_nrpages = rma_nrpages;
+    else
+ dom0_nrpages = allocate_extents(d, dom0_nrpages, rma_nrpages);

     d->tot_pages = dom0_nrpages;
     ASSERT(d->tot_pages > 0);

Note that dom0_nrpages here doesn't do anything other than confuse the
memory accounting (though we don't try to destroy dom0, so it doesn't
matter too much ATM).

good catch.. I'll simplify


In particular, we're assigning d->tot_pages without actually doing any
heap allocation.
we should end up reassigning the same value redundantly.
-JX



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

Reply via email to