Re: [Qemu-devel] [PATCH 3/9 v3] Move includes and struct definition to dump.h

2013-05-27 Thread Qiao Nuohan
On 05/22/2013 10:12 PM, Andreas Färber wrote: I fear that this patch is conflicting. Are really all of those headers actually needed for the struct you're moving? In particular I'm worried about cpu.h as well as cpu-all.h and kvm.h with indirect dependencies on cpu.h. Well, I cannot avoid

Re: [Qemu-devel] [PATCH 3/9 v3] Move includes and struct definition to dump.h

2013-05-22 Thread Andreas Färber
Am 17.05.2013 05:24, schrieb Qiao Nuohan: Move includes and definition of struct DumpState into include/sysemu/dump.h. Signed-off-by: Qiao Nuohan qiaonuo...@cn.fujitsu.com Reviewed-by: Zhang Xiaohe zhan...@cn.fujitsu.com --- dump.c| 29 -

[Qemu-devel] [PATCH 3/9 v3] Move includes and struct definition to dump.h

2013-05-16 Thread Qiao Nuohan
Move includes and definition of struct DumpState into include/sysemu/dump.h. Signed-off-by: Qiao Nuohan qiaonuo...@cn.fujitsu.com Reviewed-by: Zhang Xiaohe zhan...@cn.fujitsu.com --- dump.c| 29 - include/sysemu/dump.h | 30