On 09/18/2017 10:05 AM, Mao Zhongyi wrote: > 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. > > v3: > -patch1: update the test output in 051 [John Snow] > -patch2: update the test putput in 172 [John Snow] > > v2: > -use bool as the return type instead of int. [Markus Armbruster & Stefan > Hajnoczi] > > Cc: John Snow <js...@redhat.com> > Cc: Kevin Wolf <kw...@redhat.com> > Cc: Max Reitz <mre...@redhat.com> > Cc: Keith Busch <keith.bu...@intel.com> > Cc: Stefan Hajnoczi <stefa...@redhat.com> > Cc: "Michael S. Tsirkin" <m...@redhat.com> > Cc: Paolo Bonzini <pbonz...@redhat.com> > Cc: Gerd Hoffmann <kra...@redhat.com> > Cc: Markus Armbruster <arm...@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 ++- > tests/qemu-iotests/051.pc.out | 10 +---- > tests/qemu-iotests/172.out | 8 ---- > 14 files changed, 127 insertions(+), 161 deletions(-) >
Hi Mao Zhongyi: I've staged patches one and two here for my IDE pull request. I think patches 3-6 here would belong to Kevin. Thanks, John