2003-06-23 17:56:01

by pwitting

[permalink] [raw]
Subject: Re: Typo in Redhat 8/9 nfs start/stop script

Steve D.,

You may want to do more than that. RH9 is using 2.4.20 kernel, which by my
understanding resets the values to what I thinks is reasonable. For details
on how 2.4.20 calculates queue sizes, see this earlier post by Neil Brown:

http://sourceforge.net/mailarchive/message.php?msg_id=4229482

But the gist is, there is no longer any need for the startup script to reset
these values unless RH has disabled this feature of the kernel nfs code.

Secondly, it might be wise to automatically install a /etc/sysconfig/nfs
file. The script reads this file, but there's no default included, only be
reading and understanding the script would a user find out that this is now
the preferred way to change the thread count.

Finally, why not release an erratum that corrects the problem instead of
forcing any upgrade to RH10? It would be nice if I didn't have to add
functions to my Kickstart post-install script to correct this problem, and
instead could just upgrade the RPM in my install directory.

Thanks,

>Date: Sat, 21 Jun 2003 08:51:32 -0400
>From: Steve Dickson <[email protected]>
>
>This is fixed in the next release...
>
>SteveD.
>
>Matt Schillinger wrote:
>
>>Note for any Redhat users that need higher than default input queues for
>>NFS.
>>
>>
>># Get the initial values for the input sock queues
>># at the time of running the script.
>>if [ "$TUNE_QUEUE" = "yes" ]; then
>> RMEM_DEFAULT=`/sbin/sysctl -n net.core.rmem_default`
>> RMEM_MAX=`/sbin/sysctl -n net.core.rmem_max`
>> # 256kb recommended minimum size based on SPECsfs NFS benchmarks
>> [ -z "$NFS_QS" ] && NFS_QS=262144
>>fi
>>
>># See how we were called.
>>case "$1" in
>> start)
>> # Start daemons.
>> # Apply input queue increase for nfs server
>> if [ "$TUNE_QUEUE" = "yes" ]; then
>> /sbin/sysctl -w net.core.rmem_default=$NFSD_QS >/dev/null
>>2>&1
>> /sbin/sysctl -w net.core.rmem_max=$NFSD_QS >/dev/null 2>&1
>> fi
>>
>>
>>
>>NOTE THAT when checking that the variable has a value and setting the
>>variable, NFS_QS is used. But when setting the input queues, $NFSD_QS is
>>used.



-------------------------------------------------------
This SF.Net email is sponsored by: INetU
Attention Web Developers & Consultants: Become An INetU Hosting Partner.
Refer Dedicated Servers. We Manage Them. You Get 10% Monthly Commission!
INetU Dedicated Managed Hosting http://www.inetu.net/partner/index.php
_______________________________________________
NFS maillist - [email protected]
https://lists.sourceforge.net/lists/listinfo/nfs


2003-06-24 12:20:08

by Steve Dickson

[permalink] [raw]
Subject: Re: Typo in Redhat 8/9 nfs start/stop script

[email protected] wrote:

>You may want to do more than that. RH9 is using 2.4.20 kernel, which by my
>understanding resets the values to what I thinks is reasonable. For details
>on how 2.4.20 calculates queue sizes, see this earlier post by Neil Brown:
>
>http://sourceforge.net/mailarchive/message.php?msg_id=4229482
>
>But the gist is, there is no longer any need for the startup script to reset
>these values unless RH has disabled this feature of the kernel nfs code.
>
Good point... That QS stuff probably should be removed...

>Secondly, it might be wise to automatically install a /etc/sysconfig/nfs
>file. The script reads this file, but there's no default included, only be
>reading and understanding the script would a user find out that this is now
>the preferred way to change the thread count.
>
I kinda like it the way it is.... I don't want to make it too easy for
people to
shoot themselves in the foot.... :-) Making people read/understand the
script
before changing it seems to me like a good idea...

>Finally, why not release an erratum that corrects the problem instead of
>forcing any upgrade to RH10? It would be nice if I didn't have to add
>functions to my Kickstart post-install script to correct this problem, and
>instead could just upgrade the RPM in my install directory.
>
You don't have to upgrade the RH10 to get these changes... just
upgrade the nfs-utils package...

SteveD.



-------------------------------------------------------
This SF.Net email is sponsored by: INetU
Attention Web Developers & Consultants: Become An INetU Hosting Partner.
Refer Dedicated Servers. We Manage Them. You Get 10% Monthly Commission!
INetU Dedicated Managed Hosting http://www.inetu.net/partner/index.php
_______________________________________________
NFS maillist - [email protected]
https://lists.sourceforge.net/lists/listinfo/nfs