diff options
| author | Sebastian Huber <sebastian.huber@embedded-brains.de> | 2012-04-11 10:50:00 +0200 |
|---|---|---|
| committer | Sebastian Huber <sebastian.huber@embedded-brains.de> | 2012-04-24 14:22:57 +0200 |
| commit | 3595533de5a347ddeb5f2bb2b14a853296d23119 (patch) | |
| tree | 50cbabc595df18634cfa1abf2a902bdeb9297b51 /cpukit/score/src/threadstackallocate.c | |
| parent | nfsclient: Use UID and GID of NFS handle (diff) | |
| download | rtems-3595533de5a347ddeb5f2bb2b14a853296d23119.tar.bz2 | |
score: Remove _Stack_Adjust_size()
The increase of the stack size by CPU_STACK_ALIGNMENT in
_Thread_Stack_Allocate() is disadvantageous. This may lead to a huge
over allocation for specialized stack allocators. The
CPU_STACK_ALIGNMENT is at most 16 on all current RTEMS CPU ports. The
mimimum stack size ensured by _Stack_Ensure_minimum() must be
considerable larger than this value, otherwise stack overflows will
likely occur. Thus the _Stack_Adjust_size() is also superfluous.
Diffstat (limited to 'cpukit/score/src/threadstackallocate.c')
| -rw-r--r-- | cpukit/score/src/threadstackallocate.c | 12 |
1 files changed, 0 insertions, 12 deletions
diff --git a/cpukit/score/src/threadstackallocate.c b/cpukit/score/src/threadstackallocate.c index 9448328365..68aa649857 100644 --- a/cpukit/score/src/threadstackallocate.c +++ b/cpukit/score/src/threadstackallocate.c @@ -51,18 +51,6 @@ size_t _Thread_Stack_Allocate( the_stack_size = _Stack_Ensure_minimum( stack_size ); - /* - * Pad the requested size so we allocate enough memory - * so the context initialization can align it properly. The address - * returned the workspace allocate must be directly stored in the - * stack control block because it is later used in the free sequence. - * - * Thus it is the responsibility of the CPU dependent code to - * get and keep the stack adjust factor, the stack alignment, and - * the context initialization sequence in sync. - */ - the_stack_size = _Stack_Adjust_size( the_stack_size ); - stack_addr = (*stack_allocate_hook)( the_stack_size ); if ( !stack_addr ) |