linux-kernel - Re: [PATCH] acpi/bgrt: Don't let users configure BGRT on non X86 systems

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 linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [day] [month] [year] [list]

Date:	Wed, 03 Apr 2013 02:10:29 +0200
From:	"Rafael J. Wysocki" <rafael.j.wysocki@...el.com>
To:	"Luck, Tony" <tony.luck@...el.com>
CC:	linux-acpi@...r.kernel.org, Matthew Garrett <mjg@...hat.com>,
	Len Brown <len.brown@...el.com>,
	LKML <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] acpi/bgrt: Don't let users configure BGRT on non X86
 systems

On 3/29/2013 11:30 PM, Luck, Tony wrote:
> Fengguang Wu's 0-Day kernel build testing backend found the
> following build error for an allmodconfig build on ia64:
>
>     drivers/built-in.o: In function `show_yoffset':
>>> bgrt.c:(.text+0xe5a71): undefined reference to `bgrt_tab'
>>> bgrt.c:(.text+0xe5a91): undefined reference to `bgrt_tab'
>     drivers/built-in.o: In function `show_xoffset':
>>> bgrt.c:(.text+0xe5b51): undefined reference to `bgrt_tab'
>>> bgrt.c:(.text+0xe5b71): undefined reference to `bgrt_tab'
>     drivers/built-in.o: In function `show_type':
>>> bgrt.c:(.text+0xe5c31): undefined reference to `bgrt_tab'
>     drivers/built-in.o:bgrt.c:(.text+0xe5c51): more undefined references to `bgrt_tab' follow
>     drivers/built-in.o: In function `bgrt_init':
>     bgrt.c:(.init.text+0x8931): undefined reference to `bgrt_image'
>     bgrt.c:(.init.text+0x8932): undefined reference to `bgrt_image_size'
>     bgrt.c:(.init.text+0x8950): undefined reference to `bgrt_image'
>     bgrt.c:(.init.text+0x8960): undefined reference to `bgrt_image_size'
>
> The problem is that all these undefined names are provided by
> arch/x86/platform/efi/efi-bgrt.c - which is obviously not available
> to the ia64 build.
>
> It doesn't seem useful to provide the BGRT support for Itanium
> (many systems are headless and have no graphics at all). So
> just don't let users configure this driver on non-X86 machines.
>
> Reported-by: Fengguang Wu <fengguang.wu@...el.com>
> Acked-by: H. Peter Anvin <hpa@...or.com>
> Signed-off-by: Tony Luck <tony.luck@...el.com>

I'd take it, but it didn't seem to make it to the list for some obscure 
reason and I'd prefer it to show up there before it shows up in my tree.

Let's see if my reply makes it.

Thanks,
Rafael


> ---
>   drivers/acpi/Kconfig | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/acpi/Kconfig b/drivers/acpi/Kconfig
> index 4fc0ffc..8fcae6d 100644
> --- a/drivers/acpi/Kconfig
> +++ b/drivers/acpi/Kconfig
> @@ -395,7 +395,7 @@ config ACPI_CUSTOM_METHOD
>   
>   config ACPI_BGRT
>   	bool "Boottime Graphics Resource Table support"
> -	depends on EFI
> +	depends on EFI && X86
>           help
>   	  This driver adds support for exposing the ACPI Boottime Graphics
>   	  Resource Table, which allows the operating system to obtain

---------------------------------------------------------------------
Intel Technology Poland sp. z o.o.
z siedziba w Gdansku
ul. Slowackiego 173
80-298 Gdansk

Sad Rejonowy Gdansk Polnoc w Gdansku, 
VII Wydzial Gospodarczy Krajowego Rejestru Sadowego, 
numer KRS 101882

NIP 957-07-52-316
Kapital zakladowy 200.000 zl

This e-mail and any attachments may contain confidential material for
the sole use of the intended recipient(s). Any review or distribution
by others is strictly prohibited. If you are not the intended
recipient, please contact the sender and delete all copies.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/