On Tue, Jul 10, 2018 at 12:26:52AM +0200, Sebastian Bauer wrote: > 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?
"none" looked like a false positive when I first looked, but now I think it's not. Shouldn't it set default_display="none"? > > 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). If machines prefer "std", they should set default_display="std" explicitly. > > What is the indented target release for the patch? I'm not convinced this patch is appropriate for 3.0. If we have remaining bugs they should be fixed by setting default_display explicitly on the affected machines. > > 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. 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. > > 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. This part worries me: do we have other machines that are broken right now? -- Eduardo