From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) by sourceware.org (Postfix) with ESMTP id 901673858012 for ; Sun, 9 May 2021 21:42:04 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.3.2 sourceware.org 901673858012 Received: from mail-qt1-f199.google.com (mail-qt1-f199.google.com [209.85.160.199]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-452-EmWsRDUlPxSIm7Fn01D_QA-1; Sun, 09 May 2021 17:42:02 -0400 X-MC-Unique: EmWsRDUlPxSIm7Fn01D_QA-1 Received: by mail-qt1-f199.google.com with SMTP id d13-20020a05622a05cdb02901c2cffd946bso9221739qtb.23 for ; Sun, 09 May 2021 14:42:02 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:to:references:from:organization :message-id:date:user-agent:mime-version:in-reply-to :content-language:content-transfer-encoding; bh=M95ESugu9meRtPiNt9Kmc7EI82KIIhtDwdV+vIgqT7s=; b=CDC1x/a/hjXOmWJF+aMLTwNI1cmG59L16asODy9Jb0+DvgzNGXpPrOo8LkvnA30Zl5 Hktcc9yA4vktTGzkj3+kysCsI+1ou4Os9sI20Cu6PrzroPkjRQREKrOr1b015l+acbHy ZiPENNxuYMe5Oa+MJPqIn6iH5egYQjJ4fRzHpmxEZrr5RNzVIMQ6dU3AogALiXCqkG3V y9swlLOowKVWt0E4ff8E2chs40d8RNi4miz/jq4eigdk2lnsZQTU7bmhisbJRaF7+wgE uIsruOPA+jY+jg/FuaFHRlPzObZAoerMHO2Wqulc6yzm7DjK4RG7sMaxbyerOcgdnImP vOpw== X-Gm-Message-State: AOAM530Xej7UUg+PqJct6KStAQc7hkeU+ET8T+JsA3viKvpoFQ5n2dEP DISM601gvxEENW0WoY95HXJFNOWvZQdT+KfjWxAmzubPX/mNAg06j16bhQGRZ+4MzZxdZbk/x3G Y58n8Wz8DkfqLzijWTSn4WrCND5h7bvyBdTh/dhcYow/frcJ7YCMMHta4bABLDwv5qa7Usg== X-Received: by 2002:ae9:e884:: with SMTP id a126mr20057883qkg.421.1620596522004; Sun, 09 May 2021 14:42:02 -0700 (PDT) X-Google-Smtp-Source: ABdhPJyVoaUzf7PrEgYXcZOXKKVtmOiXv/L+gdq43gtQ5hFOW3kwaejJtQjf5uQgpRiWvPe646H4AA== X-Received: by 2002:ae9:e884:: with SMTP id a126mr20057865qkg.421.1620596521769; Sun, 09 May 2021 14:42:01 -0700 (PDT) Received: from [192.168.1.16] (198-84-214-74.cpe.teksavvy.com. [198.84.214.74]) by smtp.gmail.com with ESMTPSA id g25sm9681080qtu.93.2021.05.09.14.42.00 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Sun, 09 May 2021 14:42:01 -0700 (PDT) Subject: Re: [PATCH 12/13] nptl: Move __default_pthread_attr, __default_pthread_attr_lock into libc To: Florian Weimer , libc-alpha@sourceware.org References: <2320d7987e65bc57dbef63c8af1448810cd2f3ea.1620323953.git.fweimer@redhat.com> From: Carlos O'Donell Organization: Red Hat Message-ID: <47057759-d7a0-fad5-4107-74a58f59d1a4@redhat.com> Date: Sun, 9 May 2021 17:42:00 -0400 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.8.1 MIME-Version: 1.0 In-Reply-To: <2320d7987e65bc57dbef63c8af1448810cd2f3ea.1620323953.git.fweimer@redhat.com> X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit X-Spam-Status: No, score=-12.0 required=5.0 tests=BAYES_00, DKIMWL_WL_HIGH, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, GIT_PATCH_0, NICE_REPLY_A, RCVD_IN_DNSWL_LOW, RCVD_IN_MSPIKE_H4, RCVD_IN_MSPIKE_WL, SPF_HELO_NONE, SPF_PASS, TXREP autolearn=ham autolearn_force=no version=3.4.2 X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on server2.sourceware.org X-BeenThere: libc-alpha@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Libc-alpha mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 09 May 2021 21:42:05 -0000 On 5/6/21 2:11 PM, Florian Weimer via Libc-alpha wrote: > The GLIBC_PRIVATE exports for these symbols are expected to be > temporary. > --- > nptl/Makefile | 2 +- > nptl/Versions | 2 ++ > nptl/pthreadP.h | 8 +++++--- > nptl/vars.c | 8 ++++++-- > 4 files changed, 14 insertions(+), 6 deletions(-) > > diff --git a/nptl/Makefile b/nptl/Makefile > index b5f26c6864..f7723cb808 100644 > --- a/nptl/Makefile > +++ b/nptl/Makefile > @@ -170,6 +170,7 @@ routines = \ > sem_wait \ > tpp \ > unwind \ > + vars \ > > shared-only-routines = forward > static-only-routines = pthread_atfork > @@ -212,7 +213,6 @@ libpthread-routines = \ > pthread_sigqueue \ > pthread_timedjoin \ > pthread_tryjoin \ > - vars \ > version \ > > libpthread-shared-only-routines = \ > diff --git a/nptl/Versions b/nptl/Versions > index fb15a7e8eb..d439a023b7 100644 > --- a/nptl/Versions > +++ b/nptl/Versions > @@ -298,6 +298,8 @@ libc { > tss_set; > } > GLIBC_PRIVATE { > + __default_pthread_attr; > + __default_pthread_attr_lock; > __futex_abstimed_wait64; > __futex_abstimed_wait_cancelable64; > __init_sched_fifo_prio; > diff --git a/nptl/pthreadP.h b/nptl/pthreadP.h > index 3a6b436400..6b912f053b 100644 > --- a/nptl/pthreadP.h > +++ b/nptl/pthreadP.h > @@ -198,9 +198,11 @@ enum > > > /* Default pthread attributes. */ > -extern union pthread_attr_transparent __default_pthread_attr attribute_hidden; > -extern int __default_pthread_attr_lock attribute_hidden; > -/* Called from __libpthread_freeres to deallocate the default attribute. */ > +extern union pthread_attr_transparent __default_pthread_attr; > +libc_hidden_proto (__default_pthread_attr) > +extern int __default_pthread_attr_lock; > +libc_hidden_proto (__default_pthread_attr_lock) > +/* Called from __libc_freeres to deallocate the default attribute. */ > extern void __default_pthread_attr_freeres (void) attribute_hidden; > > /* Size and alignment of static TLS block. */ > diff --git a/nptl/vars.c b/nptl/vars.c > index 03a6cc84be..989d7930e0 100644 > --- a/nptl/vars.c > +++ b/nptl/vars.c > @@ -22,7 +22,11 @@ > > /* Default thread attributes for the case when the user does not > provide any. */ > -union pthread_attr_transparent __default_pthread_attr attribute_hidden; > +union pthread_attr_transparent __default_pthread_attr > + __attribute__ ((nocommon)); > +libc_hidden_data_def (__default_pthread_attr) > > /* Mutex protecting __default_pthread_attr. */ > -int __default_pthread_attr_lock = LLL_LOCK_INITIALIZER; > +int __default_pthread_attr_lock __attribute__ ((nocommon)) > + = LLL_LOCK_INITIALIZER; > +libc_hidden_data_def (__default_pthread_attr_lock) > -- Cheers, Carlos.