[v2,0/9] exec: Remove 'qemu/log.h' from 'exec-all.h'

Message

Philippe Mathieu-Daudé Feb. 7, 2022, 8:27 a.m. UTC

Yet another noisy/boring refactor, but it will allow us to better
split exec/ includes as being target-agnostic/specific (the next
series).

Since v1:
- Remove unrelated "hw/s390x/virtio: missing 'cpu.h'" (thuth)
- Reword some commit descriptions (thuth)

Based-on: <[e-mail]>
"softmmu: Make various objects target agnostic"

Philippe Mathieu-Daudé (9):
  hw/tpm: Clean includes
  hw/remote: Add missing include
  hw/acpi/memory_hotplug: Remove unused 'hw/acpi/pc-hotplug.h' header
  qtest: Add missing 'hw/qdev-core.h' include
  exec/ramblock: Add missing includes
  core/ptimers: Remove unnecessary 'sysemu/cpus.h' include
  target: Add missing "qemu/timer.h" include
  linux-user: Add missing "qemu/timer.h" include
  exec/exec-all: Move 'qemu/log.h' include in units requiring it

 bsd-user/signal.c                       | 1 +
 hw/acpi/memory_hotplug.c                | 1 -
 hw/arm/omap1.c                          | 1 +
 hw/core/ptimer.c                        | 1 -
 hw/display/omap_dss.c                   | 1 +
 hw/gpio/omap_gpio.c                     | 1 +
 hw/input/tsc210x.c                      | 1 +
 hw/remote/proxy-memory-listener.c       | 1 +
 hw/tpm/tpm_crb.c                        | 1 +
 hw/tpm/tpm_ppi.c                        | 4 ++--
 hw/tpm/tpm_ppi.h                        | 4 ++--
 include/exec/exec-all.h                 | 2 --
 include/exec/ramblock.h                 | 2 ++
 linux-user/i386/cpu_loop.c              | 1 +
 linux-user/ppc/cpu_loop.c               | 1 +
 monitor/misc.c                          | 1 +
 plugins/api.c                           | 1 +
 softmmu/qtest.c                         | 1 +
 softmmu/runstate.c                      | 1 +
 target/alpha/helper.c                   | 2 +-
 target/arm/cpu.c                        | 1 +
 target/arm/helper.c                     | 2 ++
 target/arm/m_helper.c                   | 1 +
 target/arm/mte_helper.c                 | 1 +
 target/avr/helper.c                     | 1 +
 target/cris/helper.c                    | 1 +
 target/hexagon/internal.h               | 2 ++
 target/hexagon/translate.h              | 1 +
 target/hppa/cpu.c                       | 1 +
 target/hppa/helper.c                    | 2 +-
 target/hppa/mem_helper.c                | 1 +
 target/hppa/op_helper.c                 | 1 +
 target/i386/helper.c                    | 1 +
 target/i386/tcg/int_helper.c            | 1 +
 target/i386/tcg/misc_helper.c           | 1 +
 target/i386/tcg/sysemu/seg_helper.c     | 1 +
 target/i386/tcg/sysemu/svm_helper.c     | 1 +
 target/m68k/op_helper.c                 | 1 +
 target/microblaze/cpu.c                 | 1 +
 target/microblaze/mmu.c                 | 1 +
 target/microblaze/op_helper.c           | 1 +
 target/mips/tcg/exception.c             | 1 +
 target/mips/tcg/sysemu/special_helper.c | 1 +
 target/mips/tcg/translate.h             | 1 +
 target/openrisc/interrupt.c             | 1 +
 target/openrisc/mmu.c                   | 1 +
 target/ppc/excp_helper.c                | 1 +
 target/ppc/misc_helper.c                | 1 +
 target/riscv/csr.c                      | 1 +
 target/s390x/tcg/excp_helper.c          | 1 +
 target/s390x/tcg/mem_helper.c           | 1 +
 target/sparc/helper.c                   | 1 +
 target/sparc/ldst_helper.c              | 1 +
 target/sparc/mmu_helper.c               | 1 +
 target/tricore/helper.c                 | 2 +-
 target/xtensa/dbg_helper.c              | 1 +
 target/xtensa/exc_helper.c              | 1 +
 target/xtensa/fpu_helper.c              | 1 +
 target/xtensa/helper.c                  | 1 +
 target/xtensa/mmu_helper.c              | 1 +
 target/xtensa/win_helper.c              | 1 +
 61 files changed, 63 insertions(+), 11 deletions(-)

Comments

Thomas Huth Feb. 21, 2022, 12:22 p.m. UTC | #1

On 07/02/2022 09.27, Philippe Mathieu-Daudé via wrote:
> Yet another noisy/boring refactor, but it will allow us to better
> split exec/ includes as being target-agnostic/specific (the next
> series).
> 
> Since v1:
> - Remove unrelated "hw/s390x/virtio: missing 'cpu.h'" (thuth)
> - Reword some commit descriptions (thuth)
> 
> Based-on: <[e-mail]>
> "softmmu: Make various objects target agnostic"
> 
> Philippe Mathieu-Daudé (9):
>    hw/tpm: Clean includes
>    hw/remote: Add missing include
>    hw/acpi/memory_hotplug: Remove unused 'hw/acpi/pc-hotplug.h' header
>    qtest: Add missing 'hw/qdev-core.h' include
>    exec/ramblock: Add missing includes
>    core/ptimers: Remove unnecessary 'sysemu/cpus.h' include
>    target: Add missing "qemu/timer.h" include
>    linux-user: Add missing "qemu/timer.h" include
>    exec/exec-all: Move 'qemu/log.h' include in units requiring it

FYI, since nobody else picked this up yet, I've added this series to my 
qtests+misc pull request today.

  Thomas