[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4 15/29] Include migration/vmstate.h less
|
From: |
Philippe Mathieu-Daudé |
|
Subject: |
Re: [Qemu-devel] [PATCH v4 15/29] Include migration/vmstate.h less |
|
Date: |
Mon, 12 Aug 2019 17:59:54 +0200 |
|
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.7.0 |
On 8/12/19 7:23 AM, Markus Armbruster wrote:
> In my "build everything" tree, changing migration/vmstate.h triggers a
> recompile of some 2700 out of 6600 objects (not counting tests and
> objects that don't depend on qemu/osdep.h).
>
> hw/hw.h supposedly includes it for convenience. Several other headers
> include it just to get VMStateDescription. The previous commit made
> that unnecessary.
>
> Include migration/vmstate.h only where it's still needed. Touching it
> now recompiles only some 1600 objects.
>
> Signed-off-by: Markus Armbruster <address@hidden>
> Reviewed-by: Alistair Francis <address@hidden>
Tested-by: Philippe Mathieu-Daudé <address@hidden>
> ---
> include/hw/acpi/tco.h | 1 -
> include/hw/block/flash.h | 1 -
> include/hw/hw.h | 1 -
> include/hw/input/hid.h | 1 -
> include/hw/pci/shpc.h | 1 +
> include/hw/ppc/spapr_ovec.h | 1 -
> include/hw/ptimer.h | 1 -
> include/hw/virtio/virtio.h | 1 +
> include/migration/cpu.h | 1 +
> include/net/net.h | 1 -
> include/qemu/fifo8.h | 1 -
> audio/audio.c | 1 +
> cpus.c | 1 +
> hw/acpi/cpu.c | 1 +
> hw/acpi/ich9.c | 1 +
> hw/acpi/memory_hotplug.c | 1 +
> hw/acpi/pcihp.c | 1 +
> hw/acpi/piix4.c | 1 +
> hw/acpi/tco.c | 2 ++
> hw/acpi/vmgenid.c | 1 +
> hw/adc/stm32f2xx_adc.c | 1 +
> hw/arm/armsse.c | 1 +
> hw/arm/highbank.c | 1 +
> hw/arm/integratorcp.c | 1 +
> hw/arm/musicpal.c | 1 +
> hw/arm/pxa2xx.c | 1 +
> hw/arm/pxa2xx_gpio.c | 1 +
> hw/arm/pxa2xx_pic.c | 1 +
> hw/arm/smmuv3.c | 1 +
> hw/arm/spitz.c | 1 +
> hw/arm/stellaris.c | 1 +
> hw/arm/strongarm.c | 1 +
> hw/arm/versatilepb.c | 1 +
> hw/arm/virt-acpi-build.c | 1 +
> hw/arm/z2.c | 1 +
> hw/audio/ac97.c | 1 +
> hw/audio/cs4231.c | 1 +
> hw/audio/cs4231a.c | 1 +
> hw/audio/es1370.c | 1 +
> hw/audio/gus.c | 1 +
> hw/audio/hda-codec.c | 1 +
> hw/audio/intel-hda.c | 1 +
> hw/audio/lm4549.c | 1 +
> hw/audio/marvell_88w8618.c | 1 +
> hw/audio/milkymist-ac97.c | 1 +
> hw/audio/pcspk.c | 1 +
> hw/audio/pl041.c | 1 +
> hw/audio/sb16.c | 1 +
> hw/audio/wm8750.c | 1 +
> hw/block/ecc.c | 1 +
> hw/block/fdc.c | 1 +
> hw/block/m25p80.c | 1 +
> hw/block/nand.c | 1 +
> hw/block/nvme.c | 1 +
> hw/block/onenand.c | 1 +
> hw/block/pflash_cfi01.c | 1 +
> hw/block/pflash_cfi02.c | 1 +
> hw/char/bcm2835_aux.c | 1 +
> hw/char/cadence_uart.c | 1 +
> hw/char/cmsdk-apb-uart.c | 1 +
> hw/char/digic-uart.c | 1 +
> hw/char/escc.c | 1 +
> hw/char/exynos4210_uart.c | 1 +
> hw/char/imx_serial.c | 1 +
> hw/char/ipoctal232.c | 1 +
> hw/char/lm32_juart.c | 1 +
> hw/char/lm32_uart.c | 1 +
> hw/char/milkymist-uart.c | 1 +
> hw/char/nrf51_uart.c | 1 +
> hw/char/parallel.c | 1 +
> hw/char/pl011.c | 1 +
> hw/char/sclpconsole-lm.c | 1 +
> hw/char/sclpconsole.c | 1 +
> hw/char/serial-isa.c | 1 +
> hw/char/serial-pci-multi.c | 1 +
> hw/char/serial-pci.c | 1 +
> hw/char/serial.c | 1 +
> hw/char/spapr_vty.c | 1 +
> hw/core/loader.c | 1 +
> hw/core/numa.c | 1 +
> hw/core/or-irq.c | 1 +
> hw/core/ptimer.c | 2 ++
> hw/core/qdev.c | 1 +
> hw/display/ads7846.c | 1 +
> hw/display/bcm2835_fb.c | 1 +
> hw/display/bochs-display.c | 1 +
> hw/display/cg3.c | 1 +
> hw/display/cirrus_vga.c | 1 +
> hw/display/dpcd.c | 1 +
> hw/display/exynos4210_fimd.c | 1 +
> hw/display/g364fb.c | 1 +
> hw/display/i2c-ddc.c | 1 +
> hw/display/jazz_led.c | 1 +
> hw/display/milkymist-tmu2.c | 1 +
> hw/display/milkymist-vgafb.c | 1 +
> hw/display/pl110.c | 1 +
> hw/display/pxa2xx_lcd.c | 1 +
> hw/display/qxl.c | 1 +
> hw/display/sii9022.c | 1 +
> hw/display/sm501.c | 1 +
> hw/display/ssd0303.c | 1 +
> hw/display/ssd0323.c | 1 +
> hw/display/tcx.c | 1 +
> hw/display/vga-isa-mm.c | 2 ++
> hw/display/vga-pci.c | 1 +
> hw/display/vga.c | 1 +
> hw/display/vmware_vga.c | 1 +
> hw/display/xlnx_dp.c | 1 +
> hw/dma/bcm2835_dma.c | 1 +
> hw/dma/i82374.c | 1 +
> hw/dma/i8257.c | 1 +
> hw/dma/pl080.c | 1 +
> hw/dma/pl330.c | 1 +
> hw/dma/pxa2xx_dma.c | 1 +
> hw/dma/rc4030.c | 1 +
> hw/dma/sparc32_dma.c | 1 +
> hw/dma/xlnx-zdma.c | 1 +
> hw/dma/xlnx-zynq-devcfg.c | 1 +
> hw/dma/xlnx_dpdma.c | 1 +
> hw/gpio/bcm2835_gpio.c | 1 +
> hw/gpio/gpio_key.c | 1 +
> hw/gpio/imx_gpio.c | 1 +
> hw/gpio/max7310.c | 1 +
> hw/gpio/mpc8xxx.c | 1 +
> hw/gpio/nrf51_gpio.c | 1 +
> hw/gpio/pl061.c | 1 +
> hw/gpio/zaurus.c | 1 +
> hw/hppa/dino.c | 1 +
> hw/i2c/aspeed_i2c.c | 1 +
> hw/i2c/core.c | 1 +
> hw/i2c/exynos4210_i2c.c | 1 +
> hw/i2c/imx_i2c.c | 1 +
> hw/i2c/microbit_i2c.c | 1 +
> hw/i2c/mpc_i2c.c | 1 +
> hw/i2c/pm_smbus.c | 2 ++
> hw/i2c/smbus_eeprom.c | 1 +
> hw/i2c/smbus_ich9.c | 1 +
> hw/i2c/smbus_slave.c | 1 +
> hw/i386/acpi-build.c | 1 +
> hw/i386/amd_iommu.c | 2 ++
> hw/i386/intel_iommu.c | 1 +
> hw/i386/kvm/clock.c | 1 +
> hw/i386/kvmvapic.c | 1 +
> hw/i386/pc.c | 1 +
> hw/i386/vmmouse.c | 2 ++
> hw/i386/xen/xen_platform.c | 1 +
> hw/i386/xen/xen_pvdevice.c | 1 +
> hw/ide/ahci-allwinner.c | 1 +
> hw/ide/ahci.c | 1 +
> hw/ide/cmd646.c | 1 +
> hw/ide/core.c | 1 +
> hw/ide/ich.c | 1 +
> hw/ide/isa.c | 1 +
> hw/ide/macio.c | 1 +
> hw/ide/microdrive.c | 1 +
> hw/ide/mmio.c | 1 +
> hw/ide/pci.c | 1 +
> hw/ide/piix.c | 1 +
> hw/ide/via.c | 1 +
> hw/input/adb-kbd.c | 1 +
> hw/input/adb-mouse.c | 1 +
> hw/input/adb.c | 1 +
> hw/input/hid.c | 2 ++
> hw/input/lm832x.c | 1 +
> hw/input/milkymist-softusb.c | 1 +
> hw/input/pckbd.c | 1 +
> hw/input/pl050.c | 1 +
> hw/input/ps2.c | 1 +
> hw/input/pxa2xx_keypad.c | 1 +
> hw/input/stellaris_input.c | 1 +
> hw/input/tsc2005.c | 1 +
> hw/input/tsc210x.c | 1 +
> hw/intc/allwinner-a10-pic.c | 1 +
> hw/intc/apic_common.c | 1 +
> hw/intc/arm_gic_common.c | 1 +
> hw/intc/arm_gicv3_common.c | 1 +
> hw/intc/arm_gicv3_its_common.c | 1 +
> hw/intc/armv7m_nvic.c | 1 +
> hw/intc/aspeed_vic.c | 1 +
> hw/intc/bcm2835_ic.c | 1 +
> hw/intc/bcm2836_control.c | 1 +
> hw/intc/exynos4210_combiner.c | 1 +
> hw/intc/exynos4210_gic.c | 1 +
> hw/intc/heathrow_pic.c | 1 +
> hw/intc/i8259_common.c | 2 ++
> hw/intc/imx_avic.c | 1 +
> hw/intc/imx_gpcv2.c | 1 +
> hw/intc/ioapic_common.c | 1 +
> hw/intc/lm32_pic.c | 1 +
> hw/intc/ompic.c | 1 +
> hw/intc/openpic.c | 1 +
> hw/intc/pl190.c | 1 +
> hw/intc/slavio_intctl.c | 1 +
> hw/intc/spapr_xive.c | 1 +
> hw/intc/xics.c | 1 +
> hw/intc/xive.c | 1 +
> hw/intc/xlnx-pmu-iomod-intc.c | 1 +
> hw/intc/xlnx-zynqmp-ipi.c | 1 +
> hw/ipack/ipack.c | 1 +
> hw/ipack/tpci200.c | 1 +
> hw/ipmi/ipmi_bmc_extern.c | 1 +
> hw/ipmi/ipmi_bmc_sim.c | 1 +
> hw/ipmi/isa_ipmi_bt.c | 1 +
> hw/ipmi/isa_ipmi_kcs.c | 1 +
> hw/isa/apm.c | 1 +
> hw/isa/i82378.c | 1 +
> hw/isa/lpc_ich9.c | 1 +
> hw/isa/pc87312.c | 1 +
> hw/isa/piix4.c | 1 +
> hw/isa/vt82c686.c | 1 +
> hw/mem/pc-dimm.c | 1 +
> hw/mips/gt64xxx_pci.c | 1 +
> hw/misc/a9scu.c | 1 +
> hw/misc/arm_l2x0.c | 1 +
> hw/misc/arm_sysctl.c | 1 +
> hw/misc/armsse-mhu.c | 1 +
> hw/misc/aspeed_scu.c | 1 +
> hw/misc/aspeed_sdmc.c | 1 +
> hw/misc/aspeed_xdma.c | 1 +
> hw/misc/bcm2835_mbox.c | 1 +
> hw/misc/bcm2835_property.c | 1 +
> hw/misc/bcm2835_rng.c | 1 +
> hw/misc/eccmemctl.c | 1 +
> hw/misc/exynos4210_clk.c | 1 +
> hw/misc/exynos4210_pmu.c | 1 +
> hw/misc/exynos4210_rng.c | 1 +
> hw/misc/imx25_ccm.c | 1 +
> hw/misc/imx31_ccm.c | 1 +
> hw/misc/imx6_ccm.c | 1 +
> hw/misc/imx6_src.c | 1 +
> hw/misc/imx6ul_ccm.c | 1 +
> hw/misc/imx7_ccm.c | 1 +
> hw/misc/iotkit-secctl.c | 1 +
> hw/misc/iotkit-sysctl.c | 1 +
> hw/misc/ivshmem.c | 1 +
> hw/misc/macio/cuda.c | 1 +
> hw/misc/macio/gpio.c | 1 +
> hw/misc/macio/mac_dbdma.c | 1 +
> hw/misc/macio/macio.c | 1 +
> hw/misc/macio/pmu.c | 1 +
> hw/misc/max111x.c | 1 +
> hw/misc/milkymist-hpdmc.c | 1 +
> hw/misc/milkymist-pfpu.c | 1 +
> hw/misc/mips_cmgcr.c | 1 +
> hw/misc/mips_cpc.c | 1 +
> hw/misc/mos6522.c | 1 +
> hw/misc/mps2-fpgaio.c | 1 +
> hw/misc/mps2-scc.c | 1 +
> hw/misc/msf2-sysreg.c | 1 +
> hw/misc/mst_fpga.c | 1 +
> hw/misc/nrf51_rng.c | 1 +
> hw/misc/pca9552.c | 1 +
> hw/misc/slavio_misc.c | 1 +
> hw/misc/tmp105.c | 1 +
> hw/misc/tmp421.c | 1 +
> hw/misc/tz-mpc.c | 1 +
> hw/misc/tz-msc.c | 1 +
> hw/misc/tz-ppc.c | 1 +
> hw/misc/vmcoreinfo.c | 1 +
> hw/misc/zynq-xadc.c | 1 +
> hw/misc/zynq_slcr.c | 1 +
> hw/net/allwinner_emac.c | 1 +
> hw/net/cadence_gem.c | 1 +
> hw/net/can/can_kvaser_pci.c | 1 +
> hw/net/can/can_mioe3680_pci.c | 1 +
> hw/net/can/can_pcm3680_pci.c | 1 +
> hw/net/can/can_sja1000.c | 1 +
> hw/net/dp8393x.c | 1 +
> hw/net/e1000.c | 1 +
> hw/net/e1000e.c | 1 +
> hw/net/eepro100.c | 1 +
> hw/net/ftgmac100.c | 1 +
> hw/net/imx_fec.c | 1 +
> hw/net/lan9118.c | 1 +
> hw/net/lance.c | 1 +
> hw/net/milkymist-minimac2.c | 1 +
> hw/net/mipsnet.c | 1 +
> hw/net/ne2000-isa.c | 1 +
> hw/net/ne2000-pci.c | 1 +
> hw/net/ne2000.c | 1 +
> hw/net/pcnet-pci.c | 1 +
> hw/net/pcnet.c | 1 +
> hw/net/rocker/rocker.c | 1 +
> hw/net/rtl8139.c | 1 +
> hw/net/smc91c111.c | 1 +
> hw/net/spapr_llan.c | 1 +
> hw/net/stellaris_enet.c | 1 +
> hw/net/sungem.c | 1 +
> hw/net/sunhme.c | 1 +
> hw/net/vmxnet3.c | 1 +
> hw/net/xgmac.c | 1 +
> hw/nvram/ds1225y.c | 1 +
> hw/nvram/eeprom93xx.c | 1 +
> hw/nvram/fw_cfg.c | 1 +
> hw/nvram/mac_nvram.c | 1 +
> hw/nvram/nrf51_nvm.c | 1 +
> hw/nvram/spapr_nvram.c | 1 +
> hw/openrisc/cputimer.c | 1 +
> hw/pci-bridge/gen_pcie_root_port.c | 1 +
> hw/pci-bridge/i82801b11.c | 1 +
> hw/pci-bridge/ioh3420.c | 1 +
> hw/pci-bridge/xio3130_downstream.c | 1 +
> hw/pci-bridge/xio3130_upstream.c | 1 +
> hw/pci-host/bonito.c | 1 +
> hw/pci-host/designware.c | 1 +
> hw/pci-host/gpex.c | 1 +
> hw/pci-host/piix.c | 1 +
> hw/pci-host/ppce500.c | 1 +
> hw/pci-host/prep.c | 1 +
> hw/pci-host/q35.c | 1 +
> hw/pci-host/versatile.c | 1 +
> hw/pci/msix.c | 1 +
> hw/pci/pci.c | 1 +
> hw/pci/pcie_aer.c | 1 +
> hw/ppc/ppc.c | 1 +
> hw/ppc/ppc4xx_pci.c | 1 +
> hw/ppc/prep_systemio.c | 1 +
> hw/ppc/rs6000_mc.c | 1 +
> hw/ppc/spapr_caps.c | 1 +
> hw/ppc/spapr_cpu_core.c | 1 +
> hw/ppc/spapr_drc.c | 1 +
> hw/ppc/spapr_iommu.c | 1 +
> hw/ppc/spapr_ovec.c | 1 +
> hw/ppc/spapr_pci.c | 1 +
> hw/ppc/spapr_rtc.c | 1 +
> hw/ppc/spapr_vio.c | 1 +
> hw/riscv/sifive_gpio.c | 1 +
> hw/s390x/event-facility.c | 1 +
> hw/s390x/sclpquiesce.c | 1 +
> hw/scsi/esp-pci.c | 1 +
> hw/scsi/esp.c | 1 +
> hw/scsi/lsi53c895a.c | 1 +
> hw/scsi/megasas.c | 1 +
> hw/scsi/mptsas.c | 1 +
> hw/scsi/scsi-bus.c | 1 +
> hw/scsi/scsi-disk.c | 1 +
> hw/scsi/spapr_vscsi.c | 1 +
> hw/scsi/vmw_pvscsi.c | 1 +
> hw/sd/bcm2835_sdhost.c | 1 +
> hw/sd/milkymist-memcard.c | 1 +
> hw/sd/pl181.c | 1 +
> hw/sd/pxa2xx_mmci.c | 1 +
> hw/sd/sd.c | 1 +
> hw/sd/sdhci.c | 1 +
> hw/sd/ssi-sd.c | 1 +
> hw/sparc/sun4m.c | 1 +
> hw/sparc/sun4m_iommu.c | 1 +
> hw/sparc64/sun4u.c | 2 ++
> hw/ssi/aspeed_smc.c | 1 +
> hw/ssi/imx_spi.c | 1 +
> hw/ssi/mss-spi.c | 1 +
> hw/ssi/pl022.c | 1 +
> hw/ssi/ssi.c | 1 +
> hw/ssi/stm32f2xx_spi.c | 1 +
> hw/ssi/xilinx_spi.c | 1 +
> hw/ssi/xilinx_spips.c | 1 +
> hw/timer/a9gtimer.c | 1 +
> hw/timer/allwinner-a10-pit.c | 1 +
> hw/timer/arm_mptimer.c | 1 +
> hw/timer/arm_timer.c | 1 +
> hw/timer/armv7m_systick.c | 1 +
> hw/timer/aspeed_rtc.c | 1 +
> hw/timer/aspeed_timer.c | 1 +
> hw/timer/cadence_ttc.c | 1 +
> hw/timer/cmsdk-apb-dualtimer.c | 1 +
> hw/timer/cmsdk-apb-timer.c | 1 +
> hw/timer/digic-timer.c | 1 +
> hw/timer/ds1338.c | 1 +
> hw/timer/exynos4210_mct.c | 1 +
> hw/timer/exynos4210_pwm.c | 1 +
> hw/timer/exynos4210_rtc.c | 1 +
> hw/timer/hpet.c | 1 +
> hw/timer/i8254_common.c | 1 +
> hw/timer/imx_epit.c | 1 +
> hw/timer/imx_gpt.c | 1 +
> hw/timer/lm32_timer.c | 1 +
> hw/timer/m48t59.c | 1 +
> hw/timer/mc146818rtc.c | 1 +
> hw/timer/milkymist-sysctl.c | 1 +
> hw/timer/mss-timer.c | 1 +
> hw/timer/nrf51_timer.c | 1 +
> hw/timer/pl031.c | 1 +
> hw/timer/pxa2xx_timer.c | 1 +
> hw/timer/slavio_timer.c | 1 +
> hw/timer/stm32f2xx_timer.c | 1 +
> hw/timer/twl92230.c | 1 +
> hw/timer/xlnx-zynqmp-rtc.c | 1 +
> hw/tpm/tpm_emulator.c | 1 +
> hw/tpm/tpm_tis.c | 1 +
> hw/usb/bus.c | 1 +
> hw/usb/ccid-card-passthru.c | 1 +
> hw/usb/dev-audio.c | 1 +
> hw/usb/dev-bluetooth.c | 1 +
> hw/usb/dev-hid.c | 1 +
> hw/usb/dev-hub.c | 1 +
> hw/usb/dev-mtp.c | 1 +
> hw/usb/dev-network.c | 1 +
> hw/usb/dev-serial.c | 1 +
> hw/usb/dev-smartcard-reader.c | 1 +
> hw/usb/dev-storage.c | 1 +
> hw/usb/dev-uas.c | 1 +
> hw/usb/dev-wacom.c | 1 +
> hw/usb/hcd-ehci-pci.c | 1 +
> hw/usb/hcd-ehci-sysbus.c | 1 +
> hw/usb/hcd-ehci.c | 1 +
> hw/usb/hcd-ohci-pci.c | 1 +
> hw/usb/hcd-ohci.c | 1 +
> hw/usb/hcd-uhci.c | 1 +
> hw/usb/hcd-xhci.c | 1 +
> hw/usb/host-libusb.c | 1 +
> hw/usb/redirect.c | 1 +
> hw/vfio/amd-xgbe.c | 1 +
> hw/vfio/ap.c | 1 +
> hw/vfio/calxeda-xgmac.c | 1 +
> hw/vfio/pci.c | 1 +
> hw/vfio/platform.c | 1 +
> hw/watchdog/cmsdk-apb-watchdog.c | 1 +
> hw/watchdog/wdt_aspeed.c | 1 +
> hw/watchdog/wdt_diag288.c | 1 +
> hw/watchdog/wdt_i6300esb.c | 1 +
> hw/watchdog/wdt_ib700.c | 1 +
> hw/xtensa/xtfpga.c | 1 +
> migration/savevm.c | 1 +
> target/s390x/machine.c | 1 +
> util/fifo8.c | 1 +
> 425 files changed, 426 insertions(+), 8 deletions(-)
- Re: [Qemu-devel] [PATCH v4 20/29] Include qemu/main-loop.h less, (continued)
- [Qemu-devel] [PATCH v4 12/29] Include hw/irq.h a lot less, Markus Armbruster, 2019/08/12
- [Qemu-devel] [PATCH v4 29/29] sysemu: Split sysemu/runstate.h off sysemu/sysemu.h, Markus Armbruster, 2019/08/12
- [Qemu-devel] [PATCH v4 26/29] Clean up inclusion of sysemu/sysemu.h, Markus Armbruster, 2019/08/12
- [Qemu-devel] [PATCH v4 21/29] Include hw/qdev-properties.h less, Markus Armbruster, 2019/08/12
- [Qemu-devel] [PATCH v4 18/29] Include hw/hw.h exactly where needed, Markus Armbruster, 2019/08/12
- [Qemu-devel] [PATCH v4 15/29] Include migration/vmstate.h less, Markus Armbruster, 2019/08/12
- Re: [Qemu-devel] [PATCH v4 15/29] Include migration/vmstate.h less,
Philippe Mathieu-Daudé <=