From: Frank Van Der Linden <frank.van.der.linden@oracle.com>
To: "Ke, Liping" <liping.ke@intel.com>
Cc: Egger <Christoph.Egger@amd.com>,
"xen-devel@lists.xensource.com" <xen-devel@lists.xensource.com>,
"gavin.maltby@oracle.com" <gavin.maltby@oracle.com>,
"Jiang, Yunhong" <yunhong.jiang@intel.com>,
Christoph@rcsinet11.oracle.com,
Keir Fraser <keir.fraser@eu.citrix.com>
Subject: Re: RE: [PATCH] x86: add CMCI software injection interface
Date: Tue, 04 May 2010 21:52:11 -0600 [thread overview]
Message-ID: <4BE0EB6B.6090804@oracle.com> (raw)
In-Reply-To: <789F9655DD1B8F43B48D77C5D30659731E41783E@shsmsx501.ccr.corp.intel.com>
On 5/4/10 7:56 PM, Ke, Liping wrote:
> Hi, Keir
>
> Yes, we do some discussion about the interface compatibility problem before submitting.
> But we found the mceinj_cpunr below is defined as unsigned int (not uint32), so we think it may have some problem in 32/64 bit machine originally.
> So we gave up the backward compatibility effort and made it as uint32_5. But if sun/oracle use this interface before, we did have some problem
>
> So Christopher and Gavin, any suggestions for this?
>
> Regards,
> criping
I have no problems with this change. dom0/Xen should match in version
anyway. There have been ABI breakages in the dom0/Xen interface before.
You can bump the xen MCE hypercall version if you wish..
- Frank
next prev parent reply other threads:[~2010-05-05 3:52 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-04-28 10:06 [PATCH] x86: add CMCI software injection interface Ke, Liping
2010-05-04 13:06 ` Keir Fraser
2010-05-05 1:56 ` Ke, Liping
2010-05-05 3:52 ` Frank Van Der Linden [this message]
2010-05-05 7:11 ` Jiang, Yunhong
2010-05-05 8:54 ` Keir Fraser
2010-05-05 9:07 ` Jiang, Yunhong
2010-05-05 9:20 ` Keir Fraser
2010-05-05 9:34 ` Jiang, Yunhong
2010-06-07 8:12 ` Jiang, Yunhong
2010-05-05 7:46 ` Jan Beulich
2010-05-05 7:53 ` Christoph Egger
2010-05-05 8:09 ` Jan Beulich
2010-05-05 8:54 ` Keir Fraser
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4BE0EB6B.6090804@oracle.com \
--to=frank.van.der.linden@oracle.com \
--cc=Christoph.Egger@amd.com \
--cc=Christoph@rcsinet11.oracle.com \
--cc=gavin.maltby@oracle.com \
--cc=keir.fraser@eu.citrix.com \
--cc=liping.ke@intel.com \
--cc=xen-devel@lists.xensource.com \
--cc=yunhong.jiang@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.