From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754823AbbGGGjj (ORCPT ); Tue, 7 Jul 2015 02:39:39 -0400 Received: from www.linutronix.de ([62.245.132.108]:58635 "EHLO Galois.linutronix.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753303AbbGGGjV (ORCPT ); Tue, 7 Jul 2015 02:39:21 -0400 Date: Tue, 7 Jul 2015 08:39:01 +0200 (CEST) From: Thomas Gleixner To: Frederic Weisbecker cc: Vatika Harlalka , Peter Zijlstra , preeti@linux.vnet.ibm.com, mingo@redhat.com, rafael.j.wysocki@intel.com, schwidefsky@de.ibm.com, paulmck@linux.vnet.ibm.com, linux-kernel@vger.kernel.org, cl@linux.com, cmetcalf@ezchip.com, bitbucket@online.de Subject: Re: [PATCH RESEND] nohz: Affining unpinned timers In-Reply-To: <20150706224948.GF4981@lerouge> Message-ID: References: <20150623083133.GA587@vatika-Inspiron-3443> <20150706165405.GD4981@lerouge> <20150706224948.GF4981@lerouge> User-Agent: Alpine 2.11 (DEB 23 2013-08-11) MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII X-Linutronix-Spam-Score: -1.0 X-Linutronix-Spam-Level: - X-Linutronix-Spam-Status: No , -1.0 points, 5.0 required, ALL_TRUSTED=-1,SHORTCIRCUIT=-0.0001 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, 7 Jul 2015, Frederic Weisbecker wrote: > On Mon, Jul 06, 2015 at 07:18:22PM +0200, Thomas Gleixner wrote: > > On Mon, 6 Jul 2015, Frederic Weisbecker wrote: > > > I hope everybody received the patch because there is a ";" after each address :-) > > > > > > On Tue, Jun 23, 2015 at 02:01:33PM +0530, Vatika Harlalka wrote: > > > > The problem addressed in this patch is about affining unpinned timers. > > > > Adaptive or Full Dynticks CPUs should not be disturbed by unnecessary > > > > jitter due to firing of such timers on them. > > > > This patch will affine timers to online CPUs which are not Full Dynticks > > > > in FULL_NOHZ configured systems. It will not bring about functional > > > > changes if NOHZ_FULL is not configured, because is_housekeeping_cpu() > > > > always returns true in CONFIG_NO_HZ_FULL=n. > > > > > > > > Signed-off by: Vatika Harlalka > > > > > > The patch looks good to me. Peter, Thomas, are you ok with it too? > > > > By some definition of OK. The overhead of this is growing and growing. > > > > We really need to make this a pull not a push model. > > I'm currently working toward that. > See "[PATCH 0/8] tick/nohz: Tick dependency quick check + cleanups" as a > first step. > > Now for this very patch, I don't know how we could make it better. Any > suggestion? Not for the time being. When I'm done with the timer wheel overhaul we should look into queueing stuff in different wheels and let the non idle cpus pull from there instead of pushing stuff at enqueue time. The latter is really silly because most of these timers are removed before they expire. Needs some thoughts, but anything which avoids the whole target cpu crap at enqueue time is better than what we have now. Thanks, tglx