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

Disable transparent huge pages #103

Closed
wants to merge 1 commit into from
Closed

Conversation

btd
Copy link

@btd btd commented Mar 23, 2016

According to Redis Administration Guide they recommend to disable this feature.

@DavidWittman
Copy link
Owner

Good idea @btd. We'll need to find a way to make this persist across a reboot. Perhaps it should be added to the init script. It would be nice to also have this feature be toggle able. Appreciate any thoughts you have on that.

@btd
Copy link
Author

btd commented Mar 23, 2016

I copied this snippet from ansible-oracle, they also add this line to grub. I am not sure how portable it is.
For my use case i usually never reboot server, just spawn new VM with fresh redis, do migration and terminate old.

@btd
Copy link
Author

btd commented Mar 23, 2016

By idea grub.conf should be regenerated when new kernel installed.

@cjamison
Copy link

I did some research into this and there seems to be a lot of problems with grub solutions and there are so many cases depending on the platform and even the hosting provider.

What seems to be the easiest thing is to add the check and setting to the init scripts. That worked for me.

@DavidWittman
Copy link
Owner

DavidWittman commented Apr 21, 2016

@cjamison mind sharing your init script changes? Thanks!

@cjamison
Copy link

I just added this before the case statement:

if test -f /sys/kernel/mm/transparent_hugepage/enabled; then
   echo never > /sys/kernel/mm/transparent_hugepage/enabled
fi

@awr
Copy link
Contributor

awr commented Mar 8, 2017

Here's an approach that seems to work across reboots affinity@f626839 based on https://docs.mongodb.com/manual/tutorial/transparent-huge-pages/

@DavidWittman
Copy link
Owner

I'm going to close this in favor of #153

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

Successfully merging this pull request may close these issues.

4 participants