[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4 04/10] block: make 'top' argument to block-co
From: |
Jeff Cody |
Subject: |
Re: [Qemu-devel] [PATCH v4 04/10] block: make 'top' argument to block-commit optional |
Date: |
Wed, 4 Jun 2014 20:26:05 -0400 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Wed, Jun 04, 2014 at 02:59:44PM -0600, Eric Blake wrote:
> On 06/04/2014 07:51 AM, Jeff Cody wrote:
> > Now that active layer block-commit is supported, the 'top' argument
> > no longer needs to be mandatory.
> >
> > Change it to optional, with the default being the active layer in the
> > device chain.
> >
> > Reviewed-by: Eric Blake <address@hidden>
> > Reviewed-by: Benoit Canet <address@hidden>
> > Signed-off-by: Jeff Cody <address@hidden>
> > ---
>
> Unrelated to my review, but I wish we had done a better job at making
> the qemu 2.0 addition of active commit introspectible. Had we made
> _this_ patch at the same time, introspection would be possible by
> creating a dummy blockdev, then attempting a block-commit that omits the
> 'top' argument (since the error message for a missing required argument
> [old qemu] is different than the message for a blockdev that can't be
> committed [new qemu]). But since this change is in a different release
> than where we supported active commit, I'm stuck coming up with some
> reliable way to do a probe of whether active commit is supported so that
> libvirt knows whether to expose active commit on to the end user.
>
Well, I think what we _really_ need is true QAPI introspection.
Unfortunately, libvirt can't rely on the QEMU version to determine API
levels. This leaves it needing to do various QAPI contortions to
figure out what is supported and what is not supported, which seems
clunky and probably not sustainable long-term.
But I agree with the gist of what you saying, I think: libvirt and
QEMU should make sure to stay in the loop with each other on
command discoverability.
With that said, 2.1 soft freeze is not that far off; what other QAPI
commands might we be saying this about once 2.1 is out?
-Jeff
- [Qemu-devel] [PATCH v4 00/10] Modify block jobs to use node-names, Jeff Cody, 2014/06/04
- [Qemu-devel] [PATCH v4 02/10] block: add helper function to determine if a BDS is in a chain, Jeff Cody, 2014/06/04
- [Qemu-devel] [PATCH v4 01/10] block: Auto-generate node_names for each BDS entry, Jeff Cody, 2014/06/04
- [Qemu-devel] [PATCH v4 04/10] block: make 'top' argument to block-commit optional, Jeff Cody, 2014/06/04
- [Qemu-devel] [PATCH v4 03/10] block: simplify bdrv_find_base() and bdrv_find_overlay(), Jeff Cody, 2014/06/04
- [Qemu-devel] [PATCH v4 05/10] block: Accept node-name arguments for block-commit, Jeff Cody, 2014/06/04
- [Qemu-devel] [PATCH v4 06/10] block: extend block-commit to accept a string for the backing file, Jeff Cody, 2014/06/04
- [Qemu-devel] [PATCH v4 08/10] block: add backing-file option to block-stream, Jeff Cody, 2014/06/04
- [Qemu-devel] [PATCH v4 07/10] block: add ability for block-stream to use node-name, Jeff Cody, 2014/06/04
- [Qemu-devel] [PATCH v4 09/10] block: Add QMP documentation for block-stream, Jeff Cody, 2014/06/04