svn commit: r260486 - head/etc/defaults

[Adrian Chadd]

Hi,

Depends if you're thinking locally or globally.

Locally - for nfs? not a big deal.

Globally - NFS, ZFS, GELI, geom/cam, NIC, etc.. suddenly your machine
could default to having a couple thousand worker threads just for a
HBA and a 10GE NIC. That's a little nuts.


-a


On 9 January 2014 08:00, Alexander Motin <mav at freebsd.org> wrote:
> On 09.01.2014 17:57, Adrian Chadd wrote:
>>
>> .. so with say, 128 core boxes showing up, is this really a good default?
>
>
> And what is the price? 16K+ of KVA per thread for thread stack, etc? 4
> threads is probably much worse default there. May be nfsd's default could be
> tuned, but obviously it should not be hardcoded value.
>
>
>> On 9 January 2014 07:55, Alexander Motin <mav at freebsd.org> wrote:
>>>
>>> Author: mav
>>> Date: Thu Jan  9 15:55:55 2014
>>> New Revision: 260486
>>> URL: http://svnweb.freebsd.org/changeset/base/260486
>>>
>>> Log:
>>>    Remove very low default limit of 4 nfsd threads.  nfsd's own default
>>> is
>>>    8 * hw.ncpu, that sounds more appropriate for these SMP/NCQ/... days.
>>>
>>> Modified:
>>>    head/etc/defaults/rc.conf
>>>
>>> Modified: head/etc/defaults/rc.conf
>>>
>>> ==============================================================================
>>> --- head/etc/defaults/rc.conf   Thu Jan  9 15:38:28 2014        (r260485)
>>> +++ head/etc/defaults/rc.conf   Thu Jan  9 15:55:55 2014        (r260486)
>>> @@ -311,7 +311,7 @@ nfs_client_enable="NO"              # This host is a
>>>   nfs_access_cache="60"          # Client cache timeout in seconds
>>>   nfs_server_enable="NO"         # This host is an NFS server (or NO).
>>>   oldnfs_server_enable="NO"      # Run the old NFS server (YES/NO).
>>> -nfs_server_flags="-u -t -n 4"  # Flags to nfsd (if enabled).
>>> +nfs_server_flags="-u -t"       # Flags to nfsd (if enabled).
>>>   mountd_enable="NO"             # Run mountd (or NO).
>>>   mountd_flags="-r"              # Flags to mountd (if NFS server
>>> enabled).
>>>   weak_mountd_authentication="NO"        # Allow non-root mount requests
>>> to be served.
>
>
>
> --
> Alexander Motin