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