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
>>>
>>> .
>>>
>>
>
>
> .
>
