On Sun, 24 Jan 2010 08:01:28 -0600
Anthony Liguori wrote:
> On 01/24/2010 04:59 AM, Avi Kivity wrote:
> > On 01/22/2010 09:03 PM, Adam Litke wrote:
> >>
> >> +static void do_async_cmd_handler(Monitor *mon, const mon_cmd_t *cmd,
> >> + const QDict *params)
> >> +{
>
On 01/24/2010 04:01 PM, Anthony Liguori wrote:
Instead of sending opaques everywhere (and having them correspond to
different types in different cases), I would prefer it if the handle
always accepted an AsyncCommandCompletion *. That makes it easier to
follow the code, since there are no opaq
On 01/24/2010 04:59 AM, Avi Kivity wrote:
On 01/22/2010 09:03 PM, Adam Litke wrote:
+static void do_async_cmd_handler(Monitor *mon, const mon_cmd_t *cmd,
+ const QDict *params)
+{
+if (monitor_ctrl_mode(mon)) {
+cmd->mhandler.cmd_async(mon, params, qm
On 01/22/2010 09:03 PM, Adam Litke wrote:
+static void do_async_cmd_handler(Monitor *mon, const mon_cmd_t *cmd,
+ const QDict *params)
+{
+if (monitor_ctrl_mode(mon)) {
+cmd->mhandler.cmd_async(mon, params, qmp_monitor_complete, mon);
+} else {
+
Qemu has a number of commands that can operate asynchronously (savevm, migrate,
etc) and it will be getting more. For these commands, the user monitor needs
to be suspended, but QMP monitors could continue to to accept other commands.
This patch introduces a new command API that isolates the detai