linux-kernel - Re: [RFC] clocksource: Add clocksource drivers menu.

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Fri, 31 May 2013 11:40:05 -0700
From:	John Stultz <john.stultz@...aro.org>
To:	Srinivas KANDAGATLA <srinivas.kandagatla@...com>
CC:	Thomas Gleixner <tglx@...utronix.de>, linux-kernel@...r.kernel.org,
	Linus Walleij <linus.walleij@...aro.org>
Subject: Re: [RFC] clocksource: Add clocksource drivers menu.

On 05/31/2013 02:59 AM, Srinivas KANDAGATLA wrote:
> From: Srinivas Kandagatla <srinivas.kandagatla@...com>
>
> This patch adds "Clocksource drivers" menu to clocksource drivers.
> The reason to add this is because, some of the clocksource Kconfig
> options like *SHED_CLK ones are selectable and they appear at random
> places in the device drivers menu.
> These options can be more than one option for multiplatform case, so I
> think it will be neat to get a menu to clocksource drivers itself.
>
> Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@...com>

Sigh. So I've done a poor job at being a maintainer[1], and sort of left 
the barn door open here.

I'd *really* like to avoid having a clocksource config menu. Looking at 
the existing RTC menu as an example, there's just a ton of random 
hardware names that the user has to try to figure out if it applies to 
their system or not. For instance, on x86 I have to filter through 90% 
of the RTC drivers that aren't ever even an option on any x86 hardware.

It just turns into a needless config headache.

And only just now did I notice that there are user-prompts in the 
drivers/clocksource/Kconfig.

I suspect the platform support options can safely select the proper 
clocksource config options without requiring the user to configure it. 
For the most part this is the case, even so folks still introduced some 
unnecessary clocksource config options (unnecessary as they don't prompt 
the user, default to y and depend on another config).

It looks like only: CLKSRC_DBX500_PRCMU & 
CLKSRC_DBX500_PRCMU_SCHED_CLOCK prompt the user right now.

Can you explain why its necessary the user has to be prompted here?

I know the situation isn't your fault, and you're just trying to clean 
things up here. However I'd much prefer removing the user-prompt and 
trying to remove unnecessary configs over creating a new config menu. 
And if there must be a user-selected config, I'd much rather it be a 
platform config option rather then some generic-seeming driver config.

thanks
-john

[1] I'm no good at flames. Linus does a much better job covering this topic:
     http://lkml.indiana.edu/hypermail/linux/kernel/1304.3/02435.html
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/