From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:47215) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TGcY3-0008WQ-Rs for qemu-devel@nongnu.org; Tue, 25 Sep 2012 17:20:09 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1TGcY0-0003UP-88 for qemu-devel@nongnu.org; Tue, 25 Sep 2012 17:20:07 -0400 Received: from mail-oa0-f45.google.com ([209.85.219.45]:33717) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TGcY0-0003SN-1F for qemu-devel@nongnu.org; Tue, 25 Sep 2012 17:20:04 -0400 Received: by oagi18 with SMTP id i18so2347809oag.4 for ; Tue, 25 Sep 2012 14:20:03 -0700 (PDT) From: Anthony Liguori In-Reply-To: <1348584532-21914-1-git-send-email-mjt@msgid.tls.msk.ru> References: <1348584532-21914-1-git-send-email-mjt@msgid.tls.msk.ru> Date: Tue, 25 Sep 2012 16:19:59 -0500 Message-ID: <876271ol4w.fsf@codemonkey.ws> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Subject: Re: [Qemu-devel] [PATCH v2] stop using stdio for monitor/serial/etc with -daemonize List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Michael Tokarev , qemu-devel@nongnu.org Cc: Peter Maydell , Stefan Hajnoczi Michael Tokarev writes: > Current code binds monitor and serial port to the guest console > unless -nographic is specified, which is okay. But when there's > no guest console (-nographic), the code tries to use stdio for > the same default devices. But it does not check for -daemonize > at the same time -- because when -daemonize is given, there's no > point at using stdin since it will be closed down the line. > However, when serial port is attached to stdin, tty control > modes are changed (switching tty to raw mode), and qemu will > switch to background, leaving the tty in bad state. > > Take -daemonize into account too, when assigning default devices, > and for -nographic -daemonize case, assign them to "null" instead. Combining -nographic and -daemonize don't make sense. I'd rather error out with this combination. I think what the user is after is -daemonize -vga none OR -daemonize -display none. Regards, Anthony Liguori > > This is https://bugs.launchpad.net/qemu/+bug/1024275 > or http://bugs.debian.org/549195 . > > While at it, reformat this code a bit to be less of a maze of > ifs and use a variable to hold common target of devices. > > This patch depends on another patch, 995ee2bf469de6, > "curses: don't initialize curses when qemu is daemonized", > by Hitoshi Mitake, which creates is_daemonized() routine. > > Signed-off-by: Michael Tokarev > --- > vl.c | 45 +++++++++++++++++++++++++++------------------ > 1 file changed, 27 insertions(+), 18 deletions(-) > > diff --git a/vl.c b/vl.c > index 48049ef..a210ff9 100644 > --- a/vl.c > +++ b/vl.c > @@ -3392,30 +3392,39 @@ int main(int argc, char **argv, char **envp) > default_sdcard = 0; > } > > - if (display_type == DT_NOGRAPHIC) { > - if (default_parallel) > - add_device_config(DEV_PARALLEL, "null"); > + /* Create default monitor, serial, parallel and virtcon devices. */ > + /* reuse optarg variable */ > + optarg = NULL; > + if (display_type != DT_NOGRAPHIC) { > + /* regular case, all devices directed to the guest console */ > + optarg = "vc:80Cx24C"; > + } else if (is_daemonized()) { > + /* nographic and daemonize, everything => null */ > + optarg = "null"; > + } else { > + /* nographic and no daemonize */ > + /* can't have both serial and virtcon on stdio */ > if (default_serial && default_monitor) { > add_device_config(DEV_SERIAL, "mon:stdio"); > } else if (default_virtcon && default_monitor) { > add_device_config(DEV_VIRTCON, "mon:stdio"); > } else { > - if (default_serial) > - add_device_config(DEV_SERIAL, "stdio"); > - if (default_virtcon) > - add_device_config(DEV_VIRTCON, "stdio"); > - if (default_monitor) > - monitor_parse("stdio", "readline"); > + optarg = "stdio"; > } > - } else { > - if (default_serial) > - add_device_config(DEV_SERIAL, "vc:80Cx24C"); > - if (default_parallel) > - add_device_config(DEV_PARALLEL, "vc:80Cx24C"); > - if (default_monitor) > - monitor_parse("vc:80Cx24C", "readline"); > - if (default_virtcon) > - add_device_config(DEV_VIRTCON, "vc:80Cx24C"); > + } > + if (optarg && default_serial) { > + add_device_config(DEV_SERIAL, optarg); > + } > + if (default_parallel) { > + /* parallel port is connected console or to null */ > + add_device_config(DEV_PARALLEL, > + display_type == DT_NOGRAPHIC ? "null" : optarg); > + } > + if (optarg && default_monitor) { > + monitor_parse(optarg, "readline"); > + } > + if (optarg && default_virtcon) { > + add_device_config(DEV_VIRTCON, optarg); > } > > socket_init(); > -- > 1.7.10.4