Re: [PATCH] libxl: remove freemem_slack

[Ian Campbell <ian.campbell@citrix.com>]

On Wed, 2015-02-25 at 18:01 -0700, Mike Latimer wrote:
> On Wednesday, February 25, 2015 03:07:25 PM Stefano Stabellini wrote:
> > freemem_slack accounts for the amount of memory to be left free in the
> > system because empirical experiments seem to demonstrate that is needed
> > for "stability reasons".
> > 
> > As we don't have any actual data on these stability issues, remove it.
> > 
> > Signed-off-by: Stefano Stabellini <stefano.stabellini@eu.citrix.com>
> 
> I just finished tested this patch in conjunction with the LIBXL_MAXMEM_CONSTANT 
> removal patch. The freemem_slack zone is no longer reserved, and that aspect 
> of the ballooning issue appears fixed.
> 
> There is still a problem with xl's freemem loop, but we can investigate that 
> further with slack out of the picture.
> 
> From my side:
> 
>   Acked-by: Mike Latimer <mlatimer@suse.com>

Me too. I think I'm right in saying that this patch isn't affected by
any of the other on going discussion, so I've applied it. Please let me
know if I was wrong!