linux-kernel - Re: [incremental-PATCH-for-Sam's-Review] ACPI: use ccflags-y instead of EXTRA

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]

Message-ID: <20090102215208.GA8494@uranus.ravnborg.org>
Date:	Fri, 2 Jan 2009 22:52:08 +0100
From:	Sam Ravnborg <sam@...nborg.org>
To:	Len Brown <lenb@...nel.org>
Cc:	Andi Kleen <andi@...stfloor.org>, linux-acpi@...r.kernel.org,
	Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: Re: [incremental-PATCH-for-Sam's-Review] ACPI: use ccflags-y instead of EXTRA_CFLAGS

On Fri, Jan 02, 2009 at 04:44:02PM -0500, Len Brown wrote:
> From: Len Brown <len.brown@...el.com>
> 
> Signed-off-by: Len Brown <len.brown@...el.com>
> ---
>  drivers/acpi/Makefile       |   11 ++---------
>  kernel/acpi/Makefile        |   11 ++---------
>  kernel/acpi/acpica/Makefile |   12 ++----------
>  3 files changed, 6 insertions(+), 28 deletions(-)
> 
> diff --git a/drivers/acpi/Makefile b/drivers/acpi/Makefile
> index 5f4096d..a4c0582 100644
> --- a/drivers/acpi/Makefile
> +++ b/drivers/acpi/Makefile
> @@ -2,15 +2,8 @@
>  # Makefile for the Linux ACPI Drivers
>  #
>  
> -export ACPI_CFLAGS
> -
> -ACPI_CFLAGS	:= -Os
> -
> -ifdef CONFIG_ACPI_DEBUG
> -  ACPI_CFLAGS	+= -DACPI_DEBUG_OUTPUT
> -endif
> -
> -EXTRA_CFLAGS	+= $(ACPI_CFLAGS)
> +ccflags-y			:= -Os
> +ccflags-$(CONFIG_ACPI_DEBUG)	+= -DACPI_DEBUG_OUTPUT

The conversion from ACPI_CFLAGS to ccflags-y is fine.
And if acpi really require -Os then this part is also OK -
I just did not get *why* acpi needs -Os

On the assumption the the -Os requirement is a hard requirement:

Acked-by: Sam Ravnborg <sam@...nborg.org>
[For the full patch - I just quoted the first part]

	Sam
--
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/