From: Avi Kivity <avi@qumranet.com>
To: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] Re: NBD server for QEMU images
Date: Wed, 13 Dec 2006 15:29:10 +0200 [thread overview]
Message-ID: <45800026.2010809@qumranet.com> (raw)
In-Reply-To: <56d259a00612130519l2b324966o2f2cd6e5c69beb10@mail.gmail.com>
Martin Guy wrote:
>> - write tons of data to nbd device, data ends up in pagecache
>> - memory gets low, kswapd wakes up, calls nbd device to actually write
>> the data
>> - nbd issues a request, which ends up on the nbd server on the same
>> machine
>> - the nbd server allocates memory
>> - memory allocation hangs waiting for kswapd
>
> In other words, it can deadlock only if you are swapping to an nbd
> device that is served by nbd-server running on the same machine and
> kernel.
No. It is possible if you issue non-O_SYNC writes.
> In the case of a qemu system swapping over nbd to a server on
> the host machine, it is the guest kernel that waits on the host kernel
> paging the nbd server in from the host's separate swap space, so no
> deadlock is possible.
>
> Practice bears this out; if you wanna stress-test it, here's a program
> that creates a low memory condition by saturating the VM.
It isn't enough to thrash the guest, you need to exhaust host memory as
well. You also need to do it faster than kswapd can write it out.
>
> Of course, this has nothing to do with the original patch, which just
> lets nbd-server interpret qemu image files ;)
>
Agreed. But mounting nbd from localhost is dangerous.
--
error compiling committee.c: too many arguments to function
next prev parent reply other threads:[~2006-12-13 13:29 UTC|newest]
Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-12-12 12:48 [Qemu-devel] NBD server for QEMU images Salvador Fandiño
2006-12-12 13:37 ` Martin Guy
2006-12-12 17:00 ` [Qemu-devel] " Salvador Fandino
2006-12-12 16:58 ` Paul Brook
2006-12-12 17:13 ` Daniel Jacobowitz
2006-12-12 17:33 ` RE : " Sylvain Petreolle
2006-12-12 17:39 ` Paul Brook
2006-12-12 18:54 ` Anthony Liguori
2006-12-12 17:41 ` RE : " Johannes Schindelin
2006-12-12 17:42 ` Daniel Jacobowitz
2006-12-12 18:41 ` [Qemu-devel] Re: RE : " Salvador Fandino
2006-12-13 12:23 ` Jan Marten Simons
2006-12-13 19:03 ` Salvador Fandino
2006-12-13 20:03 ` Jim C. Brown
2006-12-13 22:07 ` Salvador Fandino
2006-12-13 22:55 ` Jim C. Brown
2006-12-14 8:37 ` Salvador Fandino
2006-12-14 14:58 ` Jim C. Brown
2006-12-12 19:00 ` Salvador Fandino
2006-12-12 17:45 ` [Qemu-devel] " Mark Williamson
2006-12-12 19:30 ` Christian MICHON
2006-12-12 15:09 ` Anthony Liguori
2006-12-12 17:32 ` Salvador Fandino
2006-12-12 20:13 ` Anthony Liguori
2006-12-13 2:14 ` Mark Williamson
2006-12-13 11:37 ` Avi Kivity
2006-12-13 13:19 ` Martin Guy
2006-12-13 13:29 ` Avi Kivity [this message]
2006-12-13 19:14 ` Salvador Fandino
2006-12-14 8:34 ` Avi Kivity
2006-12-13 16:58 ` [Qemu-devel] " Mulyadi Santosa
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=45800026.2010809@qumranet.com \
--to=avi@qumranet.com \
--cc=qemu-devel@nongnu.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 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).