Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1491998
| From | Sam Ravnborg <sam@ravnborg.org> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | Re: [PATCH v2 1/2] config: Add new CONFIG_PROVE_LOCKING_SMALL |
| Date | 2016-09-27 17:50 +0200 |
| Message-ID | <sm2NQ-2OB-27@gated-at.bofh.it> (permalink) |
| References | <slMJ3-1gp-11@gated-at.bofh.it> <slMJ3-1gp-9@gated-at.bofh.it> <slSEO-54F-11@gated-at.bofh.it> <slZ3z-zK-13@gated-at.bofh.it> <sm21s-2jb-35@gated-at.bofh.it> |
| Organization | linux.* mail to news gateway |
On Tue, Sep 27, 2016 at 09:51:40AM -0500, Babu Moger wrote: > > On 9/27/2016 6:40 AM, Peter Zijlstra wrote: > >On Tue, Sep 27, 2016 at 06:46:25AM +0200, Sam Ravnborg wrote: > >>Since this is only relevant for sparc, and for sparc this is "select"ed, > >>then there is limited/no gain having this as a visible menu config option. > >> > >>How about adding just a simple non-visible config symbol: > >> > >>config PROVE_LOCKING_SMALL > >> bool > >> > >>The nice help text can be added to the H file, and the select > >>can be move to the sparc/Kconfig file where it really belongs. > >Yes, this should not be user selectable. I don't mind the help being > >here though. > How about this? > Moved everything to arch/sparc/Kconfig.debug. It may be not useful > to have help in > config file as it is not visible. Lets have some explanation in .h file. > I will send v3 version if you all agree. > ===================================== > diff --git a/arch/sparc/Kconfig.debug b/arch/sparc/Kconfig.debug > index 6db35fb..67e58a1 100644 > --- a/arch/sparc/Kconfig.debug > +++ b/arch/sparc/Kconfig.debug > @@ -21,4 +21,9 @@ config FRAME_POINTER > depends on MCOUNT > default y > > +config PROVE_LOCKING_SMALL > + bool > + depends on PROVE_LOCKING && SPARC > + default y > + > endmenu The idea is to have the SPAC specific stuff in arch/sparc/Kconfig, and not scattered in Kconfig files all over the tree. Therefore drop the "depends". In sparc/Kconfig you then just do: config SPARC select PROVE_LOCKING_SMALL if PROVE_LOCKING The if part is likely not needed as PROVE_LOCKING_SMALL will be ignored unless PROVE_LOCKING is enabled. Sam
Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread
[PATCH v2 1/2] config: Add new CONFIG_PROVE_LOCKING_SMALL Babu Moger <babu.moger@oracle.com> - 2016-09-27 00:40 +0200
Re: [PATCH v2 1/2] config: Add new CONFIG_PROVE_LOCKING_SMALL Sam Ravnborg <sam@ravnborg.org> - 2016-09-27 07:00 +0200
Re: [PATCH v2 1/2] config: Add new CONFIG_PROVE_LOCKING_SMALL Peter Zijlstra <peterz@infradead.org> - 2016-09-27 13:50 +0200
Re: [PATCH v2 1/2] config: Add new CONFIG_PROVE_LOCKING_SMALL Babu Moger <babu.moger@oracle.com> - 2016-09-27 17:00 +0200
Re: [PATCH v2 1/2] config: Add new CONFIG_PROVE_LOCKING_SMALL Sam Ravnborg <sam@ravnborg.org> - 2016-09-27 17:50 +0200
Re: [PATCH v2 1/2] config: Add new CONFIG_PROVE_LOCKING_SMALL Babu Moger <babu.moger@oracle.com> - 2016-09-27 18:30 +0200
csiph-web