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

Re: [Xen-devel] [PATCH 4/4] tools/xen-mceinj: Pass in GPA when injecting through MSR_MCI_ADDR



On Tue, Sep 15, 2015 at 12:08:32PM +0200, Egger, Christoph wrote:
> On 2015/09/15 12:02, Wei Liu wrote:
> > I don't know this piece of code so my comments might be stupid.
> > 
> > On Tue, Sep 15, 2015 at 04:29:40PM +0800, Haozhong Zhang wrote:
> >> This patch removes the address translation in xen-mceinj which
> >> translates the guest physical address passed-in through the argument
> >> of '-p' to the host machine address.
> >>
> > 
> > Is the translation functionality broken or superseded by hardware
> > support? What is the reason for removing this piece of (working?) code?
> 
> Neither nor. The translation done in xen-mceinj.c doesn't deal with
> memory addresses above 4G. The fix is to let the hypervisor do the
> translation.
> 

Ah, I see. I missed patch 3.

Since this tool is distributed with HV so I don't think we chance of
breakage here.

The only concern is backward compatibility in parameter. But because it
is inside tools/tests I feel it's not a requirement to maintain backward
compatibility.

Christoph, since you're more familiar with this functionality could you
kindly have a look at this patch and possible give an ack if you're
happy with it?

Wei.

> Christoph
> 
> > 
> > (I haven't looked at the code)
> > 
> > Wei.
> > 
> 
> Amazon Development Center Germany GmbH
> Krausenstr. 38
> 10117 Berlin
> Geschaeftsfuehrer: Dr. Ralf Herbrich, Christian Schlaeger
> Ust-ID: DE289237879
> Eingetragen am Amtsgericht Charlottenburg HRB 149173 B

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


 


Rackspace

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