From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ian Campbell Subject: Re: [PATCH 0/4] fix freemem loop Date: Thu, 5 Mar 2015 17:49:35 +0000 Message-ID: <1425577775.14353.1.camel@citrix.com> References: Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Stefano Stabellini Cc: Ian Jackson , xen-devel@lists.xensource.com, Wei Liu , mlatimer@suse.com List-Id: xen-devel@lists.xenproject.org On Tue, 2015-03-03 at 11:08 +0000, Stefano Stabellini wrote: > Hi all, > > this patch series fixes the freemem loop on machines with very large > amount of memory, where the current wait time is not enough. > > In order to be able to handle arbitrarly large amount of ram, we > implement in libxl_wait_for_memory_target a policy of waiting until dom0 > is making progress. What is the impact of the libxl API change made here on other callers, in particular libvirt? If it is expected that existing callers should continue to work as before (perhaps with a redundant call etc) then please state this in the relevant commit message(s). > The patch series also reverts "libxl: Wait for > ballooning if free memory is increasing", that is not actually > implemented correctly. > > > Stefano Stabellini (4): > Revert "libxl: Wait for ballooning if free memory is increasing" > libxl_wait_for_memory_target: wait as long as dom0 is making progress > freemem: remove call to libxl_wait_for_free_memory > libxl_wait_for_memory_target: wait for 2 sec at a time > > tools/libxl/libxl.c | 31 +++++++++++++++++++++++-------- > tools/libxl/xl_cmdimpl.c | 29 ++++++----------------------- > 2 files changed, 29 insertions(+), 31 deletions(-) > > Cheers, > > Stefano