[PATCH v2 00/15] Error message improvements

Markus Armbruster posted 15 patches 3 weeks, 1 day ago
Failed in applying to current master (apply log)
There is a newer version of this series
backends/cryptodev-lkcf.c   |  2 +-
backends/spdm-socket.c      |  4 ++--
backends/tpm/tpm_emulator.c | 13 +++++--------
block/blkdebug.c            |  2 +-
block/file-win32.c          |  2 +-
hw/9pfs/9p-local.c          |  2 +-
hw/9pfs/9p.c                |  3 +--
hw/acpi/core.c              |  5 ++---
hw/audio/es1370.c           |  2 +-
hw/core/loader.c            |  2 +-
hw/intc/openpic_kvm.c       |  3 +--
hw/intc/xics_kvm.c          |  5 +++--
hw/pci-host/xen_igd_pt.c    |  2 +-
hw/ppc/spapr.c              |  6 +++---
hw/remote/vfio-user-obj.c   | 18 +++++++++---------
hw/scsi/vhost-scsi.c        |  3 +--
hw/sensor/emc141x.c         |  4 ++--
hw/sensor/tmp421.c          |  4 ++--
hw/smbios/smbios.c          |  4 ++--
hw/usb/bus.c                |  5 ++---
hw/vfio/migration-multifd.c |  5 +++--
hw/virtio/vdpa-dev.c        |  4 ++--
hw/virtio/vhost-vsock.c     |  3 +--
migration/postcopy-ram.c    | 10 +++++-----
migration/rdma.c            |  3 +--
monitor/hmp-cmds-target.c   |  2 +-
net/dump.c                  |  2 +-
net/l2tpv3.c                |  6 ++----
net/slirp.c                 |  9 ++++++---
net/tap-bsd.c               |  6 +++---
net/tap-linux.c             |  2 +-
net/tap-solaris.c           |  6 +++---
net/tap.c                   |  3 +--
qga/commands-linux.c        | 11 ++++++-----
qga/commands-posix-ssh.c    | 23 +++++++++++++----------
qga/commands-win32.c        | 16 ++++++++--------
system/vl.c                 |  2 +-
target/i386/sev.c           |  6 ++----
target/ppc/kvm.c            |  5 ++---
target/riscv/kvm/kvm-cpu.c  | 11 ++++++-----
ui/gtk.c                    |  2 +-
ui/ui-qmp-cmds.c            |  4 +---
util/vfio-helpers.c         |  5 ++---
43 files changed, 114 insertions(+), 123 deletions(-)
[PATCH v2 00/15] Error message improvements
Posted by Markus Armbruster 3 weeks, 1 day ago
v2:
* PATCH 02: New, replacing old PATCH 02 [Dave]
* PATCH 03: New, replacing one of old PATCH 09's hunks
* PATCH 12: Fix patch splitting mishap [Dave]
* PATCH 13: Fix lost argument [Dave]

Markus Armbruster (15):
  error: Strip trailing '\n' from error string arguments (again)
  hw/usb: Convert to qemu_create() for a better error message
  ui: Convert to qemu_create() for simplicity and consistency
  tap-solaris: Use error_setg_file_open() for better error messages
  qga: Use error_setg_file_open() for better error messages
  hw/scsi: Use error_setg_file_open() for a better error message
  hw/virtio: Use error_setg_file_open() for a better error message
  net/tap: Use error_setg_file_open() for a better error message
  blkdebug: Use error_setg_file_open() for a better error message
  error: Use error_setg_file_open() for simplicity and consistency
  net/slirp: Improve file open error message
  error: Use error_setg_errno() to improve error messages
  error: Use error_setg_errno() for simplicity and consistency
  qga/commands-win32: Use error_setg_win32() for better error messages
  block/file-win32: Improve an error message

 backends/cryptodev-lkcf.c   |  2 +-
 backends/spdm-socket.c      |  4 ++--
 backends/tpm/tpm_emulator.c | 13 +++++--------
 block/blkdebug.c            |  2 +-
 block/file-win32.c          |  2 +-
 hw/9pfs/9p-local.c          |  2 +-
 hw/9pfs/9p.c                |  3 +--
 hw/acpi/core.c              |  5 ++---
 hw/audio/es1370.c           |  2 +-
 hw/core/loader.c            |  2 +-
 hw/intc/openpic_kvm.c       |  3 +--
 hw/intc/xics_kvm.c          |  5 +++--
 hw/pci-host/xen_igd_pt.c    |  2 +-
 hw/ppc/spapr.c              |  6 +++---
 hw/remote/vfio-user-obj.c   | 18 +++++++++---------
 hw/scsi/vhost-scsi.c        |  3 +--
 hw/sensor/emc141x.c         |  4 ++--
 hw/sensor/tmp421.c          |  4 ++--
 hw/smbios/smbios.c          |  4 ++--
 hw/usb/bus.c                |  5 ++---
 hw/vfio/migration-multifd.c |  5 +++--
 hw/virtio/vdpa-dev.c        |  4 ++--
 hw/virtio/vhost-vsock.c     |  3 +--
 migration/postcopy-ram.c    | 10 +++++-----
 migration/rdma.c            |  3 +--
 monitor/hmp-cmds-target.c   |  2 +-
 net/dump.c                  |  2 +-
 net/l2tpv3.c                |  6 ++----
 net/slirp.c                 |  9 ++++++---
 net/tap-bsd.c               |  6 +++---
 net/tap-linux.c             |  2 +-
 net/tap-solaris.c           |  6 +++---
 net/tap.c                   |  3 +--
 qga/commands-linux.c        | 11 ++++++-----
 qga/commands-posix-ssh.c    | 23 +++++++++++++----------
 qga/commands-win32.c        | 16 ++++++++--------
 system/vl.c                 |  2 +-
 target/i386/sev.c           |  6 ++----
 target/ppc/kvm.c            |  5 ++---
 target/riscv/kvm/kvm-cpu.c  | 11 ++++++-----
 ui/gtk.c                    |  2 +-
 ui/ui-qmp-cmds.c            |  4 +---
 util/vfio-helpers.c         |  5 ++---
 43 files changed, 114 insertions(+), 123 deletions(-)

-- 
2.49.0
Re: [PATCH v2 00/15] Error message improvements
Posted by Philippe Mathieu-Daudé 2 weeks, 3 days ago
On 21/11/25 13:14, Markus Armbruster wrote:

> Markus Armbruster (15):

>    hw/usb: Convert to qemu_create() for a better error message

>    hw/scsi: Use error_setg_file_open() for a better error message
>    hw/virtio: Use error_setg_file_open() for a better error message

Queued these 3, thanks.