Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754906Ab2EaSI1 (ORCPT ); Thu, 31 May 2012 14:08:27 -0400 Received: from mga14.intel.com ([143.182.124.37]:64460 "EHLO mga14.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751879Ab2EaSIZ convert rfc822-to-8bit (ORCPT ); Thu, 31 May 2012 14:08:25 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.71,315,1320652800"; d="scan'208";a="106383508" From: "Liu, Jinsong" To: Konrad Rzeszutek Wilk CC: "Luck, Tony" , "'xen-devel@lists.xensource.com'" , Borislav Petkov , "'linux-kernel@vger.kernel.org'" Subject: RE: [Xen-devel] [PATCH 1/2] xen/mce: Add mcelog support for Xen platform Thread-Topic: [Xen-devel] [PATCH 1/2] xen/mce: Add mcelog support for Xen platform Thread-Index: AQHNP1RDM9HNPWGJrkGbs3SSikmUqJbkLqPQ Date: Thu, 31 May 2012 18:08:21 +0000 Message-ID: References: <20120531161139.GA13939@phenom.dumpdata.com> <20120531173137.GA31735@phenom.dumpdata.com> In-Reply-To: <20120531173137.GA31735@phenom.dumpdata.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.239.127.40] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 8BIT MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2468 Lines: 65 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 >>> >>> 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-- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/