Re: [PATCH v3 00/18] migration: more bool+errp APIs

[Fabiano Rosas <farosas@suse.de>]

Vladimir Sementsov-Ogievskiy <vsementsov@yandex-team.ru> writes:

> v3:
> 01,02: add r-bs by Fabiano and Peter
> 03: improve comment, add r-b by Peter
> 04: add r-bs by Fabiano and Peter
> 05: new, replacement for dropped "[PATCH v2 05/16] migration: vmstate_save/load_state(): stop tracing errors"
> 06: rebase on kept trace-points, keep r-b by Peter
> 07-09: add r-b by Peter
> 10: rebase on kept trace-points, keep r-b by Peter
> 11-12: add r-b by Peter
> 13: rebase ion kept traces and logic around trace_vmstate_load_field_error() already refactored, fix indent, drop r-b
> 14: add r-b by Peter
> 15: add r-b by Peter, use "else { if () {} }"
> 16-17: new
> 18: rebased on dropped err_hint, drop r-b
>
> Vladimir Sementsov-Ogievskiy (18):
>   migration: vmstate_save_state_v: fix double error_setg
>   migration: make vmstate_save_state_v() static
>   migration: make .post_save() a void function
>   migration: vmstate_load_state(): add some newlines
>   migration: vmstate_save/load_state(): refactor tracing errors
>   migration: factor out vmstate_pre_save() from vmstate_save_state()
>   migration: factor out vmstate_save_field() from vmstate_save_state()
>   migration: factor out vmstate_pre_load() from vmstate_load_state()
>   migration: factor out vmstate_load_field() from vmstate_load_state()
>   migration: factor out vmstate_post_load() from vmstate_load_state()
>   migration: convert vmstate_subsection_save/load functions to bool
>   migration: VMStateInfo: introduce new handlers with errp
>   migration: introduce vmstate_load_vmsd() and vmstate_save_vmsd()
>   migration/cpr: move to new migration APIs
>   migration/savevm: move to new migration APIs
>   hw/s390x/css: drop use of .err_hint for vmstate
>   migration: drop VMStateField.err_hint
>   migration/vmstate-types: move to new migration APIs
>
>  docs/devel/migration/main.rst |   2 +-
>  hw/block/fdc.c                |   2 +-
>  hw/display/qxl.c              |   4 +-
>  hw/display/vga.c              |   2 +-
>  hw/display/virtio-gpu.c       |   2 +-
>  hw/display/vmware_vga.c       |   2 +-
>  hw/i386/vmmouse.c             |   2 +-
>  hw/ide/ahci.c                 |   2 +-
>  hw/intc/openpic.c             |   2 +-
>  hw/intc/spapr_xive.c          |   2 +-
>  hw/intc/xics.c                |   2 +-
>  hw/intc/xive.c                |   2 +-
>  hw/nvram/eeprom93xx.c         |   2 +-
>  hw/pci/pci.c                  |   2 +-
>  hw/pci/pcie_aer.c             |   2 +-
>  hw/ppc/spapr_iommu.c          |   2 +-
>  hw/ppc/spapr_pci.c            |   7 +-
>  hw/ppc/spapr_vio.c            |   4 +-
>  hw/s390x/css.c                |  15 +-
>  hw/usb/hcd-uhci.c             |   2 +-
>  include/migration/cpr.h       |   2 +-
>  include/migration/vmstate.h   |  73 ++--
>  migration/cpr.c               |  22 +-
>  migration/savevm.c            | 108 +++---
>  migration/trace-events        |  30 +-
>  migration/vmstate-types.c     | 642 +++++++++++++++++-----------------
>  migration/vmstate.c           | 354 +++++++++++--------
>  target/arm/machine.c          |   4 +-
>  target/ppc/machine.c          |   6 +-
>  29 files changed, 691 insertions(+), 612 deletions(-)

Queued, thanks!