[00/14] ppc/spapr: Error handling fixes and cleanups

Message

Greg Kurz Aug. 10, 2020, 4:53 p.m. UTC

This series aims to be a follow up to Markus Ambruster's massive
work to sanitize error handling. It focuses on the following:
- check return values rather than a pointer to a local Error object
- get rid of the local_err + error_propagate() boilerplate when
  possible
- propage negative errnos to vmstate instead of -1

---

Greg Kurz (14):
      spapr: Simplify error handling in spapr_phb_realize()
      spapr/xive: Rework error handling of kvmppc_xive_cpu_connect()
      spapr/xive: Rework error handling of kvmppc_xive_source_reset()
      spapr/xive: Rework error handling of kvmppc_xive_mmap()
      spapr/xive: Rework error handling of kvmppc_xive_cpu_[gs]et_state()
      spapr/xive: Rework error handling of kvmppc_xive_[gs]et_queue_config()
      spapr/xive: Rework error handling in kvmppc_xive_get_queues()
      spapr/xive: Rework error handling of kvmppc_xive_set_source_config()
      spapr/kvm: Fix error handling in kvmppc_xive_pre_save()
      spapr/xive: Fix error handling in kvmppc_xive_post_load()
      ppc/xive: Fix error handling in vmstate_xive_tctx_*() callbacks
      spapr/xive: Simplify error handling in kvmppc_xive_connect()
      ppc/xive: Simplify error handling in xive_tctx_realize()
      spapr/xive: Simplify error handling of kvmppc_xive_cpu_synchronize_state()


 hw/intc/spapr_xive_kvm.c    |  202 ++++++++++++++++++++++---------------------
 hw/intc/xive.c              |   19 ++--
 hw/ppc/spapr_pci.c          |   16 +--
 include/hw/ppc/spapr_xive.h |    8 +-
 include/hw/ppc/xive.h       |    8 +-
 5 files changed, 125 insertions(+), 128 deletions(-)

--
Greg