[PATCH 1/3] kvm, x86: introduce kvm_mce_in_progress

Hi, Marcelo-san I modified the patches sent before and made them work well on origin/uq/master tree. Resend the patches. Best Regards, Jin Dongming --- Share same error handing, and rename this function after MCIP (Machine Check

[PATCH 2/3] kvm, x86: kvm_mce_inj_* subroutins for templated error injections

Refactor codes for maintainability. Signed-off-by: Hidetoshi Seto seto.hideto...@jp.fujitsu.com Signed-off-by: Jin Dongming jin.dongm...@np.css.fujitsu.com --- target-i386/kvm.c | 111 ++--- 1 files changed, 71 insertions(+), 40 deletions(-) diff

[PATCH 3/3] kvm, x86: introduce kvm_inject_x86_mce_on

= addr, .misc = misc, }; kvm_inject_x86_mce_on(env, mce, abort_on_error); Signed-off-by: Hidetoshi Seto seto.hideto...@jp.fujitsu.com Signed-off-by: Jin Dongming jin.dongm...@np.css.fujitsu.com --- target-i386/kvm.c | 57 +--- 1 files

Re: [PATCH 3/3] kvm, x86: introduce kvm_inject_x86_mce_on

Hi, Marcelo-san (2010/12/20 20:29), Marcelo Tosatti wrote: On Fri, Dec 10, 2010 at 05:25:20PM +0900, Jin Dongming wrote: Pass a table instead of multiple args. Note: kvm_inject_x86_mce(env, bank, status, mcg_status, addr, misc, abort_on_error); is equal

[PATCH 1/3] Clean up cpu_inject_x86_mce().

Hi, all I am sorry for replying late. I modified the patches as the comments of huang-san and Marcelo-san and resend them. Thanks. Best Regards, Jin Dongming --- Clean up cpu_inject_x86_mce() for later patch. Signed-off-by: Jin Dongming

[PATCH 2/3] Add broadcast option for mce command.

bank status mcg_status addr misc (qemu) mce 1 10xbd00 0x050x1234 0x8c Signed-off-by: Jin Dongming jin.dongm...@np.css.fujitsu.com --- cpu-all.h |3 ++- hmp-commands.hx |6 +++--- monitor.c |7

[PATCH 3/3] Add function for checking mca broadcast of CPU.

Add function for checking whether current CPU support mca broadcast. Signed-off-by: Jin Dongming jin.dongm...@np.css.fujitsu.com --- target-i386/cpu.h|1 + target-i386/helper.c | 33 + target-i386/kvm.c|6 +- 3 files changed, 35 insertions

[PATCH 1/3] kvm, x86: introduce kvm_mce_in_progress

Share same error handing, and rename this function after MCIP (Machine Check In Progress) flag. Signed-off-by: Hidetoshi Seto seto.hideto...@jp.fujitsu.com Signed-off-by: Jin Dongming jin.dongm...@np.css.fujitsu.com --- target-i386/kvm.c | 15 +-- 1 files changed, 5 insertions

[PATCH 2/3] kvm, x86: kvm_mce_inj_* subroutins for templated error injections

Refactor codes for maintainability. Signed-off-by: Hidetoshi Seto seto.hideto...@jp.fujitsu.com Signed-off-by: Jin Dongming jin.dongm...@np.css.fujitsu.com --- target-i386/kvm.c | 111 ++--- 1 files changed, 71 insertions(+), 40 deletions(-) diff

[PATCH 3/3] kvm, x86: introduce kvm_inject_x86_mce_on

= addr, .misc = misc, }; kvm_inject_x86_mce_on(env, mce, abort_on_error); Signed-off-by: Hidetoshi Seto seto.hideto...@jp.fujitsu.com Signed-off-by: Jin Dongming jin.dongm...@np.css.fujitsu.com --- target-i386/kvm.c | 58 1 files

[PATCH 1/3] Clean up cpu_inject_x86_mce().

Clean up cpu_inject_x86_mce() for later patch. Signed-off-by: Jin Dongming jin.dongm...@np.css.fujitsu.com --- target-i386/helper.c | 26 -- 1 files changed, 16 insertions(+), 10 deletions(-) diff --git a/target-i386/helper.c b/target-i386/helper.c index f5c06a8

[PATCH 2/3] Add broadcast option for mce command.

. Injecting mce with this option the expected result could be gotten. Signed-off-by: Jin Dongming jin.dongm...@np.css.fujitsu.com --- cpu-all.h |3 ++- hmp-commands.hx |6 +++--- monitor.c | 18 -- target-i386/helper.c | 17

