From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 6803 invoked by alias); 1 Oct 2004 08:38:51 -0000 Mailing-List: contact libc-hacker-help@sources.redhat.com; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: libc-hacker-owner@sources.redhat.com Received: (qmail 6787 invoked from network); 1 Oct 2004 08:38:50 -0000 Received: from unknown (HELO sunsite.ms.mff.cuni.cz) (195.113.15.26) by sourceware.org with SMTP; 1 Oct 2004 08:38:50 -0000 Received: from sunsite.ms.mff.cuni.cz (sunsite.mff.cuni.cz [127.0.0.1]) by sunsite.ms.mff.cuni.cz (8.12.8/8.12.8) with ESMTP id i918cB3j003574; Fri, 1 Oct 2004 10:38:11 +0200 Received: (from jakub@localhost) by sunsite.ms.mff.cuni.cz (8.12.8/8.12.8/Submit) id i918cAIe003572; Fri, 1 Oct 2004 10:38:10 +0200 Date: Fri, 01 Oct 2004 08:38:00 -0000 From: Jakub Jelinek To: Thorsten Kukuk Cc: Glibc hackers Subject: Re: [PATCH] Add multiple inclusion guards to useldt.h Message-ID: <20041001083810.GX30497@sunsite.ms.mff.cuni.cz> Reply-To: Jakub Jelinek References: <20040925081040.GR30497@sunsite.ms.mff.cuni.cz> <20041001083338.GA16316@suse.de> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20041001083338.GA16316@suse.de> User-Agent: Mutt/1.4.1i X-SW-Source: 2004-10/txt/msg00001.txt.bz2 On Fri, Oct 01, 2004 at 10:33:39AM +0200, Thorsten Kukuk wrote: > On Sat, Sep 25, Jakub Jelinek wrote: > > > Hi! > > > > >From reading the code, I believe i686 --with-tls linuxthreads > > are now broken. > > It is still broken for me using --with-tls --without-__thread. > > All thread test cases will be killed. I know, I'm using following patch ATM, but didn't have time to figure out why exactly current CVS doesn't work, which would be good to understand so that it can be fixed for real. 2004-09-26 Jakub Jelinek * sysdeps/i386/tls.h: Only include useldt.h if THREAD_SELF is not defined. * sysdeps/i386/i686/pt-machine.h: Revert last change. --- libc/linuxthreads/sysdeps/i386/tls.h 25 Sep 2004 03:32:13 -0000 1.36 +++ libc/linuxthreads/sysdeps/i386/tls.h 26 Sep 2004 08:45:21 -0000 1.35.2.3 @@ -113,7 +113,9 @@ typedef struct # define TLS_LOAD_EBX # endif -# include "useldt.h" /* For the structure. */ +# ifndef THREAD_SELF +# include "useldt.h" /* For the structure. */ +# endif # if __ASSUME_LDT_WORKS > 0 # define TLS_DO_MODIFY_LDT_KERNEL_CHECK(doit) (doit) /* Nothing to check. */ # else --- libc/linuxthreads/sysdeps/i386/i686/pt-machine.h 25 Sep 2004 10:04:15 -0000 1.23 +++ libc/linuxthreads/sysdeps/i386/i686/pt-machine.h 26 Sep 2004 08:45:22 -0000 1.22.2.2 @@ -69,9 +69,7 @@ __compare_and_swap (long int *p, long in } #endif -/* If tls.h was included, it will include useldt.h after defining USE_TLS. - We don't want to include it here first when tls.h includes us. */ -#if __ASSUME_LDT_WORKS > 0 && !defined _TLS_H +#if __ASSUME_LDT_WORKS > 0 #include "../useldt.h" #endif Jakub