Messages in this thread | | | From | "Liu, Jinsong" <> | Subject | RE: [Xen-devel] [PATCH 1/2] xen/mce: Add mcelog support for Xen platform | Date | Thu, 31 May 2012 18:08:21 +0000 |
| |
Konrad Rzeszutek Wilk wrote: > On Thu, May 31, 2012 at 05:27:53PM +0000, Liu, Jinsong wrote: >> 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@ljsromley.bj.intel.com> >>> >>> 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. > > Are you sure about it? The comments in it speak of piggybacking on > the native MCE handling routines. But since that is not used anymore > do you need to use a different mechanism?
What is not used anymore? what's your concern about cvt_gate_to_trap? I really confused here. Could you elaborate more?
> >> 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 > > Please look at > http://git.kernel.org/?p=linux/kernel/git/konrad/xen.git;a=commit;h=894c02d298cb638f3ff04bdbf41f646a0223bf05 > as it has some of the mistakes fixed (spelling mistakes, wrong title, > etc).
Seems you have checkin my patch for vMCE injection to your tree. If yes, I quit my [Patch 3/3] sent out half hour ago.
>>> >>> 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 > > Which I think the tree is based on too.
So it would not be stuck?
Thanks, Jinsong
| |