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 default memory consumption with config and mempools #78

Closed
budrus opened this issue Mar 27, 2020 · 1 comment · Fixed by #79 or #82
Closed

Reduce default memory consumption with config and mempools #78

budrus opened this issue Mar 27, 2020 · 1 comment · Fixed by #79 or #82
Assignees
Labels
refactoring Refactor code without adding features

Comments

@budrus
Copy link
Contributor

budrus commented Mar 27, 2020

Brief feature description

The current defaults for compile time configuration and mempools are quite high. Better have reasonable defaults and let people increase the values if needed

Detailed information

With the current default configuration we need ~200 MB for the internal data structures and the default memory pools have ~ 600 MB. This is a challenge on systems which only have some hundred MBs of RAM left. Beginners could be confused and get out of memory errors on RouDi startup. We support 4096 publishers and subscribers with the default config. Because of max size allocation with fixed size containers this blows up the memory. For most users a lower number is sufficient. The mempools can be easily configured with the config file now, so having the large default does not make sense IMHO

budrus added a commit to budrus/iceoryx that referenced this issue Mar 27, 2020
Signed-off-by: Poehnl Michael (CC-AD/ESW1) <michael.poehnl@de.bosch.com>
@budrus budrus added the refactoring Refactor code without adding features label Mar 28, 2020
budrus added a commit that referenced this issue Mar 28, 2020
iox-#78: adaptations for reduction of memory consumption
@budrus budrus closed this as completed Mar 28, 2020
@mossmaurice
Copy link
Contributor

mossmaurice commented Apr 7, 2020

Reopening, to make RouDi_GTest more configurable and adapt TOML files.

@mossmaurice mossmaurice reopened this Apr 7, 2020
@mossmaurice mossmaurice self-assigned this Apr 7, 2020
mossmaurice added a commit to mossmaurice/iceoryx that referenced this issue Apr 7, 2020
…DiEnvironment

Signed-off-by: Hoinkis Simon (CC-AD/ESW1) <simon.hoinkis2@de.bosch.com>
@mossmaurice mossmaurice linked a pull request Apr 7, 2020 that will close this issue
mossmaurice added a commit to mossmaurice/iceoryx that referenced this issue Apr 7, 2020
…e to large

Signed-off-by: Hoinkis Simon (CC-AD/ESW1) <simon.hoinkis2@de.bosch.com>
mossmaurice added a commit to mossmaurice/iceoryx that referenced this issue Apr 7, 2020
…file

Signed-off-by: Hoinkis Simon (CC-AD/ESW1) <simon.hoinkis2@de.bosch.com>
@mossmaurice mossmaurice linked a pull request Apr 7, 2020 that will close this issue
mossmaurice added a commit to mossmaurice/iceoryx that referenced this issue May 5, 2020
…learer in docu

Signed-off-by: Hoinkis Simon (CC-AD/ESW1) <simon.hoinkis2@de.bosch.com>
mossmaurice added a commit that referenced this issue May 5, 2020
…s-more-configurable

Iox #78 Minor RouDi config adaptions
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactoring Refactor code without adding features
Projects
None yet
2 participants