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

add option to call mlockall in munged #22

Closed
GoogleCodeExporter opened this issue May 15, 2015 · 3 comments
Closed

add option to call mlockall in munged #22

GoogleCodeExporter opened this issue May 15, 2015 · 3 comments
Labels
Milestone

Comments

@GoogleCodeExporter
Copy link

GoogleCodeExporter commented May 15, 2015

We've found it useful in our environment to have munged lock its pages upon startup because our compute nodes are often subject to extreme memory and disk pressure. Without locking, munged sometimes gets swapped out and cannot respond to requests quickly enough. This causes the resource manager (slurm in our case) to believe that compute nodes are down when they really aren't.

I've written a patch (attached) that adds an option (-M, --mlockall) to munged so that it will lock pages on startup. It works well in our environment, so I thought it might be useful to others.

0001-Add-mlockall-option-to-munged.patch.gz

Original issue reported on code.google.com by michaelfenn87 on 22 Apr 2013 at 4:26

@GoogleCodeExporter
Copy link
Author

Sounds reasonable. Thanks for the patch!

Original comment by chris.m.dunlap on 24 Apr 2013 at 12:12

  • Changed state: Accepted
  • Added labels: Milestone-0.5.11

@GoogleCodeExporter
Copy link
Author

You're welcome, glad to contribute. :)

Michael

Original comment by michaelfenn87 on 24 Apr 2013 at 11:30

@GoogleCodeExporter
Copy link
Author

This issue was closed by 0aad599.

Original comment by chris.m.dunlap on 2 May 2013 at 8:25

  • Changed state: Fixed

@dun dun added this to the 0.5.11 milestone Jun 4, 2015
@dun dun added feature and removed auto-migrated labels Jun 5, 2015
dun added a commit that referenced this issue Jun 17, 2015
Attachment from Michael Fenn <fennm@deshawresearch.com> on 2013-04-22
for "0001-Add-mlockall-option-to-munged.patch".

https://storage.googleapis.com/google-code-attachments/munge/issue-22/comment-0/0001-Add-mlockall-option-to-munged.patch
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants