Re: [LTP] [RFC] make mce-test a submodule

Caspar Zhang
2012-04-17T02:39:01Z

On 04/17/2012 10:03 AM, Chen Gong wrote:
> 于 2012/4/9 14:02, Chen Gong 写道:
>> 于 2012/4/9 12:03, Caspar Zhang 写道:
>>> On 04/09/2012 11:10 AM, Chen Gong wrote:
>>>> 于 2012/4/6 22:33, Caspar Zhang 写道:
>>>>> On 03/21/2012 05:18 PM, Chen Gong wrote:
>>>>>> 于 2012/3/16 22:35, Andi Kleen 写道:
>>>>>>> On Fri, Mar 16, 2012 at 03:27:44PM +0800, Caspar Zhang wrote:
>>>>>>>> Hi all, I'll merge this change to master if I don't hear any
>>>>>>>> objections
>>>>>>>> from you by today.
>>>>>>> Fine for me. Copying Gong Chen who did most of the mce-test work
>>>>>>> recently
>>>>>>> too.
>>>>>>>
>>>>>>> -Andi
>>>>>> Oh, yes. Recently I make a big change on mce-test project and old
>>>>>> structure will be reorganized.
>>>>>> And I ever looked through the mce-test part in LTP. Unfortunately
>>>>>> it is
>>>>>> very old and can't run on
>>>>>> newer kernel. I hope you can keep current disabled status in LTP and
>>>>>> wait my new work is merged.
>>>>>> We can talk at that time. Is it OK to you?
>>>>> Hi Chen Gong, sorry for late reply, we're planning to remove old
>>>>> static
>>>>> coded mce-test from LTP, and use it as a submodule, which means,
>>>>> mce-test won't be static-coded any more, it's always synced up with
>>>>> your
>>>>> upstream version, we just need to do some manual ``submodule update``
>>>>> operations to checkout your version when doing a LTP release. So I'll
>>>>> merge this patch in, remove all current out-of-date codes, and wait
>>>>> for
>>>>> your notice of mce-test's next version, so that we can include it
>>>>> in our
>>>>> next LTP stable release.
>>>>>
>>>>> Thanks,
>>>>> Caspar
>>>>>
>>>> It's fine to me. So do you want to get a notice from me after the codes
>>>> are merged or you
>>>> monitor the upstream status by yourself?
>>> I'd appreciate it if you can send me a mail when the codes get merged,
>>> but I'll check upstream status as well from time to time.
>>>
>>> Thanks,
>>> Caspar
>>>
>> OK, I will. Now patch is in the review status, so I will tell you ASAP
>> after merged.
> The new patch for mce-test has been merged, but it looks a little bit wrong
> during the merge procedure (all shell scripts execution right are missed).
> I'm pushing Andi to fix it, so please don't add mce-test at current stage.
> I will tell you when it is ready.

Noted. Thanks for the update.

Thanks,
Caspar

------------------------------------------------------------------------------
Better than sec? Nothing is better than sec when it comes to
monitoring Big Data applications. Try Boundary one-second
resolution app monitoring today. Free.