[libvirt] [PATCH 00/11] qemu: Refactor the block job code

Peter Krempa pkrempa at redhat.com
Wed Apr 1 17:20:45 UTC 2015


At the expense of adding 113 lines of code, kill the ugly qemuBlockJobImpl
method and spread it's guts into separate functions.

This series additionally fixes a issue with failed drive pivot and the abort
function now returns errors if the returned event contained failure.

Peter Krempa (11):
  qemu: monitor: Extract handling of JSON block job error codes
  qemu: domain: Introduce helper to retrieve domain monitor object
  qemu: domain: Add helper to check block job support
  qemu: blockjob: Use the new helpers in qemuDomainGetBlockJobInfo
  qemu: blockjob: Split qemuDomainBlockJobSetSpeed from
    qemuDomainBlockJobImpl
  qemu: Clean up old leftovers in qemuMonitorDrivePivot
  qemu: blockPivot: Don't pause the VM any more since we don't use
    drive-reopen
  qemu: blockjob: Separate qemuDomainBlockJobAbort from
    qemuDomainBlockJobImpl
  qemu: blockPull: Refactor the rest of qemuDomainBlockJobImpl
  qemu: drivePivot: Fix assumption when 'block-job-complete' fails
  qemu: Refactor qemuDomainBlockJobAbort()

 src/qemu/qemu_domain.c       |  43 +++++
 src/qemu/qemu_domain.h       |   4 +
 src/qemu/qemu_driver.c       | 437 +++++++++++++++++++++----------------------
 src/qemu/qemu_migration.c    |   8 +-
 src/qemu/qemu_monitor.c      |  88 ++++++---
 src/qemu/qemu_monitor.h      |  35 ++--
 src/qemu/qemu_monitor_json.c | 188 +++++++++++--------
 src/qemu/qemu_monitor_json.h |  28 ++-
 tests/qemumonitorjsontest.c  |   2 +-
 9 files changed, 473 insertions(+), 360 deletions(-)

-- 
2.2.2




More information about the libvir-list mailing list