Cleanups around error_propagate() and ERRP_GUARD()

[PATCH 00/10] Cleanups around error_propagate() and ERRP_GUARD()

Markus Armbruster posted 10 patches 1 year, 5 months ago

Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20221121085054.683122-1-armbru@redhat.com

Maintainers: Paolo Bonzini <pbonzini@redhat.com>, Kevin Wolf <kwolf@redhat.com>, Hanna Reitz <hreitz@redhat.com>, John Snow <jsnow@redhat.com>, Vladimir Sementsov-Ogievskiy <vsementsov@yandex-team.ru>, "Marc-André Lureau" <marcandre.lureau@redhat.com>, Peter Maydell <peter.maydell@linaro.org>, Eduardo Habkost <eduardo@habkost.net>, Marcel Apfelbaum <marcel.apfelbaum@gmail.com>, "Philippe Mathieu-Daudé" <philmd@linaro.org>, Yanan Wang <wangyanan55@huawei.com>, "Daniel P. Berrangé" <berrange@redhat.com>, "Maciej S. Szmigiero" <maciej.szmigiero@oracle.com>, Richard Henderson <richard.henderson@linaro.org>, "Michael S. Tsirkin" <mst@redhat.com>, Elena Ufimtseva <elena.ufimtseva@oracle.com>, Jagannathan Raman <jag.raman@oracle.com>, John G Johnson <john.g.johnson@oracle.com>, David Hildenbrand <david@redhat.com>, Markus Armbruster <armbru@redhat.com>, "Dr. David Alan Gilbert" <dgilbert@redhat.com>, Zhang Chen <chen.zhang@intel.com>, Li Zhijian <lizhijian@fujitsu.com>, Jason Wang <jasowang@redhat.com>, Konstantin Kostiuk <kkostiuk@redhat.com>, Michael Roth <michael.roth@amd.com>, Marcelo Tosatti <mtosatti@redhat.com>, Gerd Hoffmann <kraxel@redhat.com>

include/io/channel.h             |  2 +-
include/qemu/config-file.h       |  2 +-
accel/kvm/kvm-all.c              |  5 +---
block/blkdebug.c                 |  4 +--
block/copy-before-write.c        |  1 -
dump/dump.c                      |  2 --
hw/arm/armsse.c                  |  3 +--
hw/arm/virt.c                    | 14 ++++------
hw/core/machine.c                |  3 +--
hw/core/qdev-properties-system.c |  5 +---
hw/core/qdev.c                   |  2 --
hw/hyperv/vmbus.c                |  8 +++---
hw/i386/pc.c                     |  5 +---
hw/pci/msi.c                     |  1 -
hw/remote/vfio-user-obj.c        |  1 -
hw/virtio/vhost-vdpa.c           |  2 +-
hw/virtio/virtio-balloon.c       | 20 +++++++-------
hw/virtio/virtio-mem.c           | 10 ++-----
iothread.c                       |  2 +-
monitor/misc.c                   | 14 +++-------
monitor/monitor.c                | 12 +++------
monitor/qmp-cmds.c               |  4 +--
net/colo-compare.c               | 13 +++------
qga/commands-win32.c             |  8 +++---
target/i386/kvm/kvm.c            |  5 +---
ui/util.c                        |  1 -
util/qemu-config.c               | 46 +++++++++++++++-----------------
util/qemu-sockets.c              | 26 +++++++-----------
util/thread-context.c            | 10 ++-----
29 files changed, 80 insertions(+), 151 deletions(-)

Expand all Fold all

[PATCH 00/10] Cleanups around error_propagate() and ERRP_GUARD()

Posted by Markus Armbruster 1 year, 5 months ago

Markus Armbruster (10):
  error: Drop some obviously superfluous error_propagate()
  error: Drop a few superfluous ERRP_GUARD()
  error: Move ERRP_GUARD() to the beginning of the function
  monitor: Simplify monitor_fd_param()'s error handling
  monitor: Use ERRP_GUARD() in monitor_init()
  qemu-config: Make config_parse_qdict() return bool
  qemu-config: Use ERRP_GUARD() where obviously appropriate
  sockets: Use ERRP_GUARD() where obviously appropriate
  qapi: Use returned bool to check for failure (again)
  io: Tidy up fat-fingered parameter name

 include/io/channel.h             |  2 +-
 include/qemu/config-file.h       |  2 +-
 accel/kvm/kvm-all.c              |  5 +---
 block/blkdebug.c                 |  4 +--
 block/copy-before-write.c        |  1 -
 dump/dump.c                      |  2 --
 hw/arm/armsse.c                  |  3 +--
 hw/arm/virt.c                    | 14 ++++------
 hw/core/machine.c                |  3 +--
 hw/core/qdev-properties-system.c |  5 +---
 hw/core/qdev.c                   |  2 --
 hw/hyperv/vmbus.c                |  8 +++---
 hw/i386/pc.c                     |  5 +---
 hw/pci/msi.c                     |  1 -
 hw/remote/vfio-user-obj.c        |  1 -
 hw/virtio/vhost-vdpa.c           |  2 +-
 hw/virtio/virtio-balloon.c       | 20 +++++++-------
 hw/virtio/virtio-mem.c           | 10 ++-----
 iothread.c                       |  2 +-
 monitor/misc.c                   | 14 +++-------
 monitor/monitor.c                | 12 +++------
 monitor/qmp-cmds.c               |  4 +--
 net/colo-compare.c               | 13 +++------
 qga/commands-win32.c             |  8 +++---
 target/i386/kvm/kvm.c            |  5 +---
 ui/util.c                        |  1 -
 util/qemu-config.c               | 46 +++++++++++++++-----------------
 util/qemu-sockets.c              | 26 +++++++-----------
 util/thread-context.c            | 10 ++-----
 29 files changed, 80 insertions(+), 151 deletions(-)

-- 
2.37.3