Skip to content
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

Heap management options #513

Merged
merged 2 commits into from
Dec 2, 2019
Merged

Conversation

pablogs9
Copy link
Contributor

This PR enables OOT apps to overwrite MEMMANG_OBJ with a custom implementantion of the heap manager.

It also enables the build system to set FREERTOS_HEAP_SIZE from CFLAGS.

@ataffanel ataffanel merged commit 1e6f2c3 into bitcraze:master Dec 2, 2019
@ataffanel
Copy link
Member

This looks good, I am merging it, thanks!

One side note about that: we are planning on removing all dynamic memory allocation from the Crazyflie firmware itself since it does not fit very well with hard real-time. In a sense it makes this PR more relevant though since only the app layer might then be interested in dynamic memory.

@pablogs9
Copy link
Contributor Author

pablogs9 commented Dec 2, 2019

I've read something about this migration to static memory, the v10 version of the kernel (merged in #510) allows this static allocation of tasks, queues, etc. Thaks for the merge :).

@pablogs9
Copy link
Contributor Author

pablogs9 commented Dec 2, 2019

@ataffanel is this PR going to be in next-release milestone? Thanks

@ataffanel
Copy link
Member

Yes sure, we release the tip of the master branch, so everything that is in master will be released.

@ataffanel ataffanel added this to the next-release milestone Dec 2, 2019
@pablogs9 pablogs9 deleted the feature/mem_management branch December 3, 2019 12:05
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

Successfully merging this pull request may close these issues.

2 participants