From mboxrd@z Thu Jan 1 00:00:00 1970 From: Markus Armbruster Subject: Why QCOW1? (was: [PATCH v2] kvm tool: add QCOW verions 1 read/write support) Date: Fri, 15 Apr 2011 08:41:28 +0200 Message-ID: References: <1302722762-30517-1-git-send-email-prasadjoshi124@gmail.com> <4DA6AA2F.2020306@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Prasad Joshi , mingo@elte.hu, kvm@vger.kernel.org, penberg@kernel.org, asias.hejun@gmail.com, gorcunov@gmail.com, levinsasha928@gmail.com, stefanha@linux.vnet.ibm.com To: Kevin Wolf Return-path: Received: from mx1.redhat.com ([209.132.183.28]:62567 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753770Ab1DOGln (ORCPT ); Fri, 15 Apr 2011 02:41:43 -0400 In-Reply-To: <4DA6AA2F.2020306@redhat.com> (Kevin Wolf's message of "Thu, 14 Apr 2011 10:02:55 +0200") Sender: kvm-owner@vger.kernel.org List-ID: Kevin Wolf writes: > Am 13.04.2011 21:26, schrieb Prasad Joshi: >> The patch only implements the basic read write support for QCOW version 1 >> images. Many of the QCOW features are not implmented, for example >> - image creation >> - snapshot >> - copy-on-write >> - encryption > > Yay, more forks, more code duplication! > > Have you thought about a way to actually share code with qemu instead of > repeating Xen's mistake of copying code, modifying it until merges are > no longer possible and then let it bitrot? > > If you shared code, you also wouldn't have to use an obsolete, but > simple-to-implement format, but could use some of the better maintained > formats of qemu, like qcow2. > > Also at least your qcow1.c is lacking the copyright header. Please add > this, otherwise you're violating the license. As discussed already, reimplementing rather than sharing can be excused, because the existing code is not ready for sharing. What hasn't been discussed much is the other half of Kevin's remark: why QCOW1? Does anyone still use that old hag in anger? The format people actually use is QCOW2, and it is properly maintained. Even for little-used QOW formats, there are more interesting choices: QED and FVD.