WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

Re: [Xen-devel] [Patch 2/3] New blktap implementation

To: Dutch Meyer <dmeyer@xxxxxxxxx>
Subject: Re: [Xen-devel] [Patch 2/3] New blktap implementation
From: Kevin Wolf <kwolf@xxxxxxx>
Date: Fri, 31 Oct 2008 10:59:38 +0100
Cc: "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>, Keir Fraser <keir.fraser@xxxxxxxxxxxxx>
Delivery-date: Fri, 31 Oct 2008 02:57:48 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <Pine.GSO.4.60.0810302011540.6387@xxxxxxxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <Pine.GSO.4.60.0810302011540.6387@xxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Thunderbird 2.0.0.12 (X11/20071114)
Dutch Meyer schrieb:
> Patch 2 of 3.
> 
> Signed-off-by: Jake Wires <jake.wires@xxxxxxxxxx>, Dutch Meyer
> <dmeyer@xxxxxxxxx>
> 
> This is a new and rewritten version of blktap that we have developed at
> Citrix.  The current version of blktap is left functionally unmodified.
> The change set consists of three patches.

You only have attached a fixqcow patch, no blktap rewrite.

Could you explain what the qcow patch is supposed to fix? I think we
need good reasons to change this code because every change increases the
differences between the tapdisk and qemu implementation of the format.
It was agreed that eventually we want to get rid of the tapdisk
implementation of the formats qemu can provide, but in the meantime we
should at least be careful not to increase differences.

If these are real fixes, maybe they should also go into upstream qemu?

Kevin

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel