From owner-freebsd-net@FreeBSD.ORG  Thu Aug 23 01:00:19 2012
Return-Path: <owner-freebsd-net@FreeBSD.ORG>
Delivered-To: freebsd-net@hub.freebsd.org
Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52])
	by hub.freebsd.org (Postfix) with ESMTP id 5E7E7106566B
	for <freebsd-net@hub.freebsd.org>; Thu, 23 Aug 2012 01:00:19 +0000 (UTC)
	(envelope-from gnats@FreeBSD.org)
Received: from freefall.freebsd.org (freefall.freebsd.org
	[IPv6:2001:4f8:fff6::28])
	by mx1.freebsd.org (Postfix) with ESMTP id 3B13F8FC08
	for <freebsd-net@hub.freebsd.org>; Thu, 23 Aug 2012 01:00:19 +0000 (UTC)
Received: from freefall.freebsd.org (localhost [127.0.0.1])
	by freefall.freebsd.org (8.14.5/8.14.5) with ESMTP id q7N10JiA000373
	for <freebsd-net@freefall.freebsd.org>; Thu, 23 Aug 2012 01:00:19 GMT
	(envelope-from gnats@freefall.freebsd.org)
Received: (from gnats@localhost)
	by freefall.freebsd.org (8.14.5/8.14.5/Submit) id q7N10J9C000372;
	Thu, 23 Aug 2012 01:00:19 GMT (envelope-from gnats)
Date: Thu, 23 Aug 2012 01:00:19 GMT
Message-Id: <201208230100.q7N10J9C000372@freefall.freebsd.org>
To: freebsd-net@FreeBSD.org
From: Navdeep Parhar <np@FreeBSD.org>
Cc: 
Subject: Re: kern/170713: [cxgb] Driver must be loaded after boot due to
 timing
 issues checking for kern.ipc.nmb* values set via /boot/loader.conf
X-BeenThere: freebsd-net@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
Reply-To: Navdeep Parhar <np@FreeBSD.org>
List-Id: Networking and TCP/IP with FreeBSD <freebsd-net.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-net>,
	<mailto:freebsd-net-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-net>
List-Post: <mailto:freebsd-net@freebsd.org>
List-Help: <mailto:freebsd-net-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-net>,
	<mailto:freebsd-net-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Thu, 23 Aug 2012 01:00:19 -0000

The following reply was made to PR kern/170713; it has been noted by GNATS.

From: Navdeep Parhar <np@FreeBSD.org>
To: bug-followup@FreeBSD.org, yanegomi@gmail.com
Cc:  
Subject: Re: kern/170713: [cxgb] Driver must be loaded after boot due to timing
 issues checking for kern.ipc.nmb* values set via /boot/loader.conf
Date: Wed, 22 Aug 2012 17:56:34 -0700

 First, note that only kern.ipc.nmbclusters is a valid tunable.  The rest 
 of the nmbXXX settings in your loader.conf have no effect.  There are 
 sysctls but no tunables for the rest.
 
 Take a look at tunable_mbinit in kern_mbuf.c -- on recent FreeBSD 
 versions it starts with nmbclusters and sizes others based on this.  You 
 can set nmbclusters really high and influence the values of the other 
 parameters.
 
 In my opinion we should have a TUNABLE_INT_FETCH for all of the nmbXXX 
 and autocalculate the ones that are not set, just like what we do for 
 nmbclusters today.
 
 static void
 tunable_mbinit(void *dummy)
 {
 	TUNABLE_INT_FETCH("kern.ipc.nmbclusters", &nmbclusters);
 
 	/* This has to be done before VM init. */
 	if (nmbclusters == 0)
 		nmbclusters = 1024 + maxusers * 64;
 	nmbjumbop = nmbclusters / 2;
 	nmbjumbo9 = nmbjumbop / 2;
 	nmbjumbo16 = nmbjumbo9 / 2;
 }
 
 
 Compare this to the tunable_mbinit in 7 and you can see why the 
 nmbclusters tunable does not affect the others -- it is updated after 
 the other values have already been calculated.
 static void
 tunable_mbinit(void *dummy)
 {
 
 	/* This has to be done before VM init. */
 	nmbclusters = 1024 + maxusers * 64;
 	nmbjumbop = nmbclusters / 2;
 	nmbjumbo9 = nmbjumbop / 2;
 	nmbjumbo16 = nmbjumbo9 / 2;
 	TUNABLE_INT_FETCH("kern.ipc.nmbclusters", &nmbclusters);
 }
 
 Regards,
 Navdeep