linux-kernel - Re: [Xen-devel] [PATCH 1/2] xen/mce: Add mcelog support for Xen platform

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Thu, 31 May 2012 13:31:37 -0400
From:	Konrad Rzeszutek Wilk <konrad.wilk@...cle.com>
To:	"Liu, Jinsong" <jinsong.liu@...el.com>
Cc:	"Luck, Tony" <tony.luck@...el.com>,
	"'xen-devel@...ts.xensource.com'" <xen-devel@...ts.xensource.com>,
	Borislav Petkov <bp@...64.org>,
	"'linux-kernel@...r.kernel.org'" <linux-kernel@...r.kernel.org>
Subject: Re: [Xen-devel] [PATCH 1/2] xen/mce: Add mcelog support for Xen
 platform

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@...romley.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?

> 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).

> 
> 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

Which I think the tree is based on too.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/