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/
Home Products Support Community News


Re: [Xen-devel] [PATCH] xl: Update memory info in xenstore when use 'xl

To: Yu Zhiguo <yuzg@xxxxxxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] xl: Update memory info in xenstore when use 'xl mem-set'
From: Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>
Date: Fri, 7 May 2010 18:42:40 +0100
Cc: "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Fri, 07 May 2010 10:43:26 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <4BE3DFBE.2010901@xxxxxxxxxxxxxx>
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>
Newsgroups: chiark.mail.xen.devel
References: <4BE3DFBE.2010901@xxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Yu Zhiguo writes ("[Xen-devel] [PATCH] xl: Update memory info in xenstore when 
use 'xl mem-set'"):
> 'xl mem-set' should update memory info in xenstore's vm info.

Your patch looks sane (although I haven't tested it - I guess you
have).  So:

Acked-by: Ian Jackson <ian.jackson@xxxxxxxxxxxxx>

However, looking at this, it's far from clear why
libxl_set_memory_target calls xc_domain_setmaxmem at all.  Can anyone
explain ?


Xen-devel mailing list