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

[Ian Campbell <Ian.Campbell@xxxxxxxxxx>]

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.


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