From patchwork Tue Sep 20 00:19:38 2011 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Anton Blanchard X-Patchwork-Id: 115401 Return-Path: X-Original-To: patchwork-incoming@ozlabs.org Delivered-To: patchwork-incoming@ozlabs.org Received: from ozlabs.org (localhost [IPv6:::1]) by ozlabs.org (Postfix) with ESMTP id 0890D100A1B for ; Tue, 20 Sep 2011 10:20:07 +1000 (EST) Received: from kryten (unknown [12.118.188.58]) (using TLSv1 with cipher DHE-RSA-AES128-SHA (128/128 bits)) (Client did not present a certificate) by ozlabs.org (Postfix) with ESMTPSA id 620EEB6FF5; Tue, 20 Sep 2011 10:19:54 +1000 (EST) Date: Tue, 20 Sep 2011 10:19:38 +1000 From: Anton Blanchard To: Peter Zijlstra Subject: Re: [PATCH 0/5] ppc64 scheduler fixes Message-ID: <20110920101938.121098ed@kryten> In-Reply-To: <1311597702.24752.1.camel@twins> References: <20110725023311.175792493@samba.org> <1311597702.24752.1.camel@twins> X-Mailer: Claws Mail 3.7.8 (GTK+ 2.24.4; i686-pc-linux-gnu) Mime-Version: 1.0 Cc: mingo@elte.hu, linuxppc-dev@lists.ozlabs.org, linux-kernel@vger.kernel.org X-BeenThere: linuxppc-dev@lists.ozlabs.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: linuxppc-dev-bounces+patchwork-incoming=ozlabs.org@lists.ozlabs.org Sender: linuxppc-dev-bounces+patchwork-incoming=ozlabs.org@lists.ozlabs.org Hi Peter, > On Mon, 2011-07-25 at 12:33 +1000, Anton Blanchard wrote: > > Here are a set of ppc64 scheduler fixes that help with some > > multi node performance issues. > > They look fine to me. I'll probably ping you when I'll rip out all > that SD_NODES_PER_DOMAIN crap for good, but until then I'm fine with > you fiddling it for ppc64. > > Acked-by: Peter Zijlstra Are you OK if Ben takes this, or would you prefer to send it via the scheduler tree? Anton --- [2/5] sched: Allow SD_NODES_PER_DOMAIN to be overridden We want to override the default value of SD_NODES_PER_DOMAIN on ppc64, so move it into linux/topology.h. Signed-off-by: Anton Blanchard --- Index: linux-2.6-work/include/linux/topology.h =================================================================== --- linux-2.6-work.orig/include/linux/topology.h 2011-07-25 11:20:02.588717796 +1000 +++ linux-2.6-work/include/linux/topology.h 2011-07-25 11:26:50.616468376 +1000 @@ -201,6 +201,10 @@ int arch_update_cpu_topology(void); .balance_interval = 64, \ } +#ifndef SD_NODES_PER_DOMAIN +#define SD_NODES_PER_DOMAIN 16 +#endif + #ifdef CONFIG_SCHED_BOOK #ifndef SD_BOOK_INIT #error Please define an appropriate SD_BOOK_INIT in include/asm/topology.h!!! Index: linux-2.6-work/kernel/sched.c =================================================================== --- linux-2.6-work.orig/kernel/sched.c 2011-07-25 11:20:09.538850173 +1000 +++ linux-2.6-work/kernel/sched.c 2011-07-25 11:26:50.626468565 +1000 @@ -6938,8 +6938,6 @@ static int __init isolated_cpu_setup(cha __setup("isolcpus=", isolated_cpu_setup); -#define SD_NODES_PER_DOMAIN 16 - #ifdef CONFIG_NUMA /**