Re: [Qemu-devel] [PATCH 2/2] qmp: revert "add set_echo implementation for qemu_chr_stdio"

[Eric Blake]

On 05/30/2012 04:01 AM, Pavel Hrdina wrote:
> This reverts commit bb002513a9bd2bff169c3d431a8f00c5b2e3aa99 because this 
> code is not used
> in order that we use readline mode for '-qmp stdio'.
> 
> Conflicts:
> 
>       qemu-char.c

This is evidence that you've rebased locally, but as none of us know
what the patch was like before you resolved your rebase conflicts, you
should drop this portion of the commit message before posting upstream.
 Conflict designations are primarily useful on public backporting
branches, rather than on initial submissions for upstream.

As to the series itself, will readline mode add overhead?  Libvirt wants
to use QMP monitor without the overhead of remembering issued commands,
and if the difference for turning on readline is significant, then it
needs to be an opt-in option and not the default.

-- 
Eric Blake   address@hidden    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

signature.asc
Description: OpenPGP digital signature