From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from [140.186.70.92] (port=39625 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1P6JU5-0001Xz-Rp for qemu-devel@nongnu.org; Thu, 14 Oct 2010 04:48:23 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1P6JU4-0003dO-KD for qemu-devel@nongnu.org; Thu, 14 Oct 2010 04:48:21 -0400 Received: from fgwmail7.fujitsu.co.jp ([192.51.44.37]:41285) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1P6JU4-0003d6-4y for qemu-devel@nongnu.org; Thu, 14 Oct 2010 04:48:20 -0400 Received: from m6.gw.fujitsu.co.jp ([10.0.50.76]) by fgwmail7.fujitsu.co.jp (Fujitsu Gateway) with ESMTP id o9E8mI17032596 for (envelope-from jin.dongming@np.css.fujitsu.com); Thu, 14 Oct 2010 17:48:19 +0900 Received: from smail (m6 [127.0.0.1]) by outgoing.m6.gw.fujitsu.co.jp (Postfix) with ESMTP id 8D09C45DE53 for ; Thu, 14 Oct 2010 17:48:18 +0900 (JST) Received: from s6.gw.fujitsu.co.jp (s6.gw.fujitsu.co.jp [10.0.50.96]) by m6.gw.fujitsu.co.jp (Postfix) with ESMTP id 658F345DE4F for ; Thu, 14 Oct 2010 17:48:18 +0900 (JST) Received: from s6.gw.fujitsu.co.jp (localhost.localdomain [127.0.0.1]) by s6.gw.fujitsu.co.jp (Postfix) with ESMTP id 44C101DB8016 for ; Thu, 14 Oct 2010 17:48:18 +0900 (JST) Received: from m002.s.css.fujitsu.com (m002.s.css.fujitsu.com [10.23.4.32]) by s6.gw.fujitsu.co.jp (Postfix) with ESMTP id E50DC1DB8015 for ; Thu, 14 Oct 2010 17:48:17 +0900 (JST) Message-ID: <4CB6C427.10202@np.css.fujitsu.com> Date: Thu, 14 Oct 2010 17:49:43 +0900 From: Jin Dongming MIME-Version: 1.0 Content-Type: text/plain; charset=ISO-2022-JP Content-Transfer-Encoding: 7bit Subject: [Qemu-devel] [PATCH 07/11] kvm, x86: unify sigbus handling, prep List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: KVM list Cc: Hidetoshi Seto , Dean Nelson , Marcelo Tosatti , "qemu-devel@nongnu.org" , Avi Kivity , Huang Ying There are 2 similar functions to handle SIGBUS: sigbus_handler(int n, struct qemu_signalfd_siginfo *siginfo, void *ctx) kvm_on_sigbus(CPUState *env, siginfo_t *siginfo) The former is used when main thread receives SIGBUS via signalfd, while latter is used when vcpu thread 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 Tested-by: Jin Dongming --- qemu-kvm.c | 41 ++++++++++++++++++++--------------------- 1 files changed, 20 insertions(+), 21 deletions(-) diff --git a/qemu-kvm.c b/qemu-kvm.c index 89ae524..b58181a 100644 --- a/qemu-kvm.c +++ b/qemu-kvm.c @@ -1219,32 +1219,28 @@ static void kvm_mce_inj_srao_broadcast(target_phys_addr_t paddr) } #endif -static void sigbus_handler(int n, struct qemu_signalfd_siginfo *siginfo, - void *ctx) +static void kvm_handle_sigbus(int code, void *vaddr) { #if defined(KVM_CAP_MCE) && defined(TARGET_I386) - if ((first_cpu->mcg_cap & MCG_SER_P) && siginfo->ssi_addr - && siginfo->ssi_code == BUS_MCEERR_AO) { - void *vaddr; + if ((first_cpu->mcg_cap & MCG_SER_P) && vaddr && code == BUS_MCEERR_AO) { ram_addr_t ram_addr; target_phys_addr_t paddr; /* Hope we are lucky for AO MCE */ - vaddr = (void *)(intptr_t)siginfo->ssi_addr; if (do_qemu_ram_addr_from_host(vaddr, &ram_addr) || !kvm_physical_memory_addr_from_ram(kvm_state, ram_addr, &paddr)) { fprintf(stderr, "Hardware memory error for memory used by " "QEMU itself instead of guest system!: %llx\n", - (unsigned long long)siginfo->ssi_addr); + (unsigned long long)vaddr); return; } kvm_mce_inj_srao_broadcast(paddr); } else #endif { - if (siginfo->ssi_code == BUS_MCEERR_AO) { + if (code == BUS_MCEERR_AO) { return; - } else if (siginfo->ssi_code == BUS_MCEERR_AR) { + } else if (code == BUS_MCEERR_AR) { hardware_memory_error(); } else { sigbus_reraise(); @@ -1252,6 +1248,11 @@ static void sigbus_handler(int n, struct qemu_signalfd_siginfo *siginfo, } } +static void sigbus_handler(int n, struct qemu_signalfd_siginfo *ssi, void *ctx) +{ + kvm_handle_sigbus(ssi->ssi_code, (void *)(intptr_t)ssi->ssi_addr); +} + static void on_vcpu(CPUState *env, void (*func)(void *data), void *data) { struct qemu_work_item wi; @@ -1377,36 +1378,34 @@ static void flush_queued_work(CPUState *env) pthread_cond_broadcast(&qemu_work_cond); } -static void kvm_on_sigbus(CPUState *env, siginfo_t *siginfo) +static void kvm_on_sigbus(CPUState *env, int code, void *vaddr) { #if defined(KVM_CAP_MCE) && defined(TARGET_I386) - void *vaddr; ram_addr_t ram_addr; target_phys_addr_t paddr; - if ((env->mcg_cap & MCG_SER_P) && siginfo->si_addr - && (siginfo->si_code == BUS_MCEERR_AR - || siginfo->si_code == BUS_MCEERR_AO)) { + if ((env->mcg_cap & MCG_SER_P) && vaddr + && (code == BUS_MCEERR_AR || code == BUS_MCEERR_AO)) { /* * If there is an MCE excpetion being processed, ignore this SRAO MCE */ - if (siginfo->si_code == BUS_MCEERR_AO && kvm_mce_in_progress(env)) { + if (code == BUS_MCEERR_AO && kvm_mce_in_progress(env)) } return; } - vaddr = (void *)siginfo->si_addr; + if (do_qemu_ram_addr_from_host(vaddr, &ram_addr) || !kvm_physical_memory_addr_from_ram(kvm_state, ram_addr, &paddr)) { fprintf(stderr, "Hardware memory error for memory used by " "QEMU itself instead of guest system!\n"); /* Hope we are lucky for AO MCE */ - if (siginfo->si_code == BUS_MCEERR_AO) { + if (code == BUS_MCEERR_AO) { return; } else { hardware_memory_error(); } } - if (siginfo->si_code == BUS_MCEERR_AR) { + if (code == BUS_MCEERR_AR) { /* Fake an Intel architectural Data Load SRAR UCR */ kvm_mce_inj_srar_dataload(env, paddr); } else { @@ -1416,9 +1415,9 @@ static void kvm_on_sigbus(CPUState *env, siginfo_t *siginfo) } else #endif { - if (siginfo->si_code == BUS_MCEERR_AO) { + if (code == BUS_MCEERR_AO) { return; - } else if (siginfo->si_code == BUS_MCEERR_AR) { + } else if (code == BUS_MCEERR_AR) { hardware_memory_error(); } else { sigbus_reraise(); @@ -1455,7 +1454,7 @@ static void kvm_main_loop_wait(CPUState *env, int timeout) switch (r) { case SIGBUS: - kvm_on_sigbus(env, &siginfo); + kvm_on_sigbus(env, siginfo.si_code, (void *)siginfo.si_addr); break; default: break; -- 1.7.1.1