[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v3 00/15] Clean up use of error_printf()
From: |
Markus Armbruster |
Subject: |
[Qemu-devel] [PATCH v3 00/15] Clean up use of error_printf() |
Date: |
Wed, 17 Apr 2019 21:06:26 +0200 |
This series cleans up two kinds of error_printf() misuse:
* Errors and warnings should be reported with error_report() and
warn_report().
* Help output should be go to stdout, not stderr.
This is obviously for 4.1. If nobody objects, I'll take the whole
series through my tree.
Based-on: <address@hidden>
v3:
* PATCH 03: Replaced by new PATCH 14 [Paolo]
* PATCH 10: Trivial conflict with "[PATCH v7 0/2] log: Make glib
logging go through QEMU" resolved.
v2:
* PATCH 02: Commit message tweaked, indentation fixed [Eric]
* PATCH 10: Indentation fixed [Marcel]
* PATCH 11: Use-after-free fixed [Patchew]
* PATCH 12: test-util-sockets.c updated along with stubs/monitor.c
* PATCH 14: Commit message typo [Eric]
Markus Armbruster (15):
qemu-img: Use error_vreport() in error_exit()
block/ssh: Do not report read/write/flush errors to the user
loader-fit: Wean off error_printf()
mips/boston: Report errors with error_report(), not error_printf()
pci: Report fatal errors with error_report(), not error_printf()
hpet: Report warnings with warn_report(), not error_printf()
vfio: Report warnings with warn_report(), not error_printf()
s390x/kvm: Report warnings with warn_report(), not error_printf()
vl: Make -machine $TYPE,help and -accel help print to stdout
monitor error: Make printf()-like functions return a value
qemu-print: New qemu_printf(), qemu_vprintf() etc.
blockdev: Make -drive format=help print to stdout
char: Make -chardev help print to stdout
char-pty: Print "char device redirected" message to stdout
monitor: Simplify how -device/device_add print help
MAINTAINERS | 2 ++
block/ssh.c | 38 +++++++-------------
block/trace-events | 3 ++
blockdev.c | 9 ++---
chardev/char-pty.c | 5 +--
chardev/char.c | 3 +-
hw/core/loader-fit.c | 62 +++++++++++++++++++--------------
hw/mips/boston.c | 6 ++--
hw/pci/pci.c | 2 +-
hw/timer/hpet.c | 2 +-
hw/vfio/pci.c | 19 ++++++----
include/monitor/monitor.h | 7 ++--
include/qemu/error-report.h | 8 ++---
include/qemu/qemu-print.h | 19 ++++++++++
monitor.c | 69 ++++++++++++++++++-------------------
qdev-monitor.c | 36 ++++++++-----------
qemu-img.c | 6 ++--
stubs/error-printf.c | 13 ++++---
stubs/monitor.c | 5 +++
target/s390x/kvm.c | 2 +-
tests/test-util-sockets.c | 1 +
util/Makefile.objs | 1 +
util/qemu-error.c | 12 ++++---
util/qemu-print.c | 42 ++++++++++++++++++++++
vl.c | 10 +++---
25 files changed, 227 insertions(+), 155 deletions(-)
create mode 100644 include/qemu/qemu-print.h
create mode 100644 util/qemu-print.c
--
2.17.2
- [Qemu-devel] [PATCH v3 00/15] Clean up use of error_printf(),
Markus Armbruster <=
- [Qemu-devel] [PATCH v3 04/15] mips/boston: Report errors with error_report(), not error_printf(), Markus Armbruster, 2019/04/17
- [Qemu-devel] [PATCH v3 14/15] char-pty: Print "char device redirected" message to stdout, Markus Armbruster, 2019/04/17
- [Qemu-devel] [PATCH v3 09/15] vl: Make -machine $TYPE, help and -accel help print to stdout, Markus Armbruster, 2019/04/17
- [Qemu-devel] [PATCH v3 01/15] qemu-img: Use error_vreport() in error_exit(), Markus Armbruster, 2019/04/17
- [Qemu-devel] [PATCH v3 06/15] hpet: Report warnings with warn_report(), not error_printf(), Markus Armbruster, 2019/04/17
- [Qemu-devel] [PATCH v3 05/15] pci: Report fatal errors with error_report(), not error_printf(), Markus Armbruster, 2019/04/17
- [Qemu-devel] [PATCH v3 12/15] blockdev: Make -drive format=help print to stdout, Markus Armbruster, 2019/04/17
- [Qemu-devel] [PATCH v3 08/15] s390x/kvm: Report warnings with warn_report(), not error_printf(), Markus Armbruster, 2019/04/17
- [Qemu-devel] [PATCH v3 15/15] monitor: Simplify how -device/device_add print help, Markus Armbruster, 2019/04/17