From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752175AbdFTU0l (ORCPT ); Tue, 20 Jun 2017 16:26:41 -0400 Received: from mail-wr0-f193.google.com ([209.85.128.193]:34166 "EHLO mail-wr0-f193.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751109AbdFTU0k (ORCPT ); Tue, 20 Jun 2017 16:26:40 -0400 Date: Tue, 20 Jun 2017 22:26:35 +0200 From: Frederic Weisbecker To: Rik van Riel Cc: Peter Zijlstra , LKML , Thomas Gleixner , Ingo Molnar Subject: Re: [PATCH 3/3] sched: Spare idle load balancing on nohz_full CPUs Message-ID: <20170620202632.GA2918@lerouge> References: <1497838322-10913-1-git-send-email-fweisbec@gmail.com> <1497838322-10913-4-git-send-email-fweisbec@gmail.com> <1497980547.20270.106.camel@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <1497980547.20270.106.camel@redhat.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 On Tue, Jun 20, 2017 at 01:42:27PM -0400, Rik van Riel wrote: > On Mon, 2017-06-19 at 04:12 +0200, Frederic Weisbecker wrote: > > Although idle load balancing obviously only concern idle CPUs, it can > > be a disturbance on a busy nohz_full CPU. Indeed a CPU can only get > > rid > > of an idle load balancing duty once a tick fires while it runs a task > > and this can take a while in a nohz_full CPU. > > > > We could fix that and escape the idle load balancing duty from the > > very > > idle exit path but that would bring unecessary overhead. Lets just > > not > > bother and leave that job to housekeeping CPUs (those outside > > nohz_full > > range). The nohz_full CPUs simply don't want any disturbance. > > > > Signed-off-by: Frederic Weisbecker > > Cc: Thomas Gleixner > > Cc: Ingo Molnar > > Cc: Rik van Riel > > Cc: Peter Zijlstra > > --- > >  kernel/sched/fair.c | 4 ++++ > >  1 file changed, 4 insertions(+) > > > > diff --git a/kernel/sched/fair.c b/kernel/sched/fair.c > > index d711093..cfca960 100644 > > --- a/kernel/sched/fair.c > > +++ b/kernel/sched/fair.c > > @@ -8659,6 +8659,10 @@ void nohz_balance_enter_idle(int cpu) > >   if (!cpu_active(cpu)) > >   return; > >   > > + /* Spare idle load balancing on CPUs that don't want to be > > disturbed */ > > + if (!is_housekeeping_cpu(cpu)) > > + return; > > + > >   if (test_bit(NOHZ_TICK_STOPPED, nohz_flags(cpu))) > >   return; > > I am not entirely convinced on this one. > > Doesn't the if (on_null_domain(cpu_rq(cpu)) test > a few lines down take care of this already? It shouldn't, since nohz_full= doesn't imply isolcpus= anymore. Of course it does if the user manually adds them. > > Do we want nohz_full to always automatically > imply that no idle balancing will happen, like > on isolated CPUs? You're making a good point in that I would prefer that nohz_full be only about the tick and let some sort of separate isolation subsystem deal with individual isolation features: nohz, workqueues, idle load balancing, etc... That's why I rather used is_housekeeping_cpu() and not !tick_nohz_full_cpu() because for now housekeepers are ~tick_nohz_full_mask but later it should be cpu_possible_mask by default or some given set of CPUs defined by the future isolation subsystem. Thanks.