mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: "Liu, Jinsong" <jinsong.liu@intel.com>
To: Borislav Petkov <bp@amd64.org>
Cc: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>,
	"'linux-kernel@vger.kernel.org'" <linux-kernel@vger.kernel.org>,
	"Luck, Tony" <tony.luck@intel.com>,
	"'xen-devel@lists.xensource.com'" <xen-devel@lists.xensource.com>
Subject: RE: [PATCH 1/3] xen/mce: Add mcelog support for Xen platform (RFC)
Date: Thu, 31 May 2012 13:00:58 +0000	[thread overview]
Message-ID: <DE8DF0795D48FD4CA783C40EC82923351FEEAF@SHSMSX101.ccr.corp.intel.com> (raw)
In-Reply-To: <20120530152948.GD4771@aftab.osrc.amd.com>

Borislav Petkov wrote:
> On Wed, May 30, 2012 at 03:20:03PM +0000, Liu, Jinsong wrote:
>>>> IMO currently there are 2 options:
>>>> 1). use the original approach (implicitly redirect /dev/mcelog to
>>>> xen_mce_chrdev_device) --> what point of this approach do you think
>>>> unreasonable? It just remove a 'static' from native mce code! 2).
>>>> use another /dev/xen-mcelog interface, with another misc minor
>>>> '226' 
>>> 
>>> The 2) is no good.
>>> 
>>> 3) What about moving the corresponding other users (so
>>> threshold_init_device), to be at late_initcall and the mce to be at
>>> late_initcall_sync
>> 
>> I can try, but only if Boris would not jump out.
>> and it can only be tested by Boris at AMD platform :(
> 
> Sure, I'll test it.
> 

Boris,

I have update patches, and they have been tested under baremetal (Intel platform) and xen platform.
So you can test it at AMD platform, and thanks ahead!

Regards,
Jinsong

      parent reply	other threads:[~2012-05-31 13:01 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-05-22  5:45 [PATCH 1/3] xen/mce: Add mcelog support for Xen platform (v2) Liu, Jinsong
2012-05-22  9:23 ` Borislav Petkov
2012-05-24 10:10   ` Liu, Jinsong
2012-05-24 10:30     ` Borislav Petkov
2012-05-24 16:15       ` Liu, Jinsong
2012-05-24 16:26         ` Borislav Petkov
2012-05-24 16:52           ` Liu, Jinsong
2012-05-24 19:10             ` [Xen-devel] " Konrad Rzeszutek Wilk
2012-05-25 18:01               ` Liu, Jinsong
2012-05-25 18:03                 ` Konrad Rzeszutek Wilk
2012-05-25 19:55                   ` Liu, Jinsong
2012-05-25 20:23                     ` Konrad Rzeszutek Wilk
2012-05-25 20:47                       ` Liu, Jinsong
2012-05-25 21:15                         ` Konrad Rzeszutek Wilk
2012-05-24 18:49         ` Konrad Rzeszutek Wilk
2012-05-25 17:56           ` Liu, Jinsong
2012-05-25 18:01             ` Konrad Rzeszutek Wilk
2012-05-25 18:55               ` Liu, Jinsong
     [not found]               ` <DE8DF0795D48FD4CA783C40EC82923351F0D53@SHSMSX101.ccr.corp.intel.com>
2012-05-28 13:36                 ` Liu, Jinsong
2012-05-29 18:39                   ` Konrad Rzeszutek Wilk
2012-05-30 15:14                     ` Liu, Jinsong
2012-05-28 14:48                 ` [PATCH 1/3] xen/mce: Add mcelog support for Xen platform (RFC) Liu, Jinsong
2012-05-29 13:38                   ` Borislav Petkov
2012-05-29 16:40                     ` Liu, Jinsong
2012-05-29 17:06                       ` Borislav Petkov
2012-05-30 14:40                         ` Liu, Jinsong
2012-05-30 15:09                     ` Liu, Jinsong
2012-05-30 15:08                       ` Konrad Rzeszutek Wilk
2012-05-30 15:20                         ` Liu, Jinsong
2012-05-30 15:29                           ` Borislav Petkov
2012-05-30 17:21                             ` Liu, Jinsong
2012-05-30 17:28                               ` Borislav Petkov
2012-05-31 13:00                             ` Liu, Jinsong [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=DE8DF0795D48FD4CA783C40EC82923351FEEAF@SHSMSX101.ccr.corp.intel.com \
    --to=jinsong.liu@intel.com \
    --cc=bp@amd64.org \
    --cc=konrad.wilk@oracle.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=tony.luck@intel.com \
    --cc=xen-devel@lists.xensource.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®