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

Replace BigBuf with a small & efficient allocator #898

Open
slurdge opened this issue Jun 14, 2020 · 3 comments
Open

Replace BigBuf with a small & efficient allocator #898

slurdge opened this issue Jun 14, 2020 · 3 comments

Comments

@slurdge
Copy link
Contributor

slurdge commented Jun 14, 2020

It shouldn't be hard to replace BigBuf with an embeded allocator:
https://github.com/dimonomid/umm_malloc

The only missing bit is requesting the biggest chunk of free memory, which is easy to implement (for traces etc.)

We can then gradually remove static memory taken & put it back to this allocator, taking care of not creating memory leaks.

End result: more memory & easier tracking (basically removing each little function taking a bunch of .data for itself...)

@iceman1001
Copy link
Collaborator

Sounds awesome!
To finally have a unified good memory allocator deviceside.

@doegox doegox transferred this issue from another repository Jun 25, 2020
@doegox doegox transferred this issue from another repository Jul 11, 2020
@doegox
Copy link
Contributor

doegox commented Aug 31, 2020

I started adding umm code here : https://github.com/RfidResearchGroup/proxmark3/tree/umm_wip
Status: it should be usable but it still needs a space and size, currently it's pointing to BigBuff space.

@doegox
Copy link
Contributor

doegox commented Aug 31, 2020

oops, @slurdge had already his own test branch, see also https://github.com/slurdge/proxmark3/tree/umm

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

No branches or pull requests

3 participants