From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933404AbXDFXWj (ORCPT ); Fri, 6 Apr 2007 19:22:39 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S933408AbXDFXWj (ORCPT ); Fri, 6 Apr 2007 19:22:39 -0400 Received: from smtp.osdl.org ([65.172.181.24]:38069 "EHLO smtp.osdl.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933404AbXDFXWi (ORCPT ); Fri, 6 Apr 2007 19:22:38 -0400 Date: Fri, 6 Apr 2007 16:22:33 -0700 From: Andrew Morton To: Jeremy Fitzhardinge Cc: Andi Kleen , virtualization@lists.osdl.org, lkml Subject: Re: [patch 18/20] clean up tsc-based sched_clock Message-Id: <20070406162233.b835bd4c.akpm@linux-foundation.org> In-Reply-To: <20070404191206.476822257@goop.org> References: <20070404191151.009821039@goop.org> <20070404191206.476822257@goop.org> X-Mailer: Sylpheed version 2.2.7 (GTK+ 2.8.17; x86_64-unknown-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org On Wed, 04 Apr 2007 12:12:09 -0700 Jeremy Fitzhardinge wrote: > Three cleanups: > - change "instable" -> "unstable" > - its better to use get_cpu_var for getting this cpu's variables > - change cycles_2_ns to do the full computation rather than just the > tsc->ns scaling. Its a simpler interface, and it makes the function > more generally useful. > > Signed-off-by: Jeremy Fitzhardinge > > --- > arch/i386/kernel/sched-clock.c | 35 +++++++++++++++++++++-------------- I'm dropping the relevant patch from Andi's tree due to it causing mysterious hangs when initscripts start ondemand. So I'll need to drop this patch and "[patch 19/20] Add a sched_clock paravirt_op". I still need to work out why that hang is happening - it is very mysterious. I got as far as working out that it was hanging on write_seqlock_irqsave(xtime_lock), then remembered that it's with CONFIG_SMP=n so I stomped off to bed in disgust. Later.