Misc QGA/monitor QMP improvements (resend)

[Qemu-devel] [PATCH 0/2] Misc QGA/monitor QMP improvements (resend)

Marc-André Lureau posted 2 patches 6 years, 8 months ago

Download series mbox

Test asan passed

Test docker-mingw@fedora passed

Test docker-clang@ubuntu failed

Test checkpatch passed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20190220154253.26982-1-marcandre.lureau@redhat.com

Maintainers: Markus Armbruster <armbru@redhat.com>, Michael Roth <mdroth@linux.vnet.ibm.com>, "Dr. David Alan Gilbert" <dgilbert@redhat.com>

monitor.c           | 33 ++++++++++++-------------------
qapi/qmp-dispatch.c | 10 ++++++++--
qga/main.c          | 47 +++++++++------------------------------------
tests/test-qga.c    | 13 +++++--------
4 files changed, 34 insertions(+), 69 deletions(-)

Expand all Fold all

[Qemu-devel] [PATCH 0/2] Misc QGA/monitor QMP improvements (resend)

Posted by Marc-André Lureau 6 years, 8 months ago

Hi,

I have those 2 simplification patches left which were part of previous
series. They are still worthwhile imho.

Thanks

Marc-André Lureau (2):
  qga: process_event() simplification
  qmp: common 'id' handling & make QGA conform to QMP spec

 monitor.c           | 33 ++++++++++++-------------------
 qapi/qmp-dispatch.c | 10 ++++++++--
 qga/main.c          | 47 +++++++++------------------------------------
 tests/test-qga.c    | 13 +++++--------
 4 files changed, 34 insertions(+), 69 deletions(-)

-- 
2.21.0.rc1

Re: [Qemu-devel] [PATCH 0/2] Misc QGA/monitor QMP improvements (resend)

Posted by Markus Armbruster 6 years, 8 months ago

Marc-André Lureau <marcandre.lureau@redhat.com> writes:

> Hi,
>
> I have those 2 simplification patches left which were part of previous
> series. They are still worthwhile imho.

They are.

> Thanks
>
> Marc-André Lureau (2):
>   qga: process_event() simplification
>   qmp: common 'id' handling & make QGA conform to QMP spec

PATCH 1 is qga only.  PATCH 2 simplifies the common monitor core, but
observable change is again qga only.  Mike, care to take these patches
through your tree?