Skip to content

core: debug: make stack size check more sensible#2183

Closed
kaspar030 wants to merge 1 commit intoRIOT-OS:masterfrom
kaspar030:make_debug_develhelp_stack_check_more_sensible
Closed

core: debug: make stack size check more sensible#2183
kaspar030 wants to merge 1 commit intoRIOT-OS:masterfrom
kaspar030:make_debug_develhelp_stack_check_more_sensible

Conversation

@kaspar030
Copy link
Copy Markdown
Contributor

When DEVELHELP is defined, DEBUG() used to bail out when the thread's stack size is smaller or equal KERNEL_CONF_STACKSIZE_PRINTF. Change that so that if stacksize is at least KERNEL_CONF_STACKSIZE_PRINTF, DEBUG() will do it's magic.

@kaspar030 kaspar030 added Area: core Area: RIOT kernel. Handle PRs marked with this with care! Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation labels Dec 12, 2014
@OlegHahm
Copy link
Copy Markdown
Member

Again? #1244

@OlegHahm
Copy link
Copy Markdown
Member

Particular: #1244 (comment)

@kaspar030
Copy link
Copy Markdown
Contributor Author

Uff... Sorry for the noise...

@kaspar030 kaspar030 closed this Dec 12, 2014
@OlegHahm
Copy link
Copy Markdown
Member

RIOT's getting too old - there's nothing new left to discuss about. ;)

@LudwigKnuepfer
Copy link
Copy Markdown
Member

Maybe we should document that line if people keep stumbling over it...

@OlegHahm
Copy link
Copy Markdown
Member

We could also implement my suggestion. ;)

Edited: fixed typo

@kaspar030 kaspar030 deleted the make_debug_develhelp_stack_check_more_sensible branch January 26, 2017 12:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: core Area: RIOT kernel. Handle PRs marked with this with care! Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants