From owner-cvs-all Sun Jun 3 17:26:31 2001 Delivered-To: cvs-all@freebsd.org Received: from freesbee.wheel.dk (freesbee.wheel.dk [193.162.159.97]) by hub.freebsd.org (Postfix) with ESMTP id E677837B403; Sun, 3 Jun 2001 17:26:24 -0700 (PDT) (envelope-from jesper@skriver.dk) Received: by freesbee.wheel.dk (Postfix, from userid 1001) id E6AD65D81; Mon, 4 Jun 2001 02:28:40 +0200 (CEST) Date: Mon, 4 Jun 2001 02:28:40 +0200 From: Jesper Skriver To: Bruce Evans Cc: Warner Losh , Michael Harnois , cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/kern uipc_accf.c uipc_mbuf.c uipc_socket2.c src/sys/netinet ip_input.c src/sys/netinet6 in6_proto.c src/sys/sys param.h Message-ID: <20010604022840.A50666@skriver.dk> References: <20010601102422.D23169@skriver.dk> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline User-Agent: Mutt/1.2.5i In-Reply-To: ; from bde@zeta.org.au on Mon, Jun 04, 2001 at 09:56:45AM +1000 X-PGP-Fingerprint: 6B88 9CE8 66E9 E631 C9C5 5EB4 22AB F0EC F956 1C31 X-PGP-Public-Key: http://freesbee.wheel.dk/~jesper/gpgkey.pub Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG On Mon, Jun 04, 2001 at 09:56:45AM +1000, Bruce Evans wrote: > On Fri, 1 Jun 2001, Jesper Skriver wrote: > > > On Fri, Jun 01, 2001 at 10:14:25AM +0200, Jesper Skriver wrote: > > > On Thu, May 31, 2001 at 07:03:56PM -0600, Warner Losh wrote: > > > > In message <868zjdjatx.fsf@mharnois.workgroup.net> Michael Harnois writes: > > > > : cc -O -pipe -march=pentiumpro -Wall -I/usr/obj/usr/src/i386/usr/include -c /usr/src/games/fortune/strfile/strfile.c > > > > : In file included from /usr/src/games/fortune/strfile/strfile.c:54: > > > > : /usr/include/sys/param.h:236: opt_param.h: No such file or directory > > > > > > > > Ah, the classic "I included opt_foo.h from a file that was exported to > > > > userland" problem. > > > > > > Bummer, I don't have time to fix it right now (probably put it in > > > a new .h file), so I'll back it out for now, and have a second go > > > when I have more time. > > > > http://people.FreeBSD.org/~jesper/opt_param.diff > > > > Should fix this, it moved NMBCLUSTERS back to > > src/sys/kern/uipc_mbuf.c, revert to the old version, > > and sets the default value for net.inet.ip.maxfragpackets and > > net.inet6.ip6.maxfragpackets to 200 instead of NMBCLUSTERS/4 > > I think you want it to be (nmbclusters / 4) Yes, but the initial value for a sysctl has to be a constant, or atleast it won't compile unless it is, Andrew Gallatin has suggested something like SYSINIT(frag, SI_SUB_MBUF, SI_ORDER_ANY, nfraginit, NULL) static void nfraginit(void *dummy) { ip_nfragpackets = nmbclusters/4; } Which I need to read more on and understand, so for now I committed it with a fixed default value of 200. > not NMBCLUSTERS/4. The variables in param.c are supposed to actually > be variables (it is always possible to change them by patching > the binary, even if they are not boot-time tunables). Using the > compile-time constants for secondary variables just gives wrong values > if the primary variables are changed, or more variables to patch/tune > manually. /Jesper -- Jesper Skriver, jesper(at)skriver(dot)dk - CCIE #5456 Work: Network manager @ AS3292 (Tele Danmark DataNetworks) Private: FreeBSD committer @ AS2109 (A much smaller network ;-) One Unix to rule them all, One Resolver to find them, One IP to bring them all and in the zone to bind them. To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message