Hi Am 23.01.24 um 10:41 schrieb Javier Martinez Canillas:
Thorsten Leemhuis <regressions@xxxxxxxxxxxxx> writes:On 23.01.24 09:53, Jani Nikula wrote:On Wed, 08 Nov 2023, Thomas Zimmermann <tzimmermann@xxxxxxx> wrote:[...]As you know, there's a platform device that represents the firmware framebuffer. The firmware drivers, such as simpledrm, bind to it. In i915 and the other native drivers we remove that platform device, so that simpledrm does not run.The problem is still not resolved. Another bug report at [1]. The commit message here points at 60aebc955949 ("drivers/firmware: Move sysfb_init() from device_initcall to subsys_initcall_sync") as regressing, and Jaak also bisected it (see Closes:). I agree the patch here is just papering over the issue, but lacking a proper fix, for months, a revert would be in order, no?Yes, I agree that this patch has to be reverted, since as you said the issue has not been fixed and is causing regressions for multiple users.
I wanted to send out a revert anyway. I'll do this in a bit. Best regards Thomas
[1] https://gitlab.freedesktop.org/drm/intel/-/issues/10133Interesting. JFYI for those that don't follow this closely: Huacai Chen proposed a fix and asked a earlier reporter to test it, but afaics heard nothing back: https://lore.kernel.org/all/CAAhV-H5eXM7FNzuRCMthAziG_jg75XwQV3grpw=sdyJ-9GXgvA@xxxxxxxxxxxxxx/It's not a fix but a debug patch for the patch author to get more info ?
-- Thomas Zimmermann Graphics Driver Developer SUSE Software Solutions Germany GmbH Frankenstrasse 146, 90461 Nuernberg, Germany GF: Ivo Totev, Andrew Myers, Andrew McDonald, Boudien Moerman HRB 36809 (AG Nuernberg)
Attachment:
OpenPGP_signature.asc
Description: OpenPGP digital signature
