* [PATCH 2/2] init/Kconfig: Fix HAVE_FUTEX_CMPXCHG to not break up the EXPERT menu
@ 2014-10-03 23:31 Josh Triplett
2014-10-03 23:36 ` Randy Dunlap
0 siblings, 1 reply; 5+ messages in thread
From: Josh Triplett @ 2014-10-03 23:31 UTC (permalink / raw)
To: Yann E. MORIN, Andrew Morton, Eric Paris, Michal Hocko,
Matt Turner, Paul Gortmaker, 蔡正龙, Tejun Heo,
Fabian Frederick, Luis R. Rodriguez, Peter Foley,
Konstantin Khlebnikov, Eric W. Biederman, H. Peter Anvin,
Oleg Nesterov, linux-kbuild, linux-kernel
commit 03b8c7b623c80af264c4c8d6111e5c6289933666 ("futex: Allow
architectures to skip futex_atomic_cmpxchg_inatomic() test") added the
HAVE_FUTEX_CMPXCHG symbol right below FUTEX. This placed it right in
the middle of the options for the EXPERT menu. However,
HAVE_FUTEX_CMPXCHG does not depend on EXPERT or FUTEX, so Kconfig stops
placing items in the EXPERT menu, and displays the remaining several
EXPERT items (starting with EPOLL) directly in the General Setup menu.
Since both users of HAVE_FUTEX_CMPXCHG only select it "if FUTEX", make
HAVE_FUTEX_CMPXCHG itself depend on FUTEX. With this change, the
subsequent items display as part of the EXPERT menu again; the EMBEDDED
menu now appears as the next top-level item in the General Setup menu,
which makes General Setup much shorter and more usable.
Signed-off-by: Josh Triplett <josh@joshtriplett.org>
---
Posting for review. I can upstream this through the tiny tree.
Personally, I'd consider this a bit of a bug in Kconfig; ideally,
Kconfig should only consider symbols with prompt strings when
considering what to display in a menu. However, in the interim, this
one-line patch drastically improves the usability of the "General Setup"
config menu.
init/Kconfig | 1 +
1 file changed, 1 insertion(+)
diff --git a/init/Kconfig b/init/Kconfig
index 31505a5..80a6907 100644
--- a/init/Kconfig
+++ b/init/Kconfig
@@ -1477,6 +1477,7 @@ config FUTEX
config HAVE_FUTEX_CMPXCHG
bool
+ depends on FUTEX
help
Architectures should select this if futex_atomic_cmpxchg_inatomic()
is implemented and always working. This removes a couple of runtime
--
2.1.1
^ permalink raw reply related [flat|nested] 5+ messages in thread* Re: [PATCH 2/2] init/Kconfig: Fix HAVE_FUTEX_CMPXCHG to not break up the EXPERT menu
2014-10-03 23:31 [PATCH 2/2] init/Kconfig: Fix HAVE_FUTEX_CMPXCHG to not break up the EXPERT menu Josh Triplett
@ 2014-10-03 23:36 ` Randy Dunlap
2014-10-03 23:47 ` Josh Triplett
0 siblings, 1 reply; 5+ messages in thread
From: Randy Dunlap @ 2014-10-03 23:36 UTC (permalink / raw)
To: Josh Triplett, Yann E. MORIN, Andrew Morton, Eric Paris,
Michal Hocko, Matt Turner, Paul Gortmaker,
蔡正龙, Tejun Heo, Fabian Frederick,
Luis R. Rodriguez, Peter Foley, Konstantin Khlebnikov,
Eric W. Biederman, H. Peter Anvin, Oleg Nesterov, linux-kbuild,
linux-kernel
On 10/03/14 16:31, Josh Triplett wrote:
> commit 03b8c7b623c80af264c4c8d6111e5c6289933666 ("futex: Allow
> architectures to skip futex_atomic_cmpxchg_inatomic() test") added the
> HAVE_FUTEX_CMPXCHG symbol right below FUTEX. This placed it right in
> the middle of the options for the EXPERT menu. However,
> HAVE_FUTEX_CMPXCHG does not depend on EXPERT or FUTEX, so Kconfig stops
> placing items in the EXPERT menu, and displays the remaining several
> EXPERT items (starting with EPOLL) directly in the General Setup menu.
>
> Since both users of HAVE_FUTEX_CMPXCHG only select it "if FUTEX", make
> HAVE_FUTEX_CMPXCHG itself depend on FUTEX. With this change, the
> subsequent items display as part of the EXPERT menu again; the EMBEDDED
> menu now appears as the next top-level item in the General Setup menu,
> which makes General Setup much shorter and more usable.
>
> Signed-off-by: Josh Triplett <josh@joshtriplett.org>
> ---
>
> Posting for review. I can upstream this through the tiny tree.
>
> Personally, I'd consider this a bit of a bug in Kconfig; ideally,
> Kconfig should only consider symbols with prompt strings when
> considering what to display in a menu. However, in the interim, this
> one-line patch drastically improves the usability of the "General Setup"
> config menu.
Good catch. Thanks.
I would prefer to see both of your patches merged quickly into 3.17 no matter
how they get there.
both patches:
Acked-by: Randy Dunlap <rdunlap@infradead.org>
> init/Kconfig | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/init/Kconfig b/init/Kconfig
> index 31505a5..80a6907 100644
> --- a/init/Kconfig
> +++ b/init/Kconfig
> @@ -1477,6 +1477,7 @@ config FUTEX
>
> config HAVE_FUTEX_CMPXCHG
> bool
> + depends on FUTEX
> help
> Architectures should select this if futex_atomic_cmpxchg_inatomic()
> is implemented and always working. This removes a couple of runtime
>
--
~Randy
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH 2/2] init/Kconfig: Fix HAVE_FUTEX_CMPXCHG to not break up the EXPERT menu
2014-10-03 23:36 ` Randy Dunlap
@ 2014-10-03 23:47 ` Josh Triplett
2014-10-04 0:00 ` Randy Dunlap
0 siblings, 1 reply; 5+ messages in thread
From: Josh Triplett @ 2014-10-03 23:47 UTC (permalink / raw)
To: Randy Dunlap
Cc: Yann E. MORIN, Andrew Morton, Eric Paris, Michal Hocko,
Matt Turner, Paul Gortmaker, 蔡正龙, Tejun Heo,
Fabian Frederick, Luis R. Rodriguez, Peter Foley,
Konstantin Khlebnikov, Eric W. Biederman, H. Peter Anvin,
Oleg Nesterov, linux-kbuild, linux-kernel
On Fri, Oct 03, 2014 at 04:36:36PM -0700, Randy Dunlap wrote:
> On 10/03/14 16:31, Josh Triplett wrote:
> > commit 03b8c7b623c80af264c4c8d6111e5c6289933666 ("futex: Allow
> > architectures to skip futex_atomic_cmpxchg_inatomic() test") added the
> > HAVE_FUTEX_CMPXCHG symbol right below FUTEX. This placed it right in
> > the middle of the options for the EXPERT menu. However,
> > HAVE_FUTEX_CMPXCHG does not depend on EXPERT or FUTEX, so Kconfig stops
> > placing items in the EXPERT menu, and displays the remaining several
> > EXPERT items (starting with EPOLL) directly in the General Setup menu.
> >
> > Since both users of HAVE_FUTEX_CMPXCHG only select it "if FUTEX", make
> > HAVE_FUTEX_CMPXCHG itself depend on FUTEX. With this change, the
> > subsequent items display as part of the EXPERT menu again; the EMBEDDED
> > menu now appears as the next top-level item in the General Setup menu,
> > which makes General Setup much shorter and more usable.
> >
> > Signed-off-by: Josh Triplett <josh@joshtriplett.org>
> > ---
> >
> > Posting for review. I can upstream this through the tiny tree.
> >
> > Personally, I'd consider this a bit of a bug in Kconfig; ideally,
> > Kconfig should only consider symbols with prompt strings when
> > considering what to display in a menu. However, in the interim, this
> > one-line patch drastically improves the usability of the "General Setup"
> > config menu.
>
> Good catch. Thanks.
>
> I would prefer to see both of your patches merged quickly into 3.17 no matter
> how they get there.
>
> both patches:
> Acked-by: Randy Dunlap <rdunlap@infradead.org>
Both of the fixes are entirely about Kconfig usability, don't affect the
built kernel, and have existed for quite a few kernel releases, so I
hadn't planned to get them into 3.17 at the last minute. I'd just
planned to submit them during the 3.18 merge window when that opens.
Do you really think they should be pushed into 3.17 at this point?
If you'd like, I could get them into 3.18 and mark them for stable,
instead.
- Josh Triplett
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH 2/2] init/Kconfig: Fix HAVE_FUTEX_CMPXCHG to not break up the EXPERT menu
2014-10-03 23:47 ` Josh Triplett
@ 2014-10-04 0:00 ` Randy Dunlap
2014-10-04 0:17 ` Josh Triplett
0 siblings, 1 reply; 5+ messages in thread
From: Randy Dunlap @ 2014-10-04 0:00 UTC (permalink / raw)
To: Josh Triplett
Cc: Yann E. MORIN, Andrew Morton, Eric Paris, Michal Hocko,
Matt Turner, Paul Gortmaker, 蔡正龙, Tejun Heo,
Fabian Frederick, Luis R. Rodriguez, Peter Foley,
Konstantin Khlebnikov, Eric W. Biederman, H. Peter Anvin,
Oleg Nesterov, linux-kbuild, linux-kernel
On 10/03/14 16:47, Josh Triplett wrote:
> On Fri, Oct 03, 2014 at 04:36:36PM -0700, Randy Dunlap wrote:
>> On 10/03/14 16:31, Josh Triplett wrote:
>>> commit 03b8c7b623c80af264c4c8d6111e5c6289933666 ("futex: Allow
>>> architectures to skip futex_atomic_cmpxchg_inatomic() test") added the
>>> HAVE_FUTEX_CMPXCHG symbol right below FUTEX. This placed it right in
>>> the middle of the options for the EXPERT menu. However,
>>> HAVE_FUTEX_CMPXCHG does not depend on EXPERT or FUTEX, so Kconfig stops
>>> placing items in the EXPERT menu, and displays the remaining several
>>> EXPERT items (starting with EPOLL) directly in the General Setup menu.
>>>
>>> Since both users of HAVE_FUTEX_CMPXCHG only select it "if FUTEX", make
>>> HAVE_FUTEX_CMPXCHG itself depend on FUTEX. With this change, the
>>> subsequent items display as part of the EXPERT menu again; the EMBEDDED
>>> menu now appears as the next top-level item in the General Setup menu,
>>> which makes General Setup much shorter and more usable.
>>>
>>> Signed-off-by: Josh Triplett <josh@joshtriplett.org>
>>> ---
>>>
>>> Posting for review. I can upstream this through the tiny tree.
>>>
>>> Personally, I'd consider this a bit of a bug in Kconfig; ideally,
>>> Kconfig should only consider symbols with prompt strings when
>>> considering what to display in a menu. However, in the interim, this
>>> one-line patch drastically improves the usability of the "General Setup"
>>> config menu.
>>
>> Good catch. Thanks.
>>
>> I would prefer to see both of your patches merged quickly into 3.17 no matter
>> how they get there.
>>
>> both patches:
>> Acked-by: Randy Dunlap <rdunlap@infradead.org>
>
> Both of the fixes are entirely about Kconfig usability, don't affect the
> built kernel, and have existed for quite a few kernel releases, so I
> hadn't planned to get them into 3.17 at the last minute. I'd just
> planned to submit them during the 3.18 merge window when that opens.
>
> Do you really think they should be pushed into 3.17 at this point?
Both of them are bugs IMO, even though they are usability/presentation bugs.
and they make the menu system confusing.
> If you'd like, I could get them into 3.18 and mark them for stable,
> instead.
Do that at a minimum, please.
--
~Randy
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH 2/2] init/Kconfig: Fix HAVE_FUTEX_CMPXCHG to not break up the EXPERT menu
2014-10-04 0:00 ` Randy Dunlap
@ 2014-10-04 0:17 ` Josh Triplett
0 siblings, 0 replies; 5+ messages in thread
From: Josh Triplett @ 2014-10-04 0:17 UTC (permalink / raw)
To: Randy Dunlap
Cc: Yann E. MORIN, Andrew Morton, Eric Paris, Michal Hocko,
Matt Turner, Paul Gortmaker, 蔡正龙, Tejun Heo,
Fabian Frederick, Luis R. Rodriguez, Peter Foley,
Konstantin Khlebnikov, Eric W. Biederman, H. Peter Anvin,
Oleg Nesterov, linux-kbuild, linux-kernel
On Fri, Oct 03, 2014 at 05:00:18PM -0700, Randy Dunlap wrote:
> On 10/03/14 16:47, Josh Triplett wrote:
> > On Fri, Oct 03, 2014 at 04:36:36PM -0700, Randy Dunlap wrote:
> >> On 10/03/14 16:31, Josh Triplett wrote:
> >>> commit 03b8c7b623c80af264c4c8d6111e5c6289933666 ("futex: Allow
> >>> architectures to skip futex_atomic_cmpxchg_inatomic() test") added the
> >>> HAVE_FUTEX_CMPXCHG symbol right below FUTEX. This placed it right in
> >>> the middle of the options for the EXPERT menu. However,
> >>> HAVE_FUTEX_CMPXCHG does not depend on EXPERT or FUTEX, so Kconfig stops
> >>> placing items in the EXPERT menu, and displays the remaining several
> >>> EXPERT items (starting with EPOLL) directly in the General Setup menu.
> >>>
> >>> Since both users of HAVE_FUTEX_CMPXCHG only select it "if FUTEX", make
> >>> HAVE_FUTEX_CMPXCHG itself depend on FUTEX. With this change, the
> >>> subsequent items display as part of the EXPERT menu again; the EMBEDDED
> >>> menu now appears as the next top-level item in the General Setup menu,
> >>> which makes General Setup much shorter and more usable.
> >>>
> >>> Signed-off-by: Josh Triplett <josh@joshtriplett.org>
> >>> ---
> >>>
> >>> Posting for review. I can upstream this through the tiny tree.
> >>>
> >>> Personally, I'd consider this a bit of a bug in Kconfig; ideally,
> >>> Kconfig should only consider symbols with prompt strings when
> >>> considering what to display in a menu. However, in the interim, this
> >>> one-line patch drastically improves the usability of the "General Setup"
> >>> config menu.
> >>
> >> Good catch. Thanks.
> >>
> >> I would prefer to see both of your patches merged quickly into 3.17 no matter
> >> how they get there.
> >>
> >> both patches:
> >> Acked-by: Randy Dunlap <rdunlap@infradead.org>
> >
> > Both of the fixes are entirely about Kconfig usability, don't affect the
> > built kernel, and have existed for quite a few kernel releases, so I
> > hadn't planned to get them into 3.17 at the last minute. I'd just
> > planned to submit them during the 3.18 merge window when that opens.
> >
> > Do you really think they should be pushed into 3.17 at this point?
>
> Both of them are bugs IMO, even though they are usability/presentation bugs.
> and they make the menu system confusing.
OK, I've pushed just these two patches (with your acks and a Cc to
stable) to tiny/kconfig, and I'll send a direct pull request for that
branch to Linus for 3.17.
- Josh Triplett
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2014-10-04 0:17 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-10-03 23:31 [PATCH 2/2] init/Kconfig: Fix HAVE_FUTEX_CMPXCHG to not break up the EXPERT menu Josh Triplett
2014-10-03 23:36 ` Randy Dunlap
2014-10-03 23:47 ` Josh Triplett
2014-10-04 0:00 ` Randy Dunlap
2014-10-04 0:17 ` Josh Triplett
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox