From: "Daniel P. Berrange" <berrange@redhat.com>
To: qemu-devel@nongnu.org
Cc: Gleb Natapov <gleb@redhat.com>
Subject: Re: [Qemu-devel] [6388] Stop VM on ENOSPC error.
Date: Wed, 25 Feb 2009 17:20:51 +0000 [thread overview]
Message-ID: <20090225172051.GT24969@redhat.com> (raw)
In-Reply-To: <49A577FD.60701@codemonkey.ws>
On Wed, Feb 25, 2009 at 10:55:25AM -0600, Anthony Liguori wrote:
> Anthony Liguori wrote:
> >Revision: 6388
> > http://svn.sv.gnu.org/viewvc/?view=rev&root=qemu&revision=6388
> >Author: aliguori
> >Date: 2009-01-21 18:59:04 +0000 (Wed, 21 Jan 2009)
> >
> >Log Message:
> >-----------
> >Stop VM on ENOSPC error. (Gleb Natapov)
> >
> >This version of the patch adds new option "werror" to -drive flag.
> >Possible values are:
> >
> >report - report errors to a guest as IO errors
> >ignore - continue as if nothing happened
> >stop - stop VM on any error and retry last command on resume
> >enospc - stop vm on ENOSPC error and retry last command on resume
> > all other errors are reported to a guest.
> >
> >Default is "report" to maintain current behaviour.
> >
>
> I recently got burnt by the default being "report". I was doing an
> installation and ran out of disk space. The guest did not do anything
> intelligible with the error reports and froze very hard (as you'd expect).
>
> Any objection to changing to default to enospc?
>From a managment POV having QEMU change its state from running to
paused behind our back is hard. You don't want to have to poll on
'info state' to see if the VM has paused, and QEMU provides us no
async notification for this yet. So at this time, if QEMU auto-pauses
we can't notice this change, and so again it'll just appear to the
user as if it froze.
If we get async notifications available via the monitor, then making
the default enospc is very sensible.
Daniel
--
|: Red Hat, Engineering, London -o- http://people.redhat.com/berrange/ :|
|: http://libvirt.org -o- http://virt-manager.org -o- http://ovirt.org :|
|: http://autobuild.org -o- http://search.cpan.org/~danberr/ :|
|: GnuPG: 7D3B9505 -o- F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 :|
next prev parent reply other threads:[~2009-02-25 17:20 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-01-21 18:59 [Qemu-devel] [6388] Stop VM on ENOSPC error Anthony Liguori
2009-02-25 16:55 ` Anthony Liguori
2009-02-25 17:04 ` Gleb Natapov
2009-02-25 17:25 ` Avi Kivity
2009-02-25 17:38 ` Jamie Lokier
2009-02-26 9:20 ` Avi Kivity
2009-02-26 9:24 ` Gleb Natapov
2009-02-25 17:34 ` Daniel P. Berrange
2009-02-25 18:31 ` Anthony Liguori
2009-02-25 18:48 ` Jamie Lokier
2009-02-25 19:29 ` Anthony Liguori
2009-02-25 18:36 ` Jamie Lokier
2009-02-25 17:11 ` Avi Kivity
2009-02-25 18:26 ` Anthony Liguori
2009-02-26 9:17 ` Avi Kivity
2009-02-25 17:20 ` Daniel P. Berrange [this message]
2009-02-25 18:08 ` [Qemu-devel] " Jan Kiszka
2009-02-25 18:27 ` [Qemu-devel] " Anthony Liguori
2009-02-25 18:51 ` Jamie Lokier
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=20090225172051.GT24969@redhat.com \
--to=berrange@redhat.com \
--cc=gleb@redhat.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).