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

[Yang Hongyang <yanghy@cn.fujitsu.com>]

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.