[Qemu-devel] [PATCH v2 0/6] Convert to realize and improve error handling

Mao Zhongyi posted 6 patches 6 years, 8 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/cover.1501827395.git.maozy.fnst@cn.fujitsu.com
Test FreeBSD passed
Test checkpatch passed
Test docker passed
Test s390x passed
There is a newer version of this series
hw/block/block.c                | 15 ++++---
hw/block/dataplane/virtio-blk.c | 12 +++---
hw/block/dataplane/virtio-blk.h |  2 +-
hw/block/fdc.c                  | 48 +++++++++------------
hw/block/nvme.c                 | 24 +++++------
hw/block/virtio-blk.c           | 17 +++-----
hw/ide/core.c                   |  7 +--
hw/ide/qdev.c                   | 94 +++++++++++++++++++----------------------
hw/scsi/scsi-disk.c             | 13 ++----
hw/usb/dev-storage.c            | 29 ++++++-------
include/hw/block/block.h        |  4 +-
include/hw/ide/internal.h       |  5 ++-
12 files changed, 125 insertions(+), 145 deletions(-)
[Qemu-devel] [PATCH v2 0/6] Convert to realize and improve error handling
Posted by Mao Zhongyi 6 years, 8 months ago
This series mainly implements the conversions of ide, floppy and nvme
device to realize. Add some error handling messages and remove the local
variable local_err, use errp to propagate the error directly. Also
fix the unusual function name.

v2:
  -use bool as the return type instead of int. [Markus Armbruster & Stefan Hajnoczi]

Cc: John Snow <jsnow@redhat.com>
Cc: Kevin Wolf <kwolf@redhat.com>
Cc: Max Reitz <mreitz@redhat.com>
Cc: Keith Busch <keith.busch@intel.com>
Cc: Stefan Hajnoczi <stefanha@redhat.com>
Cc: "Michael S. Tsirkin" <mst@redhat.com>
Cc: Paolo Bonzini <pbonzini@redhat.com>
Cc: Gerd Hoffmann <kraxel@redhat.com>
Cc: Markus Armbruster <armbru@redhat.com>

Mao Zhongyi (6):
  hw/ide: Convert DeviceClass init to realize
  hw/block/fdc: Convert to realize
  hw/block/nvme: Convert to realize
  hw/block: Fix the return type
  hw/block: Use errp directly rather than local_err
  dev-storage: Fix the unusual function name

 hw/block/block.c                | 15 ++++---
 hw/block/dataplane/virtio-blk.c | 12 +++---
 hw/block/dataplane/virtio-blk.h |  2 +-
 hw/block/fdc.c                  | 48 +++++++++------------
 hw/block/nvme.c                 | 24 +++++------
 hw/block/virtio-blk.c           | 17 +++-----
 hw/ide/core.c                   |  7 +--
 hw/ide/qdev.c                   | 94 +++++++++++++++++++----------------------
 hw/scsi/scsi-disk.c             | 13 ++----
 hw/usb/dev-storage.c            | 29 ++++++-------
 include/hw/block/block.h        |  4 +-
 include/hw/ide/internal.h       |  5 ++-
 12 files changed, 125 insertions(+), 145 deletions(-)

-- 
2.9.4