mbox

[PULL,00/41] Error reporting patches for 2016-01-13

Message ID 1452699819-26608-1-git-send-email-armbru@redhat.com
State New
Headers show

Pull-request

git://repo.or.cz/qemu/armbru.git tags/pull-error-2016-01-13

Message

Markus Armbruster Jan. 13, 2016, 3:42 p.m. UTC
The following changes since commit 649a1bbaf95adb228f1030ab0618a932bc26aa8b:

  Merge remote-tracking branch 'remotes/kvaneesh/tags/for-upstream-signed' into staging (2016-01-12 17:37:22 +0000)

are available in the git repository at:

  git://repo.or.cz/qemu/armbru.git tags/pull-error-2016-01-13

for you to fetch changes up to 5d596c245d675000ddee69e87616d537ef273be5:

  checkpatch: Detect newlines in error_report and other error functions (2016-01-13 15:16:19 +0100)

----------------------------------------------------------------
Error reporting patches for 2016-01-13

----------------------------------------------------------------
Jason J. Herne (1):
      checkpatch: Detect newlines in error_report and other error functions

Markus Armbruster (40):
      error: Document how to accumulate multiple errors
      Use error_fatal to simplify obvious fatal errors
      hw: Inline the qdev_prop_set_drive_nofail() wrapper
      hw: Don't use hw_error() for machine initialization errors
      omap: Don't use hw_error() in device init() methods
      arm_mptimer: Don't use hw_error() in realize() method
      etraxfs_eth: Don't use hw_error() in init() method
      raven: Mark use of hw_error() in realize() FIXME
      error: Don't append a newline when printing the error hint
      hw/arm/virt: Fix property "gic-version" error handling
      sysbus: Don't use hw_error() in machine_init_done_notifiers
      isa: Trivially convert remaining PCI-ISA bridges to realize()
      isa: Clean up error handling around isa_bus_new()
      isa: Clean up inappropriate hw_error()
      audio: Clean up inappropriate and unreachable use of hw_error()
      xen-hvm: Mark inappropriate error handling FIXME
      qemu-nbd: Replace BSDism <err.h> by error_report()
      error: Use error_report_err() where appropriate (again)
      error: Use error_report_err() instead of monitor_printf()
      error: Use error_report_err() instead of ad hoc prints
      error: Improve documentation
      block: Clean up "Could not create temporary overlay" error message
      qemu-nbd: Clean up "Failed to load snapshot" error message
      test-throttle: Simplify qemu_init_main_loop() error handling
      error: New error_prepend(), error_reportf_err()
      error: Don't decorate original error message when adding to it
      error: Use error_reportf_err() where it makes obvious sense
      error: Use error_prepend() where it makes obvious sense
      spapr: Use error_reportf_err()
      migration: Use error_reportf_err() instead of monitor_printf()
      qemu-io qemu-nbd: Use error_report() etc. instead of fprintf()
      error: Strip trailing '\n' from error string arguments (again)
      vmdk: Clean up control flow in vmdk_parse_extents() a bit
      vmdk: Clean up "Invalid extent lines" error message
      pci-assign: Clean up "Failed to assign" error messages
      vhdx: Fix "log that needs to be replayed" error message
      error: Clean up errors with embedded newlines (again)
      hw/s390x: Rename local variables Error *l_err to just err
      s390/sclp: Simplify control flow in sclp_realize()
      error: Consistently name Error * objects err, and not errp

 arch_init.c                        |   4 +-
 audio/audio.c                      |  11 +---
 block.c                            |  20 +++---
 block/qcow2.c                      |   5 +-
 block/qed.c                        |   5 +-
 block/sheepdog.c                   |   8 +--
 block/vhdx-log.c                   |  13 ++--
 block/vmdk.c                       |  50 +++++++++------
 blockdev.c                         |  12 ++--
 contrib/ivshmem-server/main.c      |  10 ++-
 hmp.c                              |  61 ++++++++----------
 hw/alpha/dp264.c                   |  11 ++--
 hw/alpha/typhoon.c                 |   3 +-
 hw/arm/cubieboard.c                |   9 ++-
 hw/arm/digic_boards.c              |   3 +-
 hw/arm/exynos4210.c                |  13 +---
 hw/arm/highbank.c                  |  13 ++--
 hw/arm/imx25_pdk.c                 |   2 +-
 hw/arm/integratorcp.c              |  13 +---
 hw/arm/kzm.c                       |   2 +-
 hw/arm/netduino2.c                 |   2 +-
 hw/arm/nseries.c                   |   4 +-
 hw/arm/realview.c                  |  20 ++----
 hw/arm/versatilepb.c               |  13 +---
 hw/arm/vexpress.c                  |   7 +-
 hw/arm/virt.c                      |   5 +-
 hw/arm/xilinx_zynq.c               |  28 ++------
 hw/arm/xlnx-ep108.c                |   2 +-
 hw/arm/xlnx-zynqmp.c               |   2 +-
 hw/block/dataplane/virtio-blk.c    |   8 +--
 hw/block/fdc.c                     |  15 +++--
 hw/block/nand.c                    |   2 +-
 hw/char/exynos4210_uart.c          |   9 ++-
 hw/char/serial.c                   |  14 +---
 hw/core/nmi.c                      |  10 +--
 hw/core/platform-bus.c             |  25 ++++----
 hw/core/qdev-properties-system.c   |  12 ----
 hw/core/qdev-properties.c          |   6 +-
 hw/core/qdev.c                     |   5 +-
 hw/gpio/omap_gpio.c                |  29 ++++++---
 hw/i2c/omap_i2c.c                  |   8 ++-
 hw/i386/kvm/pci-assign.c           |  16 ++---
 hw/i386/pc.c                       |  23 ++-----
 hw/i386/pc_piix.c                  |   3 +-
 hw/ide/qdev.c                      |   3 +-
 hw/intc/omap_intc.c                |  10 ++-
 hw/isa/i82378.c                    |   5 +-
 hw/isa/isa-bus.c                   |  15 +----
 hw/isa/lpc_ich9.c                  |  11 ++--
 hw/isa/pc87312.c                   |   8 +--
 hw/isa/piix4.c                     |   6 +-
 hw/isa/vt82c686.c                  |   5 +-
 hw/m68k/an5206.c                   |   4 +-
 hw/mips/mips_jazz.c                |   2 +-
 hw/mips/mips_r4k.c                 |   2 +-
 hw/net/etraxfs_eth.c               |   4 +-
 hw/pci-host/piix.c                 |   6 +-
 hw/pci-host/prep.c                 |   6 +-
 hw/ppc/e500.c                      |   4 +-
 hw/ppc/mac_newworld.c              |  11 ++--
 hw/ppc/mac_oldworld.c              |  16 +++--
 hw/ppc/prep.c                      |  11 ++--
 hw/ppc/spapr.c                     |  13 ++--
 hw/ppc/spapr_drc.c                 |   6 +-
 hw/s390x/ipl.c                     |  12 ++--
 hw/s390x/s390-skeys-kvm.c          |   2 +-
 hw/s390x/s390-skeys.c              |  19 +++---
 hw/s390x/sclp.c                    |  21 +++---
 hw/scsi/vhost-scsi.c               |   6 +-
 hw/smbios/smbios.c                 |  43 ++-----------
 hw/sparc64/sun4u.c                 |  12 ++--
 hw/timer/arm_mptimer.c             |   5 +-
 hw/tpm/tpm_tis.c                   |   2 +-
 hw/unicore32/puv3.c                |  10 ++-
 hw/usb/bus.c                       |  11 ++--
 include/hw/isa/isa.h               |   2 +-
 include/hw/qdev-properties.h       |   2 -
 include/qapi/error.h               |  68 ++++++++++++++++++--
 include/qemu/sockets.h             |   2 +-
 kvm-all.c                          |   6 +-
 migration/ram.c                    |   2 +-
 migration/savevm.c                 |  21 +++---
 monitor.c                          |   6 +-
 net/vhost-user.c                   |   6 +-
 numa.c                             |   8 +--
 qdev-monitor.c                     |   5 +-
 qemu-img.c                         |  33 ++++------
 qemu-io.c                          |   8 +--
 qemu-nbd.c                         | 128 ++++++++++++++++++++++---------------
 qga/commands-posix.c               |   2 +-
 replay/replay.c                    |   3 +-
 scripts/checkpatch.pl              |  36 +++++++++++
 target-arm/cpu.c                   |   2 +-
 target-arm/machine.c               |   4 +-
 tests/qemu-iotests/059.out         |  12 ++--
 tests/qemu-iotests/060.out         |   2 +-
 tests/qemu-iotests/069.out         |   2 +-
 tests/qemu-iotests/070.out         |   5 +-
 tests/qemu-iotests/075.out         |  14 ++--
 tests/qemu-iotests/076.out         |   6 +-
 tests/qemu-iotests/078.out         |  12 ++--
 tests/qemu-iotests/080.out         |  36 +++++------
 tests/qemu-iotests/083.out         |  34 +++++-----
 tests/qemu-iotests/088.out         |  12 ++--
 tests/qemu-iotests/092.out         |  24 +++----
 tests/qemu-iotests/103.out         |   8 +--
 tests/qemu-iotests/114.out         |   2 +-
 tests/qemu-iotests/116.out         |  14 ++--
 tests/qemu-iotests/131.out         |   2 +-
 tests/test-aio.c                   |   4 +-
 tests/test-string-output-visitor.c |   6 +-
 tests/test-thread-pool.c           |   4 +-
 tests/test-throttle.c              |  15 +----
 ui/vnc.c                           |   4 +-
 util/error.c                       |  37 ++++++++++-
 util/qemu-error.c                  |   8 +--
 util/qemu-option.c                 |   4 +-
 vl.c                               |  29 ++-------
 xen-hvm.c                          |   7 ++
 119 files changed, 733 insertions(+), 741 deletions(-)

Comments

Peter Maydell Jan. 14, 2016, 2:09 p.m. UTC | #1
On 13 January 2016 at 15:42, Markus Armbruster <armbru@redhat.com> wrote:
> The following changes since commit 649a1bbaf95adb228f1030ab0618a932bc26aa8b:
>
>   Merge remote-tracking branch 'remotes/kvaneesh/tags/for-upstream-signed' into staging (2016-01-12 17:37:22 +0000)
>
> are available in the git repository at:
>
>   git://repo.or.cz/qemu/armbru.git tags/pull-error-2016-01-13
>
> for you to fetch changes up to 5d596c245d675000ddee69e87616d537ef273be5:
>
>   checkpatch: Detect newlines in error_report and other error functions (2016-01-13 15:16:19 +0100)
>
> ----------------------------------------------------------------
> Error reporting patches for 2016-01-13
>
> ----------------------------------------------------------------

Applied, thanks.

-- PMM