Re: [Xen-devel] [PATCH] libxl: Wait for ballooning if free memory is increasing

2015-01-30 Thread Mike Latimer
On Thursday, January 29, 2015 10:14:26 AM Ian Campbell wrote: > I'm thinking it would be clearer if the comment and the condition were > logically inverted. e.g.: > > /* > * If the amount of free mem has increased on this iteration (i.e. > * some progress has been made) then reset th

Re: [Xen-devel] [PATCH] libxl: Wait for ballooning if free memory is increasing

2015-01-29 Thread Ian Campbell
On Wed, 2015-01-28 at 17:14 -0700, Mike Latimer wrote: > > > +if (free_memkb <= free_memkb_prev) { > > > +retries--; > > > > I think you need to update prev here, otherwise after one successful > > iteration the condition is always true even if progress subsequently > > stalls.

Re: [Xen-devel] [PATCH] libxl: Wait for ballooning if free memory is increasing

2015-01-28 Thread Mike Latimer
On Wednesday, January 28, 2015 01:05:25 PM Ian Campbell wrote: > On Wed, 2015-01-21 at 22:22 -0700, Mike Latimer wrote: > > Sorry for the delay. No problem! Thanks for the comments. > > @@ -2228,7 +2230,13 @@ static int freemem(uint32_t domid, > > libxl_domain_build_info *b_info)> > >

Re: [Xen-devel] [PATCH] libxl: Wait for ballooning if free memory is increasing

2015-01-28 Thread Ian Campbell
On Wed, 2015-01-21 at 22:22 -0700, Mike Latimer wrote: Sorry for the delay. > @@ -2228,7 +2230,13 @@ static int freemem(uint32_t domid, > libxl_domain_build_info *b_info) > if (rc < 0) > return rc; > > -retries--; > +/* only decrement retry count if free_m

Re: [Xen-devel] [PATCH] libxl: Wait for ballooning if free memory is increasing

2015-01-28 Thread Wei Liu
You forgot go CC relevant maintainers. Now I've done that for you. I think last time Ian and George were following. On Wed, Jan 21, 2015 at 10:22:53PM -0700, Mike Latimer wrote: > During domain startup, all required memory ballooning must complete > within a maximum window of 33 seconds (3 retrie

Re: [Xen-devel] [PATCH] libxl: Wait for ballooning if free memory is increasing

2015-01-27 Thread Mike Latimer
On Wednesday, January 21, 2015 10:22:53 PM Mike Latimer wrote: > During domain startup, all required memory ballooning must complete > within a maximum window of 33 seconds (3 retries, 11 seconds of delay). > If not, domain creation is aborted with a 'failed to free memory' error. > > In order to

[Xen-devel] [PATCH] libxl: Wait for ballooning if free memory is increasing

2015-01-21 Thread Mike Latimer
During domain startup, all required memory ballooning must complete within a maximum window of 33 seconds (3 retries, 11 seconds of delay). If not, domain creation is aborted with a 'failed to free memory' error. In order to accommodate large domains or slower hardware (which require substantially