[rtems commit] score: Simplify TLS area allocation
Sebastian Huber
sebh at rtems.org
Wed Feb 12 15:11:59 UTC 2020
Module: rtems
Branch: master
Commit: f4dbf37dd432623c345f8e19f78a4eb01fcedb8b
Changeset: http://git.rtems.org/rtems/commit/?id=f4dbf37dd432623c345f8e19f78a4eb01fcedb8b
Author: Sebastian Huber <sebastian.huber at embedded-brains.de>
Date: Mon Dec 9 10:12:14 2019 +0100
score: Simplify TLS area allocation
Use the stack area to allocate the TLS area.
Update #3835.
---
cpukit/Makefile.am | 2 +
cpukit/include/rtems/confdefs.h | 3 +-
cpukit/include/rtems/config.h | 8 +---
cpukit/include/rtems/score/stack.h | 11 ++++-
cpukit/include/rtems/score/stackimpl.h | 4 +-
cpukit/include/rtems/score/tls.h | 25 ++----------
cpukit/score/src/configstackspacesize.c | 54 +++++++++++++++++++++++++
cpukit/score/src/threadinitialize.c | 30 +++++++-------
cpukit/score/src/threadrestart.c | 2 -
cpukit/score/src/tlsallocsize.c | 72 +++++++++++++++++++++++++++++++++
cpukit/score/src/wkspace.c | 38 -----------------
11 files changed, 161 insertions(+), 88 deletions(-)
diff --git a/cpukit/Makefile.am b/cpukit/Makefile.am
index 9182919..303c0af 100644
--- a/cpukit/Makefile.am
+++ b/cpukit/Makefile.am
@@ -1009,6 +1009,7 @@ librtemscpu_a_SOURCES += score/src/ioprintf.c
librtemscpu_a_SOURCES += score/src/iovprintf.c
librtemscpu_a_SOURCES += score/src/isrisinprogress.c
librtemscpu_a_SOURCES += score/src/condition.c
+librtemscpu_a_SOURCES += score/src/configstackspacesize.c
librtemscpu_a_SOURCES += score/src/futex.c
librtemscpu_a_SOURCES += score/src/profilingisrentryexit.c
librtemscpu_a_SOURCES += score/src/mutex.c
@@ -1019,6 +1020,7 @@ librtemscpu_a_SOURCES += score/src/smpbarrierwait.c
librtemscpu_a_SOURCES += score/src/kern_tc.c
librtemscpu_a_SOURCES += score/src/libatomic.c
librtemscpu_a_SOURCES += score/src/processormaskcopy.c
+librtemscpu_a_SOURCES += score/src/tlsallocsize.c
librtemscpu_a_SOURCES += sapi/src/chainappendnotify.c
librtemscpu_a_SOURCES += sapi/src/chaingetnotify.c
librtemscpu_a_SOURCES += sapi/src/chaingetwait.c
diff --git a/cpukit/include/rtems/confdefs.h b/cpukit/include/rtems/confdefs.h
index a356a94..7fc1cf9 100644
--- a/cpukit/include/rtems/confdefs.h
+++ b/cpukit/include/rtems/confdefs.h
@@ -2726,12 +2726,13 @@ struct _reent *__getreent(void)
uint32_t rtems_minimum_stack_size =
CONFIGURE_MINIMUM_TASK_STACK_SIZE;
+ const uintptr_t _Stack_Space_size = _CONFIGURE_STACK_SPACE_SIZE;
+
/**
* This is the primary Configuration Table for this application.
*/
const rtems_configuration_table Configuration = {
CONFIGURE_EXECUTIVE_RAM_SIZE, /* required RTEMS workspace */
- _CONFIGURE_STACK_SPACE_SIZE, /* required stack space */
CONFIGURE_MAXIMUM_USER_EXTENSIONS, /* maximum dynamic extensions */
CONFIGURE_MICROSECONDS_PER_TICK, /* microseconds per clock tick */
CONFIGURE_TICKS_PER_TIMESLICE, /* ticks per timeslice quantum */
diff --git a/cpukit/include/rtems/config.h b/cpukit/include/rtems/config.h
index fac5c6e..725f45d 100644
--- a/cpukit/include/rtems/config.h
+++ b/cpukit/include/rtems/config.h
@@ -96,11 +96,6 @@ typedef struct {
uintptr_t work_space_size;
/**
- * This field specifies the size in bytes of the RTEMS thread stack space.
- */
- uintptr_t stack_space_size;
-
- /**
* This field contains the maximum number of POSIX API
* key value pairs which are configured for this application.
*
@@ -191,8 +186,7 @@ extern const rtems_configuration_table Configuration;
#define rtems_configuration_get_stack_allocator_avoids_work_space() \
(Configuration.stack_allocator_avoids_work_space)
-#define rtems_configuration_get_stack_space_size() \
- (Configuration.stack_space_size)
+uintptr_t rtems_configuration_get_stack_space_size( void );
#define rtems_configuration_get_work_space_size() \
(Configuration.work_space_size + \
diff --git a/cpukit/include/rtems/score/stack.h b/cpukit/include/rtems/score/stack.h
index 1158f34..2ad362a 100644
--- a/cpukit/include/rtems/score/stack.h
+++ b/cpukit/include/rtems/score/stack.h
@@ -58,12 +58,19 @@ typedef struct {
} Stack_Control;
/**
- * This variable contains the the minimum stack size;
+ * @brief The minimum stack size.
*
- * @note It is instantiated and set by User Configuration via confdefs.h.
+ * Application provided via <rtems/confdefs.h>.
*/
extern uint32_t rtems_minimum_stack_size;
+/**
+ * @brief The configured stack space size.
+ *
+ * Application provided via <rtems/confdefs.h>.
+ */
+extern const uintptr_t _Stack_Space_size;
+
/** @} */
#ifdef __cplusplus
diff --git a/cpukit/include/rtems/score/stackimpl.h b/cpukit/include/rtems/score/stackimpl.h
index 6070453..aafde5d 100644
--- a/cpukit/include/rtems/score/stackimpl.h
+++ b/cpukit/include/rtems/score/stackimpl.h
@@ -23,6 +23,7 @@
#include <rtems/score/stack.h>
#include <rtems/score/context.h>
+#include <rtems/score/tls.h>
#ifdef __cplusplus
extern "C" {
@@ -87,7 +88,8 @@ RTEMS_INLINE_ROUTINE bool _Stack_Is_enough(
{
size_t minimum;
- minimum = _Stack_Minimum();
+ minimum = _TLS_Get_allocation_size();
+ minimum += _Stack_Minimum();
#if ( CPU_HARDWARE_FP == TRUE ) || ( CPU_SOFTWARE_FP == TRUE )
if ( is_fp ) {
diff --git a/cpukit/include/rtems/score/tls.h b/cpukit/include/rtems/score/tls.h
index dadcb77..1b9dae1 100644
--- a/cpukit/include/rtems/score/tls.h
+++ b/cpukit/include/rtems/score/tls.h
@@ -134,30 +134,11 @@ static inline uintptr_t _TLS_Get_thread_control_block_area_size(
}
/**
- * @brief Returns the actual size that has to be allocated for this size and
- * alignment.
+ * @brief Return the TLS area allocation size.
*
- * @param size The size for the operation.
- * @param alignment The alignment for the operation.
- *
- * @return The actual allocation size.
+ * @return The TLS area allocation size.
*/
-static inline uintptr_t _TLS_Get_allocation_size(
- uintptr_t size,
- uintptr_t alignment
-)
-{
- uintptr_t allocation_size = 0;
-
- allocation_size += _TLS_Heap_align_up( size );
- allocation_size += _TLS_Get_thread_control_block_area_size( alignment );
-
-#ifndef __i386__
- allocation_size += sizeof(TLS_Dynamic_thread_vector);
-#endif
-
- return allocation_size;
-}
+uintptr_t _TLS_Get_allocation_size( void );
/**
* @brief Copies TLS size bytes from the address tls_area and returns a pointer
diff --git a/cpukit/score/src/configstackspacesize.c b/cpukit/score/src/configstackspacesize.c
new file mode 100644
index 0000000..4f0c24c
--- /dev/null
+++ b/cpukit/score/src/configstackspacesize.c
@@ -0,0 +1,54 @@
+/*
+ * SPDX-License-Identifier: BSD-2-Clause
+ *
+ * Copyright (C) 2014, 2019 embedded brains GmbH
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ * 1. Redistributions of source code must retain the above copyright
+ * notice, this list of conditions and the following disclaimer.
+ * 2. Redistributions in binary form must reproduce the above copyright
+ * notice, this list of conditions and the following disclaimer in the
+ * documentation and/or other materials provided with the distribution.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS"
+ * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
+ * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
+ * ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT OWNER OR CONTRIBUTORS BE
+ * LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
+ * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
+ * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
+ * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
+ * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
+ * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
+ * POSSIBILITY OF SUCH DAMAGE.
+ */
+
+#if HAVE_CONFIG_H
+#include "config.h"
+#endif
+
+#include <rtems/config.h>
+#include <rtems/score/stack.h>
+#include <rtems/score/tls.h>
+#include <rtems/score/thread.h>
+
+uintptr_t rtems_configuration_get_stack_space_size( void )
+{
+ uintptr_t space_size;
+
+ space_size = _Stack_Space_size;
+
+ /*
+ * In case we have a non-zero TLS size, then we need a TLS area for each
+ * thread. These areas are allocated within the stack area from the stack
+ * space. Ensure that the stack space is large enough to fulfill all requests
+ * known at configuration time (so excluding the unlimited option). It is not
+ * possible to estimate the TLS size in the configuration at compile-time.
+ * The TLS size is determined at application link-time.
+ */
+ space_size += _Thread_Initial_thread_count * _TLS_Get_allocation_size();
+
+ return space_size;
+}
diff --git a/cpukit/score/src/threadinitialize.c b/cpukit/score/src/threadinitialize.c
index 4844482..a1bc3b2 100644
--- a/cpukit/score/src/threadinitialize.c
+++ b/cpukit/score/src/threadinitialize.c
@@ -42,7 +42,7 @@ bool _Thread_Initialize(
Objects_Name name
)
{
- uintptr_t tls_size = _TLS_Get_size();
+ uintptr_t tls_size;
bool extension_status;
size_t i;
Scheduler_Node *scheduler_node;
@@ -83,6 +83,8 @@ bool _Thread_Initialize(
(char *) the_thread + add_on->source_offset;
}
+ tls_size = _TLS_Get_allocation_size();
+
/* Allocate the stack for this thread */
#if defined(RTEMS_SCORE_THREAD_ENABLE_USER_PROVIDED_STACK_VIA_API)
if ( stack_area == NULL ) {
@@ -95,6 +97,7 @@ bool _Thread_Initialize(
}
#endif
+ stack_size += tls_size;
stack_area = _Stack_Allocate( stack_size );
if ( stack_area == NULL ) {
@@ -116,6 +119,17 @@ bool _Thread_Initialize(
}
#endif
+ /* Allocate thread-local storage (TLS) area in stack area */
+ if ( tls_size > 0 ) {
+ uintptr_t tls_align;
+
+ tls_align = (uintptr_t) _TLS_Alignment;
+ the_thread->Start.tls_area = (void *)
+ ( ( (uintptr_t) stack_area + tls_align - 1 ) & ~( tls_align - 1 ) );
+ stack_size -= tls_size;
+ stack_area = (char *) stack_area + tls_size;
+ }
+
_Stack_Initialize(
&the_thread->Start.Initial_stack,
stack_area,
@@ -124,19 +138,6 @@ bool _Thread_Initialize(
scheduler_index = 0;
- /* Thread-local storage (TLS) area allocation */
- if ( tls_size > 0 ) {
- uintptr_t tls_align = _TLS_Heap_align_up( (uintptr_t) _TLS_Alignment );
- uintptr_t tls_alloc = _TLS_Get_allocation_size( tls_size, tls_align );
-
- the_thread->Start.tls_area =
- _Workspace_Allocate_aligned( tls_alloc, tls_align );
-
- if ( the_thread->Start.tls_area == NULL ) {
- goto failed;
- }
- }
-
/*
* Get thread queue heads
*/
@@ -301,7 +302,6 @@ failed:
}
#endif
- _Workspace_Free( the_thread->Start.tls_area );
_Freechain_Put(
&information->Thread_queue_heads.Free,
the_thread->Wait.spare_heads
diff --git a/cpukit/score/src/threadrestart.c b/cpukit/score/src/threadrestart.c
index aa47fef..0c68903 100644
--- a/cpukit/score/src/threadrestart.c
+++ b/cpukit/score/src/threadrestart.c
@@ -185,8 +185,6 @@ static void _Thread_Free( Thread_Control *the_thread )
*/
_Stack_Free( the_thread->Start.allocated_stack );
- _Workspace_Free( the_thread->Start.tls_area );
-
#if defined(RTEMS_SMP)
_ISR_lock_Destroy( &the_thread->Scheduler.Lock );
_ISR_lock_Destroy( &the_thread->Wait.Lock.Default );
diff --git a/cpukit/score/src/tlsallocsize.c b/cpukit/score/src/tlsallocsize.c
new file mode 100644
index 0000000..0465798
--- /dev/null
+++ b/cpukit/score/src/tlsallocsize.c
@@ -0,0 +1,72 @@
+/*
+ * SPDX-License-Identifier: BSD-2-Clause
+ *
+ * Copyright (C) 2014, 2019 embedded brains GmbH
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ * 1. Redistributions of source code must retain the above copyright
+ * notice, this list of conditions and the following disclaimer.
+ * 2. Redistributions in binary form must reproduce the above copyright
+ * notice, this list of conditions and the following disclaimer in the
+ * documentation and/or other materials provided with the distribution.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS"
+ * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
+ * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
+ * ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT OWNER OR CONTRIBUTORS BE
+ * LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
+ * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
+ * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
+ * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
+ * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
+ * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
+ * POSSIBILITY OF SUCH DAMAGE.
+ */
+
+#if HAVE_CONFIG_H
+#include "config.h"
+#endif
+
+#include <rtems/score/tls.h>
+
+static uintptr_t _TLS_Allocation_size;
+
+uintptr_t _TLS_Get_allocation_size( void )
+{
+ uintptr_t size;
+ uintptr_t allocation_size;
+ uintptr_t alignment;
+
+ size = _TLS_Get_size();
+
+ if ( size == 0 ) {
+ return 0;
+ }
+
+ allocation_size = _TLS_Allocation_size;
+
+ if ( allocation_size == 0 ) {
+ allocation_size = _TLS_Heap_align_up( size );
+ alignment = _TLS_Heap_align_up( (uintptr_t) _TLS_Alignment );
+
+ /*
+ * The stack allocator does not support aligned allocations. Allocate
+ * enough to do the alignment manually.
+ */
+ if ( alignment > CPU_HEAP_ALIGNMENT ) {
+ allocation_size += alignment;
+ }
+
+ allocation_size += _TLS_Get_thread_control_block_area_size( alignment );
+
+#ifndef __i386__
+ allocation_size += sizeof(TLS_Dynamic_thread_vector);
+#endif
+
+ _TLS_Allocation_size = allocation_size;
+ }
+
+ return allocation_size;
+}
diff --git a/cpukit/score/src/wkspace.c b/cpukit/score/src/wkspace.c
index 5a87ee4..fbdb0b0 100644
--- a/cpukit/score/src/wkspace.c
+++ b/cpukit/score/src/wkspace.c
@@ -23,7 +23,6 @@
#include <rtems/score/heapimpl.h>
#include <rtems/score/interr.h>
#include <rtems/score/threadimpl.h>
-#include <rtems/score/tls.h>
#include <rtems/posix/pthread.h>
#include <rtems/config.h>
#include <rtems/sysinit.h>
@@ -37,41 +36,6 @@
Heap_Control _Workspace_Area;
-static uintptr_t _Workspace_Space_for_TLS( uintptr_t page_size )
-{
- uintptr_t tls_size;
- uintptr_t space;
-
- tls_size = _TLS_Get_size();
-
- /*
- * In case we have a non-zero TLS size, then we need a TLS area for each
- * thread. These areas are allocated from the workspace. Ensure that the
- * workspace is large enough to fulfill all requests known at configuration
- * time (so excluding the unlimited option). It is not possible to estimate
- * the TLS size in the configuration at compile-time. The TLS size is
- * determined at application link-time.
- */
- if ( tls_size > 0 ) {
- uintptr_t tls_align = _TLS_Heap_align_up( (uintptr_t) _TLS_Alignment );
- uintptr_t tls_alloc = _TLS_Get_allocation_size( tls_size, tls_align );
-
- /*
- * Memory allocated with an alignment constraint is allocated from the end
- * of a free block. The last allocation may need one free block of minimum
- * size.
- */
- space = _Heap_Min_block_size( page_size );
-
- space += _Thread_Initial_thread_count
- * _Heap_Size_with_overhead( page_size, tls_alloc, tls_align );
- } else {
- space = 0;
- }
-
- return space;
-}
-
static void _Workspace_Initialize( void )
{
_Workspace_Handler_initialization( _Memory_Get(), _Heap_Extend );
@@ -97,8 +61,6 @@ void _Workspace_Handler_initialization(
page_size = CPU_HEAP_ALIGNMENT;
remaining = rtems_configuration_get_work_space_size();
- remaining += _Workspace_Space_for_TLS( page_size );
-
init_or_extend = _Heap_Initialize;
unified = rtems_configuration_get_unified_work_area();
overhead = _Heap_Area_overhead( page_size );
More information about the vc
mailing list