qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Anthony Liguori <anthony@codemonkey.ws>
To: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] qcow2 - safe on kill?  safe on power fail?
Date: Mon, 21 Jul 2008 17:50:10 -0500	[thread overview]
Message-ID: <488512A2.8030805@codemonkey.ws> (raw)
In-Reply-To: <48850C22.1000801@quinthar.com>

David Barrett wrote:
> Anthony Liguori wrote:
>> Andreas Schwab wrote:
>>> Anthony Liguori <anthony@codemonkey.ws> writes:
>>>  
>>>> The only ways that you can cause corruption is if the QCOW2 sector
>>>> allocation code is faulty (and you would be screwed no matter what 
>>>> here)
>>>> or if you issue a SIGTERM/SIGKILL that interrupts the code while it's
>>>> allocating a new sector.
>>>
>>> Blocking SIGTERM until the allocation is finished could close that 
>>> hole.
>>
>> Seems like a band-aid to me as SIGKILL is still an issue.  Plus it 
>> would involve modifying all disk formats, not just QCOW2.  I'd rather 
>> see proper journal support added to QCOW2 myself.
>
> Well, SIGKILL is a bit more of an extreme case.  SIGTERM seems like a 
> reasonable way to trigger a graceful shutdown (at least, I know I 
> assumed it did for a long time, whereas I'd never assume SIGKILL was 
> graceful).

It would probably be reasonable to trap SIGTERM and to have it trigger 
the equivalent of the "quit" command in the monitor.  Right now, SIGTERM 
will not result in a graceful shutdown of QEMU.

Regards,

Anthony Liguori

> -david
>
>
>

  reply	other threads:[~2008-07-21 22:50 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-03-05 21:18 [Qemu-devel] Signal handling and qcow2 image corruption David Barrett
2008-03-05 21:55 ` Anthony Liguori
2008-03-05 23:48   ` David Barrett
2008-03-06  6:57   ` Avi Kivity
2008-07-21 18:10   ` [Qemu-devel] qcow2 - safe on kill? safe on power fail? Jamie Lokier
2008-07-21 19:43     ` Anthony Liguori
2008-07-21 21:26       ` Jamie Lokier
2008-07-21 22:14         ` Anthony Liguori
2008-07-21 23:47           ` Jamie Lokier
2008-07-22  6:06           ` Avi Kivity
2008-07-22 14:08             ` Anthony Liguori
2008-07-22 14:46               ` Jamie Lokier
2008-07-22 19:11               ` Avi Kivity
2008-07-22 14:32             ` Jamie Lokier
2008-07-21 22:00       ` Andreas Schwab
2008-07-21 22:15         ` Anthony Liguori
2008-07-21 22:22           ` David Barrett
2008-07-21 22:50             ` Anthony Liguori [this message]
2008-07-22  6:07           ` Avi Kivity
2008-07-22 14:11             ` Anthony Liguori
2008-07-22 14:36               ` Avi Kivity
2008-07-22 16:16                 ` Jamie Lokier
2008-07-22 19:13                   ` Avi Kivity
2008-07-22 20:04                     ` Jamie Lokier
2008-07-22 21:25                       ` Avi Kivity
2008-07-22 14:22             ` 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=488512A2.8030805@codemonkey.ws \
    --to=anthony@codemonkey.ws \
    --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).