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] Fix that guest can not get continuous memor

To: Isaku Yamahata <yamahata@xxxxxxxxxxxxx>
Subject: RE: [Xen-ia64-devel] [PATCH] Fix that guest can not get continuous memory for vhpt
From: "Zhang, Yang" <yang.zhang@xxxxxxxxx>
Date: Mon, 2 Mar 2009 17:38:22 +0800
Accept-language: en-US
Acceptlanguage: en-US
Cc: "xen-ia64-devel@xxxxxxxxxxxxxxxxxxx" <xen-ia64-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Mon, 02 Mar 2009 01:40:08 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <20090302082135.GH21000%yamahata@xxxxxxxxxxxxx>
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>
References: <10C63FAD690C13458F0B32BCED571F1406BFDE2F@xxxxxxxxxxxxxxxxxxxxxxxxxxxx> <20090219031203.GE19443%yamahata@xxxxxxxxxxxxx> <10C63FAD690C13458F0B32BCED571F1406BFE692@xxxxxxxxxxxxxxxxxxxxxxxxxxxx> <20090219053236.GG19443%yamahata@xxxxxxxxxxxxx> <10C63FAD690C13458F0B32BCED571F1406D080C5@xxxxxxxxxxxxxxxxxxxxxxxxxxxx> <20090302082135.GH21000%yamahata@xxxxxxxxxxxxx>
Sender: xen-ia64-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AcmbD+wxM0nLO5B5SQizexC0SbsYTAABqEGw
Thread-topic: [Xen-ia64-devel] [PATCH] Fix that guest can not get continuous memory for vhpt
Isaku Yamahata wrote:
> As you described above, you used alloc_domheap_page(NULL, ) which
> means that pages for vhpt aren't accounted for a given domain.
> So the hunk in tools/python/xen/xend/image.py doesn't make sense,
> does it? Or have you found any issues without the modification to
> image.py? 
When using balloon, it does not count the memory for the vhpt in the previous 
version.
So when balloon memory from dom0, the guest always can not be launched.
because there does not have the enough memory for vhpt. And the hunk add in 
image.py is 
to fix this issue. this is different from the bug guest can not get continuous 
memory. 
Maybe i should split this hunk as an new patch.

> Hmm, the patch makes the allocating/freeing vhpt interface convoluted.
> The gut of your patch is to make the function, init_domain_vhpt(),
> not to allocate pages by preallocation. So what should be done is
> not only to add preallocate pages logic, but also to revise
> the related functions.
> Revise init_domain_vhpt() (and free_domain_vhpt()?). Maybe the
> underlying functions, thash_alloc(), thash_free() need to be
> refactored. 
Sorry, i cannot understand your meaning well. what revision should be done?
I feel that the preallocation just provides a method for init_domain_vhpt to 
get continuous
memory.  And we should not to revise this fuctions.


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