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

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

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.

Xen-devel mailing list



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