Skip to content

I think it's a bug in freeRTOS config file? #42

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
KiraVerSace opened this issue Jun 25, 2021 · 3 comments
Closed

I think it's a bug in freeRTOS config file? #42

KiraVerSace opened this issue Jun 25, 2021 · 3 comments
Labels
bug Something isn't working
Milestone

Comments

@KiraVerSace
Copy link

Please look the L102
https://github.com/stm32duino/STM32FreeRTOS/blob/master/src/FreeRTOSConfig_Default.h

here is my result I think it should change to
#define configTOTAL_HEAP_SIZE ((size_t)(&_estack - (uint32_t)&_Min_Stack_Size - &_end))

I use printf to print the value of _Min_Stack_Size it always wrong, but I add the & as &_Min_Stack_Size , it will be the right value that I set in the ld file.

SO I the heap size shoud minus the _Min_Stack_Size set to 0x400?

@fpistm fpistm added the bug Something isn't working label Jun 25, 2021
@fpistm fpistm added this to the 10.2.2 milestone Jun 25, 2021
@fpistm fpistm closed this as completed in 8b196df Jun 25, 2021
@fpistm
Copy link
Member

fpistm commented Jun 25, 2021

Thank you @KiraVerSace for pointing this issue 👍

@KiraVerSace
Copy link
Author

But I still do not know why here should add a &?

@fpistm
Copy link
Member

fpistm commented Jun 28, 2021

But I still do not know why here should add a &?

https://sourceware.org/binutils/docs/ld/Source-Code-Reference.html

Note the use of the ‘&’ operators. These are correct. Alternatively the symbols can be treated as the names of vectors or arrays and then the code will again work as expected: ```

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants