On Wed, Jun 01, 2016 at 02:38:17PM +0800, Haozhong Zhang wrote:
> On 05/27/16 17:16, Wei Liu wrote:
> > On Fri, May 27, 2016 at 05:14:08PM +0100, Wei Liu wrote:
> > > On Fri, May 27, 2016 at 10:06:31AM -0600, Jan Beulich wrote:
> > > > >>> On 27.05.16 at 17:31, <wei.l...@citrix.com> wrote:
> > > > > On Fri, May 27, 2016 at 03:06:08PM +0100, Wei Liu wrote:
> > > > >> On Fri, May 27, 2016 at 08:03:42AM -0600, Jan Beulich wrote:
> > > > >> > >>> On 27.05.16 at 15:30, <haozhong.zh...@intel.com> wrote:
> > > > >> > > Commit 26646f3 "x86/mce: translate passed-in GPA to host machine
> > > > >> > > address" and commit 4ddf474 "tools/xen-mceinj: Pass in GPA when
> > > > >> > > injecting through MSR_MCI_ADDR" forgot to consider reserved 
> > > > >> > > domain
> > > > >> > > ID and mistakenly add MC_MSRINJ_F_GPADDR flag for them, which in 
> > > > >> > > turn
> > > > >> > > causes bug reported by
> > > > >> > > http://lists.xenproject.org/archives/html/xen-devel/2016-05/msg02640.html.
> > > > >> > > 
> > > > >> > > This patch removes MC_MSRINK_F_GPADDR flag and checks this when 
> > > > >> > > injecting
> > > > >> > > to reserved domain IDs except DOMID_SELF, and treats the 
> > > > >> > > passed-in
> > > > >> > > address as host machine address.
> > > > >> > > 
> > > > >> > > Signed-off-by: Haozhong Zhang <haozhong.zh...@intel.com>
> > > > >> > 
> > > > >> > Reviewed-by: Jan Beulich <jbeul...@suse.com>
> > > > >> > 
> > > > >> 
> > > > >> Release-acked-by: Wei Liu <wei.l...@citrix.com>
> > > > > 
> > > > > And queued.
> > > > 
> > > > Please wait for a maintainer ack.
> > > > 
> > > 
> > > $ ./scripts/get_maintainer.pl -f xen/arch/x86/cpu/mcheck/mce.c
> > > Christoph Egger <cheg...@amazon.de>
> > > Liu Jinsong <jinsong....@alibaba-inc.com>
> > > Jan Beulich <jbeul...@suse.com>
> > > Andrew Cooper <andrew.coop...@citrix.com>
> > > xen-devel@lists.xen.org
> > > 
> > 
> > OK, so looking at MAINTAINERS file:
> > 
> > MACHINE CHECK (MCA) & RAS
> > M:      Christoph Egger <cheg...@amazon.de>
> > M:      Liu Jinsong <jinsong....@alibaba-inc.com>
> > S:      Supported
> > F:      xen/arch/x86/cpu/mcheck/
> > 
> > I will revert this patch now. Sorry for all the trouble!
> > 
> > Wei.
> 
> Hi Christoph and Jinsong,
> 
> Could you help to look at this patch set?
> 

I think this is a bit too late for 4.7.0.

It can be backported when Christoph and Jinsong get around to it.

Wei.

> Thanks,
> Haozhong

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

Reply via email to