qemu-devel
[Top][All Lists]
Advanced

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

Re: [Qemu-devel] [PATCH v7 00/15] qapi netdev_add introspection (post-in


From: Markus Armbruster
Subject: Re: [Qemu-devel] [PATCH v7 00/15] qapi netdev_add introspection (post-introspection cleanups subset F)
Date: Thu, 16 Jun 2016 16:57:40 +0200
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux)

Eric Blake <address@hidden> writes:

> The end is nearly in sight!  Last August, I started a process of
> cleaning up qapi on top of Markus' introspection work.  Several
> hundred emails later, and numerous intermediate series out of
> the way, I'm finally at the end of my queue of patches that were
> originally sent in v5:
> https://lists.gnu.org/archive/html/qemu-devel/2015-09/msg05410.html

So your QAPI queue is not in fact infinite?  ;-)

The series looks pretty close.  I'm confident we can get it into 2.7.

> v6 was here (yes, 5 months ago):
> https://www.mail-archive.com/address@hidden/msg343192.html
>
> Prerequisites:
> My clone/JSON visitor (weak; this series is mostly orthogonal to
> that one, if we want to swap the order of the two based on review
> complexity):
> https://www.mail-archive.com/qemu-devel%40nongnu.org/msg371555.html
>
> Once my series are in, we should focus on Marc-Andre's cleanups.

Yes.

> Ideas for further qapi work - which I have NOT coded yet:
> - convert more clients to use boxed commands
> - get rid of has_FOO when FOO is pointer
> - support default values (and get rid of has_FOO when FOO has default)
> - finally QAPI-fy device_add
> - consider swapping blockdev-add to use QAPI structs throughout the
> code-base, rather than converting back into flat QDict and using QemuOpts
> - introspection of command line options

Worthy projects.

I appreciate your patches very much, but if you keep making them faster
than I can review them, I'll get tired of QAPI.  I'd love to get around
to some serious development once in a while...



reply via email to

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