[libvirt] [PATCH 0/7] qemu: checkpoints: Collect most code in a single place

Peter Krempa pkrempa at redhat.com
Fri Sep 20 12:21:13 UTC 2019


The checkpoint code is quite complex and was dispersed in many places.
Refactor it to be in one new separate file.

I also plan to do the same to the snapshot code once this is dealt with.

Additionally aggregating all the code in one place will allow
refactoring and reuse in the incremental backup implementation.

Peter Krempa (7):
  qemu: Move, rename and export qemuDomObjFromDomain
  conf: Drop pointless 'domain' argument from
    virDomainCheckpointRedefinePrep
  conf: Drop pointless 'domain' argument from
    virDomainSnapshotRedefinePrep
  qemu: driver: Remove misplaced qemuDomainObjEndJob in
    qemuDomainCheckpointGetXMLDesc
  qemu: driver: Move checkpoint-related code to qemu_checkpoint.c
  qemu: domain: Move checkpoint related code to qemu_checkpoint.c
  qemu: driver: Don't pull in qemu_monitor_json.h directly

 src/conf/checkpoint_conf.c |   7 +-
 src/conf/checkpoint_conf.h |   3 +-
 src/conf/snapshot_conf.c   |   5 +-
 src/conf/snapshot_conf.h   |   3 +-
 src/qemu/Makefile.inc.am   |   2 +
 src/qemu/qemu_checkpoint.c | 646 ++++++++++++++++++++++++++++++++++
 src/qemu/qemu_checkpoint.h |  55 +++
 src/qemu/qemu_domain.c     | 193 ++---------
 src/qemu/qemu_domain.h     |  16 +-
 src/qemu/qemu_driver.c     | 686 ++++++++-----------------------------
 src/test/test_driver.c     |   4 +-
 11 files changed, 887 insertions(+), 733 deletions(-)
 create mode 100644 src/qemu/qemu_checkpoint.c
 create mode 100644 src/qemu/qemu_checkpoint.h

-- 
2.21.0




More information about the libvir-list mailing list