[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC][PATCH v2 00/11] QEMU Guest Agent: QMP-based host/
From: |
Jes Sorensen |
Subject: |
Re: [Qemu-devel] [RFC][PATCH v2 00/11] QEMU Guest Agent: QMP-based host/guest communication (virtagent) |
Date: |
Thu, 21 Apr 2011 11:46:45 +0200 |
User-agent: |
Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.15) Gecko/20110307 Fedora/3.1.9-0.39.b3pre.fc14 Thunderbird/3.1.9 |
On 04/18/11 17:02, Michael Roth wrote:
> These apply on top of Anthony's glib tree, commit
> 03d5927deb5e6baebaade1b4c8ff2428a85e125c currently, and can also be obtained
> from:
> git://repo.or.cz/qemu/mdroth.git qga_v2
>
> Patches 1-8 are general json/QAPI-related fixes. Anthony, please consider
> pulling these into your glib tree. The json fix-ups may need further
> evaluation, but I'm confident they're at least an improvement. The QAPI ones
> are mostly trivial fix-ups.
>
> Changes since V1:
>
> - Added guest agent worker thread to execute RPCs in the guest. With this in
> place we have a reliable timeout mechanism for hung commands, currently set
> at 30 seconds.
> - Add framework for registering init/cleanup routines for stateful RPCs to
> clean up after themselves after a timeout.
> - Added the following RPCs: guest-file-{open,close,read,write,seek},
> guest-shutdown, guest-info, and removed stubs for guest-view-file (now
> deprecated)
> - Added GUEST_AGENT_UP/GUEST_AGENT_DOWN QMP events
> - Switched to a TCP-style host-initiated 3-way handshake for channel
> negotiation, this simplifies client negotiation/interaction over the wire
> - Added configurable log level/log file/pid file options for guest agent
> - Various fixes for bugs/memory leaks and checkpatch.pl fixups
>
> ISSUES/TODOS:
>
> - Fix QMP proxy handling of error responses sent by guest agent, currently
> ignored
> - Add unit tests for guest agent wire protocol
> - Add unit tests for QMP interfaces
> - Add host-side timeout mechanism for async QMP commands
> - Return error for guest commands if guest up event has not yet been recieved
> - Make QMP param names more consistent between related commands
> - Clean up logging
>
Hi,
I had a look through this patchset and generally it looks pretty good.
There were a few nits, and I ignored all the python gibberish to avoid
getting a headache.
Did you do anything with the fsfreeze patches, or were they dropped in
the migration to qapi?
Cheers,
Jes
- Re: [Qemu-devel] [RFC][PATCH v2 15/17] guest agent: qemu-ga daemon, (continued)
- Re: [Qemu-devel] [RFC][PATCH v2 15/17] guest agent: qemu-ga daemon, Jes Sorensen, 2011/04/21
- Re: [Qemu-devel] [RFC][PATCH v2 15/17] guest agent: qemu-ga daemon, Michael Roth, 2011/04/22
- Re: [Qemu-devel] [RFC][PATCH v2 15/17] guest agent: qemu-ga daemon, Ian Molton, 2011/04/22
- Re: [Qemu-devel] [RFC][PATCH v2 15/17] guest agent: qemu-ga daemon, Jes Sorensen, 2011/04/22
- Re: [Qemu-devel] [RFC][PATCH v2 15/17] guest agent: qemu-ga daemon, Ian Molton, 2011/04/25
- Re: [Qemu-devel] [RFC][PATCH v2 15/17] guest agent: qemu-ga daemon, Jes Sorensen, 2011/04/26
[Qemu-devel] [RFC][PATCH v2 13/17] guest agent: command state class, Michael Roth, 2011/04/18
[Qemu-devel] [RFC][PATCH v2 14/17] guest agent: core marshal/dispatch interfaces, Michael Roth, 2011/04/18
[Qemu-devel] [RFC][PATCH v2 17/17] guest agent: build qemu-ga, add QEMU-wide gio dep, Michael Roth, 2011/04/18
[Qemu-devel] [RFC][PATCH v2 11/17] qmp proxy: build QEMU with qmp proxy, Michael Roth, 2011/04/18
Re: [Qemu-devel] [RFC][PATCH v2 00/11] QEMU Guest Agent: QMP-based host/guest communication (virtagent),
Jes Sorensen <=
Re: [Qemu-devel] [RFC][PATCH v2 00/11] QEMU Guest Agent: QMP-based host/guest communication (virtagent), Jes Sorensen, 2011/04/21