[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 2/3] qapi: nbd-export: allow select bitmaps by node/name p
From: |
Eric Blake |
Subject: |
Re: [PATCH v2 2/3] qapi: nbd-export: allow select bitmaps by node/name pair |
Date: |
Fri, 22 Apr 2022 15:04:48 -0500 |
User-agent: |
NeoMutt/20211029-35-db88c3 |
On Fri, Apr 08, 2022 at 11:27:42PM +0300, Vladimir Sementsov-Ogievskiy wrote:
> 17.03.2022 00:28, Eric Blake wrote:
> > > +++ b/qapi/block-export.json
> > > @@ -6,6 +6,7 @@
> > > ##
> > > { 'include': 'sockets.json' }
> > > +{ 'include': 'block-core.json' }
> > Hmm. Does this extra inclusion negatively impact qemu-storage-daemon,
> > since that is why we created block-export.json in the first place (to
> > minimize the stuff that qsd pulled in without needing all of
> > block-core.json)? In other words, would it be better to move
> > BlockDirtyBitmapOrStr to this file?
>
> Actually, looking at storage-daemon/qapi/qapi-schema.json I see
> block-cores.json.
>
> That's block.json which is not mentioned in
> storage-daemon/qapi/qapi-schema.json.
>
> So, I think it's OK to keep simple include for now.
We're early enough in the 7.1 cycle that if someone proposes a reason
why this would need to change, then we can adjust it.
So for now, I'm adding
Reviewed-by: Eric Blake <eblake@redhat.com>
and queuing this series through my NBD tree.
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3266
Virtualization: qemu.org | libvirt.org