Re: [PATCH] virtio-blk: set QUEUE_ORDERED_DRAIN by default
- From: Avi Kivity <avi@xxxxxxxxxx>
- Date: Thu, 27 Aug 2009 14:04:19 +0300
On 08/27/2009 01:43 PM, Rusty Russell wrote:
Because cache=writeback should *not* lie to the guest?Are you claiming qcow2 is unusual? I can believe snapshot is less common,Why do we need to add a feature for this?
though I use it all the time.
You'd normally have to add a feature for something like this. I don't
think this is different.
No, it should.
There are two possible semantics to cache=writeback:
- simulate a drive with a huge write cache; use fsync() to implement barriers
- tell the host that we aren't interested in data integrity, lie to the guest to get best performance
The first semantic is not very useful; guests don't expect huge write caches so you can't be sure of your integrity guarantees, and it's slower than cache=none due to double caching and extra copies. The second semantic is not useful for production, but is very useful for testing out things where you aren't worries about host crashes and you're usually rebooting the guest very often (you can't rely on guest caches, so you want the host to cache).
--
error compiling committee.c: too many arguments to function
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
- Follow-Ups:
- Re: [PATCH] virtio-blk: set QUEUE_ORDERED_DRAIN by default
- From: Rusty Russell
- Re: [PATCH] virtio-blk: set QUEUE_ORDERED_DRAIN by default
- References:
- [PATCH] virtio-blk: set QUEUE_ORDERED_DRAIN by default
- From: Christoph Hellwig
- Re: [PATCH] virtio-blk: set QUEUE_ORDERED_DRAIN by default
- From: Rusty Russell
- Re: [PATCH] virtio-blk: set QUEUE_ORDERED_DRAIN by default
- From: Avi Kivity
- Re: [PATCH] virtio-blk: set QUEUE_ORDERED_DRAIN by default
- From: Rusty Russell
- [PATCH] virtio-blk: set QUEUE_ORDERED_DRAIN by default
- Prev by Date: Re: [PATCHv4 2/2] vhost_net: a kernel-level virtio server
- Next by Date: Re: [PATCHv4 2/2] vhost_net: a kernel-level virtio server
- Previous by thread: Re: [PATCH] virtio-blk: set QUEUE_ORDERED_DRAIN by default
- Next by thread: Re: [PATCH] virtio-blk: set QUEUE_ORDERED_DRAIN by default
- Index(es):
Relevant Pages
|