Message ID | 20180815133747.25032-1-peterx@redhat.com |
---|---|
Headers | show |
Series | monitor: enable OOB by default | expand |
Peter Xu <peterx@redhat.com> writes: > Hi, Markus, Marc-Andre, > > I didn't follow up the latest discussions on the response queue or > monitor changes. For now this series still makes sense to me, so I'm > just rebasing the series to master and repost (with some additional > changes/patches to address Markus's concerns). Please let me know if > you have made any conclusion that I'm not aware of, or if this work > needs any further rebasing. > > v6 (previous version of the series might be incorrect; from this > version, I continued to use the old version counts): > - collect r-bs > - use Markus's version of build_params() [Markus] > - use \' instead of \" in Python codes [Markus] > - in qapi_event_send_*() still pass in &error_abort instead of NULL > [Markus] > - remove COMMAND_DROPPED from document too > - add missing pieces for test-qmp-cmds [Markus] > - add one patch to remove as much different code path for oob as > possible [Markus] > > TODO: > - flow control test is still missing; I still didn't consider it much, > also we possibly need to touch up monitor_puts() to use limited > sized buffer to complete the flow control of QMP channel > > Please review. Thanks, PATCH 01-03 queued. Thanks!