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

Re: [Xen-ia64-devel] [Patch] Reseve memory of domain0 (fix dom0 boot pan

To: Akio Takebe <takebe_akio@xxxxxxxxxxxxxx>
Subject: Re: [Xen-ia64-devel] [Patch] Reseve memory of domain0 (fix dom0 boot panic)
From: Isaku Yamahata <yamahata@xxxxxxxxxxxxx>
Date: Wed, 3 May 2006 23:37:32 +0900
Cc: xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Wed, 03 May 2006 07:37:46 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <96C66E7F9D1B2Dtakebe_akio@xxxxxxxxxxxxxx>
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/cgi-bin/mailman/listinfo/xen-ia64-devel>, <mailto:xen-ia64-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-ia64-devel>, <mailto:xen-ia64-devel-request@lists.xensource.com?subject=unsubscribe>
References: <77C6687C83E9AAtakebe_akio@xxxxxxxxxxxxxx> <20060501111311.GA9007%yamahata@xxxxxxxxxxxxx> <96C66E7F9D1B2Dtakebe_akio@xxxxxxxxxxxxxx>
Sender: xen-ia64-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mutt/1.4.2.1i
Hi.

On Wed, May 03, 2006 at 04:03:00PM +0900, Akio Takebe wrote:
Content-Description: Mail message body

> >Why does the region need reservation?
> >dom0/initrd images are copied by construct_dom0().
> >The dom0/initrd regions must not be used before contruct_dom0(),
> >but they can be used after construct_dom0().
> Yes, dom0/inird images are copied by construct_dom0().
> But the region is clean up by init_virtual_frametable() 
> before construct_dom0 is called.
> So the region need reservation.

You only explained the region shouldn't have been used until the copy
by construct_dom0().
Why do you waste memory of the regions?
The reservation can be accepted as a temporal work around.
But it isn't right.


> >The above change is wrong. Please backout or fix somehow.
> >With P=M model it works but it isn't right. It breaks the P2M/VP patches.
> >Please see construct_dom0().
> I attach a patch which fix the above bug.
> Am I correct?

It looks correct with respect to bp->initrd_start.

-- 
yamahata

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