From patchwork Tue May 19 10:44:48 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Florian Weimer X-Patchwork-Id: 39311 Return-Path: X-Original-To: patchwork@sourceware.org Delivered-To: patchwork@sourceware.org Received: from server2.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id 2C00B3954C79; Tue, 19 May 2020 10:44:56 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 2C00B3954C79 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sourceware.org; s=default; t=1589885096; bh=7rxYsi1V3oYB9YqOsS48SBDBpF0sslG6fzahCCbcL0c=; h=To:Subject:In-Reply-To:References:Date:List-Id:List-Unsubscribe: List-Archive:List-Post:List-Help:List-Subscribe:From:Reply-To: From; b=CCXHVWtrsyKh5GX+wUMKW2+7NXyGxbej3uvdZPO3wZ3yOTcV2RJ3bqSU5yzxj0Kja rFLAlZvlPNlcbaUkKUEmDHwcOysRwtiqWFV0SFEHu1wmxoR4aMTqEJzDcV48YHGuah +Tom8JHdFyZfFElnptF3Hi18zEWDR5ok8c/V3gF0= X-Original-To: libc-alpha@sourceware.org Delivered-To: libc-alpha@sourceware.org Received: from us-smtp-1.mimecast.com (us-smtp-2.mimecast.com [207.211.31.81]) by sourceware.org (Postfix) with ESMTP id 616DD3954452 for ; Tue, 19 May 2020 10:44:53 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.3.2 sourceware.org 616DD3954452 Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-455-VXMMmU6ZM7mZ3x74KMm2Cg-1; Tue, 19 May 2020 06:44:51 -0400 X-MC-Unique: VXMMmU6ZM7mZ3x74KMm2Cg-1 Received: from smtp.corp.redhat.com (int-mx03.intmail.prod.int.phx2.redhat.com [10.5.11.13]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id D192F100CCC1 for ; Tue, 19 May 2020 10:44:50 +0000 (UTC) Received: from oldenburg2.str.redhat.com (ovpn-113-18.ams2.redhat.com [10.36.113.18]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 2A291707B1 for ; Tue, 19 May 2020 10:44:50 +0000 (UTC) To: libc-alpha@sourceware.org Subject: [PATCH 14/19] nptl: Change type of __default_pthread_attr In-Reply-To: References: Message-Id: Date: Tue, 19 May 2020 12:44:48 +0200 User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/26.3 (gnu/linux) MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.13 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com X-Spam-Status: No, score=-4.1 required=5.0 tests=BAYES_00, DKIMWL_WL_HIGH, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, GIT_PATCH_0, RCVD_IN_DNSWL_NONE, RCVD_IN_MSPIKE_H4, RCVD_IN_MSPIKE_WL, SPF_HELO_NONE, SPF_PASS, TXREP, UNWANTED_LANGUAGE_BODY 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-Patchwork-Original-From: Florian Weimer via Libc-alpha From: Florian Weimer Reply-To: Florian Weimer Errors-To: libc-alpha-bounces@sourceware.org Sender: "Libc-alpha" union pthread_attr_transparent has always the correct size, even if pthread_attr_t has padding that is not present in struct pthread_attr. This should not result in an observable behavioral change. The existing code appears to have been correct, but it was brittle because it was not clear which functions were allowed to write to an entire pthread_attr_t argument (e.g., by copying it). Reviewed-by: Carlos O'Donell --- nptl/allocatestack.c | 2 +- nptl/nptl-init.c | 4 ++-- nptl/pthreadP.h | 2 +- nptl/pthread_attr_getstacksize.c | 2 +- nptl/pthread_create.c | 8 ++++---- nptl/pthread_getattr_default_np.c | 3 +-- nptl/pthread_setattr_default_np.c | 6 +++--- nptl/vars.c | 2 +- 8 files changed, 14 insertions(+), 15 deletions(-) diff --git a/nptl/allocatestack.c b/nptl/allocatestack.c index c94980c21c..d16f3d71f8 100644 --- a/nptl/allocatestack.c +++ b/nptl/allocatestack.c @@ -425,7 +425,7 @@ allocate_stack (const struct pthread_attr *attr, struct pthread **pdp, else { lll_lock (__default_pthread_attr_lock, LLL_PRIVATE); - size = __default_pthread_attr.stacksize; + size = __default_pthread_attr.internal.stacksize; lll_unlock (__default_pthread_attr_lock, LLL_PRIVATE); } diff --git a/nptl/nptl-init.c b/nptl/nptl-init.c index 96b1444a01..d4cf20e3d1 100644 --- a/nptl/nptl-init.c +++ b/nptl/nptl-init.c @@ -318,8 +318,8 @@ __pthread_initialize_minimal_internal (void) /* Round the resource limit up to page size. */ limit.rlim_cur = ALIGN_UP (limit.rlim_cur, pagesz); lll_lock (__default_pthread_attr_lock, LLL_PRIVATE); - __default_pthread_attr.stacksize = limit.rlim_cur; - __default_pthread_attr.guardsize = GLRO (dl_pagesize); + __default_pthread_attr.internal.stacksize = limit.rlim_cur; + __default_pthread_attr.internal.guardsize = GLRO (dl_pagesize); lll_unlock (__default_pthread_attr_lock, LLL_PRIVATE); #ifdef SHARED diff --git a/nptl/pthreadP.h b/nptl/pthreadP.h index 9c6dd41b7c..acc8e88e4a 100644 --- a/nptl/pthreadP.h +++ b/nptl/pthreadP.h @@ -199,7 +199,7 @@ enum /* Default pthread attributes. */ -extern struct pthread_attr __default_pthread_attr attribute_hidden; +extern union pthread_attr_transparent __default_pthread_attr attribute_hidden; extern int __default_pthread_attr_lock attribute_hidden; /* Size and alignment of static TLS block. */ diff --git a/nptl/pthread_attr_getstacksize.c b/nptl/pthread_attr_getstacksize.c index 346b375690..9830a635a6 100644 --- a/nptl/pthread_attr_getstacksize.c +++ b/nptl/pthread_attr_getstacksize.c @@ -33,7 +33,7 @@ __pthread_attr_getstacksize (const pthread_attr_t *attr, size_t *stacksize) if (size == 0) { lll_lock (__default_pthread_attr_lock, LLL_PRIVATE); - size = __default_pthread_attr.stacksize; + size = __default_pthread_attr.internal.stacksize; lll_unlock (__default_pthread_attr_lock, LLL_PRIVATE); } *stacksize = size; diff --git a/nptl/pthread_create.c b/nptl/pthread_create.c index 347d510707..86fbeb5218 100644 --- a/nptl/pthread_create.c +++ b/nptl/pthread_create.c @@ -612,16 +612,16 @@ __pthread_create_2_1 (pthread_t *newthread, const pthread_attr_t *attr, STACK_VARIABLES; const struct pthread_attr *iattr = (struct pthread_attr *) attr; - struct pthread_attr default_attr; + union pthread_attr_transparent default_attr; bool destroy_default_attr = false; bool c11 = (attr == ATTR_C11_THREAD); if (iattr == NULL || c11) { - int ret = __pthread_getattr_default_np ((pthread_attr_t *) &default_attr); + int ret = __pthread_getattr_default_np (&default_attr.external); if (ret != 0) return ret; destroy_default_attr = true; - iattr = &default_attr; + iattr = &default_attr.internal; } struct pthread *pd = NULL; @@ -852,7 +852,7 @@ __pthread_create_2_1 (pthread_t *newthread, const pthread_attr_t *attr, out: if (destroy_default_attr) - __pthread_attr_destroy ((pthread_attr_t *) &default_attr); + __pthread_attr_destroy (&default_attr.external); return retval; } diff --git a/nptl/pthread_getattr_default_np.c b/nptl/pthread_getattr_default_np.c index 5c99f980e2..f3ce1c2885 100644 --- a/nptl/pthread_getattr_default_np.c +++ b/nptl/pthread_getattr_default_np.c @@ -22,8 +22,7 @@ int __pthread_getattr_default_np (pthread_attr_t *out) { lll_lock (__default_pthread_attr_lock, LLL_PRIVATE); - int ret = __pthread_attr_copy (out, - (pthread_attr_t *) &__default_pthread_attr); + int ret = __pthread_attr_copy (out, &__default_pthread_attr.external); lll_unlock (__default_pthread_attr_lock, LLL_PRIVATE); return ret; } diff --git a/nptl/pthread_setattr_default_np.c b/nptl/pthread_setattr_default_np.c index eb5d24d3bd..c4cfb4e8ef 100644 --- a/nptl/pthread_setattr_default_np.c +++ b/nptl/pthread_setattr_default_np.c @@ -68,15 +68,15 @@ pthread_setattr_default_np (const pthread_attr_t *in) /* Preserve the previous stack size (see above). */ if (temp.internal.stacksize == 0) - temp.internal.stacksize = __default_pthread_attr.stacksize; + temp.internal.stacksize = __default_pthread_attr.internal.stacksize; /* Destroy the old attribute structure because it will be overwritten. */ - __pthread_attr_destroy ((pthread_attr_t *) &__default_pthread_attr); + __pthread_attr_destroy (&__default_pthread_attr.external); /* __default_pthread_attr takes ownership, so do not free attrs.internal after this point. */ - __default_pthread_attr = temp.internal; + __default_pthread_attr = temp; lll_unlock (__default_pthread_attr_lock, LLL_PRIVATE); return ret; diff --git a/nptl/vars.c b/nptl/vars.c index b88300d9b4..3696020145 100644 --- a/nptl/vars.c +++ b/nptl/vars.c @@ -22,7 +22,7 @@ /* Default thread attributes for the case when the user does not provide any. */ -struct pthread_attr __default_pthread_attr attribute_hidden; +union pthread_attr_transparent __default_pthread_attr attribute_hidden; /* Mutex protecting __default_pthread_attr. */ int __default_pthread_attr_lock = LLL_LOCK_INITIALIZER;