Skip to content

Commit

Permalink
Include sysemu/reset.h a lot less
Browse files Browse the repository at this point in the history
In my "build everything" tree, changing sysemu/reset.h triggers a
recompile of some 2600 out of 6600 objects (not counting tests and
objects that don't depend on qemu/osdep.h).

The main culprit is hw/hw.h, which supposedly includes it for
convenience.

Include sysemu/reset.h only where it's needed.  Touching it now
recompiles less than 200 objects.

Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Tested-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Message-Id: <20190812052359.30071-9-armbru@redhat.com>
  • Loading branch information
Markus Armbruster committed Aug 16, 2019
1 parent 13d4ff0 commit 71e8a91
Show file tree
Hide file tree
Showing 92 changed files with 107 additions and 2 deletions.
2 changes: 2 additions & 0 deletions hw/acpi/ich9.c
Expand Up @@ -23,13 +23,15 @@
* Contributions after 2012-01-13 are licensed under the terms of the
* GNU GPL, version 2 or (at your option) any later version.
*/

#include "qemu/osdep.h"
#include "hw/hw.h"
#include "qapi/error.h"
#include "qapi/visitor.h"
#include "hw/i386/pc.h"
#include "hw/pci/pci.h"
#include "qemu/timer.h"
#include "sysemu/reset.h"
#include "sysemu/sysemu.h"
#include "hw/acpi/acpi.h"
#include "hw/acpi/tco.h"
Expand Down
2 changes: 2 additions & 0 deletions hw/acpi/piix4.c
Expand Up @@ -18,13 +18,15 @@
* Contributions after 2012-01-13 are licensed under the terms of the
* GNU GPL, version 2 or (at your option) any later version.
*/

#include "qemu/osdep.h"
#include "hw/hw.h"
#include "hw/i386/pc.h"
#include "hw/isa/apm.h"
#include "hw/i2c/pm_smbus.h"
#include "hw/pci/pci.h"
#include "hw/acpi/acpi.h"
#include "sysemu/reset.h"
#include "sysemu/sysemu.h"
#include "qapi/error.h"
#include "qemu/range.h"
Expand Down
1 change: 1 addition & 0 deletions hw/acpi/vmgenid.c
Expand Up @@ -18,6 +18,7 @@
#include "hw/acpi/aml-build.h"
#include "hw/acpi/vmgenid.h"
#include "hw/nvram/fw_cfg.h"
#include "sysemu/reset.h"
#include "sysemu/sysemu.h"

void vmgenid_build_acpi(VmGenIdState *vms, GArray *table_data, GArray *guid,
Expand Down
1 change: 1 addition & 0 deletions hw/arm/armv7m.c
Expand Up @@ -16,6 +16,7 @@
#include "hw/loader.h"
#include "elf.h"
#include "sysemu/qtest.h"
#include "sysemu/reset.h"
#include "qemu/error-report.h"
#include "qemu/module.h"
#include "exec/address-spaces.h"
Expand Down
1 change: 1 addition & 0 deletions hw/arm/boot.c
Expand Up @@ -18,6 +18,7 @@
#include "sysemu/kvm.h"
#include "sysemu/sysemu.h"
#include "sysemu/numa.h"
#include "sysemu/reset.h"
#include "hw/boards.h"
#include "hw/loader.h"
#include "elf.h"
Expand Down
1 change: 1 addition & 0 deletions hw/arm/nseries.c
Expand Up @@ -23,6 +23,7 @@
#include "cpu.h"
#include "qemu/cutils.h"
#include "qemu/bswap.h"
#include "sysemu/reset.h"
#include "sysemu/sysemu.h"
#include "hw/arm/omap.h"
#include "hw/arm/boot.h"
Expand Down
1 change: 1 addition & 0 deletions hw/arm/omap1.c
Expand Up @@ -29,6 +29,7 @@
#include "sysemu/sysemu.h"
#include "hw/arm/soc_dma.h"
#include "sysemu/qtest.h"
#include "sysemu/reset.h"
#include "qemu/range.h"
#include "hw/sysbus.h"
#include "qemu/cutils.h"
Expand Down
1 change: 1 addition & 0 deletions hw/arm/omap2.c
Expand Up @@ -23,6 +23,7 @@
#include "qapi/error.h"
#include "cpu.h"
#include "sysemu/qtest.h"
#include "sysemu/reset.h"
#include "hw/boards.h"
#include "hw/hw.h"
#include "hw/arm/boot.h"
Expand Down
1 change: 1 addition & 0 deletions hw/arm/virt-acpi-build.c
Expand Up @@ -44,6 +44,7 @@
#include "hw/pci/pci.h"
#include "hw/arm/virt.h"
#include "sysemu/numa.h"
#include "sysemu/reset.h"
#include "kvm_arm.h"

#define ARM_SPI_BASE 32
Expand Down
1 change: 1 addition & 0 deletions hw/char/parallel.c
Expand Up @@ -31,6 +31,7 @@
#include "chardev/char-fe.h"
#include "hw/isa/isa.h"
#include "hw/char/parallel.h"
#include "sysemu/reset.h"
#include "sysemu/sysemu.h"
#include "trace.h"

Expand Down
1 change: 1 addition & 0 deletions hw/char/serial.c
Expand Up @@ -28,6 +28,7 @@
#include "chardev/char-serial.h"
#include "qapi/error.h"
#include "qemu/timer.h"
#include "sysemu/reset.h"
#include "qemu/error-report.h"
#include "trace.h"

Expand Down
1 change: 1 addition & 0 deletions hw/core/generic-loader.c
Expand Up @@ -34,6 +34,7 @@
#include "qom/cpu.h"
#include "hw/sysbus.h"
#include "sysemu/dma.h"
#include "sysemu/reset.h"
#include "hw/loader.h"
#include "qapi/error.h"
#include "qemu/module.h"
Expand Down
1 change: 1 addition & 0 deletions hw/core/loader.c
Expand Up @@ -48,6 +48,7 @@
#include "hw/hw.h"
#include "disas/disas.h"
#include "monitor/monitor.h"
#include "sysemu/reset.h"
#include "sysemu/sysemu.h"
#include "uboot_image.h"
#include "hw/loader.h"
Expand Down
1 change: 1 addition & 0 deletions hw/cris/boot.c
Expand Up @@ -29,6 +29,7 @@
#include "elf.h"
#include "boot.h"
#include "qemu/cutils.h"
#include "sysemu/reset.h"

static void main_cpu_reset(void *opaque)
{
Expand Down
1 change: 1 addition & 0 deletions hw/display/cirrus_vga.c
Expand Up @@ -35,6 +35,7 @@
#include "qemu/osdep.h"
#include "qemu/module.h"
#include "qemu/units.h"
#include "sysemu/reset.h"
#include "qapi/error.h"
#include "trace.h"
#include "hw/hw.h"
Expand Down
2 changes: 2 additions & 0 deletions hw/display/ramfb.c
Expand Up @@ -10,12 +10,14 @@
* This work is licensed under the terms of the GNU GPL, version 2 or later.
* See the COPYING file in the top-level directory.
*/

#include "qemu/osdep.h"
#include "qapi/error.h"
#include "qemu/option.h"
#include "hw/loader.h"
#include "hw/display/ramfb.h"
#include "ui/console.h"
#include "sysemu/reset.h"
#include "sysemu/sysemu.h"

struct QEMU_PACKED RAMFBCfg {
Expand Down
2 changes: 2 additions & 0 deletions hw/display/vga.c
Expand Up @@ -21,8 +21,10 @@
* OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
* THE SOFTWARE.
*/

#include "qemu/osdep.h"
#include "qemu/units.h"
#include "sysemu/reset.h"
#include "qapi/error.h"
#include "hw/hw.h"
#include "hw/display/vga.h"
Expand Down
1 change: 1 addition & 0 deletions hw/hppa/machine.c
Expand Up @@ -11,6 +11,7 @@
#include "hw/loader.h"
#include "hw/boards.h"
#include "qemu/error-report.h"
#include "sysemu/reset.h"
#include "sysemu/sysemu.h"
#include "hw/timer/mc146818rtc.h"
#include "hw/ide.h"
Expand Down
1 change: 1 addition & 0 deletions hw/i386/acpi-build.c
Expand Up @@ -48,6 +48,7 @@
#include "hw/timer/mc146818rtc_regs.h"
#include "hw/mem/memory-device.h"
#include "sysemu/numa.h"
#include "sysemu/reset.h"

/* Supported chipsets: */
#include "hw/acpi/piix4.h"
Expand Down
1 change: 1 addition & 0 deletions hw/i386/pc.c
Expand Up @@ -54,6 +54,7 @@
#include "sysemu/numa.h"
#include "sysemu/kvm.h"
#include "sysemu/qtest.h"
#include "sysemu/reset.h"
#include "kvm_i386.h"
#include "hw/xen/xen.h"
#include "hw/xen/start_info.h"
Expand Down
1 change: 1 addition & 0 deletions hw/ide/cmd646.c
Expand Up @@ -30,6 +30,7 @@
#include "hw/isa/isa.h"
#include "sysemu/sysemu.h"
#include "sysemu/dma.h"
#include "sysemu/reset.h"

#include "hw/ide/pci.h"
#include "trace.h"
Expand Down
1 change: 1 addition & 0 deletions hw/ide/piix.c
Expand Up @@ -31,6 +31,7 @@
#include "sysemu/sysemu.h"
#include "sysemu/blockdev.h"
#include "sysemu/dma.h"
#include "sysemu/reset.h"

#include "hw/ide/pci.h"
#include "trace.h"
Expand Down
1 change: 1 addition & 0 deletions hw/ide/sii3112.c
Expand Up @@ -15,6 +15,7 @@
#include "qemu/osdep.h"
#include "hw/ide/pci.h"
#include "qemu/module.h"
#include "sysemu/reset.h"
#include "trace.h"

#define TYPE_SII3112_PCI "sii3112"
Expand Down
1 change: 1 addition & 0 deletions hw/ide/via.c
Expand Up @@ -30,6 +30,7 @@
#include "qemu/module.h"
#include "sysemu/sysemu.h"
#include "sysemu/dma.h"
#include "sysemu/reset.h"

#include "hw/ide/pci.h"
#include "trace.h"
Expand Down
1 change: 1 addition & 0 deletions hw/input/lm832x.c
Expand Up @@ -23,6 +23,7 @@
#include "hw/i2c/i2c.h"
#include "qemu/module.h"
#include "qemu/timer.h"
#include "sysemu/reset.h"
#include "ui/console.h"

#define TYPE_LM8323 "lm8323"
Expand Down
2 changes: 2 additions & 0 deletions hw/input/pckbd.c
Expand Up @@ -21,13 +21,15 @@
* OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
* THE SOFTWARE.
*/

#include "qemu/osdep.h"
#include "qemu/log.h"
#include "hw/hw.h"
#include "hw/isa/isa.h"
#include "hw/i386/pc.h"
#include "hw/input/ps2.h"
#include "hw/input/i8042.h"
#include "sysemu/reset.h"
#include "sysemu/sysemu.h"

#include "trace.h"
Expand Down
2 changes: 2 additions & 0 deletions hw/input/ps2.c
Expand Up @@ -21,12 +21,14 @@
* OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
* THE SOFTWARE.
*/

#include "qemu/osdep.h"
#include "qemu/log.h"
#include "hw/hw.h"
#include "hw/input/ps2.h"
#include "ui/console.h"
#include "ui/input.h"
#include "sysemu/reset.h"
#include "sysemu/sysemu.h"

#include "trace.h"
Expand Down
1 change: 1 addition & 0 deletions hw/input/tsc2005.c
Expand Up @@ -22,6 +22,7 @@
#include "qemu/log.h"
#include "hw/hw.h"
#include "qemu/timer.h"
#include "sysemu/reset.h"
#include "ui/console.h"
#include "hw/input/tsc2xxx.h"
#include "trace.h"
Expand Down
1 change: 1 addition & 0 deletions hw/input/tsc210x.c
Expand Up @@ -23,6 +23,7 @@
#include "hw/hw.h"
#include "audio/audio.h"
#include "qemu/timer.h"
#include "sysemu/reset.h"
#include "ui/console.h"
#include "hw/arm/omap.h" /* For I2SCodec */
#include "hw/input/tsc2xxx.h"
Expand Down
1 change: 1 addition & 0 deletions hw/intc/mips_gic.c
Expand Up @@ -18,6 +18,7 @@
#include "exec/memory.h"
#include "sysemu/sysemu.h"
#include "sysemu/kvm.h"
#include "sysemu/reset.h"
#include "kvm_mips.h"
#include "hw/intc/mips_gic.h"

Expand Down
1 change: 1 addition & 0 deletions hw/intc/pnv_xive.c
Expand Up @@ -14,6 +14,7 @@
#include "target/ppc/cpu.h"
#include "sysemu/cpus.h"
#include "sysemu/dma.h"
#include "sysemu/reset.h"
#include "monitor/monitor.h"
#include "hw/ppc/fdt.h"
#include "hw/ppc/pnv.h"
Expand Down
1 change: 1 addition & 0 deletions hw/intc/spapr_xive.c
Expand Up @@ -14,6 +14,7 @@
#include "qemu/error-report.h"
#include "target/ppc/cpu.h"
#include "sysemu/cpus.h"
#include "sysemu/reset.h"
#include "monitor/monitor.h"
#include "hw/ppc/fdt.h"
#include "hw/ppc/spapr.h"
Expand Down
1 change: 1 addition & 0 deletions hw/intc/xics.c
Expand Up @@ -38,6 +38,7 @@
#include "monitor/monitor.h"
#include "hw/intc/intc.h"
#include "sysemu/kvm.h"
#include "sysemu/reset.h"

void icp_pic_print_info(ICPState *icp, Monitor *mon)
{
Expand Down
1 change: 1 addition & 0 deletions hw/intc/xive.c
Expand Up @@ -14,6 +14,7 @@
#include "target/ppc/cpu.h"
#include "sysemu/cpus.h"
#include "sysemu/dma.h"
#include "sysemu/reset.h"
#include "hw/qdev-properties.h"
#include "monitor/monitor.h"
#include "hw/ppc/xive.h"
Expand Down
1 change: 1 addition & 0 deletions hw/isa/piix4.c
Expand Up @@ -28,6 +28,7 @@
#include "hw/pci/pci.h"
#include "hw/isa/isa.h"
#include "hw/sysbus.h"
#include "sysemu/reset.h"

PCIDevice *piix4_dev;

Expand Down
1 change: 1 addition & 0 deletions hw/isa/vt82c686.c
Expand Up @@ -22,6 +22,7 @@
#include "hw/isa/apm.h"
#include "hw/acpi/acpi.h"
#include "hw/i2c/pm_smbus.h"
#include "sysemu/reset.h"
#include "sysemu/sysemu.h"
#include "qemu/module.h"
#include "qemu/timer.h"
Expand Down
1 change: 1 addition & 0 deletions hw/lm32/lm32_boards.c
Expand Up @@ -30,6 +30,7 @@
#include "lm32_hwsetup.h"
#include "lm32.h"
#include "exec/address-spaces.h"
#include "sysemu/reset.h"
#include "sysemu/sysemu.h"

typedef struct {
Expand Down
1 change: 1 addition & 0 deletions hw/lm32/milkymist.c
Expand Up @@ -27,6 +27,7 @@
#include "hw/block/flash.h"
#include "sysemu/sysemu.h"
#include "sysemu/qtest.h"
#include "sysemu/reset.h"
#include "hw/boards.h"
#include "hw/loader.h"
#include "elf.h"
Expand Down
1 change: 1 addition & 0 deletions hw/microblaze/boot.c
Expand Up @@ -31,6 +31,7 @@
#include "qemu/config-file.h"
#include "qemu/error-report.h"
#include "sysemu/device_tree.h"
#include "sysemu/reset.h"
#include "sysemu/sysemu.h"
#include "hw/loader.h"
#include "elf.h"
Expand Down
1 change: 1 addition & 0 deletions hw/mips/cps.c
Expand Up @@ -24,6 +24,7 @@
#include "hw/mips/mips.h"
#include "hw/mips/cpudevs.h"
#include "sysemu/kvm.h"
#include "sysemu/reset.h"

qemu_irq get_cps_irq(MIPSCPSState *s, int pin_number)
{
Expand Down
1 change: 1 addition & 0 deletions hw/mips/mips_fulong2e.c
Expand Up @@ -44,6 +44,7 @@
#include "hw/timer/i8254.h"
#include "exec/address-spaces.h"
#include "sysemu/qtest.h"
#include "sysemu/reset.h"
#include "qemu/error-report.h"

#define DEBUG_FULONG2E_INIT
Expand Down
1 change: 1 addition & 0 deletions hw/mips/mips_jazz.c
Expand Up @@ -48,6 +48,7 @@
#include "hw/sysbus.h"
#include "exec/address-spaces.h"
#include "sysemu/qtest.h"
#include "sysemu/reset.h"
#include "qapi/error.h"
#include "qemu/error-report.h"
#include "qemu/help_option.h"
Expand Down
1 change: 1 addition & 0 deletions hw/mips/mips_malta.c
Expand Up @@ -51,6 +51,7 @@
#include "hw/sysbus.h" /* SysBusDevice */
#include "qemu/host-utils.h"
#include "sysemu/qtest.h"
#include "sysemu/reset.h"
#include "qapi/error.h"
#include "qemu/error-report.h"
#include "hw/empty_slot.h"
Expand Down

0 comments on commit 71e8a91

Please sign in to comment.