[PATCH 3/3] Check processor version for broadcast.

Broadcast MCA signal is not supported by the CPUs whose version is below 06H_EH. Signed-off-by: Jin Dongming jin.dongm...@np.css.fujitsu.com --- target-i386/helper.c | 13 + 1 files changed, 13 insertions(+), 0 deletions(-) diff --git a/target-i386/helper.c b/target-i386/helper.c

Re: [PATCH 3/3] Check processor version for broadcast.

Hi, Huang-san (2010/11/30 17:22), Huang Ying wrote: Hi, Dongming, On Tue, 2010-11-30 at 16:15 +0800, Jin Dongming wrote: Broadcast MCA signal is not supported by the CPUs whose version is below 06H_EH. Signed-off-by: Jin Dongming jin.dongm...@np.css.fujitsu.com --- target-i386/helper.c

Re: [PATCH] Add function for clearing the requested VCPUs' mce registers.

Hi, Huang-san (2010/11/25 14:44), Huang Ying wrote: On Thu, 2010-11-25 at 13:30 +0800, Jin Dongming wrote: Hi, Huang-san (2010/11/25 10:27), Huang Ying wrote: Hi, Dongming, On Thu, 2010-11-25 at 09:14 +0800, Jin Dongming wrote: In some case of mce test, the injected data can be remained

Re: [PATCH 2/2] reset mce registers of the given VCPU or all VCPUs with mce command.

Hi, Huang-san (2010/11/25 14:55), Huang Ying wrote: On Thu, 2010-11-25 at 09:20 +0800, Jin Dongming wrote: --- a/monitor.c +++ b/monitor.c @@ -60,6 +60,7 @@ #include trace.h #endif #include qemu-kvm.h +#include kvm_x86.h //#define DEBUG //#define DEBUG_COMPLETION @@ -2277,7

[PATCH] Add function for clearing the requested VCPUs' mce registers.

to be paid attention is that the status and mcg_status of mce must be set with 0. If not, mce registers will not be cleared. Signed-off-by: Jin Dongming jin.dongm...@np.css.fujitsu.com --- arch/x86/kvm/x86.c | 28 +++- 1 files changed, 27 insertions(+), 1 deletions(-) diff

[PATCH 1/2] Add broadcast option for mce command.

. Injecting mce with this option the expected result could be gotten. Signed-off-by: Jin Dongming jin.dongm...@np.css.fujitsu.com --- hmp-commands.hx |6 ++ monitor.c | 19 ++- qemu-kvm.h|3 +++ target-i386/kvm.c |2 +- 4 files changed, 28

[PATCH 2/2] reset mce registers of the given VCPU or all VCPUs with mce command.

- all VCPUs: (qemu) mce N 00 0 00broadcast/b (Comment: N is the number of VCPU; - means no option.) Signed-off-by: Jin Dongming jin.dongm...@np.css.fujitsu.com --- hmp-commands.hx |4 +++- monitor.c | 15 ++- 2 files changed, 17

Re: [PATCH 1/2] Add broadcast option for mce command.

