[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v4 0/7] Implement a warning_report function
From: |
Alistair Francis |
Subject: |
[Qemu-devel] [PATCH v4 0/7] Implement a warning_report function |
Date: |
Wed, 12 Jul 2017 06:57:27 -0700 |
QEMU currently has a standard method to report errors with
error_repot(). This ensure a sane and standard format when printing
errors. This series is attempting to extend this functionality for
warnings and information as well.
This patch renames error_print_loc() function to be more clear, but I
didn't bother renaming the others. It seems silly to change
error_printf() to error_warning_printf() and printf is already taken so
I just left it as is.
v4:
- Small improvements
- Remove the 'error: ' prefix
...
v1:
- Convert all of the existing warning messages in QEMU
- Add a error_report*_err() functions as well
RFCv3:
- Use more detailed enum and function names
- Add wrapper functions for the info and warning reporting
RFCv2:
- Use enums for ERROR, WARN and INFO with a generic report() function
instead of adding new functions
Alistair Francis (7):
util/qemu-error: Rename error_print_loc() to be more generic
error: Functions to report warnings and informational messages
Convert error_report() to warn_report()
hw/i386: Improve some of the warning messages
char-socket: Report TCP socket waiting as information
error: Implement the warn and free Error functions
Convert error_report*_err() to warn_report*_err()
block/backup.c | 12 ++---
block/gluster.c | 3 +-
block/iscsi.c | 6 +--
block/nfs.c | 12 ++---
block/rbd.c | 6 +--
block/ssh.c | 4 +-
blockdev.c | 2 +-
chardev/char-socket.c | 4 +-
cpus.c | 2 +-
hw/9pfs/9p.c | 2 +-
hw/arm/highbank.c | 6 +--
hw/arm/imx25_pdk.c | 6 +--
hw/arm/kzm.c | 6 +--
hw/core/machine.c | 10 ++--
hw/core/qdev-properties.c | 10 ++--
hw/i386/acpi-build.c | 15 ++++--
hw/i386/kvm/pci-assign.c | 6 +--
hw/i386/pc.c | 14 +++---
hw/i386/pc_piix.c | 8 ++--
hw/i386/pc_q35.c | 8 ++--
hw/misc/aspeed_sdmc.c | 8 ++--
hw/nvram/fw_cfg.c | 2 +-
hw/pci-host/piix.c | 2 +-
hw/ppc/pnv.c | 6 +--
hw/ppc/spapr.c | 4 +-
hw/ppc/spapr_iommu.c | 2 +-
hw/scsi/scsi-bus.c | 6 +--
hw/usb/dev-smartcard-reader.c | 6 +--
hw/usb/redirect.c | 2 +-
include/qapi/error.h | 11 +++++
include/qemu/error-report.h | 7 +++
net/tap-linux.c | 2 +-
scripts/checkpatch.pl | 7 ++-
target/i386/cpu.c | 22 ++++-----
target/i386/kvm.c | 12 ++---
target/s390x/cpu_models.c | 6 +--
target/s390x/kvm.c | 4 +-
tests/test-qdev-global-props.c | 6 +--
trace/control.c | 8 ++--
util/error.c | 20 ++++++++
util/qemu-error.c | 106 ++++++++++++++++++++++++++++++++++++++---
vl.c | 20 ++++----
42 files changed, 275 insertions(+), 136 deletions(-)
--
2.11.0
- [Qemu-devel] [PATCH v4 0/7] Implement a warning_report function,
Alistair Francis <=
- [Qemu-devel] [PATCH v4 1/7] util/qemu-error: Rename error_print_loc() to be more generic, Alistair Francis, 2017/07/12
- [Qemu-devel] [PATCH v4 2/7] error: Functions to report warnings and informational messages, Alistair Francis, 2017/07/12
- [Qemu-devel] [PATCH v4 4/7] hw/i386: Improve some of the warning messages, Alistair Francis, 2017/07/12
- [Qemu-devel] [PATCH v4 6/7] error: Implement the warn and free Error functions, Alistair Francis, 2017/07/12
- [Qemu-devel] [PATCH v4 5/7] char-socket: Report TCP socket waiting as information, Alistair Francis, 2017/07/12
- [Qemu-devel] [PATCH v4 3/7] Convert error_report() to warn_report(), Alistair Francis, 2017/07/12
- [Qemu-devel] [PATCH v4 7/7] Convert error_report*_err() to warn_report*_err(), Alistair Francis, 2017/07/12