qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PULL v2 00/56] QOM devices patch queue 2013-10-31


From: Anthony Liguori
Subject: Re: [Qemu-devel] [PULL v2 00/56] QOM devices patch queue 2013-10-31
Date: Thu, 31 Oct 2013 22:03:29 +0100

On Thu, Oct 31, 2013 at 3:37 PM, Andreas Färber <address@hidden> wrote:
> Hello Anthony,
>
> This is my updated QOM devices patch queue. Please pull or provide more 
> details
> of what is not working in your build environment.

Andreas,

If you cannot be bothered to adequately test your pull requests, then
don't bother sending them.

make check fails spectacularly.  I've confirmed this on multiple
platforms on different distros.

The errors are the exact same as before.  Install some VMs and
reproduce the problem.  I just checked and it fails under 64-bit
Fedora 19.

Regards,

Anthony Liguori

> v2 is rebased, dropping two ARM patches, and the observed SD segfault had been
> fixed in 794cbc26eb94ce13c75d105eea9ff0afff56e2c2.
> Patches are not manually changed, thus intentionally not resent.
>
> Thanks,
> Andreas
>
> Cc: Anthony Liguori <address@hidden>
>
> Cc: Peter Maydell <address@hidden>
> Cc: Mian M. Hamayun <address@hidden>
> Cc: Paolo Bonzini <address@hidden>
> Cc: Michael S. Tsirkin <address@hidden>
> Cc: Stefan Hajnoczi <address@hidden>
> Cc: Edgar E. Iglesias <address@hidden>
>
> The following changes since commit b86160555f8d1fe11d6bcec393e08e645d7e1e8d:
>
>   integrator: fix Linux boot failure by emulating dbg region (2013-10-31 
> 14:00:16 +0100)
>
> are available in the git repository at:
>
>   git://github.com/afaerber/qemu-cpu.git tags/qom-devices-for-anthony
>
> for you to fetch changes up to 30a423024ffaa28920c7e36c17855d7bd0061f09:
>
>   pcmcia/pxa2xx: QOM'ify PXA2xxPCMCIAState (2013-10-31 15:12:30 +0100)
>
> ----------------------------------------------------------------
> QOM device refactorings
>
> * QTest coverage for all machines
> * QOM realize for Milkymist UART
> * QOM realize for ARM MPCore
> * device_add bug fixes and cleanups
> * QOM API extensions and cleanups
> * QOM for PCMCIA/MicroDrive (last legacy IDE device)
>
> ----------------------------------------------------------------
> Andreas Färber (49):
>       mips_mipssim: Silence BIOS loading warning for qtest
>       puv3: Turn puv3_load_kernel() into a no-op for qtest without -kernel
>       mainstone: Don't enforce use of -pflash for qtest
>       gumstix: Don't enforce use of -pflash for qtest
>       z2: Don't enforce use of -pflash for qtest
>       palm: Don't enforce loading ROM or kernel for qtest
>       omap_sx1: Don't enforce use of kernel or flash for qtest
>       exynos4_boards: Silence lack of -smp 2 warning for qtest
>       armv7m: Don't enforce use of kernel for qtest
>       axis_dev88: Don't enforce use of kernel for qtest
>       mcf5208: Don't enforce use of kernel for qtest
>       an5206: Don't enforce use of kernel for qtest
>       milkymist: Suppress -kernel/-bios/-drive error for qtest
>       shix: Drop debug output
>       shix: Don't require firmware presence for qtest
>       leon3: Don't enforce use of -bios with qtest
>       qtest: Prepare QOM machine tests
>       a9mpcore: Split off instance_init
>       arm_gic: Extract headers hw/intc/arm_gic{,_common}.h
>       a9mpcore: Embed GICState
>       a9scu: QOM cleanups
>       a9mpcore: Embed A9SCUState
>       arm_mptimer: Convert to QOM realize
>       a9mpcore: Embed ARMMPTimerState
>       a9mpcore: Convert to QOM realize
>       a9mpcore: Prepare for QOM embedding
>       a15mpcore: Split off instance_init
>       a15mpcore: Embed GICState
>       a15mpcore: Convert to QOM realize
>       a15mpcore: Prepare for QOM embedding
>       a9scu: Build only once
>       arm11mpcore: Fix typo in MemoryRegion name
>       arm11mpcore: Drop unused fields
>       arm11mpcore: Create container MemoryRegion in instance_init
>       arm11mpcore: Split off SCU device
>       arm11mpcore: Convert ARM11MPCorePriveState to QOM realize
>       realview_gic: Convert to QOM realize
>       realview_gic: Prepare for QOM embedding
>       arm11mpcore: Convert mpcore_rirq_state to QOM realize
>       arm11mpcore: Prepare for QOM embedding
>       arm11mpcore: Split off RealView MPCore
>       qdev-monitor: Clean up qdev_device_add() variable naming
>       qdev-monitor: Avoid qdev as variable name
>       qdev-monitor: Inline qdev_init() for device_add
>       pxa: Fix typo "dettach"
>       pcmcia: QOM'ify PCMCIACardState and MicroDriveState
>       microdrive: Coding Style cleanups
>       ide: Drop ide_init2_with_non_qdev_drives()
>       pcmcia/pxa2xx: QOM'ify PXA2xxPCMCIAState
>
> Antony Pavlov (1):
>       milkymist-uart: Use Device::realize instead of SysBusDevice::init
>
> Igor Mammedov (2):
>       qdev-monitor: Fix crash when device_add is called with abstract driver
>       qom: Include error.h directly in object.h
>
> Michael S. Tsirkin (2):
>       qom: Clean up struct Error references
>       qom: Add pointer to int property helpers
>
> Stefan Hajnoczi (2):
>       qdev-monitor: Unref device when device_add fails
>       qdev: Drop misleading qdev_free() function
>
>  default-configs/arm-softmmu.mak              |   1 +
>  hw/Makefile.objs                             |   1 +
>  hw/acpi/piix4.c                              |   2 +-
>  hw/arm/armv7m.c                              |  25 +--
>  hw/arm/exynos4_boards.c                      |   3 +-
>  hw/arm/gumstix.c                             |  11 +-
>  hw/arm/mainstone.c                           |   5 +-
>  hw/arm/omap_sx1.c                            |   3 +-
>  hw/arm/palm.c                                |   3 +-
>  hw/arm/z2.c                                  |   5 +-
>  hw/block/tc58128.c                           |  10 +-
>  hw/char/milkymist-uart.c                     |  24 +--
>  hw/core/qdev.c                               |  12 +-
>  hw/cpu/Makefile.objs                         |   1 +
>  hw/cpu/a15mpcore.c                           |  81 ++++-----
>  hw/cpu/a9mpcore.c                            | 120 +++++++------
>  hw/cpu/arm11mpcore.c                         | 251 +++++++-------------------
>  hw/cpu/realview_mpcore.c                     | 139 +++++++++++++++
>  hw/cris/axis_dev88.c                         |  11 +-
>  hw/ide/core.c                                |  49 ------
>  hw/ide/internal.h                            |   2 -
>  hw/ide/microdrive.c                          | 226 +++++++++++++++++-------
>  hw/intc/arm_gic_common.c                     |  18 +-
>  hw/intc/gic_internal.h                       |  80 +--------
>  hw/intc/realview_gic.c                       |  58 +++---
>  hw/lm32/milkymist.c                          |   3 +-
>  hw/m68k/an5206.c                             |   4 +
>  hw/m68k/mcf5208.c                            |   4 +
>  hw/mips/mips_mipssim.c                       |   4 +-
>  hw/misc/Makefile.objs                        |   4 +-
>  hw/misc/a9scu.c                              |  25 +--
>  hw/misc/arm11scu.c                           | 100 +++++++++++
>  hw/pci/pci-hotplug-old.c                     |   2 +-
>  hw/pci/pci_bridge.c                          |   2 +-
>  hw/pci/pcie.c                                |   2 +-
>  hw/pci/shpc.c                                |   2 +-
>  hw/pcmcia/Makefile.objs                      |   2 +
>  hw/pcmcia/pcmcia.c                           |  24 +++
>  hw/{misc/pxa2xx_pcmcia.c => pcmcia/pxa2xx.c} | 130 +++++++++++---
>  hw/s390x/virtio-ccw.c                        |   2 +-
>  hw/scsi/scsi-bus.c                           |   6 +-
>  hw/sh4/shix.c                                |  16 +-
>  hw/sparc/leon3.c                             |   3 +-
>  hw/timer/arm_mptimer.c                       |  60 ++-----
>  hw/unicore32/puv3.c                          |   4 +
>  hw/usb/bus.c                                 |   7 +-
>  hw/usb/dev-storage.c                         |   2 +-
>  hw/usb/host-legacy.c                         |   2 +-
>  hw/virtio/virtio-bus.c                       |   4 +-
>  hw/xen/xen_platform.c                        |   2 +-
>  include/hw/arm/pxa.h                         |   2 +-
>  include/hw/cpu/a15mpcore.h                   |  44 +++++
>  include/hw/cpu/a9mpcore.h                    |  37 ++++
>  include/hw/cpu/arm11mpcore.h                 |  35 ++++
>  include/hw/intc/arm_gic.h                    |  42 +++++
>  include/hw/intc/arm_gic_common.h             |  92 ++++++++++
>  include/hw/intc/realview_gic.h               |  28 +++
>  include/hw/misc/a9scu.h                      |  31 ++++
>  include/hw/misc/arm11scu.h                   |  29 +++
>  include/hw/pcmcia.h                          |  46 +++--
>  include/hw/qdev-core.h                       |   1 -
>  include/hw/timer/arm_mptimer.h               |  54 ++++++
>  include/qom/object.h                         | 104 ++++++++---
>  qdev-monitor.c                               |  63 ++++---
>  qom/object.c                                 |  60 +++++++
>  tests/Makefile                               |  26 +++
>  tests/qom-test.c                             | 253 
> +++++++++++++++++++++++++++
>  67 files changed, 1760 insertions(+), 744 deletions(-)
>  create mode 100644 hw/cpu/realview_mpcore.c
>  create mode 100644 hw/misc/arm11scu.c
>  create mode 100644 hw/pcmcia/Makefile.objs
>  create mode 100644 hw/pcmcia/pcmcia.c
>  rename hw/{misc/pxa2xx_pcmcia.c => pcmcia/pxa2xx.c} (61%)
>  create mode 100644 include/hw/cpu/a15mpcore.h
>  create mode 100644 include/hw/cpu/a9mpcore.h
>  create mode 100644 include/hw/cpu/arm11mpcore.h
>  create mode 100644 include/hw/intc/arm_gic.h
>  create mode 100644 include/hw/intc/arm_gic_common.h
>  create mode 100644 include/hw/intc/realview_gic.h
>  create mode 100644 include/hw/misc/a9scu.h
>  create mode 100644 include/hw/misc/arm11scu.h
>  create mode 100644 include/hw/timer/arm_mptimer.h
>  create mode 100644 tests/qom-test.c



reply via email to

[Prev in Thread] Current Thread [Next in Thread]