Mirrors_Framework/nuttx
1
0
Fork 0
mirror of https://github.com/apache/nuttx.git synced 2026-06-06 08:36:24 +08:00
Code Issues Actions Packages Projects Releases Wiki Activity

TLS_UNALIGNED (#2)

Browse Source 
* Implement the TLS_UNALIGNED
This commit is contained in:
Abdelatif Guettouche
2020-05-05 16:18:25 +01:00
parent b1071cca00
commit b7e7fba732
10 changed files with 59 additions and 18 deletions
Download Patch File Download Diff File Expand all files Collapse all files
arch/arm/include/tls.h
+16
View File
@@ -99,8 +99,24 @@ static inline uint32_t up_getsp(void)
static inline FAR struct tls_info_s *up_tls_info(void)
{
#ifdef CONFIG_TLS_ALIGNED
DEBUGASSERT(!up_interrupt_context());
return TLS_INFO((uintptr_t)up_getsp());
#else
FAR struct tls_info_s *info = NULL;
struct stackinfo_s stackinfo;
int ret;
DEBUGASSERT(!up_interrupt_context());
ret = sched_get_stackinfo(0, &stackinfo);
if (ret == OK)
{
info = (FAR struct tsl_info_s *)stackinfo.adj_stack_ptr;
}
return info;
#endif
}
#endif /* CONFIG_TLS */
arch/arm/src/common/arm_checkstack.c
+1 -1
View File
@@ -92,7 +92,7 @@ static size_t do_stackcheck(uintptr_t alloc, size_t size, bool int_stack)
/* Get aligned addresses of the top and bottom of the stack */
#ifdef CONFIG_TLS
#ifdef CONFIG_TLS_ALIGNED
if (!int_stack)
{
/* Skip over the TLS data structure at the bottom of the stack */
arch/arm/src/common/arm_createstack.c
+5 -3
View File
@@ -107,6 +107,7 @@ int up_create_stack(FAR struct tcb_s *tcb, size_t stack_size, uint8_t ttype)
stack_size += sizeof(struct tls_info_s);
#ifdef CONFIG_TLS_ALIGNED
/* The allocated stack size must not exceed the maximum possible for the
* TLS feature.
*/
@@ -116,6 +117,7 @@ int up_create_stack(FAR struct tcb_s *tcb, size_t stack_size, uint8_t ttype)
{
stack_size = TLS_MAXSTACK;
}
#endif
#endif
/* Is there already a stack allocated of a different size? Because of
@@ -139,7 +141,7 @@ int up_create_stack(FAR struct tcb_s *tcb, size_t stack_size, uint8_t ttype)
* If TLS is enabled, then we must allocate aligned stacks.
*/
#ifdef CONFIG_TLS
#ifdef CONFIG_TLS_ALIGNED
#ifdef CONFIG_MM_KERNEL_HEAP
/* Use the kernel allocator if this is a kernel thread */
@@ -157,7 +159,7 @@ int up_create_stack(FAR struct tcb_s *tcb, size_t stack_size, uint8_t ttype)
(uint32_t *)kumm_memalign(TLS_STACK_ALIGN, stack_size);
}
#else /* CONFIG_TLS */
#else /* CONFIG_TLS_ALIGNED */
#ifdef CONFIG_MM_KERNEL_HEAP
/* Use the kernel allocator if this is a kernel thread */
@@ -172,7 +174,7 @@ int up_create_stack(FAR struct tcb_s *tcb, size_t stack_size, uint8_t ttype)
tcb->stack_alloc_ptr = (uint32_t *)kumm_malloc(stack_size);
}
#endif /* CONFIG_TLS */
#endif /* CONFIG_TLS_ALIGNED */
#ifdef CONFIG_DEBUG_FEATURES
/* Was the allocation successful? */
arch/arm/src/common/arm_usestack.c
+1 -1
View File
@@ -90,7 +90,7 @@ int up_use_stack(struct tcb_s *tcb, void *stack, size_t stack_size)
size_t top_of_stack;
size_t size_of_stack;
#ifdef CONFIG_TLS
#ifdef CONFIG_TLS_ALIGNED
/* Make certain that the user provided stack is properly aligned */
DEBUGASSERT(((uintptr_t)stack & TLS_STACK_MASK) == 0);
arch/sim/include/tls.h
+16
View File
@@ -81,7 +81,23 @@
static inline FAR struct tls_info_s *up_tls_info(void)
{
#ifdef CONFIG_TLS_ALIGNED
return TLS_INFO((uintptr_t)__builtin_frame_address(0));
#else
FAR struct tls_info_s *info = NULL;
struct stackinfo_s stackinfo;
int ret;
DEBUGASSERT(!up_interrupt_context());
ret = sched_get_stackinfo(0, &stackinfo);
if (ret == OK)
{
info = (FAR struct tsl_info_s *)stackinfo.adj_stack_ptr;
}
return info;
#endif
}
#endif /* CONFIG_TLS */
arch/sim/src/sim/up_checkstack.c
+1 -1
View File
@@ -89,7 +89,7 @@ static size_t do_stackcheck(uintptr_t alloc, size_t size, bool int_stack)
/* Get aligned addresses of the top and bottom of the stack */
#ifdef CONFIG_TLS
#ifdef CONFIG_TLS_ALIGNED
if (!int_stack)
{
/* Skip over the TLS data structure at the bottom of the stack */
arch/sim/src/sim/up_createstack.c
+3 -1
View File
@@ -111,6 +111,7 @@ int up_create_stack(FAR struct tcb_s *tcb, size_t stack_size, uint8_t ttype)
stack_size += sizeof(struct tls_info_s);
#ifdef CONFIG_TLS_ALIGNED
/* The allocated stack size must not exceed the maximum possible for the
* TLS feature.
*/
@@ -120,6 +121,7 @@ int up_create_stack(FAR struct tcb_s *tcb, size_t stack_size, uint8_t ttype)
{
stack_size = TLS_MAXSTACK;
}
#endif
#endif
/* Move up to next even word boundary if necessary */
@@ -128,7 +130,7 @@ int up_create_stack(FAR struct tcb_s *tcb, size_t stack_size, uint8_t ttype)
/* Allocate the memory for the stack */
#ifdef CONFIG_TLS
#ifdef CONFIG_TLS_ALIGNED
stack_alloc_ptr = (FAR uint8_t *)kumm_memalign(TLS_STACK_ALIGN,
adj_stack_size);
#else /* CONFIG_TLS */
arch/sim/src/sim/up_usestack.c
+1 -1
View File
@@ -101,7 +101,7 @@ int up_use_stack(FAR struct tcb_s *tcb, FAR void *stack, size_t stack_size)
uintptr_t adj_stack_addr;
size_t adj_stack_size;
#ifdef CONFIG_TLS
#ifdef CONFIG_TLS_ALIGNED
/* Make certain that the user provided stack is properly aligned */
DEBUGASSERT(((uintptr_t)stack & TLS_STACK_MASK) == 0);
include/nuttx/tls.h
+11 -6
View File
@@ -51,8 +51,10 @@
****************************************************************************/
/* Configuration ************************************************************/
#ifndef CONFIG_TLS_LOG2_MAXSTACK
# error CONFIG_TLS_LOG2_MAXSTACK is not defined
#ifdef CONFIG_TLS_ALIGNED
# ifndef CONFIG_TLS_LOG2_MAXSTACK
# error CONFIG_TLS_LOG2_MAXSTACK is not defined
# endif
#endif
#ifndef CONFIG_TLS_NELEM
@@ -68,14 +70,17 @@
/* TLS Definitions **********************************************************/
#define TLS_STACK_ALIGN (1L << CONFIG_TLS_LOG2_MAXSTACK)
#define TLS_STACK_MASK (TLS_STACK_ALIGN - 1)
#define TLS_MAXSTACK (TLS_STACK_ALIGN)
#define TLS_INFO(sp) ((FAR struct tls_info_s *)((sp) & ~TLS_STACK_MASK))
#ifdef CONFIG_TLS_ALIGNED
# define TLS_STACK_ALIGN (1L << CONFIG_TLS_LOG2_MAXSTACK)
# define TLS_STACK_MASK (TLS_STACK_ALIGN - 1)
# define TLS_MAXSTACK (TLS_STACK_ALIGN)
# define TLS_INFO(sp) ((FAR struct tls_info_s *)((sp) & ~TLS_STACK_MASK))
#endif
/****************************************************************************
* Public Types
****************************************************************************/
/* When TLS is enabled, up_createstack() will align allocated stacks to the
* TLS_STACK_ALIGN value. An instance of the following structure will be
* implicitly positioned at the "lower" end of the stack. Assuming a
libs/libc/tls/Kconfig
+4 -4
View File
@@ -26,23 +26,23 @@ config TLS_ALIGNED
default y if BUILD_KERNEL
default n if !BUILD_KERNEL
---help---
Aligned TLS works by fetch thread information from the beginning
Aligned TLS works by fetching thread information from the beginning
of the stack memory allocation. In order to do this, the memory
must be aligned in such a way that the executing logic can simply
mask the current stack pointer to get the beginning of the stack
allocation.
The advantage of using an aligned stack is no OS interface need
be called to get the beginning of the stack. It is simply and
be called to get the beginning of the stack. It is simply an
AND operation on the current stack pointer. The disadvantages
are that the alignment (1) causes memory fragmentation which
are that the alignment (1) causes memory fragmentation which can
be a serious problem for memory limited systems, and (2) limits
the maximum size of the stack. Any mask places a limit on the
maximum size of the stack; stack sizes about that would map to
an incorrect address.
In general, CONFIG_TLS_ALIGNED is preferred for the KERNEL
build where the virtualized stack address an be aligned with
build where the virtualized stack address can be aligned with
no implications to physical memory. In other builds, the
unaligned stack implementation is usually superior.