Mateusz Guzik wrote: > On Wed, Nov 28, 2012 at 07:14:23PM -0500, Rick Macklem wrote: > > Mateusz Guzik wrote: > > > On Wed, Nov 28, 2012 at 02:47:32AM +0000, Alfred Perlstein wrote: > > > > Author: alfred > > > > Date: Wed Nov 28 02:47:31 2012 > > > > New Revision: 243645 > > > > URL: http://svnweb.freebsd.org/changeset/base/243645 > > > > > > > > Log: > > > > Don't allow minthreads > maxthreads. > > > > > > > Should not this be also checked in the kernel? Looks like > > > nfssvc_nfsd > > > is > > > trustful: > > > > > Well, since only root can do this and I can't think of why a > > sysadmin > > would use anything other than nfsd, I'm not sure it matters much? > > (But I don't see a problem with adding a sanity check in the kernel > > code.) > > > > It's nothing serious and I'm not going to insist, especially since > this > is your code. It's not my code, it's the collective's code. (For want of a better term for the FreeBSD project.;-) Also, I wasn't the author of the krpc related stuff (which these thread limits are a part of). I just cloned it from the old NFS server for the new one.
> I just prefer the kernel to be resistant if it does not > cost much. > > I had something like this in mind (untested): > http://people.freebsd.org/~mjg/patches/nfs-threads-check.diff > Since it was alfred@'s patch, I'll let him decide. As I said before, I have no problem with adding the check to the kernel. Have fun with it, rick > -- > Mateusz Guzik <mjguzik gmail.com> _______________________________________________ svn-src-all@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/svn-src-all To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"