From: Hidetoshi Seto <seto.hidetoshi@jp.fujitsu.com>
To: Marcelo Tosatti <mtosatti@redhat.com>
Cc: kvm@vger.kernel.org, qemu-devel@nongnu.org,
Huang Ying <ying.huang@intel.com>,
Dean Nelson <dnelson@redhat.com>
Subject: Re: [patch uq/master 7/8] MCE: Relay UCR MCE to guest
Date: Wed, 06 Oct 2010 10:58:36 +0900 [thread overview]
Message-ID: <4CABD7CC.6030909@jp.fujitsu.com> (raw)
In-Reply-To: <20101004185715.167557459@redhat.com>
I got some more question:
(2010/10/05 3:54), Marcelo Tosatti wrote:
> Index: qemu/target-i386/cpu.h
> ===================================================================
> --- qemu.orig/target-i386/cpu.h
> +++ qemu/target-i386/cpu.h
> @@ -250,16 +250,32 @@
> #define PG_ERROR_RSVD_MASK 0x08
> #define PG_ERROR_I_D_MASK 0x10
>
> -#define MCG_CTL_P (1UL<<8) /* MCG_CAP register available */
> +#define MCG_CTL_P (1ULL<<8) /* MCG_CAP register available */
> +#define MCG_SER_P (1ULL<<24) /* MCA recovery/new status bits */
>
> -#define MCE_CAP_DEF MCG_CTL_P
> +#define MCE_CAP_DEF (MCG_CTL_P|MCG_SER_P)
> #define MCE_BANKS_DEF 10
>
It seems that current kvm doesn't support SER_P, so injecting SRAO
to guest will mean that guest receives VAL|UC|!PCC and RIPV event
from virtual processor that doesn't have SER_P.
I think most OSes don't expect that it can receives MCE with !PCC
on traditional x86 processor without SER_P.
Q1: Is it safe to expect that guests can handle such !PCC event?
Q2: What is the expected behavior on the guest?
Q3: What happen if guest reboots itself in response to the MCE?
Thanks,
H.Seto
WARNING: multiple messages have this Message-ID (diff)
From: Hidetoshi Seto <seto.hidetoshi@jp.fujitsu.com>
To: Marcelo Tosatti <mtosatti@redhat.com>
Cc: Dean Nelson <dnelson@redhat.com>,
qemu-devel@nongnu.org, kvm@vger.kernel.org,
Huang Ying <ying.huang@intel.com>
Subject: [Qemu-devel] Re: [patch uq/master 7/8] MCE: Relay UCR MCE to guest
Date: Wed, 06 Oct 2010 10:58:36 +0900 [thread overview]
Message-ID: <4CABD7CC.6030909@jp.fujitsu.com> (raw)
In-Reply-To: <20101004185715.167557459@redhat.com>
I got some more question:
(2010/10/05 3:54), Marcelo Tosatti wrote:
> Index: qemu/target-i386/cpu.h
> ===================================================================
> --- qemu.orig/target-i386/cpu.h
> +++ qemu/target-i386/cpu.h
> @@ -250,16 +250,32 @@
> #define PG_ERROR_RSVD_MASK 0x08
> #define PG_ERROR_I_D_MASK 0x10
>
> -#define MCG_CTL_P (1UL<<8) /* MCG_CAP register available */
> +#define MCG_CTL_P (1ULL<<8) /* MCG_CAP register available */
> +#define MCG_SER_P (1ULL<<24) /* MCA recovery/new status bits */
>
> -#define MCE_CAP_DEF MCG_CTL_P
> +#define MCE_CAP_DEF (MCG_CTL_P|MCG_SER_P)
> #define MCE_BANKS_DEF 10
>
It seems that current kvm doesn't support SER_P, so injecting SRAO
to guest will mean that guest receives VAL|UC|!PCC and RIPV event
from virtual processor that doesn't have SER_P.
I think most OSes don't expect that it can receives MCE with !PCC
on traditional x86 processor without SER_P.
Q1: Is it safe to expect that guests can handle such !PCC event?
Q2: What is the expected behavior on the guest?
Q3: What happen if guest reboots itself in response to the MCE?
Thanks,
H.Seto
next prev parent reply other threads:[~2010-10-06 1:59 UTC|newest]
Thread overview: 93+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-10-04 18:54 [patch uq/master 0/8] port qemu-kvm's MCE support Marcelo Tosatti
2010-10-04 18:54 ` [Qemu-devel] " Marcelo Tosatti
2010-10-04 18:54 ` [patch uq/master 1/8] signalfd compatibility Marcelo Tosatti
2010-10-04 18:54 ` [Qemu-devel] " Marcelo Tosatti
2010-10-04 18:54 ` [patch uq/master 2/8] iothread: use signalfd Marcelo Tosatti
2010-10-04 18:54 ` [Qemu-devel] " Marcelo Tosatti
2010-10-04 18:54 ` [patch uq/master 3/8] Expose thread_id in info cpus Marcelo Tosatti
2010-10-04 18:54 ` [Qemu-devel] " Marcelo Tosatti
2010-10-04 18:54 ` [patch uq/master 4/8] kvm: x86: add mce support Marcelo Tosatti
2010-10-04 18:54 ` [Qemu-devel] " Marcelo Tosatti
2010-10-04 18:54 ` [patch uq/master 5/8] Export qemu_ram_addr_from_host Marcelo Tosatti
2010-10-04 18:54 ` [Qemu-devel] " Marcelo Tosatti
2010-10-05 12:57 ` Anthony Liguori
2010-10-05 12:57 ` [Qemu-devel] " Anthony Liguori
2010-10-05 20:13 ` Marcelo Tosatti
2010-10-05 20:13 ` [Qemu-devel] " Marcelo Tosatti
2010-10-05 20:48 ` Anthony Liguori
2010-10-05 20:48 ` [Qemu-devel] " Anthony Liguori
2010-10-04 18:54 ` [patch uq/master 6/8] Add RAM -> physical addr mapping in MCE simulation Marcelo Tosatti
2010-10-04 18:54 ` [Qemu-devel] " Marcelo Tosatti
2010-10-04 18:54 ` [patch uq/master 7/8] MCE: Relay UCR MCE to guest Marcelo Tosatti
2010-10-04 18:54 ` [Qemu-devel] " Marcelo Tosatti
2010-10-06 1:10 ` Hidetoshi Seto
2010-10-06 1:10 ` [Qemu-devel] " Hidetoshi Seto
2010-10-06 16:02 ` Marcelo Tosatti
2010-10-06 16:02 ` [Qemu-devel] " Marcelo Tosatti
2010-10-06 1:58 ` Hidetoshi Seto [this message]
2010-10-06 1:58 ` Hidetoshi Seto
2010-10-06 16:05 ` Marcelo Tosatti
2010-10-06 16:05 ` [Qemu-devel] " Marcelo Tosatti
2010-10-06 18:10 ` Dean Nelson
2010-10-06 18:10 ` [Qemu-devel] " Dean Nelson
2010-10-07 3:41 ` Hidetoshi Seto
2010-10-07 3:41 ` [Qemu-devel] " Hidetoshi Seto
2010-10-07 15:23 ` Dean Nelson
2010-10-07 15:23 ` [Qemu-devel] " Dean Nelson
2010-10-08 3:15 ` Huang Ying
2010-10-08 3:15 ` [Qemu-devel] " Huang Ying
2010-10-08 5:54 ` Hidetoshi Seto
2010-10-08 5:54 ` [Qemu-devel] " Hidetoshi Seto
2010-10-08 12:02 ` Dean Nelson
2010-10-08 12:02 ` [Qemu-devel] " Dean Nelson
2010-10-08 2:50 ` Huang Ying
2010-10-08 2:50 ` [Qemu-devel] " Huang Ying
2010-10-04 18:54 ` [patch uq/master 8/8] Add savevm/loadvm support for MCE Marcelo Tosatti
2010-10-04 18:54 ` [Qemu-devel] " Marcelo Tosatti
2010-10-05 16:31 ` [Qemu-devel] [patch uq/master 0/8] port qemu-kvm's MCE support Andreas Färber
2010-10-05 18:58 ` Chris Wright
2010-10-05 20:24 ` Marcelo Tosatti
2010-10-06 17:34 ` [patch uq/master 0/8] port qemu-kvm's MCE support (v2) Marcelo Tosatti
2010-10-06 17:34 ` [Qemu-devel] " Marcelo Tosatti
2010-10-06 17:34 ` [patch uq/master 1/8] signalfd compatibility Marcelo Tosatti
2010-10-06 17:34 ` [Qemu-devel] " Marcelo Tosatti
2010-10-06 17:34 ` [patch uq/master 2/8] iothread: use signalfd Marcelo Tosatti
2010-10-06 17:34 ` [Qemu-devel] " Marcelo Tosatti
2010-10-06 17:34 ` [patch uq/master 3/8] Expose thread_id in info cpus Marcelo Tosatti
2010-10-06 17:34 ` [Qemu-devel] " Marcelo Tosatti
2010-10-06 17:34 ` [patch uq/master 4/8] kvm: x86: add mce support Marcelo Tosatti
2010-10-06 17:34 ` [Qemu-devel] " Marcelo Tosatti
2010-10-06 19:32 ` Anthony Liguori
2010-10-06 19:32 ` [Qemu-devel] " Anthony Liguori
2010-10-06 17:34 ` [patch uq/master 5/8] Export qemu_ram_addr_from_host Marcelo Tosatti
2010-10-06 17:34 ` [Qemu-devel] " Marcelo Tosatti
2010-10-06 17:34 ` [patch uq/master 6/8] Add RAM -> physical addr mapping in MCE simulation Marcelo Tosatti
2010-10-06 17:34 ` [Qemu-devel] " Marcelo Tosatti
2010-10-06 17:34 ` [patch uq/master 7/8] MCE: Relay UCR MCE to guest Marcelo Tosatti
2010-10-06 17:34 ` [Qemu-devel] " Marcelo Tosatti
2010-10-06 17:34 ` [patch uq/master 8/8] Add savevm/loadvm support for MCE Marcelo Tosatti
2010-10-06 17:34 ` [Qemu-devel] " Marcelo Tosatti
2010-10-11 18:31 ` [patch 0/8] port qemu-kvm's MCE support (v3) Marcelo Tosatti
2010-10-11 18:31 ` [Qemu-devel] " Marcelo Tosatti
2010-10-11 18:31 ` [patch 1/8] signalfd compatibility Marcelo Tosatti
2010-10-11 18:31 ` [Qemu-devel] " Marcelo Tosatti
2010-10-11 18:31 ` [patch 2/8] iothread: use signalfd Marcelo Tosatti
2010-10-11 18:31 ` [Qemu-devel] " Marcelo Tosatti
2010-10-11 18:31 ` [patch 3/8] Expose thread_id in info cpus Marcelo Tosatti
2010-10-11 18:31 ` [Qemu-devel] " Marcelo Tosatti
2010-10-11 18:31 ` [patch 4/8] kvm: x86: add mce support Marcelo Tosatti
2010-10-11 18:31 ` [Qemu-devel] " Marcelo Tosatti
2010-10-11 18:31 ` [patch 5/8] Export qemu_ram_addr_from_host Marcelo Tosatti
2010-10-11 18:31 ` [Qemu-devel] " Marcelo Tosatti
2010-10-11 18:31 ` [patch 6/8] Add RAM -> physical addr mapping in MCE simulation Marcelo Tosatti
2010-10-11 18:31 ` [Qemu-devel] " Marcelo Tosatti
2010-10-11 18:31 ` [patch 7/8] MCE: Relay UCR MCE to guest Marcelo Tosatti
2010-10-11 18:31 ` [Qemu-devel] " Marcelo Tosatti
2010-10-11 18:31 ` [patch 8/8] Add savevm/loadvm support for MCE Marcelo Tosatti
2010-10-11 18:31 ` [Qemu-devel] " Marcelo Tosatti
2010-10-14 10:25 ` [patch 0/8] port qemu-kvm's MCE support (v3) Avi Kivity
2010-10-14 10:25 ` [Qemu-devel] " Avi Kivity
2010-10-14 16:21 ` Marcelo Tosatti
2010-10-14 16:21 ` [Qemu-devel] " Marcelo Tosatti
2010-10-17 9:32 ` [patch 0/8] port qemu-kvm's MCE support (v3 resend) Avi Kivity
2010-10-17 9:32 ` [Qemu-devel] " Avi Kivity
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=4CABD7CC.6030909@jp.fujitsu.com \
--to=seto.hidetoshi@jp.fujitsu.com \
--cc=dnelson@redhat.com \
--cc=kvm@vger.kernel.org \
--cc=mtosatti@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=ying.huang@intel.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.