On 12/04/09 12:48, Luiz Capitulino wrote:
I think we should create a new option for monitor configuration, like this:

    -mon mode={control,readline},chardev=<name>,more-options-here

So you'll create a qmp monitor socket this way:

    -chardev socket,path=/tmp/qmpsock,id=qmp,server,nowait
    -mon mode=control,chardev=qmp

Then the -monitor switch will be legacy/convinience syntax for a
readline monitor with auto-created chardev named 'monitor<nr>'.

  I like this.

  I believe it wouldn't be difficult for you to rebase your series
on top of staging with an incremental patch for QMP, right?

Rebasing to a moving target is insane. The qmp patches are committed already though, so this isn't a issue ;)

Right now I have some wip patches against current master:

   http://repo.or.cz/w/qemu/kraxel.git/shortlog/refs/heads/qmp  [1]

Not sure whenever I base this on the default series or the other way around. Either way it will be a bit messy.

cheers,
  Gerd

[1] The friendly URLs are new, arn't they?


Reply via email to