From: Sebastian Bauer <mail@sebastianbauer.info>
To: Eduardo Habkost <ehabkost@redhat.com>
Cc: Gerd Hoffmann <kraxel@redhat.com>,
Aleksandar Markovic <aleksandar.markovic@mips.com>,
"Michael S. Tsirkin" <mst@redhat.com>,
Richard Henderson <rth@twiddle.net>,
qemu-devel@nongnu.org, Alexander Graf <agraf@suse.de>,
qemu-ppc@nongnu.org, Paolo Bonzini <pbonzini@redhat.com>,
Aurelien Jarno <aurelien@aurel32.net>,
David Gibson <david@gibson.dropbear.id.au>
Subject: Re: [Qemu-devel] [PATCH v2 2/2] vga: don't pick cirrus by default
Date: Tue, 10 Jul 2018 00:26:52 +0200 [thread overview]
Message-ID: <11c6f7d4364062c42df990364bf2c2de@sebastianbauer.info> (raw)
In-Reply-To: <20180709212319.GB22336@localhost.localdomain>
Hi,
Am 2018-07-09 23:23, schrieb Eduardo Habkost:
> List of machines with default_display==NULL on those
> architectures:
>
> alpha:
> none empty machine
>
> mips:
> mipssim MIPS MIPSsim platform
> none empty machine
>
> ppc*:
> bamboo bamboo
> mpc8544ds mpc8544ds
> none empty machine
> powernv IBM PowerNV (Non-Virtualized)
> ppce500 generic paravirt e500 platform
> ref405ep ref405ep
> sam460ex aCube Sam460ex
> taihu taihu
> virtex-ml507 Xilinx Virtex ML507 reference design
>
> x86_64:
> isapc ISA-only PC
> none empty machine
> xenfv Xen Fully-virtualized PC
> xenpv Xen Para-virtualized PC
Which of these machines really require the Cirrus? The xen ones look
like that they can deal with std. The isapc is should probably stay at
the Cirrus.
Also the "none" seems to be a false-positive. I suppose they mean
"empty", i.e., no graphics card at all?
And at least the ppc ones can be canceled out, they should work with
std, the new default (expect the sam460ex which goes an own route for
now).
What is the indented target release for the patch?
If the patch is applied to 3.1 then I think there is enough time to fix
issues caused by the patch. Additionally, a warning could be put in the
ChangeLog for 3.0 that in 3.1 that the default mode will be std unless
machines define an own default. This is should be enough time for people
to complain or to fix things.
If the patch is to be applied to 3.0 then all non-ppc ones need to be
reconsidered.
The "important" ppc machines have been fixed already. I can do the
remaining if this is wanted.
Bye
Sebastian
next prev parent reply other threads:[~2018-07-09 22:27 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-07-05 6:43 [Qemu-devel] [PATCH v2 0/2] vga: make stdvga the global default Gerd Hoffmann
2018-07-05 6:43 ` [Qemu-devel] [PATCH v2 1/2] vga: set default_display for all machines using cirrus Gerd Hoffmann
2018-07-05 16:35 ` Eduardo Habkost
2018-07-05 6:43 ` [Qemu-devel] [PATCH v2 2/2] vga: don't pick cirrus by default Gerd Hoffmann
2018-07-05 16:35 ` Eduardo Habkost
2018-07-06 6:53 ` Gerd Hoffmann
2018-07-09 21:08 ` Eduardo Habkost
2018-07-09 21:23 ` Eduardo Habkost
2018-07-09 22:26 ` Sebastian Bauer [this message]
2018-07-11 15:48 ` Eduardo Habkost
2018-07-11 17:00 ` Sebastian Bauer
2018-07-11 18:43 ` Eduardo Habkost
2018-07-12 4:57 ` [Qemu-devel] [Qemu-ppc] " Thomas Huth
2018-07-12 7:37 ` Sebastian Bauer
2018-07-12 17:27 ` Eduardo Habkost
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=11c6f7d4364062c42df990364bf2c2de@sebastianbauer.info \
--to=mail@sebastianbauer.info \
--cc=agraf@suse.de \
--cc=aleksandar.markovic@mips.com \
--cc=aurelien@aurel32.net \
--cc=david@gibson.dropbear.id.au \
--cc=ehabkost@redhat.com \
--cc=kraxel@redhat.com \
--cc=mst@redhat.com \
--cc=pbonzini@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=qemu-ppc@nongnu.org \
--cc=rth@twiddle.net \
/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.