Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1172032AbdDXOEs (ORCPT ); Mon, 24 Apr 2017 10:04:48 -0400 Received: from mail-wr0-f194.google.com ([209.85.128.194]:34918 "EHLO mail-wr0-f194.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S934593AbdDXOEl (ORCPT ); Mon, 24 Apr 2017 10:04:41 -0400 Date: Mon, 24 Apr 2017 16:04:37 +0200 From: Frederic Weisbecker To: Ingo Molnar Cc: Thomas Gleixner , LKML , Peter Zijlstra , Rik van Riel , James Hartsock , Tim Wright , Pavel Machek Subject: Re: [PATCH 0/2] nohz: Deal with clock reprogram skipping issues v2 Message-ID: <20170424140436.GD21353@lerouge> References: <1492783255-5051-1-git-send-email-fweisbec@gmail.com> <20170424080835.22yjqtj6xkynx3nm@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170424080835.22yjqtj6xkynx3nm@gmail.com> User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1522 Lines: 40 On Mon, Apr 24, 2017 at 10:08:35AM +0200, Ingo Molnar wrote: > > * Frederic Weisbecker wrote: > > > As suggested by Thomas Gleixner, the second patch now integrates > > a fix in case the sanity check fails and the clockevent isn't programmed > > as expected. > > > > Frederic Weisbecker (2): > > nohz: Fix again collision between tick and other hrtimers > > tick: Make sure tick timer is active when bypassing reprogramming > > > > kernel/time/tick-sched.c | 33 ++++++++++++++++++++++++++++++--- > > kernel/time/tick-sched.h | 2 ++ > > 2 files changed, 32 insertions(+), 3 deletions(-) > > So I think one of these is causing a new warning on latest -tip: > > [ 333.341756] ------------[ cut here ]------------ > [ 333.346404] WARNING: CPU: 0 PID: 0 at kernel/time/tick-sched.c:874 __tick_nohz_idle_enter+0x461/0x490 Oh I'll never be done with that bug :) Ok I just booted your config with tip/master and didn't see the warning. But the boot seem to be stalled some time after mounting the root fs. Can you please try the following patch and tell me what it returns to you? diff --git a/kernel/time/tick-sched.c b/kernel/time/tick-sched.c index c47d135..6d72e8b 100644 --- a/kernel/time/tick-sched.c +++ b/kernel/time/tick-sched.c @@ -872,6 +872,7 @@ static ktime_t tick_nohz_stop_sched_tick(struct tick_sched *ts, goto out; WARN_ON_ONCE(1); + printk_once("basemono: %llu ts->next_tick: %llu dev->next_event: %llu\n", basemono, ts->next_tick, dev->next_event); } /*