Re: [Qemu-devel] [PATCH v12 09/12] Move related hwpoison page function to accel/kvm/ folder

[gengdongjiu]

On 2017/11/13 16:27, Paolo Bonzini wrote:
> On 13/11/2017 02:45, gengdongjiu wrote:
>> On 2017/11/10 19:32, Paolo Bonzini wrote:
>>> On 10/11/2017 20:19, Dongjiu Geng wrote:
>>>> +typedef struct HWPoisonPage {
>>>> +    ram_addr_t ram_addr;
>>>> +    QLIST_ENTRY(HWPoisonPage) list;
>>>> +} HWPoisonPage;
>>>> +
>>>
>>> Is this actually needed outside accel/kvm/kvm-all.c?
>> Paolo,
>>    Thanks for the comments, this structure is added in the 
>> accel/kvm/kvm-all.c is also OK.
>> My previous thought is that this is structure definition, so I move it to a 
>> header file.
>> If you think this structure should be added in accel/kvm/kvm-all.c, I will 
>> move it.
> 
> It can be done later; but if you have to send a v13 series, I would be
> grateful if you included this change as well.

Hi Paolo,
  I have included this changed in the v13 series, could I can get your 
"Reviewed-by" after you
reviewed it. thanks so much.

> 
> Paolo
> 
>> thanks!
>>
>>>
>>> Thanks,
>>>
>>> Paolo
>>>
>>> .
>>>
>>
> 
> 
> .
>