Re: [Qemu-devel] [PATCH 00/11] Sprint to the finish: purge QError
On Tue, 16 Jun 2015 15:08:41 +0200 Markus Armbruster wrote: > Luiz Capitulino writes: > > > On Sat, 13 Jun 2015 16:20:47 +0200 > > Markus Armbruster wrote: > > > >> After a bit over a year and many patches, QError is finally ripe. All > >> that's left of qerror.h after this series is a bunch of QERR_ macros. > >> Killing them is left for another day. > > > > Excellent! > > > > I did my best to review this series, but unfortunately my review is > > a bit weak because I'm not that familiar with some code paths > > (like QemuOpts) and I was unable to apply patches starting with patch 04/11. > > Did you apply on top of my "[PATCH v2 0/7] qdev: Mostly wean off > QError"? No, I assumed it was already merged. > > But it does look good to me: > > > > Reviewed-by: Luiz Capitulino > > Thanks! >
Re: [Qemu-devel] [PATCH 00/11] Sprint to the finish: purge QError
Luiz Capitulino writes: > On Sat, 13 Jun 2015 16:20:47 +0200 > Markus Armbruster wrote: > >> After a bit over a year and many patches, QError is finally ripe. All >> that's left of qerror.h after this series is a bunch of QERR_ macros. >> Killing them is left for another day. > > Excellent! > > I did my best to review this series, but unfortunately my review is > a bit weak because I'm not that familiar with some code paths > (like QemuOpts) and I was unable to apply patches starting with patch 04/11. Did you apply on top of my "[PATCH v2 0/7] qdev: Mostly wean off QError"? > But it does look good to me: > > Reviewed-by: Luiz Capitulino Thanks!
Re: [Qemu-devel] [PATCH 00/11] Sprint to the finish: purge QError
On Sat, 13 Jun 2015 16:20:47 +0200 Markus Armbruster wrote: > After a bit over a year and many patches, QError is finally ripe. All > that's left of qerror.h after this series is a bunch of QERR_ macros. > Killing them is left for another day. Excellent! I did my best to review this series, but unfortunately my review is a bit weak because I'm not that familiar with some code paths (like QemuOpts) and I was unable to apply patches starting with patch 04/11. But it does look good to me: Reviewed-by: Luiz Capitulino > > The diffstat looks a bit scary, but that's almost entirely due to > mechanical changes like PATCH 05. > > This series applies on top of my "[PATCH v2 0/7] qdev: Mostly wean off > QError". > > Markus Armbruster (11): > QemuOpts: Wean off qerror_report_err() > vl: Avoid qerror_report() outside QMP command handlers > vl: Use error_report() for --display errors > qerror: Eliminate QERR_DEVICE_NOT_FOUND > qerror: Clean up QERR_ macros to expand into a single string > tpm: Avoid qerror_report() outside QMP command handlers > qmp: Wean off qerror_report() > qerror: Finally unused, clean up > qerror: Move #include out of qerror.h > Include qapi/qmp/qerror.h exactly where needed > Include monitor/monitor.h exactly where needed > > audio/spiceaudio.c | 1 + > audio/wavcapture.c | 1 + > backends/hostmem.c | 1 - > backends/rng-egd.c | 11 +-- > backends/rng-random.c| 6 +- > backends/rng.c | 2 +- > backends/tpm.c | 2 +- > balloon.c| 5 +- > block.c | 4 +- > block/backup.c | 5 +- > block/commit.c | 3 +- > block/curl.c | 1 + > block/dmg.c | 1 + > block/io.c | 1 + > block/iscsi.c| 1 + > block/mirror.c | 9 +-- > block/qapi.c | 6 -- > block/qcow.c | 5 +- > block/qcow2-snapshot.c | 1 + > block/qcow2.c| 4 +- > block/qed.c | 4 +- > block/quorum.c | 5 +- > block/raw-posix.c| 2 + > block/snapshot.c | 5 +- > block/ssh.c | 2 + > block/stream.c | 5 +- > block/vhdx-log.c | 1 + > block/vmdk.c | 14 ++-- > block/vvfat.c| 1 + > blockdev-nbd.c | 6 +- > blockdev.c | 80 - > blockjob.c | 9 +-- > cpus.c | 9 +-- > dump.c | 13 ++-- > hmp.c| 24 --- > hw/9pfs/virtio-9p.c | 1 + > hw/char/serial-pci.c | 1 - > hw/char/virtio-serial-bus.c | 1 + > hw/core/nmi.c| 2 +- > hw/core/platform-bus.c | 1 - > hw/core/qdev-properties-system.c | 3 +- > hw/core/qdev-properties.c| 12 ++-- > hw/core/qdev.c | 9 +-- > hw/display/g364fb.c | 1 + > hw/display/qxl.c | 1 - > hw/display/tcx.c | 1 + > hw/dma/xilinx_axidma.c | 1 - > hw/i386/pc.c | 2 +- > hw/ide/ahci.c| 2 +- > hw/intc/openpic.c| 6 +- > hw/misc/ivshmem.c| 2 +- > hw/net/xilinx_axienet.c | 1 - > hw/pci/pci-stub.c| 3 +- > hw/pci/pci.c | 1 + > hw/pci/pcie.c| 1 - > hw/pci/shpc.c| 1 - > hw/ppc/spapr_pci.c | 8 +-- > hw/ppc/spapr_vio.c | 1 - > hw/ppc/virtex_ml507.c| 2 +- > hw/s390x/event-facility.c| 1 - > hw/s390x/s390-virtio-bus.c | 1 - > hw/s390x/s390-virtio.c | 4 +- > hw/s390x/virtio-ccw.c| 2 +- > hw/scsi/vhost-scsi.c | 1 + > hw/timer/hpet.c | 1 + > hw/tpm/tpm_passthrough.c | 1 + > hw/usb/bus.c | 1 + > hw/usb/ccid-card-emulated.c | 1 - > hw/usb/ccid-card-passthru.c | 2 +- > hw/usb/dev-network.c | 2 +- > hw/usb/dev-serial.c | 2 +- > hw/usb/dev-smartcard-reader.c| 1 - > hw/usb/dev-storage.c | 1 + > hw/usb/hcd-ehci.h| 1 - > hw/usb/host-libusb.c | 1 + > hw/usb/redirect.c| 9 +-- > hw/virtio/virtio-rng.c | 1 - > include/block/block_int.h| 2 - > include/monitor/monitor.h| 9 +-- > include/monitor/qdev.h | 5 +- > include/net/net.h| 2 +- > include/qapi/qmp/qerror.h| 85 +-