Skip to content

Commit

Permalink
Allow target or application level definition of extra TICKLESS stack …
Browse files Browse the repository at this point in the history
…size

Adding a check to let application or target force increase idle thread
stack size.
  • Loading branch information
LMESTM committed Jun 25, 2019
1 parent a8444bd commit 9e88719
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion rtos/TARGET_CORTEX/mbed_rtx_conf.h
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@
#endif

// Increase the idle thread stack size when tickless is enabled
#if defined(MBED_TICKLESS) && defined(LPTICKER_DELAY_TICKS) && (LPTICKER_DELAY_TICKS > 0)
#if defined(EXTRA_IDLE_STACK_REQUIRED) || (defined(MBED_TICKLESS) && defined(LPTICKER_DELAY_TICKS) && (LPTICKER_DELAY_TICKS > 0))
#define EXTRA_IDLE_STACK MBED_CONF_RTOS_IDLE_THREAD_STACK_SIZE_TICKLESS_EXTRA
#else
#define EXTRA_IDLE_STACK 0
Expand Down
2 changes: 1 addition & 1 deletion rtos/mbed_lib.json
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@
"value": 4096
},
"idle-thread-stack-size-tickless-extra": {
"help": "Additional size to add to the idle thread when tickless is enabled and LPTICKER_DELAY_TICKS is used",
"help": "Additional size to add to the idle thread when a specific target or application implementation requires it or in case tickless is enabled and LPTICKER_DELAY_TICKS is used",
"value": 256
},
"idle-thread-stack-size-debug-extra": {
Expand Down

2 comments on commit 9e88719

@mprse
Copy link
Contributor

@mprse mprse commented on 9e88719 Jun 25, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The change looks good, but specified macro should be also added to the affected targets in targets.json.

@LMESTM
Copy link
Contributor Author

@LMESTM LMESTM commented on 9e88719 Jun 25, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That will be done in #10701 when this one is merged as this one is a preceding PR for #10701.

Please sign in to comment.