Skip to content

OS_TCB struct size is increased to 64 bytes (from 60 bytes). With th… #2805

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

Merged
merged 1 commit into from
Sep 26, 2016
Merged

OS_TCB struct size is increased to 64 bytes (from 60 bytes). With th… #2805

merged 1 commit into from
Sep 26, 2016

Conversation

RobMeades
Copy link
Contributor

Description

With submit #2642 (stack stats) a void *argv pointer has been added to the OS_TCB struct, hence it is now 64 bytes in size and not 60.

Status

READY

Related PRs

#2786

Steps to test or reproduce

Build mbed-os for a target that uses MBED_RTOS_SINGLE_THREAD (i.e. small libraries). Since, for such a build, there is a single task, and there's no longer enough room for even one OS_TCB, rt_task_create() returns 0 during RTOS init and the system fails to get to mbed startup. With this change rt_task_create() returns 1 and the system boots correctly.

… change to introduce stats checking in commit c1728p9, OS_TCB now includes an extra pointer, *argv, hence it is larger by 4 bytes.
@sg-
Copy link
Contributor

sg- commented Sep 24, 2016

/morph test

@sg- sg- added the needs: CI label Sep 24, 2016
@mbed-bot
Copy link

Result: SUCCESS

Your command has finished executing! Here's what you wrote!

/morph test

Output

mbed Build Number: 941

All builds and test passed!

@sg- sg- merged commit 196584d into ARMmbed:master Sep 26, 2016
@0xc0170
Copy link
Contributor

0xc0170 commented Sep 26, 2016

Thanks @RobMeades for fixing this

@RobMeades
Copy link
Contributor Author

No problem :-).

@c1728p9
Copy link
Contributor

c1728p9 commented Sep 29, 2016

Thanks from me as well @RobMeades.

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

Successfully merging this pull request may close these issues.

5 participants