From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from [140.186.70.92] (port=46147 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1PdgOh-0005cP-Pp for qemu-devel@nongnu.org; Fri, 14 Jan 2011 04:56:44 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1PdgOg-0000mF-AP for qemu-devel@nongnu.org; Fri, 14 Jan 2011 04:56:43 -0500 Received: from mail-gw0-f45.google.com ([74.125.83.45]:56736) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1PdgOg-0000m0-6L for qemu-devel@nongnu.org; Fri, 14 Jan 2011 04:56:42 -0500 Received: by gwaa12 with SMTP id a12so802700gwa.4 for ; Fri, 14 Jan 2011 01:56:41 -0800 (PST) Sender: Paolo Bonzini Message-ID: <4D301DD4.3060200@redhat.com> Date: Fri, 14 Jan 2011 10:56:36 +0100 From: Paolo Bonzini MIME-Version: 1.0 References: <1293108174-24895-1-git-send-email-pbonzini@redhat.com> In-Reply-To: <1293108174-24895-1-git-send-email-pbonzini@redhat.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Subject: [Qemu-devel] Re: [PATCH 0/8] make -qmp stdio usable List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org On 12/23/2010 01:42 PM, Paolo Bonzini wrote: > This patch series is a rewritten version of the patch to disable > raw mode on the terminal when "-qmp stdio" is given on the command-line. > > The rewritten series has several advantages, including working for > text consoles ("-qmp vc2") and especially working with -chardev/-mon > without the need to add special options. This is because in this > series it is _the monitor_ who asks to disable raw mode. > > Patch 2 adds a new CharDriverState method. Patches 3/4 implement > it for stdio, while patches 5/6 implement it for text consoles. > > Patches 1, 7 and 8 are related cleanups. > > Paolo Bonzini (8): > remove broken code for tty > add qemu_chr_set_echo > move atexit(term_exit) to qemu_chr_open_stdio > add set_echo implementation for qemu_chr_stdio > create TextConsole together with the CharDeviceState > add set_echo implementation for text consoles > remove text_console_opts > fix QemuOpts leak > > console.c | 81 +++++++++++++++++++++++++++++++++++----------------------- > monitor.c | 1 + > qemu-char.c | 62 +++++++++++++++++++++++--------------------- > qemu-char.h | 2 + > 5 files changed, 85 insertions(+), 63 deletions(-) Ping? Paolo