[Qemu-devel] Re: [PATCH uq/master -v2 2/2] KVM, MCE, unpoison memory address across reboot

[Huang Ying]

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