All of lore.kernel.org
 help / color / mirror / Atom feed
From: Daniel Bareiro <dbareiro@uol.com.ar>
To: Hugh Dickins <hugh.dickins@tiscali.co.uk>
Cc: Rik van Riel <riel@redhat.com>, Avi Kivity <avi@redhat.com>,
	KVM General <kvm@vger.kernel.org>
Subject: Re: Memory usage with qemu-kvm-0.12.1.1
Date: Thu, 31 Dec 2009 12:05:27 -0300	[thread overview]
Message-ID: <20091231150527.GA16528@defiant.freesoftware> (raw)
In-Reply-To: <alpine.LSU.2.00.0912301806280.11589@sister.anvils>

[-- Attachment #1: Type: text/plain, Size: 2596 bytes --]

Hi, Hugh.

On Wednesday, 30 December 2009 18:14:30 +0000,
Hugh Dickins wrote:

> On Sun, 27 Dec 2009, Rik van Riel wrote:
> > On 12/27/2009 12:12 PM, Avi Kivity wrote:
> > > On 12/27/2009 06:45 PM, Rik van Riel wrote:
> > > > 
> > > > > If so, it doesn't copy stable@kernel.org. Is it queued for -stable?
> > > > 
> > > > I do not believe that it is queued for -stable.
> > > > 
> > > > Do performance fixes fit with -stable policy?
> > > 
> > > If it is a serious regression, I believe it fits.
> > 
> > It's probably been there since 2.6.28, though it might have been
> > introduced later with a cleanup patch.  It seems to go back at
> > least as far as March...
 
> It does look as if it would have got worse in 2.6.31 and 2.6.32
> - though I am reluctant to predict how these scans work out in
> practice from just glancing at the code!
> 
> I agree with Avi (or with Avi's implication), that it would be
> worth sending the fix to -stable - but it would be nice to hear
> if the patch (inline below) actually does fix Daniel's problem.
> It applies and builds correctly on 2.6.31.9 and 2.6.32.3.
> 
> We have certainly sent much more specialized performance fixes
> to -stable (I'm thinking of Lee's anon_vma locking), and I see
> this as more of a straightforward bugfix anyway - people get
> worried by going into swap without understanding why.
> 
> I forget which patch of mine Avi was thinking of,
> but I'm pretty sure Rik's will prove much the more relevant.

I shutdown the virtual machines yesterday and I boot them again and the
swap usage did not exceed the 9 MB / 486 MB with /proc/sys/vm/swappiness
set to 0. Now I tried restarting the host with the same running VM and,
at the moment, the use of swap is zero with /proc/sys/vm/swappiness by
default (60).

root@ubuntu:~# free
             total       used       free     shared    buffers     cached
Mem:       4060340    2299556    1760784          0     837668     133292
-/+ buffers/cache:    1328596    2731744
Swap:       497972          0     497972


What tests would be recommendable to make to reproduce the problem?

You say that the patch applies and builds correctly on 2.6.31.9 and
2.6.32.3. I would like to test it with 2.6.32.3, but I didn't find the
source code of it in [1]. It would have to be in another path?


Thanks for your reply and the patch.

Regards,
Daniel

[1] http://www.kernel.org/pub/linux/kernel/v2.6/
-- 
Fingerprint: BFB3 08D6 B4D1 31B2 72B9  29CE 6696 BF1B 14E6 1D37
Powered by Debian GNU/Linux Lenny - Linux user #188.598

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 197 bytes --]

  reply	other threads:[~2009-12-31 15:05 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-12-27 15:51 Memory usage with qemu-kvm-0.12.1.1 Daniel Bareiro
2009-12-27 16:03 ` Avi Kivity
2009-12-27 16:32   ` Rik van Riel
2009-12-27 16:38     ` Avi Kivity
2009-12-27 16:45       ` Rik van Riel
2009-12-27 17:12         ` Avi Kivity
2009-12-27 17:21           ` Rik van Riel
2009-12-30 18:14             ` Hugh Dickins
2009-12-31 15:05               ` Daniel Bareiro [this message]
2009-12-31 17:02                 ` Hugh Dickins
2009-12-31 18:36                   ` Rik van Riel
2010-01-03  6:20                     ` Avi Kivity
2010-01-01 16:59                   ` Daniel Bareiro
2009-12-27 17:00   ` Daniel Bareiro
2009-12-27 17:20     ` Avi Kivity
2009-12-27 17:33       ` Avi Kivity
2009-12-27 18:07         ` Daniel Bareiro
2010-01-03 10:08       ` Thomas Fjellstrom
2010-01-04 15:12         ` Thomas Fjellstrom
2010-01-04 17:18           ` David S. Ahern
2010-01-24  7:43           ` Avi Kivity
2009-12-29  0:49   ` Daniel Bareiro

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=20091231150527.GA16528@defiant.freesoftware \
    --to=dbareiro@uol.com.ar \
    --cc=avi@redhat.com \
    --cc=dbareiro@gmx.net \
    --cc=hugh.dickins@tiscali.co.uk \
    --cc=kvm@vger.kernel.org \
    --cc=riel@redhat.com \
    /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.