From owner-freebsd-arch@FreeBSD.ORG Wed May 14 09:13:58 2008 Return-Path: Delivered-To: freebsd-arch@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 79EFD1065677 for ; Wed, 14 May 2008 09:13:58 +0000 (UTC) (envelope-from tataz@tataz.chchile.org) Received: from smtp5-g19.free.fr (smtp5-g19.free.fr [212.27.42.35]) by mx1.freebsd.org (Postfix) with ESMTP id 174138FC12 for ; Wed, 14 May 2008 09:13:57 +0000 (UTC) (envelope-from tataz@tataz.chchile.org) Received: from smtp5-g19.free.fr (localhost.localdomain [127.0.0.1]) by smtp5-g19.free.fr (Postfix) with ESMTP id 4B70F3F6289; Wed, 14 May 2008 11:13:56 +0200 (CEST) Received: from tatooine.tataz.chchile.org (tataz.chchile.org [82.233.239.98]) by smtp5-g19.free.fr (Postfix) with ESMTP id 3018B3F6284; Wed, 14 May 2008 11:13:56 +0200 (CEST) Received: from obiwan.tataz.chchile.org (unknown [192.168.1.25]) by tatooine.tataz.chchile.org (Postfix) with ESMTP id 77C4C9D6EF; Wed, 14 May 2008 09:11:08 +0000 (UTC) Received: by obiwan.tataz.chchile.org (Postfix, from userid 1000) id 677944089; Wed, 14 May 2008 11:11:08 +0200 (CEST) Date: Wed, 14 May 2008 11:11:08 +0200 From: Jeremie Le Hen To: Marcel Moolenaar Message-ID: <20080514091108.GD70896@obiwan.tataz.chchile.org> References: <20080502070147.GE74500@obiwan.tataz.chchile.org> <8ED24288-618C-4B55-A27E-C5FAB2E046E8@mac.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <8ED24288-618C-4B55-A27E-C5FAB2E046E8@mac.com> User-Agent: Mutt/1.5.15 (2007-04-06) Cc: freebsd-arch@FreeBSD.org Subject: Re: Integration of ProPolice in FreeBSD X-BeenThere: freebsd-arch@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Discussion related to FreeBSD architecture List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 14 May 2008 09:13:58 -0000 Hi Marcel, Thank you for your comments. On Fri, May 02, 2008 at 08:52:37AM -0700, Marcel Moolenaar wrote: > Index: share/mk/bsd.sys.mk > =================================================================== > RCS file: /mnt/octobre/space/freebsd-cvs/src/share/mk/bsd.sys.mk,v > retrieving revision 1.44 > diff -u -p -r1.44 bsd.sys.mk > --- share/mk/bsd.sys.mk 22 Nov 2007 23:21:12 -0000 1.44 > +++ share/mk/bsd.sys.mk 29 Mar 2008 23:13:06 -0000 > @@ -74,5 +74,10 @@ CWARNFLAGS += -Werror > CWARNFLAGS += -Wno-unknown-pragmas > .endif > > +.if ${MK_SSP} != "no" && ${CC} != "icc" > +CFLAGS += -fstack-protector > +# Don't use -Wstack-protector as it breaks world with -Werror. > +.endif > + > # Allow user-specified additional warning flags > CFLAGS += ${CWARNFLAGS} > > > I may be better to explicitly test for GCC. I would not assume > that GCC and ICC are the only options, even if they are now. > There's a second place as well... I would make sense to test for GCC indeed but in the same file there is a number of explicit tests for ICC. Thus I prefer to keep to the current "style" at the expense of a little more work for those who will try to compile with another compiler. > Index: sys/boot/i386/Makefile.inc > =================================================================== > RCS file: /mnt/octobre/space/freebsd-cvs/src/sys/boot/i386/Makefile.inc,v > retrieving revision 1.12 > diff -u -p -r1.12 Makefile.inc > --- sys/boot/i386/Makefile.inc 28 Sep 2006 10:02:04 -0000 1.12 > +++ sys/boot/i386/Makefile.inc 28 Mar 2008 07:41:32 -0000 > @@ -24,3 +24,5 @@ BTXDIR= ${.CURDIR}/../btx > BTXLDR= ${BTXDIR}/btxldr/btxldr > BTXKERN= ${BTXDIR}/btx/btx > BTXCRT= ${BTXDIR}/lib/crt0.o > + > +.include "../Makefile.inc" > Index: sys/boot/i386/loader/Makefile > =================================================================== > RCS file: /mnt/octobre/space/freebsd-cvs/src/sys/boot/i386/loader/Makefile,v > retrieving revision 1.85 > diff -u -p -r1.85 Makefile > --- sys/boot/i386/loader/Makefile 29 May 2007 14:35:57 -0000 1.85 > +++ sys/boot/i386/loader/Makefile 16 Apr 2008 09:14:10 -0000 > @@ -1,5 +1,7 @@ > # $FreeBSD: src/sys/boot/i386/loader/Makefile,v 1.85 2007/05/29 14:35:57 > simokawa Exp $ > > +WITHOUT_SSP= > + > .include > > PROG= loader.sym > > Maybe second and third level makefiles should include > ../../Makefile.inc and ../../../Makefile.inc resp. > If, for arguments sake, we want to enable SSP in boot, > then it's best if that only requires a single knob to > be changed. This may not be a strong argument for SSP, > but with Makefile.inc in place, I don't see a possible > future in which another knob is added that controls > overall behavior (e.g. something like the Watcom option > to use argument passing in registers instead of on the > stack for i386 -- you definitely want to have that apply > to all code or none). For now, I prefer leaving it as is and let anyone else make this change. While I agree with your argument, I'm not sure this benefit is worth the complexity it adds for now, given that the only knob is WITHOUT_SSP and it would require more than a simple switch to use SSP for /boot (import SSP symbols). > Also, please make sure MK_SSP defaults to "no" on ia64. Ok, done. Best regards, -- Jeremie Le Hen < jeremie at le-hen dot org >< ttz at chchile dot org >