From owner-cvs-src@FreeBSD.ORG Mon Nov 19 21:07:53 2007 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 51E6F16A468; Mon, 19 Nov 2007 21:07:53 +0000 (UTC) (envelope-from bright@elvis.mu.org) Received: from elvis.mu.org (elvis.mu.org [192.203.228.196]) by mx1.freebsd.org (Postfix) with ESMTP id 3D55713C459; Mon, 19 Nov 2007 21:07:53 +0000 (UTC) (envelope-from bright@elvis.mu.org) Received: by elvis.mu.org (Postfix, from userid 1192) id 30E0C1A4D81; Mon, 19 Nov 2007 13:07:46 -0800 (PST) Date: Mon, 19 Nov 2007 13:07:46 -0800 From: Alfred Perlstein To: Roman Divacky Message-ID: <20071119210746.GI19166@elvis.mu.org> References: <200710161040.l9GAes88010153@repoman.freebsd.org> <20071118224216.GA29048@freebsd.org> <20071119025328.GU8950@elvis.mu.org> <20071119184111.GB1516@roadrunner.spoerlein.net> <20071119194236.GA61631@freebsd.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20071119194236.GA61631@freebsd.org> User-Agent: Mutt/1.4.2.3i Cc: cvs-src@freebsd.org, Ulrich Spoerlein Subject: Re: cvs commit: src/sys/kern subr_param.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: Mon, 19 Nov 2007 21:07:53 -0000 * Roman Divacky [071119 11:42] wrote: > On Mon, Nov 19, 2007 at 07:41:11PM +0100, Ulrich Spoerlein wrote: > > On Sun, 18.11.2007 at 18:53:28 -0800, Alfred Perlstein wrote: > > > * Roman Divacky [071118 14:42] wrote: > > > > On Tue, Oct 16, 2007 at 10:40:54AM +0000, Alfred Perlstein wrote: > > > > > Modified files: > > > > > sys/kern subr_param.c > > > > > Log: > > > > > Export maxswzone, maxbcache, maxtsiz, dfldsiz, maxdsiz, dflssiz, maxssiz, > > > > > and sgrowsiz via sysctl. > > > > > > > > why is maxssiz RD_ONLY? its used only in exec_new_vmspace() for setting the default > > > > stack area size and only if thats not set by sysentvec, so it's RW already just > > > > not defined so. > > > > > > > > also... dflssiz is not used at all, and the "default stack size" is set by maxssiz > > > > > > If you think it can be safely changed, then go for it! > > > > Could this be done for maxdsiz, too? It's kinda sad you need to reboot a > > FreeBSD system to work around the 512MB data size limit (in this day and > > age, processes can easily hit the 512MB barrier) > > as I can see the maxdsiz is used only as a barrier for rlimit.. I see no harm > in changing that to RW too.. > > I already asked kib@ if he's willing to commit this, waiting for answer... Please use a SYSCTL_PROC so some basic sanity checks could be made to keep the system functioning, basically prevent too low or insanely high values from being set. -- - Alfred Perlstein