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.

Xen-ppc-devel mailing list

Reply via email to