From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
	id S1754081AbcFPSOL (ORCPT <rfc822;w@1wt.eu>);
	Thu, 16 Jun 2016 14:14:11 -0400
Received: from bombadil.infradead.org ([198.137.202.9]:36529 "EHLO
	bombadil.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
	with ESMTP id S1751930AbcFPSOJ (ORCPT
	<rfc822;linux-kernel@vger.kernel.org>);
	Thu, 16 Jun 2016 14:14:09 -0400
Date: Thu, 16 Jun 2016 20:14:02 +0200
From: Peter Zijlstra <peterz@infradead.org>
To: "Paul E. McKenney" <paulmck@linux.vnet.ibm.com>
Cc: Thomas Gleixner <tglx@linutronix.de>,
        Arjan van de Ven <arjanvandeven@gmail.com>,
        Eric Dumazet <edumazet@google.com>, Ingo Molnar <mingo@kernel.org>,
        LKML <linux-kernel@vger.kernel.org>,
        Frederic Weisbecker <fweisbec@gmail.com>, Chris Mason <clm@fb.com>,
        Arjan van de Ven <arjan@infradead.org>,
        Linus Torvalds <torvalds@linux-foundation.org>,
        George Spelvin <linux@sciencehorizons.net>
Subject: Re: [patch 13/20] timer: Switch to a non cascading wheel
Message-ID: <20160616181402.GK30909@twins.programming.kicks-ass.net>
References: <alpine.DEB.2.11.1606141213300.5839@nanos>
 <CADyApD1L48AVT7D94wYuL89m7V+--rg59GFvHzhHMZ5uhhdaYw@mail.gmail.com>
 <alpine.DEB.2.11.1606142000150.5839@nanos>
 <20160614204225.GI30154@twins.programming.kicks-ass.net>
 <CANn89iKO4BzNOW9HCFdjbQxVbk=cwxFJ1ssRmnX+mfUnA+fD5A@mail.gmail.com>
 <alpine.DEB.2.11.1606151651520.5839@nanos>
 <CADyApD3QNoS+dkv_XhzKbk+KQv_JC5p=jXT-=bcWh1xmfyNdQw@mail.gmail.com>
 <alpine.DEB.2.11.1606151746270.5839@nanos>
 <alpine.DEB.2.11.1606161700150.5839@nanos>
 <20160616160215.GQ3923@linux.vnet.ibm.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20160616160215.GQ3923@linux.vnet.ibm.com>
User-Agent: Mutt/1.5.23.1 (2014-03-12)
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Thu, Jun 16, 2016 at 09:02:15AM -0700, Paul E. McKenney wrote:
> > 2) When we do that right, we can make the tick frequency a command line option
> >    and just have a compiled in default.
> 
> As long as there is something that tells RCU what the tick frequency
> actually is at runtime, this should not be a problem.  For example,
> in rcu_implicit_dynticks_qs(), the following:
> 
> 	rdp->rsp->jiffies_resched += 5;
> 
> Would instead need to be something like:
> 
> 	rdp->rsp->jiffies_resched += 5 * jiffies_per_tick;
> 
> Changing tick frequency at runtime would be a bit more tricky, as it would
> be tough to avoid some oddball false positives during the transition.

So the 'fun' part will be frequencies with non integer factors of 1000.
Like say HZ=300. For that we'll have to keep jiffies_remainder, and
add an extra jiffy every time that rolls over.

That would make your case slightly more interesting than you really
want I suspect.