On Thu, 2011-02-10 at 16:52 +0800, Jan Kiszka wrote: > 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! Works as expected in my testing! Best Regards, Huang Ying -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html