From owner-svn-src-all@FreeBSD.ORG  Fri Dec  3 22:44:10 2010
Return-Path: <owner-svn-src-all@FreeBSD.ORG>
Delivered-To: svn-src-all@freebsd.org
Received: from [127.0.0.1] (freefall.freebsd.org [IPv6:2001:4f8:fff6::28])
	by hub.freebsd.org (Postfix) with ESMTP id DA164106564A;
	Fri,  3 Dec 2010 22:44:09 +0000 (UTC)
	(envelope-from jkim@FreeBSD.org)
From: Jung-uk Kim <jkim@FreeBSD.org>
To: John Baldwin <jhb@freebsd.org>
Date: Fri, 3 Dec 2010 17:43:53 -0500
User-Agent: KMail/1.6.2
References: <201012032154.oB3LsADC035461@svn.freebsd.org>
	<201012031708.30232.jhb@freebsd.org>
In-Reply-To: <201012031708.30232.jhb@freebsd.org>
MIME-Version: 1.0
Content-Disposition: inline
Content-Type: text/plain;
  charset="iso-8859-1"
Content-Transfer-Encoding: 7bit
Message-Id: <201012031744.01956.jkim@FreeBSD.org>
Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org,
	src-committers@freebsd.org
Subject: Re: svn commit: r216161 - in head/sys: amd64/amd64 i386/i386
X-BeenThere: svn-src-all@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: "SVN commit messages for the entire src tree \(except for &quot;
	user&quot; and &quot; projects&quot; \)" <svn-src-all.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/svn-src-all>,
	<mailto:svn-src-all-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/svn-src-all>
List-Post: <mailto:svn-src-all@freebsd.org>
List-Help: <mailto:svn-src-all-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/svn-src-all>,
	<mailto:svn-src-all-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Fri, 03 Dec 2010 22:44:10 -0000

On Friday 03 December 2010 05:08 pm, John Baldwin wrote:
> On Friday, December 03, 2010 4:54:10 pm Jung-uk Kim wrote:
> > Author: jkim
> > Date: Fri Dec  3 21:54:10 2010
> > New Revision: 216161
> > URL: http://svn.freebsd.org/changeset/base/216161
> >
> > Log:
> >   Explicitly initialize TSC frequency.  To calibrate TSC
> > frequency, we use DELAY(9) and it may use TSC in turn if TSC
> > frequency is non-zero.
>
> We zero the BSS, so these were already zero.  This just makes the
> actual kernel file on disk larger by wasting space in .data instead
> of .bss.

Please note that I didn't touch other variables, e.g., tsc_is_broken, 
because I knew that.  However, I just wanted to do that *explicitly*.  
Anyway, it is reverted now and SVN will remember what I wanted to 
do. ;-)

BTW, if my memory serves, GCC (and all modern C compilers) put(s) 
zero-initialized variables back in .bss.

Jung-uk Kim