Hi, Huang-san (2010/11/25 10:33), Huang Ying wrote: On Thu, 2010-11-25 at 09:19 +0800, Jin Dongming wrote: [...] --- a/hmp-commands.hx +++ b/hmp-commands.hx @@ -1053,9 +1053,15 @@ ETEXI { .name = mce, +#if defined(KVM_CAP_MCE) +.args_type = cpu_index:i

Re: [PATCH] Add function for clearing the requested VCPUs' mce registers.

Hi, Huang-san (2010/11/25 10:27), Huang Ying wrote: Hi, Dongming, On Thu, 2010-11-25 at 09:14 +0800, Jin Dongming wrote: In some case of mce test, the injected data can be remained in the registers and ca affect to the result of following test cases. So add codes for clearing mce registers

[PATCH 00/11] Descriptions for patches of qemu mce.

These patches do the following changes. 1. Clean up: - Making the similar parts as one shared function. - modularizing the functions of SRAO and SRAR data setting. 2. Unify sigbus handling: - kvm_handle_sigbus can handle both cases of SIGBUS listed as

[PATCH 01/11] kvm, x86: ignore SRAO only when MCG_SER_P is available

And restruct this block to call kvm_mce_in_exception() only when it is required. Signed-off-by: Hidetoshi Seto seto.hideto...@jp.fujitsu.com Tested-by: Jin Dongming jin.dongm...@np.css.fujitsu.com --- qemu-kvm.c | 15 +-- 1 files changed, 9 insertions(+), 6 deletions(-) diff --git

[PATCH 02/11] kvm, x86: introduce kvm_do_set_mce

Share the same error handling. Signed-off-by: Hidetoshi Seto seto.hideto...@jp.fujitsu.com Tested-by: Jin Dongming jin.dongm...@np.css.fujitsu.com --- qemu-kvm.c | 31 +++ 1 files changed, 19 insertions(+), 12 deletions(-) diff --git a/qemu-kvm.c b/qemu-kvm.c index

[PATCH 03/11] kvm, x86: introduce kvm_mce_in_progress

Share same error handing, and put it in #ifdef MCE i386. Rename this function after MCIP (Machine Check In Progress) flag. Signed-off-by: Hidetoshi Seto seto.hideto...@jp.fujitsu.com Tested-by: Jin Dongming jin.dongm...@np.css.fujitsu.com --- qemu-kvm.c | 47

[PATCH 04/11] kvm, x86: kvm_mce_inj_* subroutins for templated error injections

Refactor codes for maintainability. Signed-off-by: Hidetoshi Seto seto.hideto...@jp.fujitsu.com Tested-by: Jin Dongming jin.dongm...@np.css.fujitsu.com --- qemu-kvm.c | 96 --- 1 files changed, 58 insertions(+), 38 deletions(-) diff

[PATCH 05/11] kvm, x86: introduce kvm_inject_x86_mce_on

, }; kvm_inject_x86_mce_on(env, mce, abort_on_error); Signed-off-by: Hidetoshi Seto seto.hideto...@jp.fujitsu.com Tested-by: Jin Dongming jin.dongm...@np.css.fujitsu.com --- qemu-kvm.c | 56 ++-- 1 files changed, 38 insertions(+), 18 deletions

[PATCH 06/11] kvm, x86: use target_phys_addr_t

Signed-off-by: Hidetoshi Seto seto.hideto...@jp.fujitsu.com Tested-by: Jin Dongming jin.dongm...@np.css.fujitsu.com --- qemu-kvm.c | 14 +++--- 1 files changed, 7 insertions(+), 7 deletions(-) diff --git a/qemu-kvm.c b/qemu-kvm.c index 0ba42fc..89ae524 100644 --- a/qemu-kvm.c +++ b

[PATCH 07/11] kvm, x86: unify sigbus handling, prep

receives SIGBUS. These 2 take different siginfo, but in both case required parameters are common, the code and the addr in the info. Restruct functions to take the code and the addr explicitly. Signed-off-by: Hidetoshi Seto seto.hideto...@jp.fujitsu.com Tested-by: Jin Dongming jin.dongm

[PATCH 08/11] kvm, x86: unify sigbus handling

Now kvm_handle_sigbus can handle both cases of SIGBUS. Note that env is NULL when main thread receives SIGBUS via signalfd, otherwise env points vcpu thread that receives SIGBUS. Signed-off-by: Hidetoshi Seto seto.hideto...@jp.fujitsu.com Tested-by: Jin Dongming jin.dongm...@np.css.fujitsu.com

[PATCH 09/11] kvm, x86: unify sigbus handling, post1

Explicitly duplicate blocks for next cleanup. Signed-off-by: Hidetoshi Seto seto.hideto...@jp.fujitsu.com Tested-by: Jin Dongming jin.dongm...@np.css.fujitsu.com --- qemu-kvm.c | 56 +--- 1 files changed, 33 insertions(+), 23 deletions

[PATCH 10/11] kvm, x86: unify sigbus handling, post2

Cleanup to finish unification. Signed-off-by: Hidetoshi Seto seto.hideto...@jp.fujitsu.com Tested-by: Jin Dongming jin.dongm...@np.css.fujitsu.com --- qemu-kvm.c | 41 - 1 files changed, 12 insertions(+), 29 deletions(-) diff --git a/qemu-kvm.c b/qemu

[PATCH 11/11] kvm, x86: broadcast mce depending on the cpu version

seto.hideto...@jp.fujitsu.com Tested-by: Jin Dongming jin.dongm...@np.css.fujitsu.com --- qemu-kvm.c | 63 +-- 1 files changed, 31 insertions(+), 32 deletions(-) diff --git a/qemu-kvm.c b/qemu-kvm.c index d2b2459..846f0b6 100644 --- a/qemu

Re: [PATCH] KVM: IOAPIC: only access APIC registers one dword at a time

it. Best Regards, Jin Dongming -- To unsubscribe from this list: send the line unsubscribe kvm in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html