[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH for 4.6 0/6] Prune legacy migration and move migration v2 out of daft status





On 07/22/2015 08:20 PM, Wei Liu wrote:
On Wed, Jul 22, 2015 at 12:57:59PM +0100, Andrew Cooper wrote:
On 22/07/15 12:30, Wei Liu wrote:
On Wed, Jul 22, 2015 at 12:26:19PM +0100, Andrew Cooper wrote:
On 22/07/15 12:22, Wei Liu wrote:
On Wed, Jul 22, 2015 at 11:33:06AM +0100, Ian Campbell wrote:
On Mon, 2015-07-20 at 11:37 +0100, Andrew Cooper wrote:
This series is some cleanup following the integration of migration v2
into the codebase.  It removes the legacy migration implementation
(compatability is provided with the python conversion script), and
adjusts the migration v2 docs/implementation to no longer be
experimental.
IMHO we should take this for 4.6, there is no point in shipping
obsolete/unused code and leaving it there with the xc_domain_save name
and the supported thing with a 2 suffix will only tempt people to
continue to use or build on it.

The only real risk here is breaking the build, since if it builds it
will surely work.

Wei, final call is yours.

Yes. I agree with you. But please do check this doesn't conflict with
COLOPre before applying.

Wei.
There are no conflicts I am aware of, having rebased this series myself.

Do you mean you rebase on top of full series of  COLOPre (including
those patches that have not been applied)? That's what I'm looking at.
Hongyang could still post remaining part of COLOPre within this week.

Ah - this series does interfere with that, insofar as the other bits of
COLOPre patch the legacy code to maintain the build.

One of us will have to rebase over the other.  I am not fussed which way
around that is.


Then this series needs to wait until the window for applying patches
with freeze exception is closed. Those series have priority in this
week.

Thank you Wei, I think I will remove all colo related patches from COLOPre
and send a patchset which only do pure refactoring & bug fix. So that the only
patch I need to sort out is the 7th.


Wei.

~Andrew
.


--
Thanks,
Yang.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.