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

Memory Leak in PatchBufferFixed when using boost::serialization #71

Open
klreuther opened this issue Sep 6, 2019 · 1 comment
Open

Comments

@klreuther
Copy link

klreuther commented Sep 6, 2019

Contrary to its name the fixed buffer in PatchBufferFixed is never cleared. This occurs possibly only when using boost::serialization. This leads to a steadily growing memory usage during MPI calculations as this buffer is used to store kernel and rim during ghost zone calculations.

Patch diff:
libgeodecomp/storage/patchbufferfixed.h
77d76
< buffer[indexRead].clear();

@klreuther
Copy link
Author

After deliberation, a better patch would be
57d56
< buffer[indexRead].clear()

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

No branches or pull requests

1 participant