From owner-freebsd-stable@FreeBSD.ORG Tue Jul 8 19:46:42 2014 Return-Path: Delivered-To: freebsd-stable@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id 2C698622; Tue, 8 Jul 2014 19:46:42 +0000 (UTC) Received: from esa-jnhn.mail.uoguelph.ca (esa-jnhn.mail.uoguelph.ca [131.104.91.44]) by mx1.freebsd.org (Postfix) with ESMTP id C46F229B5; Tue, 8 Jul 2014 19:46:41 +0000 (UTC) X-IronPort-Anti-Spam-Filtered: true X-IronPort-Anti-Spam-Result: AqQEAKcivFODaFve/2dsb2JhbABag2Bagm+8HIdBAYEudYQDAQEEASNWBRYOCgICDRkCWQaITQgNrmKZQReBLI1iATMHgneBTAWcPpJEggGBXiGBcw X-IronPort-AV: E=Sophos;i="5.01,626,1400040000"; d="scan'208";a="138576883" Received: from muskoka.cs.uoguelph.ca (HELO zcs3.mail.uoguelph.ca) ([131.104.91.222]) by esa-jnhn.mail.uoguelph.ca with ESMTP; 08 Jul 2014 15:46:36 -0400 Received: from zcs3.mail.uoguelph.ca (localhost.localdomain [127.0.0.1]) by zcs3.mail.uoguelph.ca (Postfix) with ESMTP id 1F9D5B3F46; Tue, 8 Jul 2014 15:46:36 -0400 (EDT) Date: Tue, 8 Jul 2014 15:46:36 -0400 (EDT) From: Rick Macklem To: Glen Barber Message-ID: <1539064411.8855953.1404848796119.JavaMail.root@uoguelph.ca> In-Reply-To: <20140708133559.GF1216@hub.FreeBSD.org> Subject: Re: 9-STABLE nfsd(8) does not auto-tune number of threads (error in the 9.3 Release Notes!) MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable X-Originating-IP: [172.17.91.202] X-Mailer: Zimbra 7.2.6_GA_2926 (ZimbraWebClient - FF3.0 (Win)/7.2.6_GA_2926) Cc: Rick Macklem , freebsd-stable@freebsd.org, =?utf-8?Q?Jos=C3=A9_Mar=C3=ADa?= Alcaide X-BeenThere: freebsd-stable@freebsd.org X-Mailman-Version: 2.1.18 Precedence: list List-Id: Production branch of FreeBSD source code List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 08 Jul 2014 19:46:42 -0000 Glen Barber wrote: > On Tue, Jul 08, 2014 at 03:06:38PM +0200, Jos=C3=A9 Mar=C3=ADa Alcaide wr= ote: > > One of the 9.3-RELEASE Release Notes (userland changes) states > > that: > >=20 > > "The default number of nfsd(8) threads has been increased from 4 to > > (8 * N), where N is the number of CPUs as reported by sysctl -n > > hw.ncpu. [r262124]" > >=20 > > Indeed, revision r262124 > > (http://svnweb.freebsd.org/base?view=3Drevision&revision=3D262124) > > removes the "-n 4" option from nfs_server_flags in > > /etc/defaults/rc.conf. > >=20 > > The change is based on the premise that nfsd(8) auto-tunes the > > number of threads as (8 * hw.ncpu). That auto-tuning was > > introduced in HEAD revision r243637 of nfsd.c > > (http://svnweb.freebsd.org/base?view=3Drevision&revision=3D243637), > > but it has not been MFC'd to 9-STABLE yet, so that release note > > *is not true*. > >=20 > > I think that r262124 should be reverted, also removing the release > > note mentioned above; or alternatively bring HEAD's nfsd.c on sync > > with 9-STABLE (probably a bad idea just before 9.3-RELEASE). > >=20 >=20 > Reverting r262124 is also not an option at this point of the release > cycle. >=20 > Rick, do you have any reservations to issuing an errata notice for > this > after the release? >=20 Well, I think that it will generate 4 threads without the "-n 4" just like it does with the "-n 4", so the only issue I see is the statement in the release notes isn't accurate. (It doesn't seem worth reverting r262124, sin= ce it doesn't really affect the outcome?) I'd suggest just taking the statement out of the release notes. rick ps: This change wasn't my commit, but I didn't see a problem with the default changing. I have fixing the nfsd(8) man page to reflect this change in default on my "to do" list. > Glen >=20 >=20