On Fri, 28 Jul 2006 20:48:31 +0200, Arjan van de Ven said: > On Fri, 2006-07-28 at 20:45 +0200, Andi Kleen wrote: > > > +ifdef CONFIG_CC_STACKPROTECTOR > > > +CFLAGS += $(call cc-ifversion, -lt, 0402, -fno-stack-protector) > > > +CFLAGS += $(call cc-ifversion, -ge, 0402, -fstack-protector) > > > > Why can't you just use the normal call cc-option for this? > > this requires gcc 4.2; cc-option is not useful for that. At least some things calling themselves 4.1.1 include it. On a Fedora Rawhide box: % gcc -v -fstack-protector hello.c Using built-in specs. Target: i386-redhat-linux Configured with: ../configure --prefix=/usr --mandir=/usr/share/man --infodir=/usr/share/info --enable-shared --enable-threads=posix --enable-checking=release --with-system-zlib --enable-__cxa_atexit --disable-libunwind-exceptions --enable-libgcj-multifile --enable-languages=c,c++,objc,obj-c++,java,fortran,ada --enable-java-awt=gtk --disable-dssi --enable-plugin --with-java-home=/usr/lib/jvm/java-1.4.2-gcj-1.4.2.0/jre --with-cpu=generic --host=i386-redhat-linux Thread model: posix gcc version 4.1.1 20060721 (Red Hat 4.1.1-13) /usr/libexec/gcc/i386-redhat-linux/4.1.1/cc1 -quiet -v hello.c -quiet -dumpbase hello.c -mtune=generic -auxbase hello -version -fstack-protector -o /tmp/ccLfn4gs.s ignoring nonexistent directory "/usr/lib/gcc/i386-redhat-linux/4.1.1/../../../../i386-redhat-linux/include" #include "..." search starts here: #include <...> search starts here: /usr/local/include /usr/lib/gcc/i386-redhat-linux/4.1.1/include /usr/include End of search list. GNU C version 4.1.1 20060721 (Red Hat 4.1.1-13) (i386-redhat-linux) compiled by GNU C version 4.1.1 20060721 (Red Hat 4.1.1-13). GGC heuristics: --param ggc-min-expand=81 --param ggc-min-heapsize=96852 Compiler executable checksum: cbc4b2991046c2b178d1ad5878ca2677 hello.c: In function 'main': hello.c:1: warning: incompatible implicit declaration of built-in function 'printf' as -V -Qy -o /tmp/cceHv3eO.o /tmp/ccLfn4gs.s GNU assembler version 2.17.50.0.3-1 (i386-redhat-linux) using BFD version 2.17.50.0.3-1 20060715 /usr/libexec/gcc/i386-redhat-linux/4.1.1/collect2 --eh-frame-hdr -m elf_i386 --hash-style=gnu -dynamic-linker /lib/ld-linux.so.2 /usr/lib/gcc/i386-redhat-linux/4.1.1/../../../crt1.o /usr/lib/gcc/i386-redhat-linux/4.1.1/../../../crti.o /usr/lib/gcc/i386-redhat-linux/4.1.1/crtbegin.o -L/usr/lib/gcc/i386-redhat-linux/4.1.1 -L/usr/lib/gcc/i386-redhat-linux/4.1.1 -L/usr/lib/gcc/i386-redhat-linux/4.1.1/../../.. /tmp/cceHv3eO.o -lgcc --as-needed -lgcc_s --no-as-needed -lc -lgcc --as-needed -lgcc_s --no-as-needed /usr/lib/gcc/i386-redhat-linux/4.1.1/crtend.o /usr/lib/gcc/i386-redhat-linux/4.1.1/../../../crtn.o
Attachment:
pgppHb2og4YZJ.pgp
Description: PGP signature
- Follow-Ups:
- Re: [patch 5/5] Add the -fstack-protector option to the CFLAGS
- From: David Miller <[email protected]>
- Re: [patch 5/5] Add the -fstack-protector option to the CFLAGS
- References:
- [patch 5/5] Add the -fstack-protector option to the CFLAGS
- From: Arjan van de Ven <[email protected]>
- Re: [patch 5/5] Add the -fstack-protector option to the CFLAGS
- From: Andi Kleen <[email protected]>
- Re: [patch 5/5] Add the -fstack-protector option to the CFLAGS
- From: Arjan van de Ven <[email protected]>
- [patch 5/5] Add the -fstack-protector option to the CFLAGS
- Prev by Date: Re: [RFC] /dev/itimer
- Next by Date: Re: metadata plugins (was Re: the " 'official' point of view" expressed by kernelnewbies.org regarding reiser4 inclusion)
- Previous by thread: Re: [patch 5/5] Add the -fstack-protector option to the CFLAGS
- Next by thread: Re: [patch 5/5] Add the -fstack-protector option to the CFLAGS
- Index(es):