[v2] qemu: track presence of blockjobs in status XML (blockdev-add saga)

[libvirt] [PATCH v2 0/4] qemu: track presence of blockjobs in status XML (blockdev-add saga)

Peter Krempa posted 4 patches 6 years, 6 months ago

Diff against v1
Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/cover.1507205097.git.pkrempa@redhat.com

src/libvirt_private.syms |  2 ++
src/qemu/qemu_domain.c   | 38 ++++++++++++++++++++++++++++++++++++++
src/qemu/qemu_domain.h   |  3 +++
src/qemu/qemu_process.c  | 22 +++++++++++++---------
src/util/virutil.c       | 20 ++++++++++++++++++++
src/util/virutil.h       |  3 +++
tests/qemuxml2xmltest.c  | 44 ++++++++++++++++++++++++++++++++++++++++++++
7 files changed, 123 insertions(+), 9 deletions(-)

Expand all Fold all

[libvirt] [PATCH v2 0/4] qemu: track presence of blockjobs in status XML (blockdev-add saga)

Posted by Peter Krempa 6 years, 6 months ago

V2:

Patches 1/4 and 3/4 are new. Patch 2/4 and 4/4 were fixed to use
virTristateBool.

Peter Krempa (4):
  util: Add functions to simplify bool->virTristate(Bool|Switch)
    assignment
  qemu: domain: Mark if no blockjobs are active in the status XML
  qemu: process: Simplify acces to individual disk when reconnecting
  qemu: process: Don't redetect backing chain on reconnect

 src/libvirt_private.syms |  2 ++
 src/qemu/qemu_domain.c   | 38 ++++++++++++++++++++++++++++++++++++++
 src/qemu/qemu_domain.h   |  3 +++
 src/qemu/qemu_process.c  | 22 +++++++++++++---------
 src/util/virutil.c       | 20 ++++++++++++++++++++
 src/util/virutil.h       |  3 +++
 tests/qemuxml2xmltest.c  | 44 ++++++++++++++++++++++++++++++++++++++++++++
 7 files changed, 123 insertions(+), 9 deletions(-)

-- 
2.14.1

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

Re: [libvirt] [PATCH v2 0/4] qemu: track presence of blockjobs in status XML (blockdev-add saga)

Posted by John Ferlan 6 years, 6 months ago


On 10/05/2017 08:06 AM, Peter Krempa wrote:
> V2:
> 
> Patches 1/4 and 3/4 are new. Patch 2/4 and 4/4 were fixed to use
> virTristateBool.
> 
> Peter Krempa (4):
>   util: Add functions to simplify bool->virTristate(Bool|Switch)
>     assignment
>   qemu: domain: Mark if no blockjobs are active in the status XML
>   qemu: process: Simplify acces to individual disk when reconnecting

^^
s/acces/access

>   qemu: process: Don't redetect backing chain on reconnect
> 
>  src/libvirt_private.syms |  2 ++
>  src/qemu/qemu_domain.c   | 38 ++++++++++++++++++++++++++++++++++++++
>  src/qemu/qemu_domain.h   |  3 +++
>  src/qemu/qemu_process.c  | 22 +++++++++++++---------
>  src/util/virutil.c       | 20 ++++++++++++++++++++
>  src/util/virutil.h       |  3 +++
>  tests/qemuxml2xmltest.c  | 44 ++++++++++++++++++++++++++++++++++++++++++++
>  7 files changed, 123 insertions(+), 9 deletions(-)
> 

Reviewed-by: John Ferlan <jferlan@redhat.com>

(series)

John

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list