From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from [140.186.70.92] (port=46853 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1Ont8c-0006qV-G9 for qemu-devel@nongnu.org; Tue, 24 Aug 2010 09:02:03 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.69) (envelope-from ) id 1Ont8b-0005HG-8o for qemu-devel@nongnu.org; Tue, 24 Aug 2010 09:02:02 -0400 Received: from mail-gx0-f173.google.com ([209.85.161.173]:59839) by eggs.gnu.org with esmtp (Exim 4.69) (envelope-from ) id 1Ont8b-0005HA-5H for qemu-devel@nongnu.org; Tue, 24 Aug 2010 09:02:01 -0400 Received: by gxk26 with SMTP id 26so447499gxk.4 for ; Tue, 24 Aug 2010 06:02:00 -0700 (PDT) Message-ID: <4C73C2BF.8050300@codemonkey.ws> Date: Tue, 24 Aug 2010 08:01:51 -0500 From: Anthony Liguori MIME-Version: 1.0 References: <1282646430-5777-1-git-send-email-kwolf@redhat.com> In-Reply-To: <1282646430-5777-1-git-send-email-kwolf@redhat.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Subject: [Qemu-devel] Re: [RFC][STABLE 0.13] Revert "qcow2: Use bdrv_(p)write_sync for metadata writes" List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Kevin Wolf Cc: stefanha@gmail.com, avi@redhat.com, mjt@tls.msk.ru, qemu-devel@nongnu.org, hch@lst.de On 08/24/2010 05:40 AM, Kevin Wolf wrote: > This reverts commit 8b3b720620a1137a1b794fc3ed64734236f94e06. > > This fix has caused severe slowdowns on recent kernels that actually do flush > when they are told so. Reverting this patch hurts correctness and means that we > could get corrupted images in case of a host crash. This means that qcow2 might > not be an option for some people without this fix. On the other hand, I get > reports that the slowdown is so massive that not reverting it would mean that > people can't use it either because it just takes ages to complete stuff. It > probably can be fixed, but not in time for 0.13.0. > > Usually, if there's a possible tradeoff between correctness and performance, I > tend to choose correctness, but I'm not so sure in this case. I'm not sure with > reverting either, which is why I post this as an RFC only. > > I hope to get some more comments on how to proceed here for 0.13. > How fundamental of an issue is this? Is this something we think we know how to fix and we just don't think there's time to fix it for 0.13? And with this fix in place, how much confidence do we have in qcow2 with respect to data integrity on power loss? We've shipped every version of QEMU since qcow2 was introduced with known data corruptions. It sucks big time. I think it's either that building an image format is a really hard problem akin to making a file system and we shouldn't be in that business or that qcow2 is bad as an image format which makes this all harder than it should be. Either way, I think we need to look seriously at enabling alternatives (i.e. external snapshots, simple sparse files, etc.). Regards, Anthony Liguori > Kevin >