[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: [RFC v0 00/15] QEMU Monitor Protocol
From: |
Avi Kivity |
Subject: |
[Qemu-devel] Re: [RFC v0 00/15] QEMU Monitor Protocol |
Date: |
Sun, 22 Nov 2009 11:41:02 +0200 |
User-agent: |
Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.4pre) Gecko/20090922 Fedora/3.0-3.9.b4.fc12 Thunderbird/3.0b4 |
On 11/19/2009 06:47 PM, Luiz Capitulino wrote:
Can you post a capture of a few monitor commands through the new protocol?
Here goes, it's a telnet session:
Looks really good, some comments below.
"""
{"QMP": {"capabilities": []}}
{ "execute": "info", "arguments": { "item": "balloon" } }
{"return": 1024}
1. I see no id attribute, but it's supported, yes?
2. I asked before for info commands to be separated into individual
commands ("query-balloon") when in machine mode. You wouldn't write a
function info(enum info_thing what), would you? What would its return
type be?
3. Quantities, for the machine protocol, should be in natural units (in
this case, bytes). The human interface can use kMGT and have some
reasonable default.
{ "execute": "info", "arguments": { "item": "balloon" } }
{"return": 512}
{ "execute": "info", "arguments": { "item": "network" } }
{"return": [{"devices": [{"name": "user.0", "info": "net=10.0.2.0, restricted=n"}, {"name": "e1000.0",
"info": "model=e1000,macaddr=52:54:00:12:34:56"}], "id": 0}]}
The internal "info" is very worrying. We need to make sure everything
is returned as an object without the need for additional parsing.
{ "execute": "migrate", "arguments": { "detach": "-d", "uri":
"tcp:localhost:4445" } }
{"error": {"class": "InvalidParameter", "data": {"parameter": "detach", "reason":
"must be an integer"}}}
I presume true and false would work here?
--
error compiling committee.c: too many arguments to function
- [Qemu-devel] [PATCH 08/15] QMP: Asynchronous events infrastructure, (continued)
- [Qemu-devel] [PATCH 08/15] QMP: Asynchronous events infrastructure, Luiz Capitulino, 2009/11/19
- [Qemu-devel] [PATCH 09/15] QMP: Introduce basic asynchronous events, Luiz Capitulino, 2009/11/19
- [Qemu-devel] [PATCH 10/15] QMP: Disable monitor print functions, Luiz Capitulino, 2009/11/19
- [Qemu-devel] [PATCH 11/15] QMP: Introduce README file, Luiz Capitulino, 2009/11/19
- [Qemu-devel] [PATCH 12/15] QMP: Introduce specification, Luiz Capitulino, 2009/11/19
- [Qemu-devel] [PATCH 13/15] QMP: Introduce qmp-events.txt, Luiz Capitulino, 2009/11/19
- [Qemu-devel] [PATCH 14/15] QMP: Introduce qmp-shell, Luiz Capitulino, 2009/11/19
- [Qemu-devel] [PATCH 15/15] QMP: Introduce vm-info, Luiz Capitulino, 2009/11/19
- [Qemu-devel] Re: [RFC v0 00/15] QEMU Monitor Protocol, Avi Kivity, 2009/11/19
Re: [Qemu-devel] [RFC v0 00/15] QEMU Monitor Protocol, Luiz Capitulino, 2009/11/19