[riot-notifications] [RIOT-OS/RIOT] sys/event/thread: add STACKSTEST flag (#16888)

Francisco notifications at github.com
Fri Sep 24 16:25:43 CEST 2021


### Contribution description

I've gotten many times scared by having the impression that my event thread is completely collapsed, but it's just because it's the only one not initialized with STACKTEST. This  PR changes the behaviour so its is.

If there is some value in making it configurable I can.

### Testing procedure

Run an application with a shell and `USEMODULE="ps event_thread"

before:

```
main(): This is RIOT! (Version: 2021.10-devel-492-g719a1-HEAD)
test_shell.
> ps
ps
	pid | name                 | state    Q | pri | stack  ( used) ( free) | base addr  | current     
	  - | isr_stack            | -        - |   - |   8192 (   -1) ( 8193) | 0x5661a720 | 0x5661a720
	  1 | idle                 | pending  Q |  15 |   8192 (  436) ( 7756) | 0x56613020 | 0x56614e7c 
	  2 | main                 | running  Q |   7 |  12288 ( 3020) ( 9268) | 0x56615020 | 0x56617e7c 
	  3 | event                | bl anyfl _ |   6 |   8192 ( 8188) (    4) | 0x5661eac0 | 0x5662091c 
	    | SUM                  |            |     |  36864 (11644) (25220)
```

now
```
main(): This is RIOT! (Version: 2021.10-devel-493-gae5e1-pr_event_thread_stacktest)
test_shell.
> ps
ps
	pid | name                 | state    Q | pri | stack  ( used) ( free) | base addr  | current     
	  - | isr_stack            | -        - |   - |   8192 (   -1) ( 8193) | 0x56588720 | 0x56588720
	  1 | idle                 | pending  Q |  15 |   8192 (  436) ( 7756) | 0x56581020 | 0x56582e7c 
	  2 | main                 | running  Q |   7 |  12288 ( 3020) ( 9268) | 0x56583020 | 0x56585e7c 
	  3 | event                | bl anyfl _ |   6 |   8192 (  960) ( 7232) | 0x5658cac0 | 0x5658e91c 
	    | SUM                  |            |     |  36864 ( 4416) (32448)
```

You can view, comment on, or merge this pull request online at:

  https://github.com/RIOT-OS/RIOT/pull/16888

-- Commit Summary --

  * <a href="https://github.com/RIOT-OS/RIOT/pull/16888/commits/ae5e10bc26fe2dce42359c58f979985e1151b9ba">sys/event/thread: add STACKSTEST flag</a>

-- File Changes --

    M sys/event/thread.c (4)

-- Patch Links --

https://github.com/RIOT-OS/RIOT/pull/16888.patch
https://github.com/RIOT-OS/RIOT/pull/16888.diff

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/16888
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20210924/531aadb0/attachment.htm>


More information about the notifications mailing list