misc: clean up includes

[PATCH 0/4] misc: clean up includes

Peter Maydell posted 4 patches 3 weeks ago

Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20260116125830.926296-1-peter.maydell@linaro.org

Maintainers: Magnus Kulke <magnus.kulke@linux.microsoft.com>, Wei Liu <wei.liu@kernel.org>, Warner Losh <imp@bsdimp.com>, Kyle Evans <kevans@freebsd.org>, Eduardo Habkost <eduardo@habkost.net>, Marcel Apfelbaum <marcel.apfelbaum@gmail.com>, "Philippe Mathieu-Daudé" <philmd@linaro.org>, Yanan Wang <wangyanan55@huawei.com>, Zhao Liu <zhao1.liu@intel.com>, Paolo Bonzini <pbonzini@redhat.com>, Fam Zheng <fam@euphon.net>, Richard Henderson <richard.henderson@linaro.org>, Helge Deller <deller@gmx.de>, "Michael S. Tsirkin" <mst@redhat.com>, Nicholas Piggin <npiggin@gmail.com>, Harsh Prateek Bora <harshpb@linux.ibm.com>, Aditya Gupta <adityag@linux.ibm.com>, Sourabh Jain <sourabhjain@linux.ibm.com>, Palmer Dabbelt <palmer@dabbelt.com>, Alistair Francis <alistair.francis@wdc.com>, Weiwei Li <liwei1518@gmail.com>, Daniel Henrique Barboza <dbarboza@ventanamicro.com>, Liu Zhiwei <zhiwei_liu@linux.alibaba.com>, Laurent Vivier <laurent@vivier.eu>, Michael Roth <michael.roth@amd.com>, Kostiantyn Kostiuk <kkostiuk@redhat.com>, "Alex Bennée" <alex.bennee@linaro.org>, Peter Maydell <peter.maydell@linaro.org>, Marcelo Tosatti <mtosatti@redhat.com>, Song Gao <gaosong@loongson.cn>

accel/mshv/irq.c                   | 3 +--
accel/mshv/mshv-all.c              | 1 -
bsd-user/bsd-mem.h                 | 3 ---
bsd-user/bsd-proc.c                | 1 -
bsd-user/freebsd/os-proc.h         | 2 --
bsd-user/qemu-bsd.h                | 1 -
hw/core/machine-qmp-cmds.c         | 1 -
hw/scsi/lasi_ncr710.h              | 1 -
hw/scsi/ncr53c710.h                | 1 -
include/hw/core/loader.h           | 1 -
include/hw/i386/tdvf.h             | 1 -
include/hw/ppc/spapr_fadump.h      | 1 -
include/hw/riscv/iommu.h           | 1 -
include/system/accel-irq.h         | 1 -
include/system/mshv.h              | 1 -
linux-user/alpha/target_proc.h     | 1 -
qga/commands-windows-ssh.h         | 1 -
qga/vss-win32/vss-debug.h          | 1 -
semihosting/arm-compat-semi-stub.c | 1 -
target/arm/cpu32-stubs.c           | 1 -
target/i386/kvm/kvm.c              | 1 -
target/i386/kvm/vmsr_energy.h      | 2 --
target/i386/mshv/mshv-cpu.c        | 1 -
target/i386/mshv/x86.c             | 1 -
target/loongarch/csr.c             | 1 -
target/riscv/csr.c                 | 1 -
tools/i386/qemu-vmsr-helper.c      | 1 -
util/cpuinfo-aarch64.c             | 1 -
28 files changed, 1 insertion(+), 33 deletions(-)

Expand all Fold all

[PATCH 0/4] misc: clean up includes

Posted by Peter Maydell 3 weeks ago

This patchset is the result of running scripts/clean-includes
as updated with my changes in the v2 patchset I just posted:
https://patchew.org/QEMU/20260116124005.925382-1-peter.maydell@linaro.org/

It makes the usual set of cleanups to #include lines:
    * Ensure .c files include qemu/osdep.h first.
    * Including it in a .h is redundant, since the .c  already includes
      it.  Drop such inclusions.
    * Likewise, including headers qemu/osdep.h includes is redundant.
      Drop these, too.

I have split the changes it proposes into five categories:
 (1) changes to mshv related files
 (2) changes to bsd-user files
 (3) changes where a .c file included a system header that
     osdep.h already provides
 (4) changes where .h files either included osdep.h or
     else some system header that osdep.h provides

The only change the updated script proposes which is not in
any patch in this series is to hw/virtio/cbor-helpers.c: I am
waiting to find out if we should put this file on the
exclude-list.

thanks
-- PMM

Peter Maydell (4):
  mshv: Clean up includes
  bsd-user: Clean up includes
  misc: Clean up includes
  all: Clean up includes

 accel/mshv/irq.c                   | 3 +--
 accel/mshv/mshv-all.c              | 1 -
 bsd-user/bsd-mem.h                 | 3 ---
 bsd-user/bsd-proc.c                | 1 -
 bsd-user/freebsd/os-proc.h         | 2 --
 bsd-user/qemu-bsd.h                | 1 -
 hw/core/machine-qmp-cmds.c         | 1 -
 hw/scsi/lasi_ncr710.h              | 1 -
 hw/scsi/ncr53c710.h                | 1 -
 include/hw/core/loader.h           | 1 -
 include/hw/i386/tdvf.h             | 1 -
 include/hw/ppc/spapr_fadump.h      | 1 -
 include/hw/riscv/iommu.h           | 1 -
 include/system/accel-irq.h         | 1 -
 include/system/mshv.h              | 1 -
 linux-user/alpha/target_proc.h     | 1 -
 qga/commands-windows-ssh.h         | 1 -
 qga/vss-win32/vss-debug.h          | 1 -
 semihosting/arm-compat-semi-stub.c | 1 -
 target/arm/cpu32-stubs.c           | 1 -
 target/i386/kvm/kvm.c              | 1 -
 target/i386/kvm/vmsr_energy.h      | 2 --
 target/i386/mshv/mshv-cpu.c        | 1 -
 target/i386/mshv/x86.c             | 1 -
 target/loongarch/csr.c             | 1 -
 target/riscv/csr.c                 | 1 -
 tools/i386/qemu-vmsr-helper.c      | 1 -
 util/cpuinfo-aarch64.c             | 1 -
 28 files changed, 1 insertion(+), 33 deletions(-)

-- 
2.47.3