From owner-cvs-src@FreeBSD.ORG Sun Mar 16 10:39:17 2008 Return-Path: Delivered-To: cvs-src@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 949251065670; Sun, 16 Mar 2008 10:39:17 +0000 (UTC) (envelope-from jroberson@chesapeake.net) Received: from webaccess-cl.virtdom.com (webaccess-cl.virtdom.com [216.240.101.25]) by mx1.freebsd.org (Postfix) with ESMTP id 766818FC14; Sun, 16 Mar 2008 10:39:17 +0000 (UTC) (envelope-from jroberson@chesapeake.net) Received: from [192.168.1.107] (cpe-24-94-75-93.hawaii.res.rr.com [24.94.75.93]) (authenticated bits=0) by webaccess-cl.virtdom.com (8.13.6/8.13.6) with ESMTP id m2GAdEsu031710; Sun, 16 Mar 2008 06:39:15 -0400 (EDT) (envelope-from jroberson@chesapeake.net) Date: Sun, 16 Mar 2008 00:39:49 -1000 (HST) From: Jeff Roberson X-X-Sender: jroberson@desktop To: Maxim Sobolev In-Reply-To: <200803160621.m2G6LUlE034766@repoman.freebsd.org> Message-ID: <20080316003849.L910@desktop> References: <200803160621.m2G6LUlE034766@repoman.freebsd.org> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII; format=flowed Cc: cvs-src@FreeBSD.org, src-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/sys eventhandler.h src/sys/kern kern_descrip.c uipc_socket.c X-BeenThere: cvs-src@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 16 Mar 2008 10:39:17 -0000 On Sun, 16 Mar 2008, Maxim Sobolev wrote: > sobomax 2008-03-16 06:21:30 UTC > > FreeBSD src repository > > Modified files: > sys/sys eventhandler.h > sys/kern kern_descrip.c uipc_socket.c > Log: > Properly set size of the file_zone to match kern.maxfiles parameter. > Otherwise the parameter is no-op, since zone by default limits number > of descriptors to some 12K entries. Attempt to allocate more ends up > sleeping on zonelimit. Why do you need to set a limit on the uma_zone when falloc() takes care of this for us by checking openfiles vs maxfiles? Thanks, Jeff > > MFC after: 2 weeks > > Revision Changes Path > 1.326 +36 -3 src/sys/kern/kern_descrip.c > 1.306 +1 -0 src/sys/kern/uipc_socket.c > 1.39 +1 -0 src/sys/sys/eventhandler.h >