From: George Dunlap <george.dunlap@eu.citrix.com>
To: xen-devel@lists.xen.org
Cc: Wei Liu <wei.liu2@citrix.com>,
Ian Campbell <ian.campbell@citrix.com>,
Wen Congyang <wency@cn.fujitsu.com>,
Jonathan Ludlam <Jonathan.Ludlam@eu.citrix.com>,
Ian Jackson <ian.jackson@citrix.com>,
Yang Hongyang <yanghy@cn.fujitsu.com>,
Dave Scott <Dave.Scott@eu.citrix.com>
Subject: Re: [PATCH RFC v2 0/3] tools: Allow building XenServer's blktap 2.5 as an external tree.
Date: Thu, 16 Apr 2015 16:57:30 +0100 [thread overview]
Message-ID: <552FDBEA.9070206@eu.citrix.com> (raw)
In-Reply-To: <1429199767-25832-1-git-send-email-george.dunlap@eu.citrix.com>
On 04/16/2015 04:56 PM, George Dunlap wrote:
> For some time, the blktap2 in-tree has bitrotted. Many years ago the
> XenServer team at Citrix forked the code into a separate repository;
> several attempts have been made to upstream those changes back into
> Xen, to no avail.
>
> The blktap code at the moment is the only source of performant vhd
> format integration. It's additionally in use by projects like the
> COLO project.
>
> This patch series removes the in-tree blktap2 code and enables xen to
> detect and build against the XenServer blktap tree. I've gotten
> agreement from the XenServer team to act as an upstream -- to accept
> patches fixing bugs, to help track down errors, and to attempt to help
> fix build breakages introduced by development.
>
> At the moment this patch is against the "blktap2" branch of
> XenServer's blktap.git. (This has been sometimes known as "blktap
> 2.5".) This branch is maintianed in order to provide a buildroot for
> OpenStack, and has also been used by the CentOS xen packages for
> several years now.
>
> The series consists of three patches:
>
> 1/3 Remove the blktap2 tree and temporarily disable it in config
>
> 2/3 Port libxl to use the upstream blktap2.
>
> 3/3 Detect blktap2 and enable it by default if detected.
>
> The first patch is pretty massive and mostly not very interesting;
> I'll be sending a modified patch with the interesting bits. The full
> patch series can be found here:
>
> git://xenbits.xen.org/people/gdunlap/xen.git out/blktap25/rfc-v2
>
> This series has some niggles (noted in the patches themselves), and so
> it not yet ready to apply as-is.
>
> One of the big ones is that the blktap headers are borked: you can't
> actually include them from where they get installed. This is
> something I'm going to try to address upstream.
I forgot to add,
Difference vs v1: build as an external library, rather than cloning and
building it internally.
-George
prev parent reply other threads:[~2015-04-16 15:57 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-04-16 15:56 [PATCH RFC v2 0/3] tools: Allow building XenServer's blktap 2.5 as an external tree George Dunlap
2015-04-16 15:56 ` [PATCH RFC v2 1/3] tools: Remove in-tree blktap2 George Dunlap
2015-04-16 16:45 ` Ian Jackson
2015-04-16 15:56 ` [PATCH RFC v2 2/3] libxl: Port libxl_blktap2.c to blktap2.5 API George Dunlap
2015-04-16 16:45 ` Ian Jackson
2015-04-16 15:56 ` [PATCH RFC v2 3/3] tools/configure: Detect out-of-tree blktap2 and use by default if available George Dunlap
2015-04-16 15:57 ` George Dunlap [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=552FDBEA.9070206@eu.citrix.com \
--to=george.dunlap@eu.citrix.com \
--cc=Dave.Scott@eu.citrix.com \
--cc=Jonathan.Ludlam@eu.citrix.com \
--cc=ian.campbell@citrix.com \
--cc=ian.jackson@citrix.com \
--cc=wei.liu2@citrix.com \
--cc=wency@cn.fujitsu.com \
--cc=xen-devel@lists.xen.org \
--cc=yanghy@cn.fujitsu.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.