mbox series

[0/5] bulk: Have object_child_foreach() take Error* and return boolean

Message ID 20230216122524.67212-1-philmd@linaro.org
Headers show
Series bulk: Have object_child_foreach() take Error* and return boolean | expand

Message

Philippe Mathieu-Daudé Feb. 16, 2023, 12:25 p.m. UTC 
The objective of this series is to allow object_child_foreach()
and object_child_foreach_recursive() propagate Error to their
callers.
Instead of adding temporary helpers with the new prototype, and
converting per subsystem, a bulk conversion is done as a single
patch... The changes aren't mechanical, but quite obvious.

Philippe Mathieu-Daudé (5):
  hw/nmi: Have nmi_monitor_handler() return a boolean indicating error
  spapr/ddw: Remove confuse return value in spapr_phb_get_free_liobn()
  bulk: Have object_child_foreach() take Error* and return boolean
  hw/nmi: Simplify nmi_monitor_handle() and do_nmi()
  hw/ppc/pnv_bmc: Simplify pnv_bmc_find()

 chardev/char-mux.c              |  6 ++--
 chardev/char.c                  |  6 ++--
 gdbstub/gdbstub.c               |  8 +++---
 hw/acpi/cxl.c                   |  7 +++--
 hw/acpi/viot.c                  |  6 ++--
 hw/arm/boot.c                   |  6 ++--
 hw/arm/virt-acpi-build.c        |  8 +++---
 hw/core/machine-qmp-cmds.c      | 18 ++++++------
 hw/core/nmi.c                   | 44 +++++------------------------
 hw/core/qdev.c                  |  8 ++++--
 hw/core/sysbus.c                | 10 +++----
 hw/cpu/cluster.c                |  7 +++--
 hw/display/virtio-gpu-udmabuf.c |  7 +++--
 hw/hppa/machine.c               |  3 +-
 hw/i386/acpi-build.c            | 16 +++++------
 hw/i386/intel_iommu.c           |  7 +++--
 hw/i386/pc.c                    | 13 ++++-----
 hw/i386/sgx.c                   |  7 ++---
 hw/i386/x86.c                   |  3 +-
 hw/intc/m68k_irqc.c             |  4 ++-
 hw/m68k/q800.c                  |  4 ++-
 hw/mem/memory-device.c          | 25 ++++++++---------
 hw/mem/pc-dimm.c                |  8 +++---
 hw/misc/macio/gpio.c            |  4 ++-
 hw/misc/mos6522.c               |  6 ++--
 hw/ppc/pnv.c                    | 18 ++++++------
 hw/ppc/pnv_bmc.c                | 28 ++++++++----------
 hw/ppc/pnv_xscom.c              |  6 ++--
 hw/ppc/spapr.c                  |  3 +-
 hw/ppc/spapr_pci.c              | 50 ++++++++++++++++-----------------
 hw/ppc/spapr_rtas_ddw.c         | 17 ++++++-----
 hw/rdma/vmw/pvrdma_main.c       |  7 +++--
 hw/s390x/s390-virtio-ccw.c      |  4 ++-
 hw/virtio/virtio-balloon.c      |  7 ++---
 include/hw/nmi.h                |  3 +-
 include/qom/object.h            | 28 ++++++++++--------
 iothread.c                      |  8 +++---
 monitor/hmp-cmds.c              |  6 ++--
 qom/object.c                    | 36 ++++++++++++------------
 qom/qom-hmp-cmds.c              |  7 +++--
 scsi/pr-manager.c               |  8 +++---
 softmmu/physmem.c               | 13 +++++----
 softmmu/qdev-monitor.c          |  9 +++---
 tests/qtest/fuzz/generic_fuzz.c |  6 ++--
 ui/dbus-chardev.c               | 14 ++++-----
 util/nvdimm-utils.c             |  8 +++---
 46 files changed, 256 insertions(+), 271 deletions(-)