[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 0/7] Poison some more target-specific defines
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH v2 0/7] Poison some more target-specific defines |
Date: |
Fri, 16 Jun 2017 18:19:11 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.1.0 |
On 16/06/2017 18:15, Thomas Huth wrote:
> This is quite similar to trying to poison CONFIG_USER_ONLY ... every
> common file that depends on include/exec/memory.h or
> /include/exec/cpu-common.h then does not compile anymore - and as far as
> I can see, this can't be fixed with some trivial "#ifdef NEED_CPU_H"s in
> most cases anymore.
Oh, indeed. :( I'll put removing ram_addr_t from memory.h on my todo
list. I think most of those can go under NEED_CPU_H or can be replaced
with size_t.
Those parts of cpu-common.h instead should move to
include/exec/ram_addr.h sooner rather than later.
Paolo
> Since you've asked on IRC for a list of files which do not compile
> anymore in this case, here we go (some of them are also quite easy to
> fix, I think, but some need some bigger reworks, I guess):
- [Qemu-devel] [PATCH 4/7] include/exec/poison: Mark CONFIG_KVM as poisoned, too, (continued)
- [Qemu-devel] [PATCH 4/7] include/exec/poison: Mark CONFIG_KVM as poisoned, too, Thomas Huth, 2017/06/16
- [Qemu-devel] [PATCH 5/7] cpu: Introduce a wrapper for tlb_flush() that can be used in common code, Thomas Huth, 2017/06/16
- [Qemu-devel] [PATCH 7/7] Makefile: Move bootdevice.o to common-obj-y, Thomas Huth, 2017/06/16
- [Qemu-devel] [PATCH 6/7] include/exec/poison: Mark CONFIG_SOFTMMU as poisoned, Thomas Huth, 2017/06/16
- Re: [Qemu-devel] [PATCH v2 0/7] Poison some more target-specific defines, Paolo Bonzini, 2017/06/16