On 2011-02-10 01:27, Huang Ying wrote: >>> @@ -1882,6 +1919,7 @@ int kvm_arch_on_sigbus_vcpu(CPUState *en >>> hardware_memory_error(); >>> } >>> } >>> + kvm_hwpoison_page_add(ram_addr); >>> >>> if (code == BUS_MCEERR_AR) { >>> /* Fake an Intel architectural Data Load SRAR UCR */ >>> @@ -1926,6 +1964,7 @@ int kvm_arch_on_sigbus(int code, void *a >>> "QEMU itself instead of guest system!: %p\n", addr); >>> return 0; >>> } >>> + kvm_hwpoison_page_add(ram_addr); >>> kvm_mce_inj_srao_memscrub2(first_cpu, paddr); >>> } else >>> #endif >>> >>> >> >> Looks fine otherwise. Unless that simplification makes sense, I could >> offer to include this into my MCE rework (there is some minor conflict). >> If all goes well, that series should be posted during this week. Please have a look at git://git.kiszka.org/qemu-kvm.git queues/kvm-upstream and tell me if it works for you and your signed-off still applies. Thanks, Jan
Attachment:
signature.asc
Description: OpenPGP digital signature