Date: Fri, 3 Dec 2010 17:43:53 -0500 From: Jung-uk Kim <jkim@FreeBSD.org> To: John Baldwin <jhb@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 Message-ID: <201012031744.01956.jkim@FreeBSD.org> In-Reply-To: <201012031708.30232.jhb@freebsd.org> References: <201012032154.oB3LsADC035461@svn.freebsd.org> <201012031708.30232.jhb@freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
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
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201012031744.01956.jkim>