linux-kernel - Re: [git-pull -tip] Add -Werror to arch/x86/ files only and fix warnings

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:	Wed, 15 Apr 2009 13:13:55 +0200
From:	Ingo Molnar <mingo@...e.hu>
To:	Jaswinder Singh Rajput <jaswinder@...nel.org>
Cc:	x86 maintainers <x86@...nel.org>, Sam Ravnborg <sam@...nborg.org>,
	Andrew Morton <akpm@...ux-foundation.org>,
	LKML <linux-kernel@...r.kernel.org>
Subject: Re: [git-pull -tip] Add -Werror to arch/x86/ files only and fix
	warnings


* Jaswinder Singh Rajput <jaswinder@...nel.org> wrote:

> Hello Ingo,
> 
> As per your suggestion, I added -Werror for x86/files only and
> fix all remaining warnings on 32-bit and 64-bit defconfig and
> allnoconfig.
> 
> Here is pull request:
> The following changes since commit 3965887240cb092f0d568b53165da6a215c7937a:
>   Ingo Molnar (1):
>         Merge branch 'linus'
> 
> are available in the git repository at:
> 
>   git://git.kernel.org/pub/scm/linux/kernel/git/jaswinder/linux-2.6-tip.git master
> 
> Jaswinder Singh Rajput (2):
>       x86: Adding -Werror to arch/x86/ files only
>       x86: k8.h convert node_to_k8_nb_misc() from a macro to an inline function
> 
>  arch/x86/boot/Makefile                 |    3 +++
>  arch/x86/boot/compressed/Makefile      |    3 +++
>  arch/x86/crypto/Makefile               |    3 +++
>  arch/x86/ia32/Makefile                 |    3 +++
>  arch/x86/include/asm/k8.h              |   11 ++++++++---
>  arch/x86/kernel/Makefile               |    3 +++
>  arch/x86/kernel/acpi/Makefile          |    3 +++
>  arch/x86/kernel/acpi/realmode/Makefile |    3 +++
>  arch/x86/kernel/apic/Makefile          |    3 +++
>  arch/x86/kernel/cpu/Makefile           |    3 +++
>  arch/x86/kernel/cpu/cpufreq/Makefile   |    3 +++
>  arch/x86/kernel/cpu/mcheck/Makefile    |    3 +++
>  arch/x86/kernel/cpu/mtrr/Makefile      |    3 +++
>  arch/x86/kvm/Makefile                  |    3 +++
>  arch/x86/lguest/Makefile               |    3 +++
>  arch/x86/lib/Makefile                  |    3 +++
>  arch/x86/math-emu/Makefile             |    3 +++
>  arch/x86/mm/Makefile                   |    3 +++
>  arch/x86/mm/kmemcheck/Makefile         |    3 +++
>  arch/x86/oprofile/Makefile             |    3 +++
>  arch/x86/pci/Makefile                  |    3 +++
>  arch/x86/power/Makefile                |    3 +++
>  arch/x86/vdso/Makefile                 |    4 +++-
>  arch/x86/video/Makefile                |    3 +++
>  arch/x86/xen/Makefile                  |    5 ++++-
>  25 files changed, 81 insertions(+), 5 deletions(-)
> 
> Complete diff:
> diff --git a/arch/x86/boot/Makefile b/arch/x86/boot/Makefile
> index 658bc52..aca7d2f 100644
> --- a/arch/x86/boot/Makefile
> +++ b/arch/x86/boot/Makefile
> @@ -22,6 +22,9 @@ ROOT_DEV	:= CURRENT
>  
>  SVGA_MODE	:= -DSVGA_MODE=NORMAL_VGA
>  
> +# Make all warnings into errors

=> 'Turn all warnings into errors'

> +ccflags-y := -Werror
> +
>  targets		:= vmlinux.bin setup.bin setup.elf bzImage
>  targets		+= fdimage fdimage144 fdimage288 image.iso mtools.conf
>  subdir-		:= compressed

> --- a/arch/x86/mm/Makefile
> +++ b/arch/x86/mm/Makefile
> @@ -1,3 +1,6 @@
> +# Make all warnings into errors
> +ccflags-y			:= -Werror
> +
>  obj-y	:=  init.o init_$(BITS).o fault.o ioremap.o extable.o pageattr.o mmap.o \
>  	    pat.o pgtable.o gup.o

i'd suggest to right-align this obj-y rule as well, to move it in 
line with the other lines.

Looks good otherwise.

Sam, any second thoughts?

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