From patchwork Mon Nov 11 09:49:43 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Simon Marchi (Code Review)" X-Patchwork-Id: 35786 Received: (qmail 91164 invoked by alias); 11 Nov 2019 09:49:52 -0000 Mailing-List: contact libc-alpha-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: libc-alpha-owner@sourceware.org Delivered-To: mailing list libc-alpha@sourceware.org Received: (qmail 91139 invoked by uid 89); 11 Nov 2019 09:49:52 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-26.9 required=5.0 tests=BAYES_00, GIT_PATCH_0, GIT_PATCH_1, GIT_PATCH_2, GIT_PATCH_3, KAM_SHORT autolearn=ham version=3.3.1 spammy= X-HELO: mx1.osci.io X-Gerrit-PatchSet: 1 Date: Mon, 11 Nov 2019 04:49:43 -0500 From: "Florian Weimer (Code Review)" To: libc-alpha@sourceware.org Cc: Florian Weimer Message-ID: Auto-Submitted: auto-generated X-Gerrit-MessageType: newchange Subject: [review] support: Add support_set_small_thread_stack_size X-Gerrit-Change-Id: I1cf79a469984f8f30a4a947ee9ec2a5e74de8926 X-Gerrit-Change-Number: 608 X-Gerrit-ChangeURL: X-Gerrit-Commit: 5c91be8b33d9b0c8fc3c4c1800b1d571b5706dd8 References: Reply-To: fweimer@redhat.com, fweimer@redhat.com, libc-alpha@sourceware.org MIME-Version: 1.0 Content-Disposition: inline User-Agent: Gerrit/3.0.3-76-gf8b6da0ab5 Change URL: https://gnutoolchain-gerrit.osci.io/r/c/glibc/+/608 ...................................................................... support: Add support_set_small_thread_stack_size And support_small_stack_thread_attribute Change-Id: I1cf79a469984f8f30a4a947ee9ec2a5e74de8926 --- M support/Makefile A support/support_set_small_thread_stack_size.c A support/support_small_stack_thread_attribute.c M support/xthread.h 4 files changed, 94 insertions(+), 0 deletions(-) diff --git a/support/Makefile b/support/Makefile index ca238ee..a6081bb 100644 --- a/support/Makefile +++ b/support/Makefile @@ -63,7 +63,9 @@ support_quote_string \ support_record_failure \ support_run_diff \ + support_set_small_thread_stack_size \ support_shared_allocate \ + support_small_stack_thread_attribute \ support_subprocess \ support_test_compare_blob \ support_test_compare_failure \ diff --git a/support/support_set_small_thread_stack_size.c b/support/support_set_small_thread_stack_size.c new file mode 100644 index 0000000..23189fd --- /dev/null +++ b/support/support_set_small_thread_stack_size.c @@ -0,0 +1,30 @@ +/* Set a small stack size, but still large enough for glibc's needs. + Copyright (C) 2019 Free Software Foundation, Inc. + This file is part of the GNU C Library. + + The GNU C Library is free software; you can redistribute it and/or + modify it under the terms of the GNU Lesser General Public + License as published by the Free Software Foundation; either + version 2.1 of the License, or (at your option) any later version. + + The GNU C Library is distributed in the hope that it will be useful, + but WITHOUT ANY WARRANTY; without even the implied warranty of + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + Lesser General Public License for more details. + + You should have received a copy of the GNU Lesser General Public + License along with the GNU C Library; if not, see + . */ + +#include +#include +#include + +void +support_set_small_thread_stack_size (pthread_attr_t *attr) +{ + /* Some architectures have too small values for PTHREAD_STACK_MIN + which cannot be used for creating threads. Ensure that the stack + size is at least 256 KiB. */ + xpthread_attr_setstacksize (attr, MAX (256 * 1024, PTHREAD_STACK_MIN)); +} diff --git a/support/support_small_stack_thread_attribute.c b/support/support_small_stack_thread_attribute.c new file mode 100644 index 0000000..a65097d --- /dev/null +++ b/support/support_small_stack_thread_attribute.c @@ -0,0 +1,54 @@ +/* A thread attribute with a small stack. + Copyright (C) 2019 Free Software Foundation, Inc. + This file is part of the GNU C Library. + + The GNU C Library is free software; you can redistribute it and/or + modify it under the terms of the GNU Lesser General Public + License as published by the Free Software Foundation; either + version 2.1 of the License, or (at your option) any later version. + + The GNU C Library is distributed in the hope that it will be useful, + but WITHOUT ANY WARRANTY; without even the implied warranty of + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + Lesser General Public License for more details. + + You should have received a copy of the GNU Lesser General Public + License along with the GNU C Library; if not, see + . */ + +#include +#include + +static void * +allocate (void *closure) +{ + pthread_attr_t *result = malloc (sizeof (*result)); + xpthread_attr_init (result); + support_set_small_thread_stack_size (result); + return result; +} + +static void +deallocate (void *ptr, void *closure) +{ + xpthread_attr_destroy (ptr); + free (ptr); +} + +static void *small_stack_attr; + +pthread_attr_t * +support_small_stack_thread_attribute (void) +{ + return allocate_once (&small_stack_attr, allocate, deallocate, NULL); +} + +static void __attribute__ ((destructor)) +fini (void) +{ + if (small_stack_attr != NULL) + { + deallocate (small_stack_attr, NULL); + small_stack_attr = NULL; + } +} diff --git a/support/xthread.h b/support/xthread.h index 829397c..f39720c 100644 --- a/support/xthread.h +++ b/support/xthread.h @@ -75,6 +75,14 @@ void xpthread_attr_setguardsize (pthread_attr_t *attr, size_t guardsize); +/* Set the stack size in ATTR to a small value, but still large enough + to cover most internal glibc stack usage. */ +void support_set_small_thread_stack_size (pthread_attr_t *attr); + +/* Return a pointer to a thread attribute which requests a small + stack. The caller must not free this pointer. */ +pthread_attr_t *support_small_stack_thread_attribute (void); + /* This function returns non-zero if pthread_barrier_wait returned PTHREAD_BARRIER_SERIAL_THREAD. */ int xpthread_barrier_wait (pthread_barrier_t *barrier);