From owner-cvs-all@FreeBSD.ORG Thu Apr 27 17:06:22 2006 Return-Path: X-Original-To: cvs-all@FreeBSD.org Delivered-To: cvs-all@FreeBSD.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id AC11D16A401; Thu, 27 Apr 2006 17:06:22 +0000 (UTC) (envelope-from brdavis@odin.ac.hmc.edu) Received: from odin.ac.hmc.edu (Odin.AC.HMC.Edu [134.173.32.75]) by mx1.FreeBSD.org (Postfix) with ESMTP id 44D8343D45; Thu, 27 Apr 2006 17:06:22 +0000 (GMT) (envelope-from brdavis@odin.ac.hmc.edu) Received: from odin.ac.hmc.edu (localhost.localdomain [127.0.0.1]) by odin.ac.hmc.edu (8.13.0/8.13.0) with ESMTP id k3RH6Luu028362; Thu, 27 Apr 2006 10:06:21 -0700 Received: (from brdavis@localhost) by odin.ac.hmc.edu (8.13.0/8.13.0/Submit) id k3RH6LEt028361; Thu, 27 Apr 2006 10:06:21 -0700 Date: Thu, 27 Apr 2006 10:06:21 -0700 From: Brooks Davis To: Scott Long Message-ID: <20060427170621.GC12971@odin.ac.hmc.edu> References: <200604261942.k3QJgdnb086990@repoman.freebsd.org> <20060427102026.GC20828@garage.freebsd.pl> <20060427163351.GB12971@odin.ac.hmc.edu> <4450F84D.4080601@samsco.org> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="gr/z0/N6AeWAPJVB" Content-Disposition: inline In-Reply-To: <4450F84D.4080601@samsco.org> User-Agent: Mutt/1.4.1i X-Virus-Scanned: by amavisd-new X-Spam-Status: No, hits=1.0 required=8.0 tests=NO_COST autolearn=no version=2.63 X-Spam-Level: * X-Spam-Checker-Version: SpamAssassin 2.63 (2004-01-11) on odin.ac.hmc.edu Cc: src-committers@FreeBSD.org, Pawel Jakub Dawidek , Brooks Davis , cvs-src@FreeBSD.org, cvs-all@FreeBSD.org, John-Mark Gurney Subject: Re: cvs commit: src/sys/kern sched_4bsd.c X-BeenThere: cvs-all@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: CVS commit messages for the entire tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 27 Apr 2006 17:06:22 -0000 --gr/z0/N6AeWAPJVB Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Thu, Apr 27, 2006 at 10:58:53AM -0600, Scott Long wrote: > Brooks Davis wrote: > >On Thu, Apr 27, 2006 at 12:20:26PM +0200, Pawel Jakub Dawidek wrote: > > > >>On Wed, Apr 26, 2006 at 07:42:39PM +0000, John-Mark Gurney wrote: > >>+> jmg 2006-04-26 19:42:38 UTC > >>+>=20 > >>+> FreeBSD src repository > >>+>=20 > >>+> Modified files: > >>+> sys/kern sched_4bsd.c=20 > >>+> Log: > >>+> move remaining sysctl into the kern.sched tree... > >>[...] > >>+> -SYSCTL_INT(_kern, OID_AUTO, ccpu, CTLFLAG_RD, &ccpu, 0, ""); > >>+> +SYSCTL_INT(_kern_sched, OID_AUTO, ccpu, CTLFLAG_RD, &ccpu, 0, ""); > >>[...] > >> > >>Too fast. > >> > >># grep -r kern.ccpu /usr/src/ > > > > > >We should retain the old location as well under COMPAT_FREEBSD[456]. It > >doesn't cost anything and we should treat the sysctl tree like any other > >API to the extent possible. >=20 > Well, sysctl's are a second class API. The ease by they can be changed,= =20 > as well as the completely uncontrolled nature that they are developed=20 > in, means that we really start to hamstring ourselves if we declare them > as a first class API. What we should do instead is define a subset of > the sysctl tree that is stable and guaranteed to exist, and declare that > the rest are transient and therefore not part of the API. Doing this by > popular vote is impractical, what we need is someone to sit down and=20 > audit what is there now and make appropriate recommendations. sysctls=20 > that are used by libkvm are a good candidate, for example, but random > driver sysctls might not be. I agree we don't want to be too restrictive here, but for cases like this where there's effectively no cost I think we should at least try to retain compatibility under #ifs. If we break that compatibility at some later date due to actually changing the data or because something else needs to use that location in the tree, that's probably OK. -- Brooks --=20 Any statement of the form "X is the one, true Y" is FALSE. PGP fingerprint 655D 519C 26A7 82E7 2529 9BF0 5D8E 8BE9 F238 1AD4 --gr/z0/N6AeWAPJVB Content-Type: application/pgp-signature Content-Disposition: inline -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.2.1 (GNU/Linux) iD8DBQFEUPoMXY6L6fI4GtQRAndlAJ4gXRoEjYEgrTXABLmbOOyMVxK57wCgpJlD BNK8OUVugyAQlkHb/IQ9QFc= =e/14 -----END PGP SIGNATURE----- --gr/z0/N6AeWAPJVB--