All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jes Sorensen <Jes.Sorensen@redhat.com>
To: Peter Maydell <peter.maydell@linaro.org>
Cc: jan.kiszka@siemens.com, qemu-devel@nongnu.org
Subject: [Qemu-devel] Re: [PATCH 1/5] Introduce -display argument
Date: Mon, 14 Mar 2011 13:41:58 +0100	[thread overview]
Message-ID: <4D7E0D16.8070405@redhat.com> (raw)
In-Reply-To: <AANLkTi=32R1NroUj8VWk7wz+RtE0ABTLHDuYPRPJ0Vuv@mail.gmail.com>

On 03/14/11 13:40, Peter Maydell wrote:
> On 14 March 2011 12:28,  <Jes.Sorensen@redhat.com> wrote:
>> This patch introduces a -display argument which consolidates the
>> setting of the display mode. Valid options are:
>> sdl/curses/default/nographic
> 
> I wasn't expecting to see "nographic" as an option here -- my
> preference would be to retain the existing '-nographic' switch
> for backwards-compatibility, but '-display foo' should only
> affect the video type. (ie the equivalent of '-nographic' ought
> to be a set of options including '-display none', '-serial mon:stdio'
> and so on, not a single '-display:nographic').

It would be easy to remove it from the -display option, but I think it
is fair to keep it in. I was pretty much trying to cover every DT_foo
option we have. If nographic is going away as a display option, we
should make it in the code that it is for legacy support only.

Cheers,
Jes

  reply	other threads:[~2011-03-14 12:42 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-03-14 12:28 [Qemu-devel] [PATCH 0/5] Introduce -display and make VNC optional Jes.Sorensen
2011-03-14 12:28 ` [Qemu-devel] [PATCH 1/5] Introduce -display argument Jes.Sorensen
2011-03-14 12:40   ` [Qemu-devel] " Peter Maydell
2011-03-14 12:41     ` Jes Sorensen [this message]
2011-03-14 13:37   ` Anthony Liguori
2011-03-14 12:28 ` [Qemu-devel] [PATCH 2/5] Introduce -display none Jes.Sorensen
2011-03-14 12:28 ` [Qemu-devel] [PATCH 3/5] error message if user specifies SDL cmd line option when SDL is disabled Jes.Sorensen
2011-03-14 12:28 ` [Qemu-devel] [PATCH 4/5] error message if user specifies curses on cmd line when curses " Jes.Sorensen
2011-03-14 12:28 ` [Qemu-devel] [PATCH 5/5] Make VNC support optional 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=4D7E0D16.8070405@redhat.com \
    --to=jes.sorensen@redhat.com \
    --cc=jan.kiszka@siemens.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.