From owner-freebsd-arch@FreeBSD.ORG Mon Mar 22 14:55:32 2010 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 9DBBE1065676; Mon, 22 Mar 2010 14:55:32 +0000 (UTC) (envelope-from scottl@samsco.org) Received: from pooker.samsco.org (pooker.samsco.org [168.103.85.57]) by mx1.freebsd.org (Postfix) with ESMTP id 347668FC23; Mon, 22 Mar 2010 14:55:31 +0000 (UTC) Received: from phobos.samsco.home (phobos.samsco.home [192.168.254.11]) (authenticated bits=0) by pooker.samsco.org (8.14.3/8.14.3) with ESMTP id o2MEtPqV065712; Mon, 22 Mar 2010 08:55:25 -0600 (MDT) (envelope-from scottl@samsco.org) Mime-Version: 1.0 (Apple Message framework v1077) Content-Type: text/plain; charset=us-ascii From: Scott Long In-Reply-To: <201003220941.10525.jhb@freebsd.org> Date: Mon, 22 Mar 2010 08:55:25 -0600 Content-Transfer-Encoding: quoted-printable Message-Id: <8E9F405D-0140-4C67-B7BD-94714E2DD109@samsco.org> References: <201003100812.29749.jhb@freebsd.org> <20100322123408.16671ijbvmcyux80@webmail.leidinger.net> <201003220941.10525.jhb@freebsd.org> To: John Baldwin X-Mailer: Apple Mail (2.1077) X-Spam-Status: No, score=-1.0 required=3.8 tests=ALL_TRUSTED autolearn=unavailable version=3.3.0 X-Spam-Checker-Version: SpamAssassin 3.3.0 (2010-01-18) on pooker.samsco.org Cc: Alexander Leidinger , "Robert N. M. Watson" , freebsd-arch@freebsd.org Subject: Re: CTF patch for testing/review (was: Re: is dtrace usable?) 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: Mon, 22 Mar 2010 14:55:32 -0000 On Mar 22, 2010, at 7:41 AM, John Baldwin wrote: > On Monday 22 March 2010 7:34:08 am Alexander Leidinger wrote: >> Redirecting from stable@ to arch@... >>=20 >> Quoting John Baldwin (from Wed, 10 Mar 2010 = 08:12:29=20 > -0500): >>=20 >>> On Wednesday 10 March 2010 5:34:22 am Alexander Leidinger wrote: >>>> Quoting "Robert N. M. Watson" (from Tue, 9 = Mar >>>> 2010 16:39:09 +0000): >>>>=20 >>>>>=20 >>>>> On Mar 9, 2010, at 2:16 PM, Alexander Leidinger wrote: >>>>>=20 >>>>>>> =46rom this you can see that sys.mk is included and parsed = before >>> 'Makefile', >>>>>>> so the WITH_CTF=3Dyes is not set until after sys.mk has been = parsed. >>>>>>=20 >>>>>> I think we need to find a different solution for this. The need = to >>>>>> specify WITH_CTF at the command line is very error prone. :( >>>>>=20 >>>>> You are neither the first person to have made this observation, = nor >>>>> the first person to have failed to propose a solution in the form = of >>>>> a patch :-). >>=20 >> Ok, here is the proposal in form of a patch. :-) >> http://www.leidinger.net/test/ctf.diff >>=20 >>> Unfortunately the ctf stuff breaks static binaries. I think that if = =20 >>> that were >>> fixed we would simply enable it by default and be done. >>=20 >> The patch is: >> - enabling CTF stuff by default for the kernel >> - allows to disable the CTF stuff for the kernel by defining NO_CTF >> - *not* enabling the CTF stuff by default for libs and progs >> (if someone tells me how to distinguish the build for static >> stuff from dynamic stuff, I can have a look to enable it for >> the dynamic case) >> - allows to enable the CTF stuff for the userland by defining >> WITH_CTF as before >=20 > I think this patch looks very interesting. I think in some ways it = would be=20 > nice to make CTF "opt-in" though instead of "opt-out". I think the = current=20 > patch would enable CTF when building ports, for example. I think = instead it=20 > should default to not building CTF, but require an ENABLE_CTF (instead = of=20 > NO_CTF) to be set, and set that in bsd.kern.mk if WITH_CTF is defined. >=20 I have a patch at Yahoo that makes WITH_CTF settable from the kernel = config file, thus making it opt-in. I'd prefer this as well to opt-out. = Give me a little bit to dig it up and polish it for review. Scott