Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 23 Jun 2010 19:56:46 -0700
From:      Juli Mallett <jmallett@FreeBSD.org>
To:        "M. Warner Losh" <imp@bsdimp.com>
Cc:        svn-src-head@freebsd.org, svn-src-all@freebsd.org, xcllnt@mac.com, src-committers@freebsd.org, nwhitehorn@freebsd.org
Subject:   Re: svn commit: r209490 - head/sys/kern
Message-ID:  <AANLkTinYyrwpUpdfO8CUv7VuEROg3GZqmpXdHWm5nHgK@mail.gmail.com>
In-Reply-To: <20100623.202647.867924114552092597.imp@bsdimp.com>
References:  <201006232326.o5NNQEtK011793@svn.freebsd.org> <3659E546-80A0-4E82-8112-40B093728D81@mac.com>  <20100623.202647.867924114552092597.imp@bsdimp.com>

next in thread | previous in thread | raw e-mail | index | archive | help
On Wed, Jun 23, 2010 at 19:26, M. Warner Losh <imp@bsdimp.com> wrote:

> : > # =A0 =A0define HZ 1000
> : > # =A0else
> : > # =A0 =A0define HZ 100
>
> Shouldn't this be defined in some file in ARCH/include instead?

Didn't it used to be?  I think the problem with doing that is that
then someone inevitably writes code that uses HZ instead of the
(tunable) hz.  Of course, you can hide it behind an obscure #ifdef and
even give it an intimidating name.



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?AANLkTinYyrwpUpdfO8CUv7VuEROg3GZqmpXdHWm5nHgK>