[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v2 15/15] KVM, MCE, unpoison memory address across r
From: |
Jan Kiszka |
Subject: |
[Qemu-devel] [PATCH v2 15/15] KVM, MCE, unpoison memory address across reboot |
Date: |
Fri, 18 Feb 2011 11:11:26 +0100 |
From: Huang Ying <address@hidden>
In Linux kernel HWPoison processing implementation, the virtual
address in processes mapping the error physical memory page is marked
as HWPoison. So that, the further accessing to the virtual
address will kill corresponding processes with SIGBUS.
If the error physical memory page is used by a KVM guest, the SIGBUS
will be sent to QEMU, and QEMU will simulate a MCE to report that
memory error to the guest OS. If the guest OS can not recover from
the error (for example, the page is accessed by kernel code), guest OS
will reboot the system. But because the underlying host virtual
address backing the guest physical memory is still poisoned, if the
guest system accesses the corresponding guest physical memory even
after rebooting, the SIGBUS will still be sent to QEMU and MCE will be
simulated. That is, guest system can not recover via rebooting.
In fact, across rebooting, the contents of guest physical memory page
need not to be kept. We can allocate a new host physical page to
back the corresponding guest physical address.
This patch fixes this issue in QEMU-KVM via calling qemu_ram_remap()
to clear the corresponding page table entry, so that make it possible
to allocate a new page to recover the issue.
[ Jan: rebasing and tiny cleanups]
Signed-off-by: Huang Ying <address@hidden>
Signed-off-by: Jan Kiszka <address@hidden>
---
target-i386/kvm.c | 36 ++++++++++++++++++++++++++++++++++++
1 files changed, 36 insertions(+), 0 deletions(-)
diff --git a/target-i386/kvm.c b/target-i386/kvm.c
index 44e5504..7b7105d 100644
--- a/target-i386/kvm.c
+++ b/target-i386/kvm.c
@@ -173,7 +173,40 @@ static int get_para_features(CPUState *env)
}
#endif /* CONFIG_KVM_PARA */
+typedef struct HWPoisonPage {
+ ram_addr_t ram_addr;
+ QLIST_ENTRY(HWPoisonPage) list;
+} HWPoisonPage;
+
+static QLIST_HEAD(, HWPoisonPage) hwpoison_page_list =
+ QLIST_HEAD_INITIALIZER(hwpoison_page_list);
+
+static void kvm_unpoison_all(void *param)
+{
+ HWPoisonPage *page, *next_page;
+
+ QLIST_FOREACH_SAFE(page, &hwpoison_page_list, list, next_page) {
+ QLIST_REMOVE(page, list);
+ qemu_ram_remap(page->ram_addr, TARGET_PAGE_SIZE);
+ qemu_free(page);
+ }
+}
+
#ifdef KVM_CAP_MCE
+static void kvm_hwpoison_page_add(ram_addr_t ram_addr)
+{
+ HWPoisonPage *page;
+
+ QLIST_FOREACH(page, &hwpoison_page_list, list) {
+ if (page->ram_addr == ram_addr) {
+ return;
+ }
+ }
+ page = qemu_malloc(sizeof(HWPoisonPage));
+ page->ram_addr = ram_addr;
+ QLIST_INSERT_HEAD(&hwpoison_page_list, page, list);
+}
+
static int kvm_get_mce_cap_supported(KVMState *s, uint64_t *mce_cap,
int *max_banks)
{
@@ -233,6 +266,7 @@ int kvm_arch_on_sigbus_vcpu(CPUState *env, int code, void
*addr)
hardware_memory_error();
}
}
+ kvm_hwpoison_page_add(ram_addr);
kvm_mce_inject(env, paddr, code);
} else
#endif /* KVM_CAP_MCE */
@@ -263,6 +297,7 @@ int kvm_arch_on_sigbus(int code, void *addr)
"QEMU itself instead of guest system!: %p\n", addr);
return 0;
}
+ kvm_hwpoison_page_add(ram_addr);
kvm_mce_inject(first_cpu, paddr, code);
} else
#endif /* KVM_CAP_MCE */
@@ -571,6 +606,7 @@ int kvm_arch_init(KVMState *s)
fprintf(stderr, "e820_add_entry() table is full\n");
return ret;
}
+ qemu_register_reset(kvm_unpoison_all, NULL);
return 0;
}
--
1.7.1
- [Qemu-devel] [PATCH v2 00/15] [uq/master] Patch queue, part IV (MCE edition), Jan Kiszka, 2011/02/18
- [Qemu-devel] [PATCH v2 10/15] x86: Run qemu_inject_x86_mce on target VCPU, Jan Kiszka, 2011/02/18
- [Qemu-devel] [PATCH v2 06/15] Synchronize VCPU states before reset, Jan Kiszka, 2011/02/18
- [Qemu-devel] [PATCH v2 05/15] x86: Optionally avoid injecting AO MCEs while others are pending, Jan Kiszka, 2011/02/18
- [Qemu-devel] [PATCH v2 13/15] kvm: x86: Fail kvm_arch_init_vcpu if MCE initialization fails, Jan Kiszka, 2011/02/18
- [Qemu-devel] [PATCH v2 12/15] kvm: x86: Clean up kvm_setup_mce, Jan Kiszka, 2011/02/18
- [Qemu-devel] [PATCH v2 14/15] Add qemu_ram_remap, Jan Kiszka, 2011/02/18
- [Qemu-devel] [PATCH v2 08/15] kvm: Rename kvm_arch_process_irqchip_events to async_events, Jan Kiszka, 2011/02/18
- [Qemu-devel] [PATCH v2 11/15] kvm: x86: Consolidate TCG and KVM MCE injection code, Jan Kiszka, 2011/02/18
- [Qemu-devel] [PATCH v2 01/15] x86: Account for MCE in cpu_has_work, Jan Kiszka, 2011/02/18
- [Qemu-devel] [PATCH v2 15/15] KVM, MCE, unpoison memory address across reboot,
Jan Kiszka <=
- [Qemu-devel] [PATCH v2 04/15] x86: Refine error reporting of MCE injection services, Jan Kiszka, 2011/02/18
- [Qemu-devel] [PATCH v2 02/15] x86: Perform implicit mcg_status reset, Jan Kiszka, 2011/02/18
- [Qemu-devel] [PATCH v2 07/15] kvm: x86: Move MCE functions together, Jan Kiszka, 2011/02/18
- [Qemu-devel] [PATCH v2 03/15] x86: Small cleanups of MCE helpers, Jan Kiszka, 2011/02/18
- [Qemu-devel] [PATCH v2 09/15] kvm: x86: Inject pending MCE events on state writeback, Jan Kiszka, 2011/02/18
- [Qemu-devel] Re: [PATCH v2 00/15] [uq/master] Patch queue, part IV (MCE edition), Marcelo Tosatti, 2011/02/22