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
