[libvirt] [PATCH 8/8] qemu: monitor: Separate probing for active block commit

Peter Krempa pkrempa at redhat.com
Fri Aug 24 11:50:46 UTC 2018


On Thu, Aug 23, 2018 at 18:11:09 -0400, John Ferlan wrote:
> 
> 
> On 08/15/2018 07:52 AM, Peter Krempa wrote:
> > Extract the code used to probe for the functionality so that it does not
> > litter the code used for actual work.
> > 
> > Signed-off-by: Peter Krempa <pkrempa at redhat.com>
> > ---
> >  src/qemu/qemu_monitor.c      |  2 +-
> >  src/qemu/qemu_monitor_json.c | 58 ++++++++++++++++++++++++++------------------
> >  src/qemu/qemu_monitor_json.h |  3 +++
> >  3 files changed, 39 insertions(+), 24 deletions(-)
> > 
> 
> This one somehow feels related to the other series I reviewed where the
> *top was or wasn't present to determine whether capability was there,
> but I see that it'd still be useful - just the irony of it all.

Your feelings are spot on. I was doing this commit when I realized
that the detection is pointless at this point and cleaned it up right
away so that we don't have to carry it around forever.

The removal of the detection is orthogonal to this cleanup though and
since we can't delete the detection code completely at this point thus
the other patches were sent in a separate series.

> 
> Reviewed-by: John Ferlan <jferlan at redhat.com>
> 
> John
> 
> 
> > diff --git a/src/qemu/qemu_monitor.c b/src/qemu/qemu_monitor.c
> > index 9bc7aa9ed1..a60e78d967 100644
> > --- a/src/qemu/qemu_monitor.c
> > +++ b/src/qemu/qemu_monitor.c
> 
> NB: My gitk view shows a few lines above here the call to
> qemuMonitorJSONBlockCommit has a second row of args that are not aligned
> properly...  Trivialities.

That will be fixed later. I'm adding few new args for block-commit.

That was not published yet though.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20180824/3d688842/attachment-0001.sig>


More information about the libvir-list mailing list