All of lore.kernel.org
 help / color / mirror / Atom feed
From: Stefano Stabellini <stefano.stabellini@eu.citrix.com>
To: Ian Campbell <ian.campbell@citrix.com>
Cc: wei.liu2@citrix.com, xen-devel@lists.xen.org,
	ian.jackson@eu.citrix.com, Mike Latimer <mlatimer@suse.com>,
	Stefano Stabellini <stefano.stabellini@eu.citrix.com>
Subject: Re: freemem-slack and large memory environments
Date: Wed, 25 Feb 2015 11:39:55 +0000	[thread overview]
Message-ID: <alpine.DEB.2.02.1502251139330.23507@kaball.uk.xensource.com> (raw)
In-Reply-To: <1424796841.20243.5.camel@citrix.com>

On Tue, 24 Feb 2015, Ian Campbell wrote:
> On Tue, 2015-02-24 at 16:06 +0000, Stefano Stabellini wrote:
> > > Now that we autodetect the use of dom0_mem and set autoballooning
> > > correctly perhaps we should just revert a39b5bc64?
> > 
> > We could do that and theoretically it makes perfect sense, but it would
> > result in an even bigger waste of memory.
> 
> Would it, even though we now detect dom0_mem usage and do the right
> thing? I thought a39b5bc64 was a workaround for autoballooning=1
> in /etc/xen/xl.conf when dom0 was used.
> 
> 
> > I think we should either introduce an hard upper limit for
> > freemem-slack as Mike suggested, or remove freemem-slack altogether and
> > properly fix any issues caused by lack of memory in the system (properly
> > account memory usage).
> > After all we are just at the beginning of the release cycle, it is the
> > right time to do this.
> 
> I'm all in favour of someone doing this, similarly to
> http://bugs.xenproject.org/xen/bug/23
> 
> Who is going to do that (either one)?

I am OK with sending the patch for both

 
> > 
> > 
> > > Ian.
> > > 
> > > > > 
> > > > > It seems that there are two approaches to resolve this:
> > > > > 
> > > > >  - Introduce a hard limit on freemem-slack to avoid unnecessarily large
> > > > > reservations
> > > > >  - Increase the retry count during domain creation to ensure enough time is
> > > > > set aside for any cycles spent freeing memory for freemem-slack (on the test
> > > > > machine, doubling the retry count to 6 is the minimum required)
> > > > > 
> > > > > Which is the best approach (or did I miss something)?
> > > > 
> > > > Sorry - forgot to CC relevant maintainers.
> > > > 
> > > > -Mike
> > > 
> > > 
> 
> 

  reply	other threads:[~2015-02-25 11:39 UTC|newest]

Thread overview: 41+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-02-10  1:27 freemem-slack and large memory environments Mike Latimer
2015-02-10 21:34 ` Mike Latimer
2015-02-13 11:13   ` Wei Liu
2015-02-13 23:16     ` Mike Latimer
2015-02-18 14:10   ` Ian Campbell
2015-02-24 16:06     ` Stefano Stabellini
2015-02-24 16:54       ` Ian Campbell
2015-02-25 11:39         ` Stefano Stabellini [this message]
2015-02-25 12:00           ` Ian Campbell
2015-02-25 14:03             ` Ian Campbell
2015-02-25 14:09               ` Stefano Stabellini
2015-02-26 15:36                 ` Mike Latimer
2015-02-26 15:57                   ` Ian Campbell
2015-02-26 17:38                     ` Mike Latimer
2015-02-26 17:47                       ` Ian Campbell
2015-02-26 20:38                         ` Mike Latimer
2015-02-27 10:17                           ` Ian Campbell
2015-02-27 11:05                             ` Stefano Stabellini
2015-02-26 17:53                       ` Stefano Stabellini
2015-02-26 20:45                         ` Mike Latimer
2015-02-26 23:30                           ` Mike Latimer
2015-02-27 10:21                             ` Ian Campbell
2015-02-27 10:52                             ` Stefano Stabellini
2015-02-27 15:28                               ` Mike Latimer
2015-02-27 18:29                                 ` Mike Latimer
2015-02-28  0:31                                   ` Mike Latimer
2015-03-02 10:12                                     ` Stefano Stabellini
2015-03-02 10:44                                       ` Jan Beulich
2015-03-02 12:13                                         ` Stefano Stabellini
2015-03-02 13:04                                           ` Jan Beulich
     [not found]                                           ` <54F46DDB020000780006505B@suse.com>
2015-03-02 22:49                                             ` Mike Latimer
2015-03-02 11:42                                     ` Ian Campbell
2015-03-02 15:19                                       ` Stefano Stabellini
2015-03-02 16:04                                         ` Ian Campbell
2015-03-02 16:15                                           ` Stefano Stabellini
2015-03-02 22:49                                             ` Mike Latimer
2015-03-03 10:02                                               ` Ian Campbell
2015-03-03 10:32                                                 ` Stefano Stabellini
2015-03-03 10:40                                               ` Stefano Stabellini
2015-02-27  8:24                         ` Jan Beulich
2015-02-27 10:52                           ` Stefano Stabellini

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=alpine.DEB.2.02.1502251139330.23507@kaball.uk.xensource.com \
    --to=stefano.stabellini@eu.citrix.com \
    --cc=ian.campbell@citrix.com \
    --cc=ian.jackson@eu.citrix.com \
    --cc=mlatimer@suse.com \
    --cc=wei.liu2@citrix.com \
    --cc=xen-devel@lists.xen.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.