Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751834AbbFNBos (ORCPT ); Sat, 13 Jun 2015 21:44:48 -0400 Received: from e38.co.us.ibm.com ([32.97.110.159]:39814 "EHLO e38.co.us.ibm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751377AbbFNBom (ORCPT ); Sat, 13 Jun 2015 21:44:42 -0400 X-Helo: d03dlp03.boulder.ibm.com X-MailFrom: preeti@linux.vnet.ibm.com X-RcptTo: linux-kernel@vger.kernel.org Message-ID: <557CDC82.9080903@linux.vnet.ibm.com> Date: Sun, 14 Jun 2015 07:14:34 +0530 From: Preeti U Murthy User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.6.0 MIME-Version: 1.0 To: Frederic Weisbecker , LKML CC: Peter Zijlstra , Thomas Gleixner , Christoph Lameter , Ingo Molnar , Viresh Kumar , Rik van Riel Subject: Re: [PATCH 4/8] nohz: Remove idle task special case References: <1434044168-23173-1-git-send-email-fweisbec@gmail.com> <1434044168-23173-5-git-send-email-fweisbec@gmail.com> In-Reply-To: <1434044168-23173-5-git-send-email-fweisbec@gmail.com> Content-Type: text/plain; charset=ISO-8859-6 Content-Transfer-Encoding: 7bit X-TM-AS-MML: disable X-Content-Scanned: Fidelis XPS MAILER x-cbid: 15061401-0029-0000-0000-00000A87C2BB Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2583 Lines: 71 On 06/11/2015 11:06 PM, Frederic Weisbecker wrote: > This is a leftover from old days to avoid conflicts with dynticks idle > code. Now full dynticks and idle dynticks are better integrated and > interact without known issue. I am sorry but I fail to understand why the check on idle task was there in the first place in the below code paths. It would help if you could clarify this in the changelog as well. > > So lets remove it. > > Cc: Christoph Lameter > Cc: Ingo Molnar > Cc; John Stultz > Cc: Peter Zijlstra > Cc: Preeti U Murthy > Cc: Rik van Riel > Cc: Thomas Gleixner > Cc: Viresh Kumar > Signed-off-by: Frederic Weisbecker > --- > kernel/time/tick-sched.c | 8 +++----- > 1 file changed, 3 insertions(+), 5 deletions(-) > > diff --git a/kernel/time/tick-sched.c b/kernel/time/tick-sched.c > index 812f7a3..324482f 100644 > --- a/kernel/time/tick-sched.c > +++ b/kernel/time/tick-sched.c > @@ -208,10 +208,8 @@ void __tick_nohz_full_check(void) > struct tick_sched *ts = this_cpu_ptr(&tick_cpu_sched); > > if (tick_nohz_full_cpu(smp_processor_id())) { > - if (ts->tick_stopped && !is_idle_task(current)) { > - if (!can_stop_full_tick()) can_stop_full_tick() would have bailed out if the current task was idle, since it checks for the number of tasks being greater than 1 to restart the tick. So why was the check is_idle_task() introduced earlier ? > - tick_nohz_restart_sched_tick(ts, ktime_get()); > - } > + if (ts->tick_stopped && !can_stop_full_tick()) > + tick_nohz_restart_sched_tick(ts, ktime_get()); > } > } > > @@ -710,7 +708,7 @@ static void tick_nohz_full_stop_tick(struct tick_sched *ts) > #ifdef CONFIG_NO_HZ_FULL > int cpu = smp_processor_id(); > > - if (!tick_nohz_full_cpu(cpu) || is_idle_task(current)) > + if (!tick_nohz_full_cpu(cpu)) If the current task was indeed idle, the check on ts->inidle would have succeeded in tick_irq_exit() and we would not have reached this function at all, isn't it? So here too I am unable to understand why we had it in the first place. Regards Preeti U Murthy > return; > > if (!ts->tick_stopped && ts->nohz_mode == NOHZ_MODE_INACTIVE) > -- 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/