Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934670AbaKMXVZ (ORCPT ); Thu, 13 Nov 2014 18:21:25 -0500 Received: from mail-qa0-f42.google.com ([209.85.216.42]:57594 "EHLO mail-qa0-f42.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933928AbaKMXVX (ORCPT ); Thu, 13 Nov 2014 18:21:23 -0500 MIME-Version: 1.0 In-Reply-To: References: <1415388855-35074-1-git-send-email-anatol.pomozov@gmail.com> <20141110095325.GC12126@ulmo> Date: Thu, 13 Nov 2014 15:21:22 -0800 Message-ID: Subject: Re: [PATCH] timekeeping: Move persistent clock registration code from ARM to kernel From: John Stultz To: Thomas Gleixner Cc: Anatol Pomozov , Thierry Reding , Stephen Warren , Daniel Lezcano , Russell King , LKML , "linux-tegra@vger.kernel.org" , Tony Lindgren , Mark Rutland Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Nov 13, 2014 at 2:46 PM, Thomas Gleixner wrote: > On Mon, 10 Nov 2014, Anatol Pomozov wrote: >> On Mon, Nov 10, 2014 at 1:53 AM, Thierry Reding >> wrote: >> > On Fri, Nov 07, 2014 at 11:34:15AM -0800, Anatol Pomozov wrote: >> >> ARM timekeeping functionality allows to register persistent/boot clock dynamically. >> >> This code is arch-independent and can be useful on other plaforms as well. >> >> >> >> As a byproduct of this change, tegra20_timer becomes ARM64 compatible. >> >> >> >> Tested: backported the change to chromeos-3.14 kernel ran on tegra 64bit >> >> board, made sure high-resolution clock works. >> > >> > Using this on an upstream kernel doesn't work, though, because 64-bit >> > ARM doesn't implement struct delay_timer which the driver needs since >> > v3.17. >> > >> > But I suppose the delay timer infrastructure could be moved into the >> > core similar to the persistent and boot clock as this patch does. >> >> Thanks. It makes sense, I will send it in a separate patch, once this >> one will be reviewed. On our kernel I haven't seen this issue as we >> still use 3.14. > > That's why you should test/compile your stuff on latest greatest and > not on a year old conglomorate of unknown provenance. :) > > Aside of that I really wonder why we need that persistent_clock stuff > at all. We already have mechanisms to register persistent clocks AKA > RTCs after the early boot process and update the wall clock time > before we actually need it. Nothing in early boot depends on correct > wall clock at all. > > So instead of adding more extra persistent clock nonsense, can we just > move all of that to the place where it belongs, i.e. RTC? Sigh.. I've got this on an eventual todo list.. The big problem though is that the RTC infrastructure can't be called with irqs off, so its not as optimal for measuring suspend time. Some of the suspend-time measurement with clocksources that don't halt is interesting here. So we need to add to the RTC infrastructure special accessors that are safe when irqs are off, and we can then deprecate the persistent clock bits. There's still evaluation quirks with setting the time earlier in boot or not (possibly some rng effects as well there), but that could be worked out if we had the suspend timing via safe RTC interfaces sorted. thanks -john -- 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/