kvm.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Takuya Yoshikawa <yoshikawa.takuya@oss.ntt.co.jp>
To: Avi Kivity <avi@redhat.com>
Cc: Jan Kiszka <jan.kiszka@siemens.com>,
	Marcelo Tosatti <mtosatti@redhat.com>, kvm <kvm@vger.kernel.org>,
	Takuya Yoshikawa <takuya.yoshikawa@gmail.com>
Subject: Re: [PATCH] KVM: x86: Drop unused return value of kvm_mmu_remove_some_alloc_mmu_pages
Date: Tue, 06 Dec 2011 19:52:29 +0900	[thread overview]
Message-ID: <4EDDF3ED.3090704@oss.ntt.co.jp> (raw)
In-Reply-To: <4EDDF14F.70804@redhat.com>

(2011/12/06 19:41), Avi Kivity wrote:
> On 12/06/2011 12:40 PM, Avi Kivity wrote:
>>>
>>> Is there any need for mmu_shrink()?
>>
>> Without it a user can easily pin large amounts of kernel memory by
>> filling guest memory with page tables and shadowing them all.
>
> It would make an interesting test case, btw, to try to crash the kernel
> this way, and exercise the shrinker.
>

If nobody is working on this mmu_shrink() issues, I will do something.

But what I saw from the code is that the current zapper is effective
for the cases there are some unsync children.

So we may need to zap rather forcibly for tdp.

	Takuya

      reply	other threads:[~2011-12-06 10:51 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-12-02 17:35 [PATCH] KVM: x86: Drop unused return value of kvm_mmu_remove_some_alloc_mmu_pages Jan Kiszka
2011-12-04 16:48 ` Avi Kivity
2011-12-06  9:56 ` Takuya Yoshikawa
2011-12-06 10:40   ` Avi Kivity
2011-12-06 10:41     ` Avi Kivity
2011-12-06 10:52       ` Takuya Yoshikawa [this message]

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=4EDDF3ED.3090704@oss.ntt.co.jp \
    --to=yoshikawa.takuya@oss.ntt.co.jp \
    --cc=avi@redhat.com \
    --cc=jan.kiszka@siemens.com \
    --cc=kvm@vger.kernel.org \
    --cc=mtosatti@redhat.com \
    --cc=takuya.yoshikawa@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).