All of lore.kernel.org
 help / color / mirror / Atom feed
From: Avi Kivity <avi@qumranet.com>
To: Zdenek Kabelac <zdenek.kabelac@gmail.com>
Cc: kvm-devel@lists.sourceforge.net
Subject: Re: Qemu-kvm is leaking my memory ???
Date: Mon, 24 Mar 2008 12:09:31 +0200	[thread overview]
Message-ID: <47E77DDB.7050709@qumranet.com> (raw)
In-Reply-To: <c4e36d110803231606s57c72d48oc1ee6244e9bc6dcb@mail.gmail.com>

Zdenek Kabelac wrote:
> 2008/3/23, Avi Kivity <avi@qumranet.com>:
>   
>> Avi Kivity wrote:
>>  >
>>  > I see the same issue too now, and am investigating.
>>  >
>>
>>
>> The attached patch should fix the issue.  It is present in 2.6.25-rc6
>>  only, and not in kvm.git, which is why few people noticed it.
>>
>>     
>
> Hi
>
> Tested - and actually seeing no difference in my case of memory leak.
> Still it looks like over 30M per execution of qemu is lost.
> (tested with fresh 2.6.25-rc6 with your patch)
>
>   

Can you double check? 2.6.25-rc6 definitely leaks without, and here it 
doesn't with the patch.

> Also now I'd have said that before my dmsetup status loop test case
> was not causing big problems and it was just enough to run another
> dmsetup to unblock the loop - now it's usually leads to some wierd end
> of qemu itself - will explore more....
>
> So it's probably fixing some bug - and exposing another.
>
> As I said before - in my debuger it was looping in page_fault hadler -
> i.e.  memory should be paged_in - but as soon as the handler return to
> the code to continue memcopy - new page_fault is invoked and pointer &
> couters are not changed.

I'll add some code to make it possible to enable the mmu tracer in runtime.

-- 
Do not meddle in the internals of kernels, for they are subtle and quick to panic.


-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/

  reply	other threads:[~2008-03-24 10:09 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-03-14 22:14 Qemu-kvm is leaking my memory ??? Zdenek Kabelac
2008-03-16 13:46 ` Avi Kivity
2008-03-19 15:05   ` Zdenek Kabelac
2008-03-19 15:56     ` Avi Kivity
2008-03-19 17:31       ` Zdenek Kabelac
2008-03-19 17:40         ` Avi Kivity
2008-03-23  9:26           ` Zdenek Kabelac
2008-03-23 10:22             ` Avi Kivity
2008-03-23 12:20               ` Avi Kivity
2008-03-23 23:06                 ` Zdenek Kabelac
2008-03-24 10:09                   ` Avi Kivity [this message]
2008-03-24 16:18                     ` Avi Kivity
2008-03-24 21:42                       ` Zdenek Kabelac

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=47E77DDB.7050709@qumranet.com \
    --to=avi@qumranet.com \
    --cc=kvm-devel@lists.sourceforge.net \
    --cc=zdenek.kabelac@gmail.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.