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. Paolo > thanks! > >> >> Thanks, >> >> Paolo >> >> . >> >