From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751963AbdHGUgs (ORCPT ); Mon, 7 Aug 2017 16:36:48 -0400 Received: from mx0a-001b2d01.pphosted.com ([148.163.156.1]:56207 "EHLO mx0a-001b2d01.pphosted.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751743AbdHGUgq (ORCPT ); Mon, 7 Aug 2017 16:36:46 -0400 Date: Mon, 7 Aug 2017 13:36:39 -0700 From: "Paul E. McKenney" To: Prarit Bhargava Cc: John Stultz , lkml , Mark Salyzyn , Jonathan Corbet , Petr Mladek , Sergey Senozhatsky , Steven Rostedt , Thomas Gleixner , Stephen Boyd , Andrew Morton , Greg Kroah-Hartman , Christoffer Dall , Deepa Dinamani , Ingo Molnar , Joel Fernandes , Kees Cook , Peter Zijlstra , Geert Uytterhoeven , "Luis R. Rodriguez" , Nicholas Piggin , "Jason A. Donenfeld" , Olof Johansson , Josh Poimboeuf , linux-doc@vger.kernel.org Subject: Re: [PATCH v4] printk: Add monotonic, boottime, and realtime timestamps Reply-To: paulmck@linux.vnet.ibm.com References: <1502121162-27981-1-git-send-email-prarit@redhat.com> <9452a611-aefb-543e-e2f7-32301b29a2cc@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) X-TM-AS-GCONF: 00 x-cbid: 17080720-2213-0000-0000-00000205FFB3 X-IBM-SpamModules-Scores: X-IBM-SpamModules-Versions: BY=3.00007503; HX=3.00000241; KW=3.00000007; PH=3.00000004; SC=3.00000217; SDB=6.00899006; UDB=6.00449939; IPR=6.00679211; BA=6.00005516; NDR=6.00000001; ZLA=6.00000005; ZF=6.00000009; ZB=6.00000000; ZP=6.00000000; ZH=6.00000000; ZU=6.00000002; MB=3.00016578; XFM=3.00000015; UTC=2017-08-07 20:36:44 X-IBM-AV-DETECTION: SAVI=unused REMOTE=unused XFE=unused x-cbparentid: 17080720-2214-0000-0000-0000572481A0 Message-Id: <20170807203639.GY3730@linux.vnet.ibm.com> X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10432:,, definitions=2017-08-07_13:,, signatures=0 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 spamscore=0 suspectscore=0 malwarescore=0 phishscore=0 adultscore=0 bulkscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.0.1-1706020000 definitions=main-1708070340 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Aug 07, 2017 at 04:06:09PM -0400, Prarit Bhargava wrote: > > > On 08/07/2017 02:47 PM, John Stultz wrote: > > On Mon, Aug 7, 2017 at 11:04 AM, Prarit Bhargava wrote: > >> On 08/07/2017 12:52 PM, John Stultz wrote: > >>> Still not quite following why you're updating all the defconfigs. I'd > >>> make sure the Kconfig default settings are right, and leave updating > >>> the defconfig to arch/device maintainers. It adds a lot of noise to > >>> the patch. > >> > >> Hmm ... I thought it was up to the patch submitter to make sure that > >> 'make defconfig' still worked? Are you sure I can leave that broken? > >> > >> /me *really* doesn't want to get yelled at by every arch maintainer. > > > > No. Don't break systems, but at the same time, can't you use the > > default value in Kconfig to set it properly so the old defconfig > > settings don't really matter? > > > > Apologies if I've not followed the issue properly, but it is odd, as > > I'm not sure I can think of a patch I've seen before that had so much > > defconfig noise in it. Again, I've not looked into it closely, so it > > may just be my own ignorance, but it makes me suspect there is a > > better way. > > > > peterz? Want to offer a suggestion? The issue is that I'm changing a bool > config option to an int and that impacts all the arch's defconfigs. John points > out that this is a lot of churn and we're both wondering if there's a better way > to do the configs. The usual approach is to keep the old bool Kconfig option, and add another int Kconfig option that depends on the original one. The tests for the int value get a bit more complex, but one way to handle this is to define a cpp macro something like the following: #ifdef CONFIG_OLD_OPTION #define CPP_NEW_OPTION 0 #else #define CPP_NEW_OPTION CONFIG_NEW_OPTION #endif Then use CPP_NEW_OPTION, where zero means disabled and other numbers select the available options. Adjust to suit depending on what values mean what. Another approach is to make the range of the new Kconfig option depend on the old option: config NEW_OPTION int "your description here" range 1 5 if OLD_OPTION range 0 0 if !OLD_OPTION default 0 help your help here Again, adjust to suit depending on what values mean what. Thanx, Paul