From: Marcelo Tosatti <marcelo@kvack.org>
To: Anthony Liguori <anthony@codemonkey.ws>
Cc: Marcelo Tosatti <marcelo@kvack.org>,
kvm-devel <kvm-devel@lists.sourceforge.net>,
Avi Kivity <avi@qumranet.com>
Subject: Re: [PATCH] virtio-balloon: do not attempt to release more than available pages
Date: Wed, 5 Mar 2008 16:39:41 -0300 [thread overview]
Message-ID: <20080305193941.GA13856@dmt> (raw)
In-Reply-To: <47CEE9E0.3080204@codemonkey.ws>
> >I guess the description was not clear, you understood the opposite.
> >
> >The problem is when the target for total guest pages (not balloon target
> >size) is set to be larger than the amount of total pages the guest has
> >booted with. What happens then is that the driver tries to release pages
> >from the balloon, without checking if there are any:
> >
>
> target in the config space is target balloon size, not target for total
> guest pages. So how is it ever possible for this condition to occur?
Set the target in QEMU to be larger than guest ram size. The config
space variable will be set negatively, so guest attempts to release
pages from the balloon.
-------------------------------------------------------------------------
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/
next prev parent reply other threads:[~2008-03-05 19:39 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-03-05 16:28 [PATCH] virtio-balloon: do not attempt to release more than available pages Marcelo Tosatti
2008-03-05 16:59 ` Avi Kivity
2008-03-05 18:12 ` Marcelo Tosatti
2008-03-05 18:13 ` Avi Kivity
2008-03-05 18:43 ` Anthony Liguori
2008-03-05 19:39 ` Marcelo Tosatti [this message]
2008-03-05 19:42 ` Anthony Liguori
2008-03-06 7:06 ` Avi Kivity
2008-03-05 18:42 ` Anthony Liguori
2008-03-05 22:39 ` Rusty Russell
2008-03-08 19:06 ` Marcelo Tosatti
2008-03-11 0:26 ` Rusty Russell
2008-03-11 0:52 ` Anthony Liguori
2008-03-11 11:54 ` Rusty Russell
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=20080305193941.GA13856@dmt \
--to=marcelo@kvack.org \
--cc=anthony@codemonkey.ws \
--cc=avi@qumranet.com \
--cc=kvm-devel@lists.sourceforge.net \
/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