From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752187AbbDCGVN (ORCPT ); Fri, 3 Apr 2015 02:21:13 -0400 Received: from mail-wi0-f176.google.com ([209.85.212.176]:36812 "EHLO mail-wi0-f176.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750936AbbDCGVM (ORCPT ); Fri, 3 Apr 2015 02:21:12 -0400 Date: Fri, 3 Apr 2015 08:21:07 +0200 From: Ingo Molnar To: John Stultz Cc: Peter Zijlstra , lkml , Thomas Gleixner , Prarit Bhargava , Richard Cochran Subject: Re: [PATCH 21/21] time: Rework debugging variables so they aren't global Message-ID: <20150403062107.GB12625@gmail.com> References: <1427945681-29972-1-git-send-email-john.stultz@linaro.org> <1427945681-29972-22-git-send-email-john.stultz@linaro.org> <20150402074739.GP23123@twins.programming.kicks-ass.net> <20150402075151.GA24591@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org * John Stultz wrote: > On Thu, Apr 2, 2015 at 12:51 AM, Ingo Molnar wrote: > > > > * Peter Zijlstra wrote: > > > >> On Wed, Apr 01, 2015 at 08:34:41PM -0700, John Stultz wrote: > >> > Ingo suggested that the timekeeping debugging variables > >> > recently added should not be global, and should be tied > >> > to the timekeeper's read_base. > >> > >> But why? its the same hardware clock for both tkr's. Surely if one goes > >> funny the other will too. > >> > >> It doesn't make sense to duplicate this. > > > > Well, could it be moved to the timekeeper data structure? What I was > > opposed to was making it super-global, after all the (nice) effort we > > made to tidy up global data structures in this area. > > Ok. I'll rework this then and resend. Though if there's no objections > to Xunlei's portion of the series, would you mind merging those into > -tip? I'd like to make sure they get some time in -next before the > merge window opens. Yeah, it all looks good to me, I've picked up all but this one patch from your series and will push it out once it has gone through testing. I updated some of the titles and changelogs and added a few more Cc:s where it seemed appropriate. Thanks, Ingo