Date: Thu, 11 Mar 2010 08:30:05 +0000 (UTC) From: Juli Mallett <jmallett@FreeBSD.org> To: src-committers@freebsd.org, svn-src-user@freebsd.org Subject: svn commit: r205004 - user/jmallett/octeon/sys/mips/cavium Message-ID: <201003110830.o2B8U5Vu014314@svn.freebsd.org>
next in thread | raw e-mail | index | archive | help
Author: jmallett Date: Thu Mar 11 08:30:05 2010 New Revision: 205004 URL: http://svn.freebsd.org/changeset/base/205004 Log: I would've expected the Octeon to have a double-counting timer as the code suggests, but at least on my CAM-0100 this seems to not be the case. Pass 0 to mips_timer_init_params for double_count instead of 1 -- now timings on my system match wall clock. The tick code really should be updated for 64-bit counter reads on the CPUs that support them -- I wonder if the t_delta wrongness is due to inaccuracy from the hand-rolled counter wrapping. Modified: user/jmallett/octeon/sys/mips/cavium/octeon_machdep.c Modified: user/jmallett/octeon/sys/mips/cavium/octeon_machdep.c ============================================================================== --- user/jmallett/octeon/sys/mips/cavium/octeon_machdep.c Thu Mar 11 08:03:56 2010 (r205003) +++ user/jmallett/octeon/sys/mips/cavium/octeon_machdep.c Thu Mar 11 08:30:05 2010 (r205004) @@ -760,7 +760,7 @@ platform_start(__register_t a0, __regist kdb_enter(KDB_WHY_BOOTFLAGS, "Boot flags requested debugger"); #endif platform_counter_freq = octeon_get_clock_rate(); - mips_timer_init_params(platform_counter_freq, 1); + mips_timer_init_params(platform_counter_freq, 0); } /* impSTART: This stuff should move back into the Cavium SDK */
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201003110830.o2B8U5Vu014314>