qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH v4 00/28] Add qapi-to-JSON and clone visitors


From: Markus Armbruster
Subject: Re: [Qemu-devel] [PATCH v4 00/28] Add qapi-to-JSON and clone visitors
Date: Mon, 13 Jun 2016 10:26:17 +0200
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux)

Eric Blake <address@hidden> writes:

> On 06/03/2016 06:09 AM, Markus Armbruster wrote:
>> Most patches are ready as is or with minor touch-ups.
>> 
>> I'd probably drop PATCH 01.  The file name collision no longer exists,
>> and moving files just to improve their names a bit doesn't seem worth
>> the bother.
>
> It's a bit more churn to the rest of the series, but I can live with it.
> It's also more related to the JSON visitor than the clone visitor, so
> even if I do want to keep it, it makes more sense to keep it with the
> second half of the series.

Thanks.

>> There are memory leaks in PATCH 12 and 24 (see review of PATCH 24).
>> 
>> PATCH 13 needs more work than I'm comfortable to do on commit.  Mostly
>> missing comment updates, but also a crash bug.
>
> I've got that fix ready to post.
>
>> 
>> On PATCH 21, I have minor interface design doubts.
>> 
>> The only difficult question is what to do about invalid UTF-8 strings
>> and non-finite numbers.  Patches before PATCH 29 show opinions, but a
>> decision isn't made until PATCH 29.  Good, because it leaves most of
>> this series not blocked by the debate on what we should do.
>> 
>> I guess I could take PATCH 02-12 now, and let you respin the rest.  But
>> perhaps its easier if you respin all of them.
>
> I'm posting v5 of 2-15 now, then focusing on your comments on the JSON
> visitor for a later posting.

Posted as "[PATCH v5 00/15] Add clone visitor".  Will review as soon as
I can.

> I also think my subset F series (adding a 'box' parameter) is relatively
> orthogonal, if you want to start reviewing that:
> https://lists.gnu.org/archive/html/qemu-devel/2016-05/msg03569.html
>
> I'd still LOVE to get blockdev-add working for ALL block devices before
> 2.7 soft freeze (there's patches for NBD that I need to review, and I
> know the gluster folks were waiting on qapi patches to be able to use a
> SocketAddress union as the basis for the gluster branch of the
> blockdev-add union - and my qapi patches are almost all the way there to
> permitting it).

I'd very much like to finish this huge task, too.  Let's try.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]