Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751437AbbKMWZw (ORCPT ); Fri, 13 Nov 2015 17:25:52 -0500 Received: from mga14.intel.com ([192.55.52.115]:18992 "EHLO mga14.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750733AbbKMWZv (ORCPT ); Fri, 13 Nov 2015 17:25:51 -0500 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.20,289,1444719600"; d="scan'208";a="819492212" Date: Fri, 13 Nov 2015 14:24:38 -0800 From: Jacob Pan To: Thomas Gleixner Cc: Peter Zijlstra , Ingo Molnar , John Stultz , LKML , Arjan van de Ven , Srinivas Pandruvada , Len Brown , Rafael Wysocki , Eduardo Valentin , Paul Turner , jacob.jun.pan@linux.intel.com Subject: Re: [PATCH 2/4] timer: relax tick stop in idle entry Message-ID: <20151113142438.3144d47d@icelake> In-Reply-To: References: <1447444387-23525-1-git-send-email-jacob.jun.pan@linux.intel.com> <1447444387-23525-3-git-send-email-jacob.jun.pan@linux.intel.com> Organization: OTC X-Mailer: Claws Mail 3.9.3 (GTK+ 2.24.23; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 4323 Lines: 77 On Fri, 13 Nov 2015 15:22:16 -0500 (EST) Thomas Gleixner wrote: > > > On Fri, 13 Nov 2015, Jacob Pan wrote: > > > Upon entering idle, we can turn off tick if the next timeout > > is exactly one tick away. Otherwise, we could enter inner idle loop > > with tick still enabled, without resched set, the tick will continue > > during idle therefore less optimal in terms of energy savings. > > This does not make any sense at all. > > next_tick is the next required tick event. If it's exactly ONE tick > away why should we go through the hassle of stopping it? Just to > cancel the timer and then set it to the same value again? Oh well. > I have been trying to understand this code, please help. Here is my theory and the ftrace of an injection period where tick did not stop. (sorry about the long lines). My comments are after [JP] cat-1993 [000] 30.093405: sched_cfs_idle_inject: action:0 throttled:1 [JP] injection timer expired, set forced idle flag, call scheduler cat-1993 [000] 30.093406: hrtimer_expire_exit: hrtimer=0xffff88003de0cc20 cat-1993 [000] 30.093406: hrtimer_start: hrtimer=0xffff88003de0cc20 function=idle_inject_timer_fn/0x0 expires=29993055250 softexpires=29993055250 cat-1993 [000] 30.093407: hrtimer_cancel: hrtimer=0xffff88003dfce400 cat-1993 [000] 30.093407: hrtimer_expire_entry: hrtimer=0xffff88003dfce400 now=29988042960 function=tick_sched_timer/0x0 cat-1993 [000] 30.093407: function: tick_sched_timer cat-1993 [000] 30.093422: function: tick_sched_do_timer cat-1993 [000] 30.093422: function: tick_do_update_jiffies64 cat-1993 [000] 30.093433: function: tick_sched_handle.isra.15 cat-1993 [000] 30.093447: sched_stat_runtime: comm=cat pid=1993 runtime=1058498 [ns] vruntime=6695549826 [ns] cat-1993 [000] 30.093449: hrtimer_expire_exit: hrtimer=0xffff88003dfce400 cat-1993 [000] 30.093449: hrtimer_start: hrtimer=0xffff88003dfce400 function=tick_sched_timer/0x0 expires=29989000000 softexpires=29989000000 cat-1993 [000] 30.093450: function: tick_program_event cat-1993 [000] 30.093460: sched_waking: comm=rcu_preempt pid=7 prio=120 target_cpu=002 cat-1993 [000] 30.093461: sched_wake_idle_without_ipi: cpu=2 cat-1993 [000] 30.093463: sched_cfs_idle_inject: action:1 throttled:1 [JP] CFS pick_next_task_fair sees forced idle, pick no task to run. cat-1993 [000] 30.093463: sched_stat_runtime: comm=cat pid=1993 runtime=16122 [ns] vruntime=6695565948 [ns] cat-1993 [000] 30.093464: sched_switch: cat:1993 [120] R ==> swapper/0:0 [120] -0 [000] 30.093465: function: tick_nohz_idle_enter -0 [000] 30.093473: bprint: __tick_nohz_idle_enter: JPAN: __tick_nohz_idle_enter 803 -0 [000] 30.093473: bprint: __tick_nohz_idle_enter: JPAN: can_stop_idle_tick 743 [JP] can_stop_idle_tick() checks ok to stop tick -0 [000] 30.093474: bprint: __tick_nohz_idle_enter: JPAN: tick_nohz_stop_sched_tick 609 delta 1000000 [JP] but sees delta is exactly 1 tick away. didn't stop tick. -0 [000] 30.093475: function: tick_check_broadcast_expired -0 [000] 30.094366: function: tick_irq_enter -0 [000] 30.094367: function: tick_check_oneshot_broadcast_this_cpu -0 [000] 30.094372: function: tick_nohz_stop_idle -0 [000] 30.094387: hrtimer_cancel: hrtimer=0xffff88003dfce400 [JP] enter repeated tick sched in inner idle loop since !need_resched() > Thanks, > > tglx > > [Jacob Pan] -- 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/