linux-kernel - Re: [PATCH] [BUILD_FAILURE] 2.6.25-rc2-mm1

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: Windows password security audit tool. GUI, reports in PDF.

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

Date:	Sat, 23 Feb 2008 00:41:23 -0500
From:	Len Brown <lenb@...nel.org>
To:	Randy Dunlap <randy.dunlap@...cle.com>
Cc:	Sam Ravnborg <sam@...nborg.org>,
	Nish Aravamudan <nish.aravamudan@...il.com>,
	Kamalesh Babulal <kamalesh@...ux.vnet.ibm.com>,
	Andrew Morton <akpm@...ux-foundation.org>,
	linux-kernel@...r.kernel.org, linux-acpi@...r.kernel.org,
	Andy Whitcroft <apw@...dowen.org>
Subject: Re: [PATCH] [BUILD_FAILURE] 2.6.25-rc2-mm1 - Build Failure at acpi_os

works for me!

applied.

thanks,
-len

ps.  CONFIG_ACPI_CUSTOM_DSDT's only use is to guard the use of
CONFIG_ACPI_CUSTOM_DSDT_FILE:

#ifdef CONFIG_ACPI_CUSTOM_DSDT
#include CONFIG_ACPI_CUSTOM_DSDT_FILE
#endif

we could get rid of it if cpp could so something like

#if (CONFIG_ACPI_CUSTOM_DSDT_FILE != "")
#include CONFIG_ACPI_CUSTOM_DSDT_FILE
#endif

but it doesn't look like cpp has a concept of strings in expressions.

On Friday 22 February 2008 14:25, Randy Dunlap wrote:
> Let's see what the ACPI people think about this change.
> 
> Thanks, Sam.
> ---
> From: Randy Dunlap <randy.dunlap@...cle.com>
> 
> Make ACPI_CUSTOM_DSDT boolean config symbol a hidden and derived
> value, based on the value of ACPI_CUSTOM_DSDT_FILE (string).
> Only the latter is presented to the user as a config option.
> 
> This fixes problems with "make randconfig" setting ACPI_CUSTOM_DSDT
> but leaving ACPI_CUSTOM_DSDT_FILE empty/blank.
> 
> Signed-off-by: Randy Dunlap <randy.dunlap@...cle.com>
> ---
>  drivers/acpi/Kconfig |   19 +++++++++----------
>  1 file changed, 9 insertions(+), 10 deletions(-)
> 
> --- linux-2.6.25-rc2-git5.orig/drivers/acpi/Kconfig
> +++ linux-2.6.25-rc2-git5/drivers/acpi/Kconfig
> @@ -283,24 +283,23 @@ config ACPI_TOSHIBA
>  	  If you have a legacy free Toshiba laptop (such as the Libretto L1
>  	  series), say Y.
>  
> -config ACPI_CUSTOM_DSDT
> -	bool "Include Custom DSDT"
> +config ACPI_CUSTOM_DSDT_FILE
> +	string "Custom DSDT Table file to include"
> +	default ""
>  	depends on !STANDALONE
> -	default n 
>  	help
>  	  This option supports a custom DSDT by linking it into the kernel.
>  	  See Documentation/acpi/dsdt-override.txt
>  
> -	  If unsure, say N.
> -
> -config ACPI_CUSTOM_DSDT_FILE
> -	string "Custom DSDT Table file to include"
> -	depends on ACPI_CUSTOM_DSDT
> -	default ""
> -	help
>  	  Enter the full path name to the file which includes the AmlCode
>  	  declaration.
>  
> +	  If unsure, don't enter a file name.
> +
> +config ACPI_CUSTOM_DSDT
> +	bool
> +	default ACPI_CUSTOM_DSDT_FILE != ""
> +
>  config ACPI_CUSTOM_DSDT_INITRD
>  	bool "Read Custom DSDT from initramfs"
>  	depends on BLK_DEV_INITRD
> --
> 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/
> 
--
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/