qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Anthony Liguori <anthony@codemonkey.ws>
To: Virtbie <virtbie@shiftmail.org>
Cc: Kevin Wolf <kwolf@redhat.com>, qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] Is cache=writeback safe yet?
Date: Mon, 20 Feb 2012 09:06:21 -0600	[thread overview]
Message-ID: <4F42616D.30800@codemonkey.ws> (raw)
In-Reply-To: <4F425643.2@shiftmail.org>

On 02/20/2012 08:18 AM, Virtbie wrote:
> Dear qemuers,
> thanks for your exellent software.
>
> I would like to use cache=writeback, but I still can't understand if this is
> safe or not in case of power loss.

"Safe" is too simplistic of a view.  The documentation in qemu-options.hx 
probably needs to be revisited.

cache=writeback emulates a large disk cache much the same as every modern hard 
drive has a builtin cache.

The only real difference is that the host cache is very, very large.  Some 
modern file systems did not take the necessary steps to ensure consistency when 
a volatile write cache is present (ext3 up until very recently when barrier=1 
became default).

In practice, this didn't create a huge issue because disk write caches are 
flushed often.  Ted T'so has written a lot about the practical bits here.

But since the host cache is very large, and may not be flushed for many minutes 
after the initial write, this can exacerbate the problem.

In short, if you're using a recent kernel with ext3 or ext4, cache=writeback is 
absolutely safe.  If you're using an older version of ext3, cache=writeback is 
still safe but ext3 itself isn't.  cache=writeback can make the situation worse.

cache=writethrough presents a non-volatile cache which makes even older ext3 
filesystems safe.

Regards,

Anthony Liguori

> In particular with virtio-blk on bare LVM device.
>
> Qemu manpage still says cache=writeback isn't safe, but I seem to find some
> discordant information reading around.
>
> Is the WCE+volatile flag exposed to the guest already? I am logging into a
> 2.6.38 guest and trying to find an indicator of disk cache in
> /sys/block/vda/device but I am not able to. There is an obscure very long
> "features" bitmask which I don't know what it shows. I think this mode is not
> safe until WBC is exposed, AFAIU, right?
>
>
> Also please have a look at this:
> https://events.linuxfoundation.org/slides/2011/linuxcon-japan/lcj2011_hajnoczi.pdf
> page "Caching modes in Qemu"
> In the table it is written that writeback has "Guest disk write cache = on".
> Does that mean that the guest can commit to real platters by issuing a flush on
> the virtual device?
> So the problem lies in the fact that no guest will ever spontaneously issue the
> flush because they don't see a wce=1?
>
>
> There is another thing I don't understand:
> I think I read somewhere that cache=none is safer than cache=writeback. Is that
> true? I think that both have a writeback cache, one is in the physical disks
> attached to the host, the other is in the host's page cache, so they should have
> about the same level of safety. What do you think?
>
>
> Lastly, regarding the newly introduced cache=directsync. Do I understand
> correctly that it is supposed to be as safe as cache=writeback but also as slow
> as cache=writeback, for writes, and in addition it cannot use the host pagecache
> for reads, is that correct?
>
>
> BTW I also wanted to say I very much like the simplification proposed by Anthony
> Liguori on 06/29/2011:
>
>> Thinking twice about this, shouldn't we just move to a simplified model:
>>
>> -drive file=foo.img,cache=[on|off],hd0 -device virtio-blk-pci,drive=hd0,wce=1
>
> splitting the cache features in two (host side / guest side) would very much
> clarify what is happening in Qemu, for us ignorant users.
>
>
> Thank you
> Vb.
>

  reply	other threads:[~2012-02-20 15:06 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-02-20 14:18 [Qemu-devel] Is cache=writeback safe yet? Virtbie
2012-02-20 15:06 ` Anthony Liguori [this message]
2012-02-20 15:43   ` Virtbie
2012-02-20 17:52     ` Virtbie
2012-02-20 15:20 ` Kevin Wolf
2012-02-20 15:29   ` Peter Maydell
2012-02-20 15:56     ` Kevin Wolf
2012-02-20 16:08       ` Peter Maydell
2012-02-20 17:03         ` Paolo Bonzini
2012-02-20 17:10           ` Peter Maydell

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=4F42616D.30800@codemonkey.ws \
    --to=anthony@codemonkey.ws \
    --cc=kwolf@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=virtbie@shiftmail.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).