From patchwork Fri Sep 6 19:32:05 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Paul E. McKenney" X-Patchwork-Id: 2855081 Return-Path: X-Original-To: patchwork-linux-kbuild@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork1.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.19.201]) by patchwork1.web.kernel.org (Postfix) with ESMTP id D78489F3DC for ; Fri, 6 Sep 2013 19:32:20 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id 41BE820328 for ; Fri, 6 Sep 2013 19:32:20 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 05606202E8 for ; Fri, 6 Sep 2013 19:32:19 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751764Ab3IFTcR (ORCPT ); Fri, 6 Sep 2013 15:32:17 -0400 Received: from e9.ny.us.ibm.com ([32.97.182.139]:37961 "EHLO e9.ny.us.ibm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750709Ab3IFTcP (ORCPT ); Fri, 6 Sep 2013 15:32:15 -0400 Received: from /spool/local by e9.ny.us.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Fri, 6 Sep 2013 15:32:14 -0400 Received: from d01dlp03.pok.ibm.com (9.56.250.168) by e9.ny.us.ibm.com (192.168.1.109) with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted; Fri, 6 Sep 2013 15:32:11 -0400 Received: from b01cxnp22036.gho.pok.ibm.com (b01cxnp22036.gho.pok.ibm.com [9.57.198.26]) by d01dlp03.pok.ibm.com (Postfix) with ESMTP id AD3B8C90046; Fri, 6 Sep 2013 15:32:10 -0400 (EDT) Received: from d03av06.boulder.ibm.com (d03av06.boulder.ibm.com [9.17.195.245]) by b01cxnp22036.gho.pok.ibm.com (8.13.8/8.13.8/NCO v10.0) with ESMTP id r86JWAAK35258490; Fri, 6 Sep 2013 19:32:10 GMT Received: from d03av06.boulder.ibm.com (loopback [127.0.0.1]) by d03av06.boulder.ibm.com (8.14.4/8.13.1/NCO v10.0 AVout) with ESMTP id r86JZ3UZ012743; Fri, 6 Sep 2013 13:35:06 -0600 Received: from paulmck-ThinkPad-W500 ([9.70.82.208]) by d03av06.boulder.ibm.com (8.14.4/8.13.1/NCO v10.0 AVin) with ESMTP id r86JZ2qx012701; Fri, 6 Sep 2013 13:35:02 -0600 Received: by paulmck-ThinkPad-W500 (Postfix, from userid 1000) id CAAEE385F55; Fri, 6 Sep 2013 12:32:05 -0700 (PDT) Date: Fri, 6 Sep 2013 12:32:05 -0700 From: "Paul E. McKenney" To: Geert Uytterhoeven Cc: "linux-kernel@vger.kernel.org" , Ingo Molnar , laijs@cn.fujitsu.com, dipankar@in.ibm.com, Andrew Morton , Mathieu Desnoyers , josh@joshtriplett.org, niv@us.ibm.com, Thomas Gleixner , Peter Zijlstra , Steven Rostedt , David Howells , edumazet@google.com, darren@dvhart.com, =?iso-8859-1?Q?Fr=E9d=E9ric?= Weisbecker , sbw@mit.edu, Linux-Arch , linux-kbuild Subject: Re: [PATCH tip/core/rcu 8/9] nohz_full: Add full-system-idle state machine Message-ID: <20130906193205.GW3966@linux.vnet.ibm.com> Reply-To: paulmck@linux.vnet.ibm.com References: <20130820024700.GA31075@linux.vnet.ibm.com> <1376966841-31774-1-git-send-email-paulmck@linux.vnet.ibm.com> <1376966841-31774-8-git-send-email-paulmck@linux.vnet.ibm.com> <20130906173047.GT3966@linux.vnet.ibm.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) X-TM-AS-MML: No X-Content-Scanned: Fidelis XPS MAILER x-cbid: 13090619-7182-0000-0000-000008532B42 Sender: linux-kbuild-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kbuild@vger.kernel.org X-Spam-Status: No, score=-9.3 required=5.0 tests=BAYES_00, RCVD_IN_DNSWL_HI, RP_MATCHES_RCVD, UNPARSEABLE_RELAY autolearn=unavailable version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP On Fri, Sep 06, 2013 at 08:50:41PM +0200, Geert Uytterhoeven wrote: > On Fri, Sep 6, 2013 at 7:30 PM, Paul E. McKenney > wrote: > >> Furthermore, it seems only hexagon, metag, mips, and x86 set NR_CPUS to 1 > >> if !SMP. On other architectures, NR_CPUS is not defined and presumed to be 0. > > > > Would it make sense to require that NR_CPUS=1 for !SMP? > > Yes, this looks reasonable to me. > > > I tried creating a NR_CPUS_REALLY as follows: > > > > config NR_CPUS_REALLY > > int "Fixed version of NR_CPUS" > > default NR_CPUS if NR_CPUS > > default 1 if !NR_CPUS > > > > But this still gave a warning on the first "default" even though it > > was not in effect. I also tried using Kconfig "if": > > IIRC, it tries to use the first default first, so the below may work > (the "if SMP" is probably not needed): > > config NR_CPUS_REALLY > int "Fixed version of NR_CPUS" > default 1 if !SMP > default NR_CPUS if SMP Seemed like a good idea, but I still get: make O=/tmp/e ARCH=m68k defconfig GEN /tmp/e/Makefile *** Default configuration is based on 'multi_defconfig' kernel/time/Kconfig:140:warning: 'NR_CPUS_REALLY': number is invalid # # configuration written to .config # Diff below in case I messed something up. > > Defining NR_CPUS=1 if !SMP is looking pretty good to me just now. > > This would probably have other benefits -- I cannot be the only > > person who ever wanted this. ;-) > > Sure. I just didn't want to create patches for all architectures without > having a discussion first. > > And it would be nice if it cuould be done in a central place, without > touching all architectures. Agreed, should it prove possible. ;-) Thanx, Paul ------------------------------------------------------------------------ --- To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html diff --git a/kernel/time/Kconfig b/kernel/time/Kconfig index 3381f09..cb7a932 100644 --- a/kernel/time/Kconfig +++ b/kernel/time/Kconfig @@ -134,6 +134,11 @@ config NO_HZ_FULL_ALL Note the boot CPU will still be kept outside the range to handle the timekeeping duty. +config NR_CPUS_REALLY + int "Fixed version of NR_CPUS" + default 1 if !SMP + default NR_CPUS if SMP + config NO_HZ_FULL_SYSIDLE bool "Detect full-system idle state for full dynticks system" depends on NO_HZ_FULL @@ -160,7 +165,7 @@ config NO_HZ_FULL_SYSIDLE config NO_HZ_FULL_SYSIDLE_SMALL int "Number of CPUs above which large-system approach is used" depends on NO_HZ_FULL_SYSIDLE - range 1 NR_CPUS + range 1 NR_CPUS_REALLY default 8 help The full-system idle detection mechanism takes a lazy approach