RE: [PATCH 1/2] xen/mce: Add mcelog support for Xen platform

[Liu, Jinsong]

Konrad Rzeszutek Wilk wrote:
> On Thu, May 31, 2012 at 12:57:44PM +0000, Liu, Jinsong wrote:
>>> From 1a7951d6ca01d7f2c9dd2bdb6de5f8e7fdcb8bbd Mon Sep 17 00:00:00
>>> 2001 
>> From: root <root@xxxxxxxxxxxxxxxxxxxxxx>
> 
> Also your git author is busted. I fixed it up for you but
> you might want to run 'git config --user.name' and such

Ah, I forgot it. Thanks for fix it.

> 
>> Date: Fri, 1 Jun 2012 03:12:51 +0800
>> Subject: [PATCH 1/2] xen/mce: Add mcelog support for Xen platform
> 
> What about the cvt_gate_to_trap? Does that need something similar
> to "xen/mce: Register native mce handler as vMCE bounce back point"
> ?

That's vMCE injection logic.
anyway, I will present new round of patch according to Boris and your commends, as following
[Patch 1/3] xen-mce-Add-mcelog-support-for-Xen-platform.patch
[Patch 2/3] X86-MCE-AMD-Adjust-initcall-sequence-for-xen.patch
[Patch 3/3] Register-native-mce-handler-as-vMCE-bounce-back-poin.patch

Patch 1/3 and 2/3 are for mcelog
Patch 3/3 are for vMCE injection

> 
> I stuck all these patches on devel/mce.v2 on
> git://git.kernel.org/pub/scm/linux/kernel/git/konrad/xen.git

Hmm, we discussed base-tree problem before, and decided to use linus latest tree.
I pull this morning and rebase my patches based on
git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
c/s a01ee165a132fadb57659d26246e340d6ac53265


Thanks,
Jinsong
> --- Begin Message ---
> On Fri, Apr 06, 2012 at 07:13:43AM +0000, Liu, Jinsong wrote:
> > Konrad,
> > 
> > I'm rebasing Xen mcelog and cpu online/offline logic now.
> > I'm not quite sure which tree is proper as base tree, has your Xen PM patches all in kernel tree now? if yes, maybe we can directly rebased on latest kernel tree, your opinion?
>
> Use the linus tree. The latest has the xen pm patches.
>
> > 
> > and, if we rebased on your pvops
> > 1. what's the git address (git://git.kernel.org/pub/scm/linux/kernel/git/konrad/xen.git ?)
> > 2. which branch (linux-next ?)
> > (considering your PM patches (which cpu hotplug need reuse some logic) has been in kernel ?)
> > 
> > Thanks,
> > Jinsong
>
> --- End Message ---