From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1MgcaQ-0008Nc-PJ for qemu-devel@nongnu.org; Thu, 27 Aug 2009 06:52:10 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1MgcaL-0008Jh-F0 for qemu-devel@nongnu.org; Thu, 27 Aug 2009 06:52:09 -0400 Received: from [199.232.76.173] (port=59091 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1MgcaL-0008JM-4W for qemu-devel@nongnu.org; Thu, 27 Aug 2009 06:52:05 -0400 Received: from ozlabs.org ([203.10.76.45]:47319) by monty-python.gnu.org with esmtps (TLS-1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.60) (envelope-from ) id 1MgcaJ-0005WL-RZ for qemu-devel@nongnu.org; Thu, 27 Aug 2009 06:52:04 -0400 From: Rusty Russell Date: Thu, 27 Aug 2009 20:21:55 +0930 References: <20090825181120.GA4863@lst.de> In-Reply-To: <20090825181120.GA4863@lst.de> MIME-Version: 1.0 Content-Type: Text/Plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Content-Disposition: inline Message-Id: <200908272021.56219.rusty@rustcorp.com.au> Subject: [Qemu-devel] Re: Notes on block I/O data integrity List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Christoph Hellwig Cc: qemu-devel@nongnu.org, kvm@vger.kernel.org On Wed, 26 Aug 2009 03:41:20 am Christoph Hellwig wrote: > As various people wanted to know how the various data integrity patches > I've send out recently play together here's a small writeup on what > issues we have in QEMU and how to fix it: Classic mail. Thanks for the massive and coherent clue injection! > Action plan for the guest drivers: >=20 > - virtio-blk needs to advertise ordered queue by default. > This makes cache=3Dwritethrough safe on virtio. =46rom a guest POV, that's "we don't know, let's say we're ordered because = that may make us safer". Of course, it may not help: how much does it cost to drain the queue? The bug, IMHO is that we *should* know. And in future I'd like to fix that, either by adding an VIRTIO_BLK_F_ORDERED feature, or a VIRTIO_BLK_F_UNORDER= ED feature. > Action plan for QEMU: >=20 > - IDE needs to set the write cache enabled bit > - virtio needs to implement a cache flush command and advertise it > (also needs a small change to the host driver) So, virtio-blk needs to be enhanced for this as well. Thanks! Rusty.