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 conf

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 > Reviewed-by: Zhang Xiaohe > --- > dump.c| 29 - > include/sysemu/dump.h | 30

[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 Reviewed-by: Zhang Xiaohe --- dump.c| 29 - include/sysemu/dump.h | 30 ++ 2 files changed, 30 insertions(+), 29 de