From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753232AbdFMKIW (ORCPT ); Tue, 13 Jun 2017 06:08:22 -0400 Received: from terminus.zytor.com ([65.50.211.136]:42621 "EHLO terminus.zytor.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752480AbdFMKIU (ORCPT ); Tue, 13 Jun 2017 06:08:20 -0400 Date: Tue, 13 Jun 2017 03:03:27 -0700 From: tip-bot for Frederic Weisbecker Message-ID: Cc: fweisbec@gmail.com, hpa@zytor.com, mingo@kernel.org, alexander.levin@verizon.com, peterz@infradead.org, linux-kernel@vger.kernel.org, tim@binbash.co.uk, torvalds@linux-foundation.org, abdhalee@linux.vnet.ibm.com, hartsjc@redhat.com, tglx@linutronix.de, riel@redhat.com Reply-To: fweisbec@gmail.com, alexander.levin@verizon.com, peterz@infradead.org, mingo@kernel.org, hpa@zytor.com, abdhalee@linux.vnet.ibm.com, torvalds@linux-foundation.org, tim@binbash.co.uk, linux-kernel@vger.kernel.org, riel@redhat.com, hartsjc@redhat.com, tglx@linutronix.de In-Reply-To: <1497326654-14122-1-git-send-email-fweisbec@gmail.com> References: <1497326654-14122-1-git-send-email-fweisbec@gmail.com> To: linux-tip-commits@vger.kernel.org Subject: [tip:timers/nohz] nohz: Fix spurious warning when hrtimer and clockevent get out of sync Git-Commit-ID: d4af6d933ccffd24286528f04d5c39e702c8580f X-Mailer: tip-git-log-daemon Robot-ID: Robot-Unsubscribe: Contact to get blacklisted from these emails MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Content-Type: text/plain; charset=UTF-8 Content-Disposition: inline Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Commit-ID: d4af6d933ccffd24286528f04d5c39e702c8580f Gitweb: http://git.kernel.org/tip/d4af6d933ccffd24286528f04d5c39e702c8580f Author: Frederic Weisbecker AuthorDate: Tue, 13 Jun 2017 06:04:14 +0200 Committer: Ingo Molnar CommitDate: Tue, 13 Jun 2017 08:45:43 +0200 nohz: Fix spurious warning when hrtimer and clockevent get out of sync The sanity check ensuring that the tick expiry cache (ts->next_tick) is actually in sync with the hardware clock (dev->next_event) makes the wrong assumption that the clock can't be programmed later than the hrtimer deadline. In fact the clock hardware can be programmed later on some conditions such as: * The hrtimer deadline is already in the past. * The hrtimer deadline is earlier than the minimum delay supported by the hardware. Such conditions can be met when we program the tick, for example if the last jiffies update hasn't been seen by the current CPU yet, we may program the hrtimer to a deadline that is earlier than ktime_get() because last_jiffies_update is our timestamp base to compute the next tick. As a result, we can randomly observe such warning: WARNING: CPU: 5 PID: 0 at kernel/time/tick-sched.c:794 tick_nohz_stop_sched_tick kernel/time/tick-sched.c:791 [inline] Call Trace: tick_nohz_irq_exit tick_irq_exit irq_exit exiting_irq smp_call_function_interrupt smp_call_function_single_interrupt call_function_single_interrupt Therefore, let's rather make sure that the tick expiry cache is sync'ed with the tick hrtimer deadline, against which it is not supposed to drift away. The clock hardware instead has its own will and can't be used as a reliable comparison point. Reported-and-tested-by: Sasha Levin Reported-and-tested-by: Abdul Haleem Signed-off-by: Frederic Weisbecker Cc: James Hartsock Cc: Linus Torvalds Cc: Peter Zijlstra Cc: Rik van Riel Cc: Thomas Gleixner Cc: Tim Wright Link: http://lkml.kernel.org/r/1497326654-14122-1-git-send-email-fweisbec@gmail.com [ Minor readability edit. ] Signed-off-by: Ingo Molnar --- kernel/time/tick-sched.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/kernel/time/tick-sched.c b/kernel/time/tick-sched.c index 9d31f1e..2046009 100644 --- a/kernel/time/tick-sched.c +++ b/kernel/time/tick-sched.c @@ -768,7 +768,7 @@ static ktime_t tick_nohz_stop_sched_tick(struct tick_sched *ts, /* Skip reprogram of event if its not changed */ if (ts->tick_stopped && (expires == ts->next_tick)) { /* Sanity check: make sure clockevent is actually programmed */ - if (likely(dev->next_event <= ts->next_tick)) + if (tick == KTIME_MAX || ts->next_tick == hrtimer_get_expires(&ts->sched_timer)) goto out; WARN_ON_ONCE(1); @@ -806,8 +806,10 @@ static ktime_t tick_nohz_stop_sched_tick(struct tick_sched *ts, goto out; } + hrtimer_set_expires(&ts->sched_timer, tick); + if (ts->nohz_mode == NOHZ_MODE_HIGHRES) - hrtimer_start(&ts->sched_timer, tick, HRTIMER_MODE_ABS_PINNED); + hrtimer_start_expires(&ts->sched_timer, HRTIMER_MODE_ABS_PINNED); else tick_program_event(tick, 1); out: