I had a pair of patches making save_snapshot/load_snapshot() return a boolean (like Markus recent qdev/QOM cleanup), then realized Daniel already has series changing migration/, so I rebased my patches in the first part of his v6 series: "migration: bring improved savevm/loadvm/delvm to QMP" https://lists.gnu.org/archive/html/qemu-devel/2020-10/msg02158.html
I included patch #1/#3 from Daniel, #2 is my first patch, and my second patch is squashed with Daniel's #3. Daniel if you find these patches worthwhile, please consider them while merging your series (or respining). Regards, Phil. Daniel P. Berrangé (2): block: push error reporting into bdrv_all_*_snapshot functions migration: stop returning errno from load_snapshot() Philippe Mathieu-Daudé (1): migration: Make save_snapshot() return bool, not 0/-1 include/block/snapshot.h | 14 +++---- include/migration/snapshot.h | 18 +++++++- block/monitor/block-hmp-cmds.c | 7 ++-- block/snapshot.c | 77 +++++++++++++++++----------------- migration/savevm.c | 72 ++++++++++++------------------- monitor/hmp-cmds.c | 9 +--- replay/replay-debugging.c | 6 +-- replay/replay-snapshot.c | 4 +- softmmu/vl.c | 2 +- tests/qemu-iotests/267.out | 10 ++--- 10 files changed, 105 insertions(+), 114 deletions(-) -- 2.26.2
