qemu: domain: Private data handling refactors

[libvirt] [PATCH 0/6] qemu: domain: Private data handling refactors

Peter Krempa posted 6 patches 6 years, 1 month ago

Download series mbox

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

Test syntax-check passed

src/qemu/qemu_domain.c | 218 ++++++++++++++++++++++++++++++-------------------
1 file changed, 134 insertions(+), 84 deletions(-)

Expand all Fold all

[libvirt] [PATCH 0/6] qemu: domain: Private data handling refactors

Posted by Peter Krempa 6 years, 1 month ago

Extracted from a huge work-in-progres series since these look
stand-alone.

Peter Krempa (6):
  qemu: Add qemu functions for storage source private data handling
  qemu: domain: Split out formating of Job data from private data
    formatter
  qemu: domain: Don't overwrite job type in private data
  qemu: domain: Return early in qemuDomainObjPrivateXMLFormatJob
  qemu: domain: Use virXMLFormatElement in
    qemuDomainObjPrivateXMLFormatJob
  qemu: domain: Extract parsing of job-related private XML

 src/qemu/qemu_domain.c | 218 ++++++++++++++++++++++++++++++-------------------
 1 file changed, 134 insertions(+), 84 deletions(-)

-- 
2.16.2

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

Re: [libvirt] [PATCH 0/6] qemu: domain: Private data handling refactors

Posted by John Ferlan 6 years, 1 month ago


On 03/01/2018 12:59 PM, Peter Krempa wrote:
> Extracted from a huge work-in-progres series since these look
> stand-alone.
> 
> Peter Krempa (6):
>   qemu: Add qemu functions for storage source private data handling
>   qemu: domain: Split out formating of Job data from private data
>     formatter
>   qemu: domain: Don't overwrite job type in private data
>   qemu: domain: Return early in qemuDomainObjPrivateXMLFormatJob
>   qemu: domain: Use virXMLFormatElement in
>     qemuDomainObjPrivateXMLFormatJob
>   qemu: domain: Extract parsing of job-related private XML
> 
>  src/qemu/qemu_domain.c | 218 ++++++++++++++++++++++++++++++-------------------
>  1 file changed, 134 insertions(+), 84 deletions(-)
> 

Series,

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

BTW: Your patch 1 has some similarities to Michal's patch 5 for the
persistent reservations.

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