Re: [Xen-devel] [PATCH v2] tools/xen-mceinj: support AMD

["Liu, Jinsong" <jinsong.liu@xxxxxxxxx>]

Ian Campbell wrote:
> On Fri, 2013-05-31 at 09:30 +0100, Jan Beulich wrote:
>>>>> On 30.05.13 at 16:33, "Egger, Christoph" <chegger@xxxxxxxxx>
>>>>> wrote: 
>>> a8206654c64f:patches chegger$ cat xen_mceinj.diff
>>> commit a277555e158c87aed34196f72eba0a4cf8f0fb38
>>> Author: Christoph Egger <chegger@xxxxxxxxx>
>>> Date:   Wed Feb 27 14:52:19 2013 +0000
>>> 
>>>     xen-mceinj: Support AMD. Add -e option.
>>> 
>>>     Add support for AMD.
>>>     Add -e option to raise an exception.
>>> 
>>>     Signed-off-by: Christoph Egger <chegger@xxxxxxxxx>
>> 
>> Apart from needing an ack from someone at Intel,
> 
> I'm not sure this needs to be all that strictly enforced for what is
> AIUI basically a devtest tool. Having the ACK would be better than not
> having it, but this patch has IIRC been posted several times over a 6
> month period without comment from Intel and there's no reason to hold
> it up indefinitely waiting for them IMHO.
> 
> I'm not sure if Intel were CCd prior to this particular posting -- if
> not then we should give them a chance to respond but otherwise I think
> we should take this patch once your other comments are addressed.
> 
> Ian.

Sorry, my fault.
Have sent out comments.

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