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]

Message-ID: <DE8DF0795D48FD4CA783C40EC82923352010D2@SHSMSX101.ccr.corp.intel.com>
Date:	Fri, 1 Jun 2012 07:57:51 +0000
From:	"Liu, Jinsong" <jinsong.liu@...el.com>
To:	Konrad Rzeszutek Wilk <konrad.wilk@...cle.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

Konrad Rzeszutek Wilk wrote:
>>>> 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?
> 
> Well, the mce.c is not involved anymore. So if it we are piggybacking
> on the native MCE handling routines - those routines
> (do_machine_check) won't deliever the data to your driver anymore?
> B/c the do_machine_check is doing mce_log which spools data. But your
> driver is using a different system to de-spool the data - and it does
> not use the mcelog structure array.

native mce is still there, we didn't mask it.
what is not used is native mce_log, w/ guest info (like guest address) when running at xen platform,
instead we use xen_mce_log, a self-contained logic getting real physical error info from xen hypervisor.
that's just what we expected.

> 
> .. snip.
>>>> 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?
> 
> I've no idea what you mean here. Could you elaborate please?

What I meant here is, it should not meet confliction when you patched our patches to latest linus tree.
(Per my understanding, you meet confliction when patched, right? any misunderstanding please tell me)

Thanks,
Jinsong--
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/