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

Reduce memory overhead #298

Closed
nikic opened this issue Mar 13, 2018 · 5 comments
Closed

Reduce memory overhead #298

nikic opened this issue Mar 13, 2018 · 5 comments

Comments

@nikic
Copy link
Collaborator

nikic commented Mar 13, 2018

Currently each cache entry has about ~256 bytes overhead (cache entry, cache slot, pool, pool block, key string header). Small pools are currently allocated in 512 byte increments, so the minimum entry size is 512 bytes.

There is clearly some optimization potential here...

@nikic
Copy link
Collaborator Author

nikic commented Mar 13, 2018

First optimizations implemented in #299, saving 32 bytes per entry.

@nikic
Copy link
Collaborator Author

nikic commented Mar 13, 2018

Two more optimizations have been implemented in #300 and #301, saving another 32 bytes.

@nikic
Copy link
Collaborator Author

nikic commented Mar 15, 2018

Between other changes, #302 saves another 8 bytes.

@nikic
Copy link
Collaborator Author

nikic commented Oct 8, 2018

#305 implements some major memory savings. Depending on the case they range between a couple percent to a factor of three.

@nikic
Copy link
Collaborator Author

nikic commented Nov 17, 2018

Closing this, as the major work happened in #305. There are still improvements that can be made, but it's peanuts in comparison.

@nikic nikic closed this as completed Nov 17, 2018
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

1 participant