[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 20/50] target-xtensa: make cpu-qom.h not target
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH 20/50] target-xtensa: make cpu-qom.h not target specific |
Date: |
Tue, 17 May 2016 18:42:09 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.0 |
On 17/05/2016 16:24, Max Filippov wrote:
>> > Make XtensaCPU an opaque type within cpu-qom.h, and move all definitions
>> > of private methods, as well as all type definitions that require knowledge
>> > of the layout to cpu.h. Conversely, move all definitions needed to
>> > define a class to cpu-qom.h. This helps making files independent of
>> > NEED_CPU_H if they only need to pass around CPU pointers.
> XtensaCPUClass doesn't need definition of XtensaConfig that pulled a lot
> of related structures, macros and enums from cpu.h. Mere
> typedef struct XtensaConfig XtensaConfig;
> above the XtensaCPUClass could be enough. How about the following?
Fine, I'll test your patch and replace mine with it.
Paolo
- [Qemu-devel] [PATCH 10/50] target-m68k: make cpu-qom.h not target specific, (continued)
- [Qemu-devel] [PATCH 10/50] target-m68k: make cpu-qom.h not target specific, Paolo Bonzini, 2016/05/16
- [Qemu-devel] [PATCH 16/50] target-sh4: make cpu-qom.h not target specific, Paolo Bonzini, 2016/05/16
- [Qemu-devel] [PATCH 03/50] log: do not use CONFIG_USER_ONLY, Paolo Bonzini, 2016/05/16
- [Qemu-devel] [PATCH 25/50] mips: use MIPSCPU instead of CPUMIPSState, Paolo Bonzini, 2016/05/16
- [Qemu-devel] [PATCH 13/50] target-ppc: do not use target_ulong in cpu-qom.h, Paolo Bonzini, 2016/05/16
- [Qemu-devel] [PATCH 12/50] target-mips: make cpu-qom.h not target specific, Paolo Bonzini, 2016/05/16
- [Qemu-devel] [PATCH 26/50] ppc: use PowerPCCPU instead of CPUPPCState, Paolo Bonzini, 2016/05/16
- [Qemu-devel] [PATCH 20/50] target-xtensa: make cpu-qom.h not target specific, Paolo Bonzini, 2016/05/16
- [Qemu-devel] [PATCH 18/50] target-tricore: make cpu-qom.h not target specific, Paolo Bonzini, 2016/05/16
- [Qemu-devel] [PATCH 11/50] target-microblaze: make cpu-qom.h not target specific, Paolo Bonzini, 2016/05/16
- [Qemu-devel] [PATCH 36/50] cpu: move endian-dependent load/store functions to cpu-all.h, Paolo Bonzini, 2016/05/16
- [Qemu-devel] [PATCH 23/50] sh4: include cpu-qom.h in files that require SuperHCPU, Paolo Bonzini, 2016/05/16
- [Qemu-devel] [PATCH 14/50] target-ppc: make cpu-qom.h not target specific, Paolo Bonzini, 2016/05/16
- [Qemu-devel] [PATCH 21/50] arm: include cpu-qom.h in files that require ARMCPU, Paolo Bonzini, 2016/05/16
- [Qemu-devel] [PATCH 30/50] explicitly include linux/kvm.h, Paolo Bonzini, 2016/05/16
- [Qemu-devel] [PATCH 27/50] arm: remove useless cpu.h inclusion, Paolo Bonzini, 2016/05/16