From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:51711) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1fdTfv-00087d-7x for qemu-devel@nongnu.org; Thu, 12 Jul 2018 00:57:56 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1fdTfs-0004vK-4d for qemu-devel@nongnu.org; Thu, 12 Jul 2018 00:57:55 -0400 References: <20180705064348.3162-1-kraxel@redhat.com> <20180705064348.3162-3-kraxel@redhat.com> <20180705163501.GN7451@localhost.localdomain> <20180706065342.pggd2qgd4nyv35ys@sirius.home.kraxel.org> <20180709210833.GC7451@localhost.localdomain> <20180709212319.GB22336@localhost.localdomain> <11c6f7d4364062c42df990364bf2c2de@sebastianbauer.info> <20180711154850.GT7451@localhost.localdomain> <2b7f5bdfd86278524b4540f512eecbc7@sebastianbauer.info> <20180711184311.GX7451@localhost.localdomain> From: Thomas Huth Message-ID: <6f4a875e-a85f-1926-88af-91a260e334a7@redhat.com> Date: Thu, 12 Jul 2018 06:57:46 +0200 MIME-Version: 1.0 In-Reply-To: <20180711184311.GX7451@localhost.localdomain> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [Qemu-ppc] [PATCH v2 2/2] vga: don't pick cirrus by default List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Eduardo Habkost , Sebastian Bauer Cc: Aleksandar Markovic , "Michael S. Tsirkin" , qemu-devel@nongnu.org, qemu-ppc@nongnu.org, Gerd Hoffmann , Paolo Bonzini , David Gibson , Richard Henderson On 11.07.2018 20:43, Eduardo Habkost wrote: > On Wed, Jul 11, 2018 at 07:00:54PM +0200, Sebastian Bauer wrote: >> Am 2018-07-11 17:48, schrieb Eduardo Habkost: >>> "none" looked like a false positive when I first looked, but now >>> I think it's not. Shouldn't it set default_display=3D"none"? >> >> I think that there is some other logic burried that these machine does= n't >> get a graphics display. But overall it is indeed not clearly defined. >> >> But see below. >> >>>> 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 unle= ss >>>> machines define an own default. This is should be enough time for >>>> people to >>>> complain or to fix things. >>> I don't think we will really make user-visible changes: we can >>> simply work to keep existing behavior, but the difference is that >>> this will be implemented by setting default_display explicitly on >>> all machines. >> >> Even if all machines were using explicit default settings the patch wi= ll >> affect machines that are not inside the QEMU tree. If the patch is to = be >> applied as it is these are affected. To warn users (or devs in this ca= se) >> about this, an entry in the ChangeLog would be appropriate. >=20 > What do you mean by "machines that are not inside the QEMU tree"? > MachineClass registration is not an API for external use. >=20 >> >>>> If the patch is to be applied to 3.0 then all non-ppc ones need to b= e >>>> reconsidered. >>>> The "important" ppc machines have been fixed already. I can do the >>>> remaining >>>> if this is wanted. >>> This part worries me: do we have other machines that are broken >>> right now? >> >> I don't know which of them are broken or how this can be elaborated, b= ut >> they are potentially affected. For instance, the sam460ex platform doe= sn't >> care about this setting, I can say that it is not broken. Other platfo= rms >> like the mac apparently were broken (and fixed in the meantime). It is= hard >> to tell which of them are really broken without someone that knows the >> platform trying it and telling it. 'Make check' did catch only one sin= gle >> case. It could also be that nobody cares about other affected machines= . >> >> Overall I think the patch is an improvement over the previous state as >> preferring the Cirrus doesn't make much sense if most machines don't p= refer >> it. The more I think over it, the more I think that the concept needs >> further fine-tuning though (not necessarily in this patch). >=20 > I'm not sure yet if most machines with default_display=3D=3DNULL > don't want Cirrus. I'm also unsure if any of the machines from > that list will break if we start using VGA by default. I think most machines with default_display =3D=3D NULL simply do not want any graphic cards at all. So we likely document that default_display =3D=3D NULL means no graphic card, and fix the machines that have other assumptions if necessary. >> [...] I still think that the most common value ca= n be a >> default (strictly bailing out when it is not available unlike it is do= ne >> now), but this is a matter of taste I guess. >=20 > This is doable too, if we have a clear consensus on what would be > a reasonable default on TYPE_MACHINE. Personally I prefer the > default on TYPE_MACHINE to be "none". Yes, please, no magic default hardware for all machines. That will only cause confusion and problems in the future. People who add new boards will keep forgetting to set a value in their new boards and then you later wonder whether that was on purpose or by accident =3D=3D> If there = is no default display set, this should simply mean "no display". Thomas