From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752787Ab0CRWTz (ORCPT ); Thu, 18 Mar 2010 18:19:55 -0400 Received: from e32.co.us.ibm.com ([32.97.110.150]:43147 "EHLO e32.co.us.ibm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751450Ab0CRWTx (ORCPT ); Thu, 18 Mar 2010 18:19:53 -0400 Subject: Re: [RFC][PATCH] Convert alpha to use clocksource From: john stultz To: Ivan Kokshaysky Cc: Richard Henderson , lkml , Thomas Gleixner , Matt Turner In-Reply-To: <20100318214030.GA8934@jurassic.park.msu.ru> References: <1268877702-2236-1-git-send-email-johnstul@us.ibm.com> <4BA23974.5030503@twiddle.net> <1268934923.1855.8.camel@work-vm> <20100318214030.GA8934@jurassic.park.msu.ru> Content-Type: text/plain; charset="UTF-8" Date: Thu, 18 Mar 2010 15:19:18 -0700 Message-ID: <1268950758.4171.51.camel@localhost.localdomain> Mime-Version: 1.0 X-Mailer: Evolution 2.28.1 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, 2010-03-19 at 00:40 +0300, Ivan Kokshaysky wrote: > On Thu, Mar 18, 2010 at 10:55:23AM -0700, john stultz wrote: > > On Thu, 2010-03-18 at 07:32 -0700, Richard Henderson wrote: > > > On 03/17/2010 07:01 PM, John Stultz wrote: > > > > Alpha has a tsc like rpcc counter that it uses to manage time. > > > > This can be converted to an actual clocksource instead of utilizing > > > > the arch_gettimeoffset method that is really only there for legacy > > > > systems with no continuous counter. > > > > > > With 8 seconds or less between roll-overs, do you actually consider > > > this a continuous counter? I don't. I suggest this be left alone. > > > > The timekeeping code handles this (although the shift value I picked may > > need some adjustment - what is the expected counter freq range on > > alpha?). The ACPI PM counter which is very common on x86 is only 24 bits > > and rolls over in ~5 seconds. It works fine. > > Yeah, that looks cool. I'm typing this on the 800MHz UP1500 running > 2.6.34-rc1 plus your patch, and the timekeeping works fine so far. Nice! Thanks for testing! Another benefit that I forgot to mention, is that NTP adjustments will be made directly against the counter, instead of being made against the tick. This avoids possible small errors at tick time if the intertick interval doesn't match the actual tick length. > Though, even after a glance over the clocksource code, I've not > gotten yet to how one could estimate the "shift" value... > Any hints? Yea, selecting a good shift is obnoxious. You want to pick the largest value of shift, so that it can be finely adjusted by ntp, but that creates large mult values, which can cause overflows for large cycle intervals. A function to solve this was actually recently added, but simply I forgot to use it. :P I actually should rework the register function so you just give it a clocksource and a freq and it sets up both mult and shift for you. > And if I recall correctly, production alphas have been clocked > in the range of 60-1250 MHz. Cool. The shift value I picked should be ok then. But below is an updated version that uses the self-calculated method. thanks -john >>From 078e91d14cb5e82a3fab00cf74faf45f44336590 Mon Sep 17 00:00:00 2001 From: John Stultz Date: Wed, 17 Mar 2010 12:43:11 -0700 Subject: [PATCH] Convert alpha to use clocksources instead of arch_gettimeoffset Alpha has a tsc like rpcc counter that it uses to manage time. This can be converted to an actual clocksource instead of utilizing the arch_gettimeoffset method that is really only there for legacy systems with no continuous counter. Further cleanups could be made if alpha converted to the clockevent model. I've not tested or compiled this code. Any help from the maintainers would be greatly appreciated CC: Thomas Gleixner CC: Richard Henderson CC: Ivan Kokshaysky CC: Matt Turner Signed-off-by: John Stultz --- arch/alpha/Kconfig | 4 -- arch/alpha/kernel/time.c | 69 ++++++++++++++++++++------------------------- 2 files changed, 31 insertions(+), 42 deletions(-) diff --git a/arch/alpha/Kconfig b/arch/alpha/Kconfig index 75291fd..793c269 100644 --- a/arch/alpha/Kconfig +++ b/arch/alpha/Kconfig @@ -51,10 +51,6 @@ config GENERIC_TIME bool default y -config ARCH_USES_GETTIMEOFFSET - bool - default y - config ZONE_DMA bool default y diff --git a/arch/alpha/kernel/time.c b/arch/alpha/kernel/time.c index 5d08266..a2be26b 100644 --- a/arch/alpha/kernel/time.c +++ b/arch/alpha/kernel/time.c @@ -51,6 +51,7 @@ #include #include #include +#include #include "proto.h" #include "irq_impl.h" @@ -301,6 +302,34 @@ rpcc_after_update_in_progress(void) return rpcc(); } +#ifndef CONFIG_SMP +/* Until and unless we figure out how to get cpu cycle counters + in sync and keep them there, we can't use the rpcc. */ +static cycle_t read_rpcc(struct clocksource *cs) +{ + cycle_t ret = (cycle_t)rpcc(); + return ret; +} + +static struct clocksource clocksource_rpcc = { + .name = "rpcc", + .rating = 300, + .read = read_rpcc, + .mask = CLOCKSOURCE_MASK(32), + .flags = CLOCK_SOURCE_IS_CONTINUOUS +}; + +static inline void register_rpcc_clocksource(long cycle_freq) +{ + clocksource_calc_mult_shift(&clocksource_rpcc, cycle_freq, 4); + clocksource_register(&clocksource_rpcc); +} +#else /* !CONFIG_SMP */ +static inline void register_rpcc_clocksource(long cycle_freq) +{ +} +#endif /* !CONFIG_SMP */ + void __init time_init(void) { @@ -391,6 +420,8 @@ time_init(void) __you_loose(); } + register_rpcc_clocksource(cycle_freq); + state.last_time = cc1; state.scaled_ticks_per_cycle = ((unsigned long) HZ << FIX_SHIFT) / cycle_freq; @@ -402,44 +433,6 @@ time_init(void) } /* - * Use the cycle counter to estimate an displacement from the last time - * tick. Unfortunately the Alpha designers made only the low 32-bits of - * the cycle counter active, so we overflow on 8.2 seconds on a 500MHz - * part. So we can't do the "find absolute time in terms of cycles" thing - * that the other ports do. - */ -u32 arch_gettimeoffset(void) -{ -#ifdef CONFIG_SMP - /* Until and unless we figure out how to get cpu cycle counters - in sync and keep them there, we can't use the rpcc tricks. */ - return 0; -#else - unsigned long delta_cycles, delta_usec, partial_tick; - - delta_cycles = rpcc() - state.last_time; - partial_tick = state.partial_tick; - /* - * usec = cycles * ticks_per_cycle * 2**48 * 1e6 / (2**48 * ticks) - * = cycles * (s_t_p_c) * 1e6 / (2**48 * ticks) - * = cycles * (s_t_p_c) * 15625 / (2**42 * ticks) - * - * which, given a 600MHz cycle and a 1024Hz tick, has a - * dynamic range of about 1.7e17, which is less than the - * 1.8e19 in an unsigned long, so we are safe from overflow. - * - * Round, but with .5 up always, since .5 to even is harder - * with no clear gain. - */ - - delta_usec = (delta_cycles * state.scaled_ticks_per_cycle - + partial_tick) * 15625; - delta_usec = ((delta_usec / ((1UL << (FIX_SHIFT-6-1)) * HZ)) + 1) / 2; - return delta_usec * 1000; -#endif -} - -/* * In order to set the CMOS clock precisely, set_rtc_mmss has to be * called 500 ms after the second nowtime has started, because when * nowtime is written into the registers of the CMOS clock, it will -- 1.6.0.4