Skip to content

Support configuring stack size in RTOS-less build #6813

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
ccli8 opened this issue May 4, 2018 · 9 comments
Closed

Support configuring stack size in RTOS-less build #6813

ccli8 opened this issue May 4, 2018 · 9 comments

Comments

@ccli8
Copy link
Contributor

ccli8 commented May 4, 2018

Description

  • Type: Enhancement
  • Priority: Major

mbed-os sha:
mbed-os-5.8.3

I am evaluating Mbed Cloud 1.3.1 on NUMAKER_PFM_M487/GCC_ARM. In mbed-bootloader, it is built with RTOS disabled. With stack size hard-coded to 0x800 in linker script file, I meet stack overflow on printf() call. For RTOS enabled, I can change main thread's stack size through mbed_app.json (MBED_CONF_APP_MAIN_STACK_SIZE). But for RTOS disabled, it seems there's no way except modifying linker script file.

@0xc0170
Copy link
Contributor

0xc0170 commented May 4, 2018

@SenRamakri @bulislaw Please review
[Mirrored to Jira]

@bulislaw
Copy link
Member

bulislaw commented May 8, 2018

Hi @ccli8 you are right it doesn't make sense to have 2k stack and huge heap. We should be able to configure it with some sane default. We'll try to have a look into that soonish.
[Mirrored to Jira]

@andrewleech
Copy link
Contributor

andrewleech commented May 10, 2018

On an barely related note, is there an official easy to build without RTOS?
Is .mbedignore the rtos folder it, or is there a better way?
[Mirrored to Jira]

@bulislaw
Copy link
Member

bulislaw commented May 11, 2018

Right now we don't support it officially. We acknowledge it's a legitimate use case so we try not to break it, but we don't actively maintain it. We may revisit that for next major release.
[Mirrored to Jira]

@ccli8
Copy link
Contributor Author

ccli8 commented Jul 5, 2018

Any update?

Besides, same as rtos-less main stack size, it is also needed to make rtos kernel stack size configurable. I meet stack overrun with them in Mbed Cloud.
[Mirrored to Jira]

@0xc0170
Copy link
Contributor

0xc0170 commented Jul 13, 2018

@ccli8 Isn't this trying to address this issue : #7238 ?
[Mirrored to Jira]

@ccli8
Copy link
Contributor Author

ccli8 commented Jul 16, 2018

@0xc0170 Yes, #7238 is addressing the issue.
[Mirrored to Jira]

@adbridge
Copy link
Contributor

adbridge commented Oct 4, 2018

Internal Jira reference: https://jira.arm.com/browse/IOTCORE-261

@ccli8
Copy link
Contributor Author

ccli8 commented Dec 12, 2018

Done with #8039

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants