Re: [patch] Improve documentation around FreeBSD Kernel Tuning

On Tue, Jan 03, 2012 at 05:02:57PM -0500, Andrew Dunstan wrote:
>
>
> On 01/03/2012 04:49 PM, Brad Davis wrote:
> > Hi,
> >
> > I have a patch that improves the documentation for FreeBSD Kernel Tuning:
> >
> > - Show a # prompt instead of $ to indicate a root shell is needed
> > - Remove the -w flag to sysctl since it is not needed anymore and just silently ignored
> > - Encourage the user to set the read-only sysctls in /boot/loader.conf, instead of setting them manually in the loader.
> >
> > I have put these in a github fork of the repo, but I am new to git. So I apologize if this is incorrect.
> >
> > https://github.com/so14k/postgres/commit/12c03bdb2967346e7ad9ce0bdd3db8dfcf81507e
> >
> >
>
> Instead of a URL, please just email us the diff as an attachment.
> Normally we prefer these in context diff format, although it doesn't
> matter so much for such a small patch.

Sorry about that.. it is attached below.

Thanks,
Brad Davis

diff against doc/src/sgml/runtime.sgml

797,799c797,799
< <prompt>$</prompt> <userinput>sysctl -w kern.ipc.shmall=32768</userinput>
< <prompt>$</prompt> <userinput>sysctl -w kern.ipc.shmmax=134217728</userinput>
< <prompt>$</prompt> <userinput>sysctl -w kern.ipc.semmap=256</userinput>
---
> <prompt>#</prompt> <userinput>sysctl kern.ipc.shmall=32768</userinput>
> <prompt>#</prompt> <userinput>sysctl kern.ipc.shmmax=134217728</userinput>
> <prompt>#</prompt> <userinput>sysctl kern.ipc.semmap=256</userinput>
807,815c807,815
< <command>sysctl</command> is concerned, but can be changed
< before boot using the <command>loader</command> prompt:
< <screen>
< <prompt>(loader)</prompt> <userinput>set kern.ipc.semmni=256</userinput>
< <prompt>(loader)</prompt> <userinput>set kern.ipc.semmns=512</userinput>
< <prompt>(loader)</prompt> <userinput>set kern.ipc.semmnu=256</userinput>
< </screen>
< Similarly these can be saved between reboots in
< <filename>/boot/loader.conf</filename>.
---
> <command>sysctl</command> is concerned, but can be set in
> <filename>/boot/loader.conf</filename>:
> <programlisting>
> kern.ipc.semmni=256
> kern.ipc.semmns=512
> kern.ipc.semmnu=256
> </programlisting>
> After modifying these values a reboot is required for the new
> settings to take affect.