lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Tue, 10 Aug 2021 14:51:26 +0200 From: Linus Walleij <linus.walleij@...aro.org> To: Javier Martinez Canillas <javierm@...hat.com>, Arnd Bergmann <arnd@...db.de> Cc: linux-kernel <linux-kernel@...r.kernel.org>, dri-devel <dri-devel@...ts.freedesktop.org>, Thomas Zimmermann <tzimmermann@...e.de>, Peter Robinson <pbrobinson@...il.com>, kernel test robot <lkp@...el.com>, Borislav Petkov <bp@...e.de>, Dinh Nguyen <dinguyen@...nel.org>, Greg Kroah-Hartman <gregkh@...uxfoundation.org>, He Ying <heying24@...wei.com>, John Stultz <john.stultz@...aro.org>, Krzysztof Kozlowski <krzysztof.kozlowski@...onical.com>, Nicolas Saenz Julienne <nsaenz@...nel.org> Subject: Re: [PATCH] drivers/firmware: fix sysfb depends to prevent build failures On Thu, Jul 22, 2021 at 6:42 AM Javier Martinez Canillas <javierm@...hat.com> wrote: > The Generic System Framebuffers support is built when the COMPILE_TEST > option is enabled. But this wrongly assumes that all the architectures > declare a struct screen_info. > > This is true for most architectures, but at least the following do not: > arc, m68k, microblaze, openrisc, parisc and s390. > > By attempting to make this compile testeable on all architectures, it > leads to linking errors as reported by the kernel test robot for parisc: > > All errors (new ones prefixed by >>): > > hppa-linux-ld: drivers/firmware/sysfb.o: in function `sysfb_init': > (.init.text+0x24): undefined reference to `screen_info' > >> hppa-linux-ld: (.init.text+0x28): undefined reference to `screen_info' > > To prevent these errors only allow sysfb to be built on systems that are > going to need it, which are x86 BIOS and EFI. > > The EFI Kconfig symbol is used instead of (ARM || ARM64 || RISC) because > some of these architectures only declare a struct screen_info if EFI is > enabled. And also, because the sysfb code is only used for EFI on these > architectures. For !EFI the "simple-framebuffer" device is registered by > OF when parsing the Device Tree Blob (if a DT node for this is defined). > > Reported-by: kernel test robot <lkp@...el.com> > Signed-off-by: Javier Martinez Canillas <javierm@...hat.com> Reviewed-by: Linus Walleij <linus.walleij@...aro.org> I don't know who picks up firmware patches though, I would send them to soc@...nel.org if nothing else works. Yours, Linus Walleij
Powered by blists - more mailing lists