From: Anthony Liguori <anthony@codemonkey.ws>
To: Peter Maydell <peter.maydell@linaro.org>
Cc: Jes Sorensen <Jes.Sorensen@redhat.com>, qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] Re: [PATCH v2] Make VNC support optional
Date: Fri, 11 Mar 2011 09:50:03 -0600 [thread overview]
Message-ID: <4D7A44AB.4030006@codemonkey.ws> (raw)
In-Reply-To: <AANLkTinaAkMNibbU3dQNCQ87HcKqitsbwsNQUheBy7K6@mail.gmail.com>
On 03/11/2011 09:11 AM, Peter Maydell wrote:
> On 11 March 2011 14:55, Anthony Liguori<anthony@codemonkey.ws> wrote:
>> On 03/11/2011 08:54 AM, Jes Sorensen wrote:
>>> On 03/11/11 15:39, Anthony Liguori wrote:
>>>> So what ends up being the default display if VNC and SDL are both
>>>> disabled? Have you tested this?
>>> Then you cry :) actually you just don't get video output, a bit like if
>>> you run with -nographic, except it doesn't try to force taking over
>>> stdio. This is intentional btw :)
>> Hrm, that doesn't sound very safe to me. Forcing -nographic would be better
>> IMHO.
> Personally I'd rather -nographic didn't take over stdio either:
> it should just disable graphics, and if you want serial on your
> stdio you can say "-serial stdio"; it's a bit non-orthogonal
> for an option which is about how we handle video output to have
> a non-overridable side-effect of redirecting the serial port.
>
> (Also the "-nographic" running of serial over stdio doesn't let
> you kill qemu with ^C, the way "-serial stdio" does, which is
> just annoyingly inconsistent...)
So if we want to have another mode that has different characteristics,
that's fine, but it should be selectable via the command line regardless
of how the build is configured.
I don't like the idea of a magic mode that is unlocked when other
features are disabled because then there's no good way to test such a
feature in a full build.
Regards,
Anthony Liguori
next prev parent reply other threads:[~2011-03-11 15:50 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-03-11 14:37 [Qemu-devel] [PATCH v2] Make VNC support optional Jes.Sorensen
2011-03-11 14:39 ` [Qemu-devel] " Anthony Liguori
2011-03-11 14:54 ` Jes Sorensen
2011-03-11 14:55 ` Anthony Liguori
2011-03-11 15:05 ` Jes Sorensen
2011-03-11 15:11 ` Peter Maydell
2011-03-11 15:50 ` Anthony Liguori [this message]
2011-03-11 16:30 ` Jes Sorensen
2011-03-11 17:09 ` Anthony Liguori
2011-03-11 16:35 ` Jan Kiszka
2011-03-14 9:35 ` Jes Sorensen
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4D7A44AB.4030006@codemonkey.ws \
--to=anthony@codemonkey.ws \
--cc=Jes.Sorensen@redhat.com \
--cc=peter.maydell@linaro.org \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).