From owner-cvs-all@FreeBSD.ORG Mon Aug 23 18:58:28 2004 Return-Path: Delivered-To: cvs-all@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 3310516A4CE; Mon, 23 Aug 2004 18:58:28 +0000 (GMT) Received: from itchy.rabson.org (mailgate.nlsystems.com [80.177.232.242]) by mx1.FreeBSD.org (Postfix) with ESMTP id 2BB2743D45; Mon, 23 Aug 2004 18:58:27 +0000 (GMT) (envelope-from dfr@nlsystems.com) Received: from ns0.nlsystems.com (ns0.nlsystems.com [80.177.232.243]) by itchy.rabson.org (8.12.11/8.12.11) with ESMTP id i7NIwMpp053935; Mon, 23 Aug 2004 19:58:22 +0100 (BST) (envelope-from dfr@nlsystems.com) From: Doug Rabson To: Marcel Moolenaar Date: Mon, 23 Aug 2004 19:58:41 +0100 User-Agent: KMail/1.6.2 References: <200408231530.i7NFU5bu082414@repoman.freebsd.org> <1093283234.16672.2.camel@builder02.qubesoft.com> <20040823181827.GA706@dhcp50.pn.xcllnt.net> In-Reply-To: <20040823181827.GA706@dhcp50.pn.xcllnt.net> MIME-Version: 1.0 Content-Disposition: inline Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Message-Id: <200408231958.41081.dfr@nlsystems.com> X-Spam-Status: No, hits=0.0 required=5.0 tests=none autolearn=no version=2.63 X-Spam-Checker-Version: SpamAssassin 2.63 (2004-01-11) on itchy.rabson.org X-Virus-Scanned: clamd / ClamAV version 0.75.1, clamav-milter version 0.75c on itchy.rabson.org X-Virus-Status: Clean cc: cvs-src@FreeBSD.org cc: src-committers@FreeBSD.org cc: cvs-all@FreeBSD.org Subject: Re: cvs commit: src/lib/libc/gen tls.c X-BeenThere: cvs-all@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: CVS commit messages for the entire tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 23 Aug 2004 18:58:28 -0000 On Monday 23 August 2004 19:18, Marcel Moolenaar wrote: > On Mon, Aug 23, 2004 at 06:47:15PM +0100, Doug Rabson wrote: > > On Mon, 2004-08-23 at 16:30, Doug Rabson wrote: > > > dfr 2004-08-23 15:30:05 UTC > > > > > > FreeBSD src repository > > > > > > Modified files: > > > lib/libc/gen tls.c > > > Log: > > > Make sure we allocate at least enough space for the TCB for > > > variant I TLS. > > > > This is the real fix for the static binaries segfaulting on alpha. > > With this we could re-add the change to crt1.c which enables TLS > > for static binaries. Given that we don't yet fully support TLS for > > alpha in either rtld or libpthread, I don't propose to change > > lib/csu/alpha/crt1.c again for 5.3. > > However, if we revert the backout of crt1.c, then we know that all > shared executables will work the moment we fix rtld and libpthread. > If we keep the call to _init_tls() removed from crt1.c then every > binary linked on a 5.3 machine needs to be relinked the moment we > do add proper TLS support. So, to minimize (future) burden I suggest > we change crt1.c back for 5.3 as well. Otherwise this will just be > another exception on Alpha... But then again, none of those static binaries are likely to include TLS (since its currently disabled for alpha). Dynamic binaries are not affected because when rtld gets involved, it handles the TLS initialisation istelf - _init_tls() is only used for static binaries and is empty for dynamic. > > The above obviously conditional upon re@ approval. I await the RE's judgement :-)