Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S261802AbVEPSOU (ORCPT ); Mon, 16 May 2005 14:14:20 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S261777AbVEPSM2 (ORCPT ); Mon, 16 May 2005 14:12:28 -0400 Received: from omx2-ext.sgi.com ([192.48.171.19]:1180 "EHLO omx2.sgi.com") by vger.kernel.org with ESMTP id S261786AbVEPSLZ (ORCPT ); Mon, 16 May 2005 14:11:25 -0400 Date: Mon, 16 May 2005 11:09:05 -0700 (PDT) From: Christoph Lameter To: john stultz cc: lkml , Tim Schmielau , George Anzinger , albert@users.sourceforge.net, Ulrich Windl , Dominik Brodowski , David Mosberger , Andi Kleen , paulus@samba.org, schwidefsky@de.ibm.com, keith maanthey , Chris McDermott , Max Asbock , mahuja@us.ibm.com, Nishanth Aravamudan , Darren Hart , "Darrick J. Wong" , Anton Blanchard , donf@us.ibm.com, mpm@selenic.com, benh@kernel.crashing.org, linux-ia64@vger.kernel.org Subject: Re: IA64 implementation of timesource for new time of day subsystem In-Reply-To: <1116264858.26990.39.camel@cog.beaverton.ibm.com> Message-ID: References: <1116029796.26454.2.camel@cog.beaverton.ibm.com> <1116029872.26454.4.camel@cog.beaverton.ibm.com> <1116029971.26454.7.camel@cog.beaverton.ibm.com> <1116030058.26454.10.camel@cog.beaverton.ibm.com> <1116030139.26454.13.camel@cog.beaverton.ibm.com> <1116264858.26990.39.camel@cog.beaverton.ibm.com> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2891 Lines: 60 On Mon, 16 May 2005, john stultz wrote: > Actually that shouldn't be necessary. Look at my arch-x86-64 patch or > vsyscall-i386 patch for how the arch_vsyscall_gtod_update() function is > used. It provides an arch specific hook called by the timeofday core to > provide the information you desire. > > Please let me know if it is not sufficient for some reason. Obviously this wont work since you cannot execute C code nor functions in an ia64 fastcall. I need the variables exported. > > I would recommend to add jitter compensation to the time sources. Otherwise > > each ITC/TSC like timesource will have to implement that on its own. > > Just to clarify for others, this is the same unsynced cpu cycle counter > problem that affects the TSC on i386 and x86-64. ia64 gets around the > problem by checking on every call to gettimeofday() if the ITC value is > less then the ITC value used on the previous call to gettimeofday(). If > the value is less (ie: would result in time going backwards) it just > uses the last value to calculate time. It then uses cmpxchg to > atomically update the last ITC value. Nope. You are way off here. Unsynched cpu cycle counters lead to the ITC timesource not being registered. > The problem I have with this is it that if the ITCs are not synced, they > really are not good timesources. If one cpu's ITC is behind another, the > net result of the above algorithm is cpu 2 will always just use cpu 1's > last calculated time. This could cause jumps in time when a process > moves from cpu2 to cpu1. Note again that the use of cmpxchg is NOT covering the case of ITCs not being synced. If the ITCs are not synced then no timesource will be established for ITC! This is the case of ITC's running synchronous but at a tiny offset. The startup on IA64 syncs the ITCs but cannot guarantee a complete sync. There may be a small offset of a few clock ticks. The cmpxchg is needed to compensate for that small offset. I imagine that other architectures have similar issues. > Since it only affects the TSC and ITC, I think keeping the decision to > use cmpxchg in the timesource code, as you've implemented with the ITC > is the best way to go. If you really want to you can special case the > arch specific fsyscall code by switching on the time source .name, and > that would allow you to use a similar cmpxchg algorithm there as well. Again this will not work on IA64 since it does the fast system calls in a different way. Clock jitter can affect multiple clock sources that may fluctuate in a minor way due to a variety of influences. Jitter compensation may help in these situations. - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/